From d3e6481b7197bb69209e15da7e08a597dfefdec2 Mon Sep 17 00:00:00 2001
From: evenisse <evenisse@13f79535-47bb-0310-9956-ffa450edef68>
Date: Sat, 6 Mar 2004 22:41:56 +0000
Subject: [PATCH] Fix possible NumberFormatException

git-svn-id: https://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk@114799 13f79535-47bb-0310-9956-ffa450edef68
---
 .../maven/jcoveragereport/Coverage.java       | 26 ++++++++++++++++---
 .../apache/maven/jcoveragereport/Package.java | 24 ++++++++++++++---
 2 files changed, 43 insertions(+), 7 deletions(-)

diff --git a/jcoverage/src/main/org/apache/maven/jcoveragereport/Coverage.java b/jcoverage/src/main/org/apache/maven/jcoveragereport/Coverage.java
index 83b8c35f..29949718 100644
--- a/jcoverage/src/main/org/apache/maven/jcoveragereport/Coverage.java
+++ b/jcoverage/src/main/org/apache/maven/jcoveragereport/Coverage.java
@@ -26,7 +26,7 @@ import java.util.HashMap;
 
 /**
  * @author Emmanuel Venisse
- * @version $Id: Coverage.java,v 1.3 2004/03/02 15:07:06 evenisse Exp $
+ * @version $Id: Coverage.java,v 1.4 2004/03/06 22:41:56 evenisse Exp $
  */
 public class Coverage
 {
@@ -129,8 +129,17 @@ public class Coverage
         {
             for (Iterator iter = getClasses().iterator(); iter.hasNext(); )
             {
-                    Clazz theClass = (Clazz) iter.next();
-                    total += new Double(theClass.getBranchRate()).floatValue();
+                Clazz theClass = (Clazz) iter.next();
+                double rate = 0;
+                try
+                {
+                    rate = new Double(theClass.getBranchRate()).floatValue();
+                }
+                catch(NumberFormatException e)
+                {
+                    rate = 0;
+                }
+                total += rate;
             }
         }
 
@@ -143,7 +152,16 @@ public class Coverage
         for (Iterator iter = getClasses().iterator(); iter.hasNext(); )
         {
             Clazz theClass = (Clazz) iter.next();
-            total += new Double(theClass.getLineRate()).floatValue();
+            double rate = 0;
+            try
+            {
+                rate = new Double(theClass.getLineRate()).floatValue();
+            }
+            catch(NumberFormatException e)
+            {
+                rate = 0;
+            }
+            total += rate;
         }
         return total / getClasses().size();
     }
diff --git a/jcoverage/src/main/org/apache/maven/jcoveragereport/Package.java b/jcoverage/src/main/org/apache/maven/jcoveragereport/Package.java
index c6df731f..e5dfa931 100644
--- a/jcoverage/src/main/org/apache/maven/jcoveragereport/Package.java
+++ b/jcoverage/src/main/org/apache/maven/jcoveragereport/Package.java
@@ -24,7 +24,7 @@ import java.util.List;
 
 /**
  * @author Emmanuel Venisse
- * @version $Id: Package.java,v 1.4 2004/03/02 15:07:06 evenisse Exp $
+ * @version $Id: Package.java,v 1.5 2004/03/06 22:41:56 evenisse Exp $
  */
 public class Package
 {
@@ -98,7 +98,16 @@ public class Package
             for (Iterator iter = getClasses().iterator(); iter.hasNext(); )
             {
                 Clazz theClass = (Clazz) iter.next();
-                total += new Double(theClass.getBranchRate()).floatValue();
+                double rate = 0;
+                try
+                {
+                    rate = new Double(theClass.getBranchRate()).floatValue();
+                }
+                catch(NumberFormatException e)
+                {
+                    rate = 0;
+                }
+                total += rate;
             }
 
             total /= getClasses().size();
@@ -114,7 +123,16 @@ public class Package
         for (Iterator iter = getClasses().iterator(); iter.hasNext(); )
         {
             Clazz theClass = (Clazz) iter.next();
-            total += new Double(theClass.getLineRate()).floatValue();
+            double rate = 0;
+            try
+            {
+                rate = new Double(theClass.getLineRate()).floatValue();
+            }
+            catch(NumberFormatException e)
+            {
+                rate = 0;
+            }
+            total += rate;
         }
 
         return (total / getClasses().size());