Make sure we don't cross more than max text lines

mcharters · mcharters · commit c823063ca3f4 · 2016-01-29T16:01:49.000-05:00
Sometimes when text is arranged into lines it can get messed up and have
chunks on more lines than we actually recognize as lines. Make sure that
we don't get array out of bounds exceptions.
diff --git a/src/main/java/technology/tabula/detectors/NurminenDetectionAlgorithm.java b/src/main/java/technology/tabula/detectors/NurminenDetectionAlgorithm.java
@@ -535,7 +535,7 @@ private TextEdges getTextEdges(List<Line> lines) {
                             TextChunk last = edgeChunks.get(edgeChunks.size() - 1);
 
                             TextEdge edge = new TextEdge(key, first.getTop(), key, last.getBottom());
-                            edge.intersectingTextRowCount = edgeChunks.size();
+                            edge.intersectingTextRowCount = Math.min(edgeChunks.size(), lines.size());
 
                             leftTextEdges.add(edge);
                         }
@@ -553,7 +553,7 @@ private TextEdges getTextEdges(List<Line> lines) {
                             TextChunk last = edgeChunks.get(edgeChunks.size() - 1);
 
                             TextEdge edge = new TextEdge(key, first.getTop(), key, last.getBottom());
-                            edge.intersectingTextRowCount = edgeChunks.size();
+                            edge.intersectingTextRowCount = Math.min(edgeChunks.size(), lines.size());
 
                             midTextEdges.add(edge);
                         }
@@ -571,7 +571,7 @@ private TextEdges getTextEdges(List<Line> lines) {
                             TextChunk last = edgeChunks.get(edgeChunks.size() - 1);
 
                             TextEdge edge = new TextEdge(key, first.getTop(), key, last.getBottom());
-                            edge.intersectingTextRowCount = edgeChunks.size();
+                            edge.intersectingTextRowCount = Math.min(edgeChunks.size(), lines.size());
 
                             rightTextEdges.add(edge);
                         }
@@ -588,7 +588,7 @@ private TextEdges getTextEdges(List<Line> lines) {
                 TextChunk last = edgeChunks.get(edgeChunks.size() - 1);
 
                 TextEdge edge = new TextEdge(key, first.getTop(), key, last.getBottom());
-                edge.intersectingTextRowCount = edgeChunks.size();
+                edge.intersectingTextRowCount = Math.min(edgeChunks.size(), lines.size());
 
                 leftTextEdges.add(edge);
             }
@@ -601,7 +601,7 @@ private TextEdges getTextEdges(List<Line> lines) {
                 TextChunk last = edgeChunks.get(edgeChunks.size() - 1);
 
                 TextEdge edge = new TextEdge(key, first.getTop(), key, last.getBottom());
-                edge.intersectingTextRowCount = edgeChunks.size();
+                edge.intersectingTextRowCount = Math.min(edgeChunks.size(), lines.size());
 
                 midTextEdges.add(edge);
             }
@@ -614,7 +614,7 @@ private TextEdges getTextEdges(List<Line> lines) {
                 TextChunk last = edgeChunks.get(edgeChunks.size() - 1);
 
                 TextEdge edge = new TextEdge(key, first.getTop(), key, last.getBottom());
-                edge.intersectingTextRowCount = edgeChunks.size();
+                edge.intersectingTextRowCount = Math.min(edgeChunks.size(), lines.size());
 
                 rightTextEdges.add(edge);
             }
