Better number parsing, handles scientific notation.

broadinstitute · Oct 28, 2024 · 5114be0 · 5114be0
1 parent a2b6201
commit 5114be0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/src/main/java/org/broadinstitute/gpp/rigerj/RigerJMain.java b/src/main/java/org/broadinstitute/gpp/rigerj/RigerJMain.java
@@ -199,11 +199,12 @@ private static List<HairpinData> createHairpinDatas() throws IOException {
                 break;
             }
             String[] cells = line.split("\t", 0);
-            hairpinDatas.add(new HairpinData(new Integer(cells[3]),
+            int rank = Double.valueOf(cells[3]).intValue();
+            hairpinDatas.add(new HairpinData(rank,
                                              cells[0],
-                                             new Double(cells[2]),
+                                             Double.parseDouble(cells[2]),
                                              cells[1],
-                                             cells.length == 5 ? new Double(cells[4]) : 1));
+                                             cells.length == 5 ? Double.parseDouble(cells[4]) : 1));
 
         }
         return hairpinDatas;

diff --git a/src/test/java/org/broadinstitute/gpp/rigerj/impl/RigerImplTest.java b/src/test/java/org/broadinstitute/gpp/rigerj/impl/RigerImplTest.java
@@ -68,9 +68,10 @@ private static void initializeHairpinInputs() throws IOException {
                 break;
             }
             String[] cells = line.split("\t", 0);
-            hairpinInputs.add(new HairpinData(new Integer(cells[3]),
+            int rank = Double.valueOf(cells[3]).intValue();
+            hairpinInputs.add(new HairpinData(rank,
                                               cells[0],
-                                              new Double(cells[2]),
+                                              Double.parseDouble(cells[2]),
                                               cells[1],
                                               1));
         }