feat(hippy-vue, hippy-react): changeTryConvertNumberCompatibility

Author: zoomchan-cxj

packages/hippy-react/src/utils/index.ts

@@ -38,7 +38,8 @@ function unicodeToChar(text: string): string {
 /**
  * Convert to string as possible
  */
-const numberRegEx = new RegExp('^[+-]?\\d+(\\.\\d+)?$');
+const numberRegEx = new RegExp('^[+-]?\\d*\\.?\\d*([Ee][+-]?\\d+)?$');
+const notEmptyRegEx = new RegExp('^.+$');
 /**
  * Try to convert something to number
  *
@@ -48,7 +49,7 @@ function tryConvertNumber(input: any) {
   if (typeof input === 'number') {
     return input;
   }
-  if (typeof input === 'string' && numberRegEx.test(input)) {
+  if (typeof input === 'string' && numberRegEx.test(input) && notEmptyRegEx.test(input)) {
     try {
       return parseFloat(input);
     } catch (err) {

packages/hippy-vue/src/util/__tests__/index.test.js

@@ -61,8 +61,15 @@ test('tryConvertNumber output test', (t) => {
   t.is(util.tryConvertNumber(123), 123);
   t.is(util.tryConvertNumber('123'), 123);
   t.is(util.tryConvertNumber('abc'), 'abc');
-  t.is(util.tryConvertNumber('123abc'), 123);
+  t.is(util.tryConvertNumber('123abc'), '123abc');
   t.is(util.tryConvertNumber('abc123'), 'abc123');
+  t.is(util.tryConvertNumber('12e3'), 12000);
+  t.is(util.tryConvertNumber('123.12'), 123.12);
+  t.is(util.tryConvertNumber('123.'), 123);
+  t.is(util.tryConvertNumber('.123'), 0.123);
+  t.is(util.tryConvertNumber('.123.'), '.123.');
+  t.is(util.tryConvertNumber('.123.1'), '.123.1');
+  t.is(util.tryConvertNumber(''), '');
   const obj = {};
   t.is(util.tryConvertNumber(obj), obj);
 });

packages/hippy-vue/src/util/index.js

@@ -86,15 +86,15 @@ function capitalizeFirstLetter(str) {
 /**
  * Convert string to number as possible
  */
-// const numberRegEx = new RegExp('^[+-]?(\\d+)?(\\.\\d+)?$');
+const numberRegEx = new RegExp('^[+-]?\\d*\\.?\\d*([Ee][+-]?\\d+)?$');
+const notEmptyRegEx = new RegExp('^.+$');
 function tryConvertNumber(str) {
   if (typeof str === 'number') {
     return str;
   }
-  if (typeof str === 'string') {
+  if (typeof str === 'string' && numberRegEx.test(str) && notEmptyRegEx.test(str)) {
     try {
-      const parsedNumber = parseFloat(str);
-      if (!Number.isNaN(parsedNumber)) return parsedNumber;
+      return parseFloat(str);
     } catch (err) {
       // pass
     }