您是否使用返回的值?
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
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輔助重構,但可能並不總是理解是否應該使用返回值。
>試試看!
如果您不需要,請勿分配返回值。
如果該方法應返回有意義的東西,請確保您使用它。
>關係
埃文·德米科利(Evan Demicoli)的照片
>
Martin Golding
本文是CodeSmell系列的一部分。
以上是代碼氣味 - 未使用的本地任務的詳細內容。更多資訊請關注PHP中文網其他相關文章!