ホームページ >ウェブフロントエンド >jsチュートリアル >コード臭い - 未使用のローカル割り当て
返された値を使用していますか?
TL; DR:使用しない値の割り当てを避けてください問題
これにより、他の人を混乱させ、コードを維持するのが難しくなります。 時々、デバッグするときは、より良い検査のために時間変数を割り当てることができます。
これは、実行をオブジェクトプロパティに割り当てるときにも起こります
しかし、従うのは難しいです。割り当てを削除すると、副作用のみが残っているため、変異オブジェクトの兆候でもあります。
変異オブジェクトは、予期しない副作用を引き起こす可能性があり、変更を追跡するのが難しくなります。 サンプルコード
間違っている
右
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[x]自動
function updateUserName(user, newname) { user.name = newname; // Just side effects without explicit return } function performMaintenance(existingUser) { updateUserName(existingUser, "Bobby Peru"); // Other tasks }ほとんどのリンジターは、これを問題としてフラグを立てます。
bloaters
レベル
AI世代
AI検出
AIアシストのリファクタリングは、明確な命令を使用して、未使用の変数にフラグを立てて削除を提案できますが、返品値が使用されるべきかどうかは常に理解できない場合があります。
必要がない場合は返品値を割り当てないでください。
メソッドが意味のあるものを返す必要がある場合は、必ず使用してください。
関係
ソフトウェアエンジニアリング素晴らしい引用
この記事は、codeSmellシリーズの一部です。
以上がコード臭い - 未使用のローカル割り当ての詳細内容です。詳細については、PHP 中国語 Web サイトの他の関連記事を参照してください。