返された値を使用していますか?
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
レベル
[x]初心者
コードが実際のロジックを正確に反映している場合、清潔で効率的なままです。
未使用の割り当てはこの接続を破り、意図を理解してコードを維持することを困難にします。
AI世代
AIツールは、意図を誤解しているときに、使用されていない変動的な割り当てを生成することはめったにありません。
AI検出
AIアシストのリファクタリングは、明確な命令を使用して、未使用の変数にフラグを立てて削除を提案できますが、返品値が使用されるべきかどうかは常に理解できない場合があります。
試してみてください!
覚えておいてください:AIアシスタントは多くの間違いを犯します
結論
変異後の未使用の変数は、ノイズと混乱を引き起こします
必要がない場合は返品値を割り当てないでください。
メソッドが意味のあるものを返す必要がある場合は、必ず使用してください。
関係
コードの臭い209-副作用
Maxi Contieri
#webdev
#javascript
#beginners
#プログラミング
コードの匂い176-エッセンスの変化
Maxi Contieri
#webdev
#javascript
#beginners
#プログラミング
コードの匂い154-変数が多すぎます
Maxi contieri・7月31日22年7月
#webdev
#beginners
#プログラミング
#チュートリアル
コード臭い107-変数の再利用
maxi contieri
#oop
#プログラミング
#webdev
#チュートリアル
詳細情報
変異体の邪悪な力
maxi contieri・12月19 '20
#webdev
#プログラミング
#突然変異
#oop
免責事項
コードの匂いは私の意見です。
クレジット
写真によるエヴァン・デミコリの写真
コードを解読するために努力を払わなければならない場合は、書き直す必要があります。
マーティンゴールディング
ソフトウェアエンジニアリング素晴らしい引用
maxi contieri・12月28 '20
#codenewbie
#プログラミング
#Quotes
#ソフトウェア
この記事は、codeSmellシリーズの一部です。
コードの臭い部分を見つける方法
Maxi Contieri
#codenewbie
#チュートリアル
#Codequality
#beginners
以上がコード臭い - 未使用のローカル割り当ての詳細内容です。詳細については、PHP 中国語 Web サイトの他の関連記事を参照してください。