function findDuplicates(arr) {
let duplicates = [];
for (let i = 0; i < arr.length; i++) {
for (let j = i + 1; j < arr.length; j++) {
if (arr[i] === arr[j] && !duplicates.includes(arr[i])) {
duplicates.push(arr[i]);
}
}
}
return duplicates;
}
我们的工具可能会提供如下反馈：
性能问题：嵌套循环导致时间复杂度为O(n^2)，可能在大型数组上性能较差。
优化建议：考虑使用Set或对象来提高效率，将时间复杂度降低到O(n)。
改进后的代码示例：
function findDuplicates(arr) {
const seen = new Set();
const duplicates = new Set();
for (const item of arr) {
if (seen.has(item)) {
duplicates.add(item);
} else {
seen.add(item);
}
}
return Array.from(duplicates);
}
示例2：提高可读性
考虑以下代码：
function calc(a,b,c){let r;if(c=='add'){r=a+b;}else if(c=='sub'){r=a-b;}else if(c=='mul'){r=a*b;}else if(c=='div'){r=a/b;}return r;}
我们的工具可能会提供如下建议：
可读性问题：代码压缩在一行，难以阅读和维护。
命名问题：变量名不够描述性。
结构问题：使用多个if-else语句不如switch语句清晰。
改进后的代码示例：
function calculate(firstNumber, secondNumber, operation) {
switch (operation) {
case 'add':
return firstNumber + secondNumber;
case 'subtract':
return firstNumber - secondNumber;
case 'multiply':
return firstNumber * secondNumber;
case 'divide':
return firstNumber / secondNumber;
default:
throw new Error('Invalid operation');
}
}
