Merge pull request #58 from nigrosimone/main

perf: micro-optimization avoid multiple call to map
dimdenGD · Nov 17, 2024 · 5d68ff6 · 5d68ff6
2 parents 7ff6a36 + 50e6706
commit 5d68ff6
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/router.js b/src/router.js
@@ -386,10 +386,10 @@ module.exports = class Router extends EventEmitter {
         if(this._paramCallbacks.size > 0) {
             return new Promise(async resolve => {
                 for(let param in req.params) {
-                    if(this._paramCallbacks.has(param) && !req._gotParams.has(param)) {
+                    const pcs = this._paramCallbacks.get(param);
+                    if(pcs && !req._gotParams.has(param)) {
                         req._gotParams.add(param);
-                        const pcs = this._paramCallbacks.get(param);
-                        for(let i = 0; i < pcs.length; i++) {
+                        for(let i = 0, len = pcs.length; i < len; i++) {
                             const fn = pcs[i];
                             await new Promise(resolveRoute => {
                                 const next = (thingamabob) => {