Now catching empty arrays in getCentroid

josemduarte · josemduarte · commit 80a3eaaacaac · 2017-01-21T12:31:29.000-08:00
diff --git a/biojava-structure/src/main/java/org/biojava/nbio/structure/Calc.java b/biojava-structure/src/main/java/org/biojava/nbio/structure/Calc.java
@@ -777,6 +777,10 @@ public static final void shift(Group group, Atom a) {
 	 * @return an Atom representing the Centroid of the set of atoms
 	 */
 	public static final Atom getCentroid(Atom[] atomSet) {
+		
+		// if we don't catch this case, the centroid returned is (NaN,NaN,NaN), which can cause lots of problems down the line
+		if (atomSet.length==0) 
+			throw new IllegalArgumentException("Atom array has length 0, can't calculate centroid!");
 
 		double[] coords = new double[3];
 
