CodeSOD: 为论点增添内容 笔记

CodeSOD: 为论点增添内容

大卫遇到了一个令人费解的拉取请求,其中包含大量关于一个看似简单的更改的评论:向一个 JavaScript 函数添加一个参数。原始函数已经有七个参数,这可能暗示了使用对象进行重构的机会。最初的修订引入了一个与新参数 notArg8 相关的条件语句。如果 notArg8 为 true,则代码将 arg8: !notArg8 传递给 context。作者无法理解这种复杂的方法,因为可以直接添加参数。大卫评论了糟糕的代码,但原始开发人员停止回应。一位初级开发人员批准了令人困惑的代码更改,但从未合并。另一位开发人员随后简化了解决方案,仅添加了 arg8 并将其传递给 context。简化的代码工作正常,通过了测试,让所有人都满意。原始开发人员的意图和复杂条件背后的逻辑仍然是个谜。结尾暗示了原始开发人员的设想仍未实现。