be: clearNumber

devezhao · Aug 8, 2023 · 837605e · 837605e
1 parent 8aebaa8
commit 837605e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/main/java/cn/devezhao/persist4j/record/RecordVisitor.java b/src/main/java/cn/devezhao/persist4j/record/RecordVisitor.java
@@ -11,6 +11,8 @@
 import cn.devezhao.persist4j.engine.ID;
 import org.apache.commons.lang.StringUtils;
 import org.apache.commons.lang.math.NumberUtils;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
 import java.io.StringReader;
 import java.math.BigDecimal;
@@ -29,6 +31,8 @@
  */
 public class RecordVisitor {
 
+	private static final Log LOG = LogFactory.getLog(RecordVisitor.class);
+
 	public static final String DATE_FORMAT_STRING = "yyyy-MM-dd";
 	public static final String DATETIME_FORMAT_STRING = "yyyy-MM-dd HH:mm:ss";
 
@@ -162,7 +166,11 @@ public static String getLiteralByValue(Field field, Object value) {
 	 */
 	protected static String clearNumber(String num) {
 		if (StringUtils.isBlank(num)) return "0";
-		return num.replace(",", "").trim();
+		num = num.replace(",", "").trim();
+		if (NumberUtils.isNumber(num)) return num;
+
+		LOG.warn("Bad number format : " + num);
+		return "0";
 	}
 
 	/**

diff --git a/src/test/java/cn/devezhao/persist4j/record/RecordVisitorTest.java b/src/test/java/cn/devezhao/persist4j/record/RecordVisitorTest.java
@@ -10,5 +10,6 @@ public void testClearNumber() {
         System.out.println(RecordVisitor.clearNumber("123d "));
         System.out.println(RecordVisitor.clearNumber("1,123 "));
         System.out.println(RecordVisitor.clearNumber("111,1,11,,123.00 "));
+        System.out.println(RecordVisitor.clearNumber("NaN"));
     }
 }