chore: improve performance (#2868)

alibaba · Feb 25, 2022 · 0fd17ea · 0fd17ea
1 parent edcc954
commit 0fd17ea
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/packages/path/src/index.ts b/packages/path/src/index.ts
@@ -16,6 +16,16 @@ const isMatcher = Symbol('PATH_MATCHER')
 
 const isValid = (val: any) => val !== undefined && val !== null
 
+const isSimplePath = (val: string) =>
+  val.indexOf('*') === -1 &&
+  val.indexOf('~') === -1 &&
+  val.indexOf('[') === -1 &&
+  val.indexOf(']') === -1 &&
+  val.indexOf(',') === -1 &&
+  val.indexOf(':') === -1 &&
+  val.indexOf(' ') === -1 &&
+  val[0] !== '.'
+
 const isAssignable = (val: any) =>
   typeof val === 'object' || typeof val === 'function'
 
@@ -140,7 +150,7 @@ const parse = (pattern: Pattern, base?: Pattern) => {
       tree: pattern.tree,
     }
   } else if (isStr(pattern)) {
-    if (!pattern)
+    if (!pattern) {
       return {
         entire: '',
         segments: [],
@@ -149,6 +159,17 @@ const parse = (pattern: Pattern, base?: Pattern) => {
         haveExcludePattern: false,
         isMatchPattern: false,
       }
+    }
+    if (isSimplePath(pattern)) {
+      return {
+        entire: pattern,
+        segments: pattern.split('.'),
+        isRegExp: false,
+        isWildMatchPattern: false,
+        haveExcludePattern: false,
+        isMatchPattern: false,
+      }
+    }
     const parser = new Parser(pattern, Path.parse(base))
     const tree = parser.parse()
     if (!parser.isMatchPattern) {

diff --git a/packages/reactive/src/handlers.ts b/packages/reactive/src/handlers.ts
@@ -165,6 +165,7 @@ export const collectionHandlers = {
 
 export const baseHandlers: ProxyHandler<any> = {
   get(target, key, receiver) {
+    if (!key) return
     const result = target[key] // use Reflect.get is too slow
     if (typeof key === 'symbol' && wellKnownSymbols.has(key)) {
       return result