代码气味 - 未使用的本地任务-js教程-PHP中文网

代码气味 - 未使用的本地任务

Mary-Kate Olsen

发布： 2025-01-30 02:31:08

原创

677 人浏览过

您是否使用返回的值？

tl; dr：避免分配您从未使用过的值。

>

问题

死亡代码

不清楚的意图
维护开销
隐藏的复杂性
调试剩菜
不良范围
注释的代码
Mustability
解决方案

>删除未使用的作业。

语境

>本地变量

但永远不要使用它时，您会在代码中创建不必要的混乱。

这可能会使他人感到困惑，并使代码更难维护。> 有时，在调试时，您可以分配时间变量以进行更好的检查。

> 当您将执行分配给

对象属性

时，这也会发生这种情况。

>这也是突变对象的标志，因为如果您删除了分配，则仅保留副作用。

突变物体会引起意外的副作用，从而使跟踪变化更加困难。

示例代码

错误的

正确的

检测

[x]自动

function updateUserName(user, newname) {
  user.name = newname;
  return user;
}

function performMaintenance(existingUser) {
  let updatedUser = updateUserName(existingUser, "Bobby Peru");
  // Other tasks
}
// The variable updatedUser is never used

登录后复制

>您可以使用静态分析工具或代码评论检测到这种气味，该工具检查分配后未使用的变量。

function updateUserName(user, newname) {
  user.name = newname;
  // Just side effects without explicit return
}

function performMaintenance(existingUser) {
  updateUserName(existingUser, "Bobby Peru");
  // Other tasks
}

登录后复制

大多数林格将其标记为一个问题。

标签

bloaters

等级

[x]初学者