Bau Kod - Tugas tempatan yang tidak digunakan

adakah anda menggunakan nilai yang dikembalikan?

tl; DR: Elakkan memberikan nilai yang tidak pernah anda gunakan.

Masalah

• kod mati
• niat tidak jelas
• overhead penyelenggaraan
• kerumitan tersembunyi
• Debugging Leftovers
• scoping buruk
• kod berkomentar
• mutabiliti

Penyelesaian

1. Keluarkan tugasan yang tidak digunakan.
2. cuba mengelakkan kesan sampingan.

Konteks

Apabila anda memberikan nilai kepada pembolehubah tempatan tetapi tidak pernah menggunakannya, anda membuat kekacauan yang tidak perlu dalam kod anda.

Ini boleh mengelirukan orang lain dan menjadikan kod lebih sukar untuk dikekalkan.

Kadang -kadang, apabila debugging anda boleh menetapkan pembolehubah temporal untuk pemeriksaan yang lebih baik.

Ini juga berlaku apabila anda menetapkan pelaksanaan ke objek

objek tetapi lebih sukar untuk diikuti.

Ia juga merupakan tanda objek bermutasi kerana jika anda mengeluarkan tugasan, hanya kesan sampingan yang kekal.

Objek bermutasi boleh menyebabkan kesan sampingan yang tidak dijangka, menjadikannya lebih sukar untuk mengesan perubahan.

Kod contoh

Salah

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

Betul

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

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

Pengesanan

[x] Automatic

anda boleh mengesan bau ini menggunakan alat analisis statik atau ulasan kod yang memeriksa pembolehubah yang tidak digunakan selepas tugasan.

Kebanyakan linter bendera ini sebagai isu.

Tag

bloaters

Tahap

[x] pemula

Mengapa Bosisnya Penting

Ia tetap bersih dan cekap apabila kod anda dengan tepat mencerminkan logik dunia nyata.

Tugasan yang tidak digunakan memecahkan sambungan ini, menjadikannya lebih sukar untuk memahami niat dan mengekalkan kod tersebut.

Generasi AI

Alat AI jarang menjana tugasan berubah -ubah yang tidak digunakan apabila mereka salah faham niat.

Pengesanan AI

refactoring dibantu AI dengan arahan yang jelas boleh membenderakan pembolehubah yang tidak digunakan dan mencadangkan penyingkiran, tetapi mungkin tidak selalu memahami jika nilai pulangan harus digunakan.

Cuba mereka!

ingat: pembantu AI membuat banyak kesilapan

Kesimpulan

Pembolehubah yang tidak digunakan selepas mutasi membuat bunyi dan kekeliruan.

Jangan berikan nilai pulangan jika anda tidak memerlukannya.

Jika kaedah harus mengembalikan sesuatu yang bermakna, pastikan anda menggunakannya.

Hubungan

Kod Smell 209 - Kesan Sampingan

maxi contieri ・ 28 Apr '23

#webdev #javascript #Beginners #programming

Kod Bau 176 - Perubahan dalam Essence

maxi contieri ・ 2 Nov '22

#webdev #javascript #Beginners #programming

Kod Bau 154 - Terlalu banyak pembolehubah

maxi contieri ・ 31 Jul '22

#webdev #Beginners #programming #tutorial

Kod Smell 107 - Pembolehubah semula digunakan

maxi contieri ・ 1 Dis '21

#oop #programming #webdev #tutorial

Maklumat lanjut

kuasa jahat mutan

maxi contieri ・ 19 Dis '20

#webdev #programming #mutation #oop

Penafian

Kod bau adalah pendapat saya.

Kredit

foto oleh Evan Demicoli pada Unsplash

---

Jika anda perlu menghabiskan usaha untuk menguraikan kod, anda harus menulis semula.

Martin Golding

Kejuruteraan Perisian Petikan Besar

maxi contieri ・ 28 Dis '20

#codenewbie #programming #QUOTES #Software

---

Artikel ini adalah sebahagian daripada siri Codesmell.

bagaimana mencari bahagian bau kod anda

maxi contieri ・ 21 Mei '21

#codenewbie #tutorial #codequality #Beginners

Atas ialah kandungan terperinci Bau Kod - Tugas tempatan yang tidak digunakan. Untuk maklumat lanjut, sila ikut artikel berkaitan lain di laman web China PHP!