属于博物馆而不是存储库的代码-js教程-PHP中文网

属于博物馆而不是存储库的代码

碧海醫心

发布： 2025-01-14 08:00:18

原创

266人浏览过

属于博物馆而不是存储库的代码

代码之美：实用性胜于炫技

我们都见过那些代码——结构复杂，晦涩难懂，更像是博物馆展品而非工程代码。初看或许令人惊叹，但一旦需要调试，便会让人望而生畏。与其追求代码的炫技，不如专注于代码的实用性。记住，漂亮的代码必须首先是有效的代码。

为何要避免过度追求“聪明”的代码？如何避免？让我们深入探讨。

过度设计的陷阱

开发者为何编写过于复杂的代码？

满足成就感: 编写巧妙的代码能带来智力上的满足感，如同完成一项智力挑战。
哗众取宠: 这种代码或许能给部分人留下深刻印象，但维护起来却极其困难。
炫耀技术: 真正的技术精湛不在于创造复杂性，而在于化繁为简，高效地解决问题。

案例分析：避免过度设计的两种方式

案例一： “令人费解”的工厂函数

以下代码虽然精巧，但却难以理解：

const createmultiplier = (x) => (y) => (z) => x * y * z;
const multiply = createmultiplier(2)(3);
console.log(multiply(4)); // 输出 24

登录后复制

三层嵌套函数仅仅是为了实现三个数的乘法？这无疑增加了代码的复杂度。更简洁易懂的写法如下：

function multiplyThreeNumbers(x, y, z) {
  return x * y * z;
}
console.log(multiplyThreeNumbers(2, 3, 4)); // 输出 24

登录后复制

案例二：冗长的Promise链

以下代码功能虽然实现，但其复杂的Promise链结构却降低了可读性和可维护性：

fetch(url)
  .then((response) => response.json())
  .then((data) =>
    data.map((item) =>
      item.isactive
        ? { ...item, status: "active" }
        : { ...item, status: "inactive" }
    )
  )
  .then((updateddata) =>
    updateddata.reduce(
      (acc, curr) =>
        curr.status === "active"
          ? { ...acc, active: [...acc.active, curr] }
          : { ...acc, inactive: [...acc.inactive, curr] },
      { active: [], inactive: [] }
    )
  )
  .then((finalresult) => console.log(finalresult))
  .catch((error) => console.error(error));

登录后复制

改进后的代码：

async function processData(url) {
  try {
    const response = await fetch(url);
    const data = await response.json();
    const updatedData = data.map(item => ({...item, status: item.isActive ? 'active' : 'inactive'}));
    const finalResult = updatedData.reduce((acc, curr) => {
      (curr.status === 'active' ? acc.active : acc.inactive).push(curr);
      return acc;
    }, {active: [], inactive: []});
    console.log(finalResult);
  } catch (error) {
    console.error(error);
  }
}
processData(url);

登录后复制

将复杂逻辑分解成更小的、更易于理解的步骤，极大地提升了代码的可读性。

简洁代码的优势

记住，代码是沟通的工具，而非个人表达的载体。简洁的代码带来诸多好处：