Skip to content

Commit 953782e

Browse files
committed
Combine the two iterations for bracketJoin into a single iteration
Change-type: patch
1 parent 830f2a2 commit 953782e

1 file changed

Lines changed: 7 additions & 8 deletions

File tree

src/index.ts

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -408,17 +408,16 @@ const bracketJoin = (arr: string[][], separator: string): string[] => {
408408
return arr[0];
409409
}
410410
const resultArr: string[] = [];
411-
const mappedArr = arr.map((subArr) => {
412-
if (subArr.length > 1) {
413-
return `(${subArr.join('')})`;
414-
}
415-
return subArr[0];
416-
});
417-
for (let i = 0; i < mappedArr.length; i++) {
411+
for (let i = 0; i < arr.length; i++) {
418412
if (i !== 0) {
419413
resultArr.push(separator);
420414
}
421-
resultArr.push(mappedArr[i]);
415+
const subArr = arr[i];
416+
if (subArr.length > 1) {
417+
resultArr.push(`(${subArr.join('')})`);
418+
} else {
419+
resultArr.push(subArr[0]);
420+
}
422421
}
423422
return resultArr;
424423
};

0 commit comments

Comments
 (0)