X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fws%2Fdbsources%2FPDBRestClient.java;h=f406702503392fab4b210ee027c865a809d08b7e;hb=a624bae1db1ec7edde062bacfe3890c1e018209f;hp=1a80d65233879594158efd01295f2ecd8b58ca24;hpb=fd235cc7542d6e308e054a8a76580a6c511ae631;p=jalview.git diff --git a/src/jalview/ws/dbsources/PDBRestClient.java b/src/jalview/ws/dbsources/PDBRestClient.java index 1a80d65..f406702 100644 --- a/src/jalview/ws/dbsources/PDBRestClient.java +++ b/src/jalview/ws/dbsources/PDBRestClient.java @@ -23,16 +23,24 @@ import com.sun.jersey.api.client.config.ClientConfig; import com.sun.jersey.api.client.config.DefaultClientConfig; import com.sun.jersey.api.json.JSONConfiguration; +/** + * A rest client for querying the Search endpoing of the PDB REST API + * + * @author tcnofoegbu + * + */ public class PDBRestClient { - private String pdbSearchEndpoint = "http://wwwdev.ebi.ac.uk/pdbe/search/pdb/select?"; + private static String PDB_SEARCH_ENDPOINT = "http://wwwdev.ebi.ac.uk/pdbe/search/pdb/select?"; + + private static int DEFAULT_RESPONSE_SIZE = 200; /** * Takes a PDBRestRequest object and returns a response upon execution * * @param pdbRestRequest - * the pdbRequest to be sent - * @return the pdbResponse object for the given pdbRequest + * the PDBRestRequest instance to be processed + * @return the pdbResponse object for the given request */ public PDBRestResponse executeRequest(PDBRestRequest pdbRestRequest) { @@ -41,45 +49,52 @@ public class PDBRestClient Boolean.TRUE); Client client = Client.create(clientConfig); - String query = pdbRestRequest.getFieldToSearchBy() - + pdbRestRequest.getSearchTerm() - + ((pdbRestRequest.isAllowEmptySeq()) ? "" - : " AND molecule_sequence:['' TO *]"); - String wantedFields = getPDBDocFieldsAsCommaDelimitedString(pdbRestRequest .getWantedFields()); - - String responseSize = (pdbRestRequest.getResponseSize() == 0) ? "200" - : String.valueOf(pdbRestRequest.getResponseSize()); + int responseSize = (pdbRestRequest.getResponseSize() == 0) ? DEFAULT_RESPONSE_SIZE + : pdbRestRequest.getResponseSize(); String sortParam = (pdbRestRequest.getFieldToSortBy() == null || pdbRestRequest - .getFieldToSortBy().trim().isEmpty()) ? "" - : (pdbRestRequest + .getFieldToSortBy().trim().isEmpty()) ? "" : (pdbRestRequest .getFieldToSortBy() + (pdbRestRequest.isAscending() ? " asc" : " desc")); - WebResource webResource = client.resource(pdbSearchEndpoint) + // Build request parameters for the REST Request + WebResource webResource = client.resource(PDB_SEARCH_ENDPOINT) .queryParam("wt", "json").queryParam("fl", wantedFields) - .queryParam("rows", responseSize) - .queryParam("q", query) + .queryParam("rows", String.valueOf(responseSize)) + .queryParam("q", pdbRestRequest.getQuery()) .queryParam("sort", sortParam); + + // Execute the REST request ClientResponse clientResponse = webResource.accept( MediaType.APPLICATION_JSON).get(ClientResponse.class); + // Get the JSON string from the response object String responseString = clientResponse.getEntity(String.class); + + // Check the response status and report exception if one occurs if (clientResponse.getStatus() != 200) { + String errorMessage = ""; if (clientResponse.getStatus() == 400) { - throw new RuntimeException(parseJsonExceptionString(responseString)); + errorMessage = parseJsonExceptionString(responseString); + throw new RuntimeException(errorMessage); } else { - throw new RuntimeException("Failed : HTTP error code : " - + clientResponse.getStatus()); + errorMessage = "Failed : HTTP error code : " + + clientResponse.getStatus(); + throw new RuntimeException(errorMessage); } } + + // Make redundant objects eligible for garbage collection to conserve + // memory clientResponse = null; client = null; + + // Process the response and return the result to the caller. return parsePDBJsonResponse(responseString, pdbRestRequest); } @@ -87,45 +102,52 @@ public class PDBRestClient * Process error response from PDB server if/when one occurs. * * @param jsonResponse - * the json string containing error message from the server - * @return the processed error message from the json string + * the JSON string containing error message from the server + * @return the processed error message from the JSON string */ public static String parseJsonExceptionString(String jsonErrorResponse) { - String errorMessage = "RunTime error"; + StringBuilder errorMessage = new StringBuilder( + "\n============= PDB Rest Client RunTime error =============\n"); try { JSONParser jsonParser = new JSONParser(); JSONObject jsonObj = (JSONObject) jsonParser.parse(jsonErrorResponse); JSONObject errorResponse = (JSONObject) jsonObj.get("error"); - errorMessage = errorResponse.get("msg").toString(); - JSONObject responseHeader = (JSONObject) jsonObj .get("responseHeader"); - errorMessage += responseHeader.get("params").toString(); + JSONObject paramsObj = (JSONObject) responseHeader.get("params"); + String status = responseHeader.get("status").toString(); + String message = errorResponse.get("msg").toString(); + String query = paramsObj.get("q").toString(); + String fl = paramsObj.get("fl").toString(); + + errorMessage.append("Status: ").append(status).append("\n"); + errorMessage.append("Message: ").append(message).append("\n"); + errorMessage.append("query: ").append(query).append("\n"); + errorMessage.append("fl: ").append(fl).append("\n"); } catch (ParseException e) { e.printStackTrace(); } - return errorMessage; + return errorMessage.toString(); } /** - * Parses json response string from PDB REST API to a PDBRestResponse - * instance. The parsed response is dynamic and based upon some of the request - * parameters. + * Parses the JSON response string from PDB REST API. The response is dynamic + * hence, only fields specifically requested for in the 'wantedFields' + * parameter is fetched/processed * * @param pdbJsonResponseString - * the json string to be parsed + * the JSON string to be parsed * @param pdbRestRequest * the request object which contains parameters used to process the - * json string + * JSON string * @return */ @SuppressWarnings("unchecked") public static PDBRestResponse parsePDBJsonResponse( - String pdbJsonResponseString, - PDBRestRequest pdbRestRequest) + String pdbJsonResponseString, PDBRestRequest pdbRestRequest) { PDBRestResponse searchResult = new PDBRestResponse(); List result = null; @@ -148,10 +170,8 @@ public class PDBRestClient .hasNext();) { JSONObject doc = docIter.next(); - // if (doc.get("molecule_sequence") != null) - // { - result.add(searchResult.new PDBResponseSummary(doc, pdbRestRequest)); - // } + result.add(searchResult.new PDBResponseSummary(doc, + pdbRestRequest)); } searchResult.setNumberOfItemsFound(numFound); searchResult.setResponseTime(queryTime); @@ -161,16 +181,16 @@ public class PDBRestClient { e.printStackTrace(); } - return searchResult; } /** - * Takes a collection of PDBDocField and converts it into a comma delimited - * string. + * Takes a collection of PDBDocField and converts its 'code' Field values into + * a comma delimited string. * * @param pdbDocfields - * @return + * the collection of PDBDocField to process + * @return the comma delimited string from the pdbDocFields collection */ public static String getPDBDocFieldsAsCommaDelimitedString( Collection pdbDocfields) @@ -183,15 +203,16 @@ public class PDBRestClient { returnedFields.append(",").append(field.getCode()); } - returnedFields.deleteCharAt(0); + returnedFields.deleteCharAt(0); result = returnedFields.toString(); } return result; } /** - * Determines the column index for the pdb id in the summary table. The pdb id - * serves as a unique identifier for a given row in the summary table + * Determines the column index for 'PDB Id' Fields in the dynamic summary + * table. The PDB Id serves as a unique identifier for a given row in the + * summary table * * @param wantedFeilds * the available table columns in no particular order @@ -200,20 +221,24 @@ public class PDBRestClient public static int getPDBIdColumIndex( Collection wantedFeilds, boolean hasRefSeq) { - int pdbFeildIndex = hasRefSeq ? 1 : 0; + + // If a reference sequence is attached then start counting from 1 else + // start from zero + int pdbFeildIndexCounter = hasRefSeq ? 1 : 0; + for (PDBDocField feild : wantedFeilds) { if (feild.equals(PDBDocField.PDB_ID)) { - break; + break; // Once PDB Id index is determined exit iteration } - ++pdbFeildIndex; + ++pdbFeildIndexCounter; } - return pdbFeildIndex; + return pdbFeildIndexCounter; } /** - * Represents the fields retrievable from a PDB Document response + * This enum represents the fields available in the PDB JSON response * */ public enum PDBDocField @@ -253,14 +278,14 @@ public class PDBRestClient "Max observed residues", "max_observed_residues"), ORG_SCI_NAME( "Organism scientific name", "organism_scientific_name"), SUPER_KINGDOM( "Super kingdom", "superkingdom"), RANK("Rank", "rank"), CRYSTALLISATION_PH( - "Crystallisation Ph", "crystallisation_ph"), BIO_FUNCTION( - "Biological Function", "biological_function"), BIO_PROCESS( - "Biological Process", "biological_process"), BIO_CELL_COMP( + "Crystallisation Ph", "crystallisation_ph"), BIOLOGICAL_FUNCTION( + "Biological Function", "biological_function"), BIOLOGICAL_PROCESS( + "Biological Process", "biological_process"), BIOLOGICAL_CELL_COMPONENT( "Biological Cell Component", "biological_cell_component"), COMPOUND_NAME( "Compound Name", "compound_name"), COMPOUND_ID("Compound Id", "compound_id"), COMPOUND_WEIGHT("Compound Weight", - "compound_weight"), COMP_SYS_NAME("Compound Systematic Name", - "compound_systematic_name"), INTERACTING_LIG( + "compound_weight"), COMPOUND_SYSTEMATIC_NAME( + "Compound Systematic Name", "compound_systematic_name"), INTERACTING_LIG( "Interacting Ligands", "interacting_ligands"), JOURNAL( "Journal", "journal"), ALL_AUTHORS("All Authors", "all_authors"), EXPERIMENTAL_DATA_AVAILABLE( "Experiment Data Available", "experiment_data_available"), DIFFRACTION_PROTOCOL( @@ -318,4 +343,4 @@ public class PDBRestClient return name; } } -} +} \ No newline at end of file