引言

为了更好地理解 JavaScript 的复杂之处，编写出更干净、更高效、更可靠的代码，我们将深入探讨 10 个即使是经验丰富的 JavaScript 开发者也容易犯的错误。

这篇文章将揭示这些错误背后的原因，并提供相应的解决方案，帮助你避免这些陷阱，提升你的 JavaScript 编码水平，最终写出更加优秀、更具可维护性的代码。

1. 忽略使用 Object.freeze 或 Object.seal

错误：

未能阻止对应该保持不变的对象进行修改可能会导致意想不到的副作用，尤其是在大型复杂应用程序中。开发人员可能会无意中修改本应为常量的对象，从而导致难以追踪的错误。

ini
 代码解读
复制代码
const config = { theme: 'dark' };
Object.freeze(config); // 冻结对象
config.theme = 'light'; // 无效，在严格模式下会抛出错误

如何避免：

在处理配置对象或任何应该保持不变的数据结构时，实现 Object.freeze()。此做法有助于防止意外更改并保持应用程序的一致性。

php
 代码解读
复制代码
const settings = Object.freeze({
  apiEndpoint: 'https://api.example.com',
  timeout: 5000
});

2. 在事件监听器中错误地管理 this

错误：

事件监听器可能会让开发人员对 this 关键字感到困惑，它通常指的是触发事件的 DOM 元素，而不是预期的上下文。这种误解会导致错误，方法在错误的上下文中被调用。

javascript
 代码解读
复制代码
button.addEventListener('click', function() {
  this.handleClick(); // 错误：this 指的是按钮元素，而不是预期对象
});

如何避免：

箭头函数没有自己的 this，因此它们从其封闭上下文继承它。或者，使用 bind() 显式设置上下文。

javascript
 代码解读
复制代码
button.addEventListener('click', () => {
  this.handleClick(); // 正确的上下文
});

// 或者使用 bind()
button.addEventListener('click', function() {
  this.handleClick(); // 使用 bind() 后的正确上下文
}.bind(this));

3. 函数功能过多

错误：

将过多的职责合并到一个函数中会使你的代码变得复杂且难以维护。这样的函数难以测试、调试和修改，增加了引入错误的风险。

csharp
 代码解读
复制代码
function handleUserAction(event) {
  // 在一个函数中处理验证、UI 更新、API 调用等
}

如何避免：

将函数分解成更小、更专注的函数。这不仅简化了每个函数，而且使代码更容易理解和维护。

scss
 代码解读
复制代码
function validateInput(value) { /* ... */ }
function updateUI() { /* ... */ }
function submitData(value) { /* ... */ }
function handleUserAction(event) {
  const input = event.target.value;
  if (validateInput(input)) {
    updateUI();
    submitData(input);
  }
}

4. 忽略函数逻辑中的边缘情况

错误：

经验丰富的开发者可能会忽略边缘情况，导致函数在某些情况下失败或行为异常。这会导致仅在特定条件下出现的错误，使其难以重现和修复。

bash
 代码解读
复制代码
function getUserById(users, id) {
  return users.find(user => user.id === id);
}

const users = [{ id: 1, name: 'Alice' }, { id: 2, name: 'Bob' }];
console.log(getUserById(users, 1)); // { id: 1, name: 'Alice' }
console.log(getUserById(users, 3)); // undefined，如果未处理可能会导致问题

如何避免：

在你的函数中加入错误处理和验证，以处理边缘情况和意外输入。

javascript
 代码解读
复制代码
function getUserById(users, id) {
  const user = users.find(user => user.id === id);
  if (!user) {
    throw new Error('用户未找到');
  }
  return user;
}

try {
  console.log(getUserById(users, 1)); // { id: 1, name: 'Alice' }
  console.log(getUserById(users, 3)); // 抛出错误
} catch (error) {
  console.error(error.message); // '用户未找到'
}

5. 忽略用户输入的清理

错误：

即使是经验丰富的开发者有时也会忽略输入清理，这会导致安全漏洞，如跨站脚本 (XSS)。未经清理的输入可以被攻击者利用来执行恶意脚本。

ini
 代码解读
复制代码
const userInput = "<img src='x' onerror='alert(1)'>";
document.body.innerHTML = userInput; // 易受 XSS 攻击

如何避免：

始终清理和验证用户输入，以防止安全问题。使用专门为此目的设计的库，例如 DOMPurify，在处理或显示用户生成的内容之前对其进行清理和保护。

ini
 代码解读
复制代码
const sanitizedInput = DOMPurify.sanitize(userInput);
document.body.innerHTML = sanitizedInput; // 安全使用

6. 忽略闭包的性能影响

错误：

闭包可以捕获并保留对变量的引用，如果未妥善管理，可能会导致内存泄漏。这会影响应用程序的性能并随着时间的推移增加内存使用量。

ini
 代码解读
复制代码
function createEventHandlers(elements) {
  const handlers = [];
  for (let i = 0; i < elements.length; i++) {
    // 每个闭包都捕获了整个 'elements' 数组
    handlers.push(function() {
      console.log('元素被点击：', elements[i].textContent);
    });
  }
  return handlers;
}

const elements = document.querySelectorAll('.list-item');
const handlers = createEventHandlers(elements);

如何避免：

谨慎使用闭包，尤其是在长生命周期的对象或函数中。确保闭包不会无意中保留不再需要的海量数据或引用。

ini
 代码解读
复制代码
function createEventHandlers(elements) {
  const handlers = [];
  for (let i = 0; i < elements.length; i++) {
    const element = elements[i]; // 仅捕获必要的元素
    handlers.push(function() {
      console.log('元素被点击：', element.textContent);
    });
  }
  return handlers;
}

const elements = document.querySelectorAll('.list-item');
const handlers = createEventHandlers(elements);

// 现在，每个处理程序只保留对其需要的单个元素的引用，而不是整个数组

7. 不对高频事件进行去抖动或节流

错误：

在没有去抖动或节流的情况下处理高频事件，如 scroll、resize 或 keypress，会导致函数调用过多，降低性能和用户体验。

javascript
 代码解读
复制代码
window.addEventListener('resize', handleResize); // 在每次调整大小事件中被调用

如何避免：

实现去抖动或节流，以限制事件处理程序执行的速率。这减少了函数调用次数，提高了应用程序的性能。

javascript
 代码解读
复制代码
function debounce(fn, delay) {
  let timeout;
  return function(...args) {
    clearTimeout(timeout);
    timeout = setTimeout(() => fn.apply(this, args), delay);
  };
}
window.addEventListener('resize', debounce(handleResize, 200));

8. 没有利用解构的强大功能

错误：

在处理对象或数组时忘记使用解构会导致冗长、重复的代码，降低可读性并增加出错的风险。

ini
 代码解读
复制代码
const person = { name: 'John', age: 30, job: 'Developer' };
const name = person.name;
const age = person.age;

如何避免：

利用解构简化从对象或数组中提取值。此技术提高了代码可读性并减少了样板代码。

arduino
 代码解读
复制代码
const { name, age, job } = person;

9. 异步代码中错误的错误处理

错误：

未能正确处理异步代码中的错误会导致未处理的 promise 拒绝，从而导致应用程序行为异常和用户体验不佳。

javascript
 代码解读
复制代码
async function fetchData() {
  const response = await fetch('https://api.example.com/data');
  const data = await response.json();
  return data;
}
fetchData().then(data => console.log(data)); // 缺少错误处理

如何避免：

始终使用 try/catch 块（与 async/await 一起使用）或 catch()（与 promises 一起使用）来处理异步代码中的错误。这确保了错误被正确捕获和管理。

javascript
 代码解读
复制代码
async function fetchData() {
  try {
    const response = await fetch('https://api.example.com/data');
    if (!response.ok) {
      throw new Error('网络响应不正常');
    }
    const data = await response.json();
    return data;
  } catch (error) {
    console.error('获取数据错误：', error);
  }
}

10. 忽略代码组织的最佳实践

错误：

经验丰富的开发者可能会忽略代码组织的最佳实践，导致代码库过于庞大和难以维护。糟糕的组织会使代码更难理解、扩展和调试。

csharp
 代码解读
复制代码
// 单片代码示例
function app() {
  function handleRequest() { /* ... */ }
  function renderUI() { /* ... */ }
  // 更多代码在此处
}

如何避免：

采用代码组织的最佳实践，例如将代码模块化成可重用组件或模块，使用清晰的命名约定并保持一致的目录结构。这种方法提高了代码的可维护性和可扩展性。

javascript
 代码解读
复制代码
// 模块化代码示例
// 文件：api.js
export function fetchData() { /* ... */ }
// 文件：ui.js
export function renderUI() { /* ... */ }
// 文件：app.js
import { fetchData } from './api';
import { renderUI } from './ui';
function app() {
  fetchData();
  renderUI();
}

结论

这些错误突出了即使是经验丰富的开发者也会遇到的微妙但重要的挑战。通过注意这些陷阱并遵循最佳实践，你可以编写更干净、更高效、更安全的 JavaScript 代码。

最后，如果本文的内容对你有启发，帮我点个赞，关注一波，分享出去，也许你的转发能给别人带来一点帮助。

作者：前端宝哥
链接：https://juejin.cn/post/7412672705301921803
来源：稀土掘金
著作权归作者所有。商业转载请联系作者获得授权，非商业转载请注明出处。