长函数应拆分为单一职责的小函数，提升可读性和可测试性；2. 重复代码需提取公共逻辑，利用高阶函数或模板函数统一处理；3. 多参数函数建议合并为带默认值的配置对象，并使用解构赋值；4. 全局变量滥用可通过模块化、闭包或类封装来避免副作用；5. 嵌套过深可用async/await、提前返回和流水线拆分降低复杂度；6. 魔法值应替换为命名常量、枚举或集中配置，增强语义与维护性。持续重构可提升JavaScript项目质量。
代码坏味道是影响JavaScript项目可维护性和可读性的常见问题。识别并改进这些坏味道，能显著提升代码质量。以下是一些典型的坏味道及其重构建议。
1. 长函数与职责不清
当一个函数超过20行、承担多个职责或嵌套层级过深时，就应引起警惕。这类函数难以测试和复用。
改进建议：
- 将大函数拆分为多个小函数，每个函数只做一件事
- 使用具名函数表达式增强可读性
- 提取条件判断为独立函数，如 isUserEligible(user) 而非内联复杂逻辑
2. 重复代码（Duplicated Code）
相同或相似的代码块出现在多个地方，一旦需求变更，需多处修改，易出错。
重构方式：
- 提取公共逻辑到工具函数或类方法中
- 利用高阶函数处理数组操作，如 map、filter、reduce 替代 for 循环
- 使用模板函数统一结构，差异部分通过参数传入
3. 过多参数与参数对象混乱
函数参数超过3个时，调用容易出错，且难以记忆顺序。
优化策略：
- 将相关参数合并为一个配置对象
- 为对象参数设置默认值，提高容错性
- 使用解构赋值简化内
部访问
部访问例如：
function createUser({ name, age = 18, role = 'user' }) { ... }
4. 全局变量滥用
全局变量容易被意外修改，导致副作用和调试困难。
解决方案：
- 使用模块化（ES Module 或 CommonJS）隔离作用域
- 通过闭包封装私有变量
- 用类或对象组织状态，避免暴露在全局
5. 嵌套过深（Callback Hell 或过度嵌套）
多层回调或条件嵌套使代码横向发展，阅读困难。
改善方法：
- 使用 async/await 替代回调函数
- 提前返回（guard clauses）减少 else 分支
- 将嵌套逻辑拆解为流水线操作
比如用 if (!condition) return; 提前退出，降低缩进层级。
6. 硬编码与魔法值
直接在代码中写字符串、数字等常量，缺乏语义且不易维护。
处理方式：
- 定义有意义的常量替代魔法值
- 将配置项集中管理
- 使用枚举或对象字面量表示固定集合
例如：const STATUS = { PENDING: 'pending', DONE: 'done' };
基本上就这些常见的坏味道和对应重构手法。关键是持续关注代码的可读性与扩展性，小步调整，逐步提升整体质量。
