Changeset 31355 for other-projects/hathitrust/wcsa

Timestamp:

2017-01-24T23:08:13+13:00 (7 years ago)

Author:

davidb

Message:

Changed to using containsKey rather than get to avoid null pointer cast problem

File:

• other-projects/hathitrust/wcsa/vol-checker/src/org/hathitrust/extractedfeatures/VolumeCheck.java (modified) (3 diffs)

other-projects/hathitrust/wcsa/vol-checker/src/org/hathitrust/extractedfeatures/VolumeCheck.java

@@ -256 +256 @@
         String cgi_convert_col = request.getParameter("convert-col");
         
-        System.err.println("**** cgi_ids = '" + cgi_ids + "'");
-        System.err.println("**** cgi_convert_col = '" + cgi_convert_col + "'");
-        
         if (cgi_ids != null) {
             response.setContentType("application/json");
@@ -270 +267 @@
             for (int i=0; i<ids_len; i++) {
                 String id = ids[i];
-                
-                boolean exists = id_check_.get(id);
+
+                boolean exists = id_check_.containsKey(id);
                 
                 if (i>0) {
@@ -286 +283 @@
             
             String id = cgi_id;
-            boolean exists = id_check_.get(id);
+            boolean exists = id_check_.containsKey(id);
             pw.append("{'" + id + "':" + exists + "}");
         }
         else if (cgi_download_id != null) {
             String download_id = cgi_download_id;
-            boolean exists = id_check_.get(download_id);
+            boolean exists = id_check_.containsKey(download_id);
             if (!exists) {
                 // Error