Bagaimana cara meyakinkan rekan tim saya bahwa kita tidak boleh mengabaikan peringatan kompiler?


51

Saya mengerjakan proyek besar (lebih seperti kombinasi puluhan proyek mini yang tidak dapat dipisahkan dengan mudah karena manajemen ketergantungan yang buruk, tapi itu diskusi yang berbeda) di Jawa menggunakan gerhana. Kami telah mematikan sejumlah peringatan dari pengaturan kompiler dan proyek masih memiliki lebih dari 10.000 peringatan.

Saya seorang pendukung besar untuk mencoba mengatasi semua peringatan, memperbaikinya semua jika mungkin, dan bagi mereka yang melihat ke dalam dan dianggap aman, tekan mereka. (Hal yang sama berlaku untuk obsesi keagamaan saya dengan menandai semua metode yang diimplementasikan / diganti sebagai @Override). Argumen terbesar saya adalah bahwa secara umum peringatan membantu Anda menemukan bug potensial selama waktu kompilasi. Mungkin dari 99 dari 100 kali, peringatannya tidak signifikan, tapi saya pikir kepala yang menggaruk itu menghemat satu kali mencegah bug besar, itu semua sepadan. (Alasan saya yang lain adalah OCD saya yang jelas dengan kode bersih).

Namun, banyak rekan tim saya tampaknya tidak peduli. Saya sesekali memperbaiki peringatan ketika saya menemukan mereka (tapi Anda tahu itu sulit ketika Anda menyentuh kode yang ditulis oleh rekan kerja). Sekarang dengan lebih banyak peringatan daripada kelas, keuntungan dari peringatan sangat diminimalkan, karena ketika peringatan adalah hal yang biasa, tidak ada yang akan repot-repot melihat semuanya.

Bagaimana saya bisa meyakinkan rekan tim saya (atau kekuatan yang ada) bahwa peringatan perlu ditangani (atau ditekan ketika diselidiki sepenuhnya)? Atau haruskah saya meyakinkan diri sendiri bahwa saya gila?

Terima kasih

(PS Saya lupa menyebutkan apa yang akhirnya mendorong saya untuk mengirim pertanyaan ini adalah saya sedih melihat bahwa saya memperbaiki peringatan lebih lambat daripada yang dihasilkan)


5
Semua jenis: rawtype, impor yang tidak digunakan, variabel yang tidak digunakan, metode pribadi yang tidak digunakan, penindasan yang tidak perlu, tidak dicentang, pemain yang tidak perlu, kondisi yang tidak perlu (selalu benar atau salah), metode menimpa yang tidak terotomatisasi, referensi untuk kelas / metode yang sudah usang dll.

1
«Saya ingin mulai menerapkan analisis" kode "statis pada peta game, dan memperlakukan peringatan sebagai kesalahan. »Kutipan terbaru dari John Carmack. Jika Anda gila, Anda adalah bagian dari Anda. Sebenarnya, kita bertiga.
deadalnix

1
@ Ray: Ini adalah peringatan dari analisis kode Eclipse, saya tidak yakin apakah Anda bisa mendapatkan semuanya dari vanilla javac.
yatima2975

4
tetapi Anda tahu itu sulit ketika Anda menyentuh kode yang ditulis oleh rekan kerja - jika tempat kerja Anda memiliki masalah teritorial seperti ini, saya menganggap itu sebagai tanda bahaya.
JSB ձոգչ

1
@deadalnix: menjadi pria C ++, saya hanya punya satu cara untuk melakukan sesuatu -Wall -Wextra -Werror(yaitu, mengaktifkan sebagian besar peringatan yang tersedia, memperlakukan semuanya sebagai kesalahan). Eclipse C ++ hampir tidak dapat digunakan lagi: /
Matthieu M.

Jawaban:


37

Anda dapat melakukan dua hal.

  1. Tekankan bahwa peringatan ada karena suatu alasan. Penulis kompiler tidak memasukkan mereka karena mereka kejam. Orang-orang di industri kami umumnya sangat membantu. Banyak peringatan sangat membantu.

  2. Kumpulkan riwayat kegagalan spektakuler yang muncul dari peringatan yang diabaikan. Pencarian web untuk "Perhatikan peringatan kompiler" mengembalikan beberapa anekdot.

Obsesi Anda @Overridebukanlah "obsesi". Itu adalah hal yang baik. Pernah salah mengeja nama metode?


8
Saya ingin menambahkan bahwa Anda tidak bisa membuat orang lain peduli, jadi bersikaplah pragmatis dan pilihlah pertempuran Anda dengan baik.
Daniel Werner

@Aniel: sedih, tapi benar. Jika mereka tidak peduli, bahkan jika mereka tahu (atau setidaknya percaya ) bahwa Anda benar, maka ini bukan tugas dengan pandangan yang cerah.
Joachim Sauer

2
Ada banyak waktu ketika peringatan sebenarnya hanya peringatan. Anda bisa waspada, tetapi saya sering lebih suka menghabiskan waktu menulis kasus uji yang jauh lebih spesifik untuk basis kode Anda.
ghayes

Saya pikir OP mencari untuk membuat rekan kerja berhenti bersikap menolak. Banyak peringatan yang diberikan tidak harus ditanggapi, tidak juga seharusnya semuanya! Tetapi menjadi blase dan membiarkan 10.000 dari mereka masuk ke basis kode bukanlah hal yang baik. Peringatan harus dilihat, dipertimbangkan, dan jika tidak berlaku, peringatan tersebut dapat dimatikan atau anotasi Penekan diterapkan.

3
Saya baru-baru ini membereskan beberapa peringatan dalam proyek open source dan menemukan bug yang jelas yang dilaporkan melalui peringatan: if (error = 0)bukannya if (error == 0). Selain itu, banyak peringatan juga membuatnya lebih mudah untuk menemukan kesalahan kompiler tanpa harus mengarungi rim peringatan.
Hugo

12

Bacaan yang relevan di sini . C ++, tetapi masih relevan. Saya terutama menyukai contoh ini (komentar kode adalah milik saya):

int searchArray(int to_search[], int len, int to_find)
{
    int i;
    for( i = 0; i < len; ++i )
    {       
        if ( to_search[i] == to_find )
        {
            return i;
        }
    }
    // should be returning designated 'not found' value (0, -1, etc)
}

Sering kali, peringatan akan berarti perbedaan antara aplikasi mogok dengan kesalahan yang sebenarnya akan membantu Anda melacak cacat desain dan memperbaikinya (seperti pengindeksan yang aman ) atau aplikasi membuat asumsi dan kemudian benar-benar berperilaku salah atau cara yang salah (yang akan terjadi dengan typecasting tidak aman ). Demikian pula, mereka juga menunjukkan kode cacat atau mati yang dapat dihapus (blok kode tidak dapat dijangkau, variabel yang tidak digunakan), yang dapat dianggap optimasi kode, atau tidak diperhitungkan - untuk kasus yang akan muncul dalam pengujian, seperti contoh di atas untuk C ++. Perhatikan bahwa contoh ini menghasilkan kesalahan waktu kompilasi di Jawa.

Jadi, memperbaiki peringatan memiliki (setidaknya) beberapa keuntungan bagi manajemen:

  1. Lebih sedikit peluang kode berperilaku tidak semestinya dengan data yang dimasukkan pengguna yang, bagi atasan yang peduli tentang citra perusahaan dan bagaimana menangani data kliennya, harus menjadi Masalah Besar (tm). Menabrak untuk mencegah pengguna melakukan sesuatu yang konyol lebih baik daripada tidak menabrak dan membiarkan mereka melakukannya.
  2. Jika mereka ingin merombak personel, orang bisa masuk ke basis kode bebas-peringatan (dan tidak terlalu panik atau banyak mengeluh ..). Mungkin ini akan mengurangi jumlah gerutuan yang terjadi, atau pendapat tentang kelestarian atau kualitas kontribusi Tim X untuk Proyek Y.
  3. Mengoptimalkan kode dengan menghapus peringatan kompiler, meskipun tidak memberikan peningkatan signifikan pada waktu berjalan, akan meningkatkan banyak skor saat melakukan pengujian:
    • Setiap skor cakupan kode kemungkinan akan meningkat.
    • Pengujian batas dan kasus uji yang tidak valid kemungkinan akan lebih sering berhasil (karena typecasting aman yang disebutkan di atas , antara lain).
    • Ketika sebuah test case gagal, Anda tidak perlu memikirkan apakah itu karena salah satu peringatan kompiler di file sumber itu.
    • Mudah diperdebatkan bahwa nol peringatan kompiler lebih baik daripada ribuan. Tidak ada peringatan atau kesalahan yang berbicara tentang kualitas kode, sehingga Anda dapat berargumen bahwa kualitas kode meningkatkan lebih sedikit peringatan yang ada.
  4. Jika Anda tidak memiliki peringatan kompiler, dan satu atau lebih diperkenalkan, jauh lebih mudah untuk mengetahui siapa yang menyebabkan mereka muncul dalam basis kode. Jika Anda memiliki kebijakan "tidak ada peringatan", itu akan membantu mereka mengidentifikasi orang yang tidak melakukan pekerjaan dengan baik, mungkin? Menulis kode bukan hanya tentang membuat sesuatu bekerja, ini tentang membuatnya bekerja dengan baik .

Perhatikan bahwa saya masih seorang junior dev yang hanya tahu sedikit tentang manajemen proyek, jadi jika saya telah mengatakan sesuatu yang salah tolong perbaiki pemikiran saya dan beri saya kesempatan untuk mengedit sebelum Anda menurunkan saya dari keberadaan :)


2
dipikirkan dengan sangat baik melalui respons. Terima kasih. Contoh yang Anda berikan akan menjadi kesalahan daripada peringatan di Jawa. :)

@ Ray: Ah, cukup adil. Sudah setahun sejak saya melakukan dev Java, jadi saya terikat untuk mengabaikan sesuatu. Saya akan mengedit posting saya untuk menyebutkan itu. Terima kasih!
darvids0n

Sudah lama sejak saya melakukan C ++. Apa fungsi yang dikembalikan jika Anda mencapai komentar di akhir? Apakah 0? Perilaku tidak terdefinisi?
MatrixFrog

1
@MatrixFrog: Tidak ditentukan. Dapat berupa int arbitrer, yang akan menyebabkan semua jenis nasties. Kutipan dari jawaban ini : " C ++ 03 §6.6.3 / 2: Mengalir di akhir fungsi sama dengan pengembalian tanpa nilai; ini menghasilkan perilaku yang tidak ditentukan dalam fungsi pengembalian nilai. "
darvids0n

7

Saya pernah mengerjakan proyek dengan karakteristik serupa di masa lalu. Yang satu ini pada mulanya ditulis di Jawa 1.4. Setelah Java 5 dengan obat generik keluar, orang dapat membayangkan jumlah peringatan yang dilemparkan oleh kompiler untuk setiap penggunaan API Koleksi.

Mungkin perlu beberapa waktu untuk menghilangkan semua peringatan, dengan mulai menggunakan obat generik. Itu adalah faktor yang harus diperhitungkan, tetapi ketika Anda harus meyakinkan seseorang (terutama manajer Anda) bahwa itu perlu diperbaiki, Anda akan memerlukan data yang sulit, seperti

waktu yang dihabiskan untuk bug dalam legacy atau kode yang tidak sesuai standar (standar adalah standar pengkodean proyek Anda) yang bisa dihindari.

Anda bisa terus menyajikan tagihan yang menunjukkan bagaimana Anda kehilangan waktu dan uang dengan mengabaikan peringatan "tertentu", dan seseorang akan mendapatkan ide. Bagian kuncinya adalah bahwa tidak semua peringatan layak dilihat, setidaknya tidak segera; dengan kata-kata yang lebih sederhana, Anda perlu menetapkan prioritas peringatan yang perlu ditangani segera. Untuk mulai dengan, pertimbangkan yang berkaitan dengan bug yang Anda lihat di proyek Anda.

Anda juga dapat membuat catatan di pelacak bug, ketika Anda memperbaiki bug, bahwa bug tersebut bisa dihindari dengan tidak mengabaikan peringatan (atau mungkin dengan menjalankan PMD atau FindBugs jika Anda memiliki CI atau membangun sistem). Selama ada cukup banyak bug yang dapat diperbaiki dengan mengindahkan peringatan kompiler, poin Anda tentang melihat peringatan kompiler akan valid. Kalau tidak, itu adalah keputusan bisnis, dan biasanya tidak sepadan dengan waktu yang dihabiskan untuk melawan pertempuran ini.


Ya persis. Saya pikir ledakan awal jumlah pengecualian terjadi selama transisi 1.4-> 1.5, dan sejak itu, tidak ada lagi yang memperhatikan peringatan karena terlalu banyak ...

2

Jika Anda berhasil dan tim Anda memutuskan untuk mematuhi peringatan, Anda harus mengambil pendekatan langkah-bijaksana. Tidak ada yang bisa dan akan 10000 peringatan dalam satu waktu. Jadi, Anda dapat memilih yang paling penting (bug yang memiliki probabilitas tinggi) dan menonaktifkan yang kurang penting. Jika sudah diperbaiki, tambah lagi tingkat peringatan. Selain itu, Anda bisa mulai dengan FindBugs, yang memperingatkan kode yang hampir selalu berupa bug.


2
Atau fokus pada memperbaiki peringatan ketika Anda melihat kode (kelas baru seharusnya tidak memiliki peringatan, kelas apa pun yang Anda modifikasi harus memiliki lebih sedikit peringatan).
Pasang kembali Monica

Pertanyaan serius: Bagaimana Anda tahu mana yang paling mungkin menghasilkan bug yang sebenarnya?
MatrixFrog

1

Saya sepenuhnya setuju dengan Anda, itu adalah praktik terbaik untuk membersihkan peringatan kompilasi sebanyak mungkin. Anda menyebutkan bahwa tim Anda menggunakan Eclipse sebagai alat pengembangan. Eclipse adalah alat yang sangat baik untuk membantu Anda membersihkan kode dan membuat konsistensi gaya kode.

  1. tentukan 'Simpan tindakan' untuk setiap proyek Java Anda, seperti memformat kode, variabel lokal yang tidak digunakan, mengatur impor (saya pikir tidak ada yang keberatan dengan pembersihan semacam itu).
  2. mendefinisikan opsi kompilasi spesifik proyek untuk setiap proyek. Misalnya, kompilasi level (jika kode Anda harus kompatibel dengan 1.4 untuk membersihkan peringatan yang terkait dengan generik), tugas tidak memiliki efek (misalnya 'x = x') dan sebagainya. Anda dan rekan satu tim Anda dapat membuat kesepakatan untuk item-item itu, dan Eclipse akan melaporkannya sebagai 'Kesalahan' dalam fase pengembangan Anda setelah Anda memiliki kesepakatan untuk kesalahan kompilasi.

Eclipse akan membuat beberapa file properti untuk preferensi tersebut di bawah folder .settings, Anda dapat menyalinnya ke proyek Java lainnya dan memeriksanya di SCM sebagai bagian dari kode sumber.

Anda dapat memeriksa kode Eclipse untuk melihat bagaimana pengembang Eclipse melakukannya.


1

Anda memiliki dua cara untuk menghilangkan semua peringatan (dan saya setuju itu dapat menyembunyikan bug halus):

  1. Yakinkan seluruh tim bahwa ini adalah hal terbaik untuk dilakukan, dan minta mereka memperbaikinya.
  2. Yakinkan bos Anda bahwa ini adalah hal terbaik untuk dilakukan, dan buatlah itu wajib. Maka mereka perlu memperbaikinya.

Saya percaya bahwa 1 tidak layak dalam kasus Anda lagi. Mungkin ini juga akan memakan waktu lama karena banyaknya peringatan yang akan ditampilkan dalam output produktivitas sehingga manajemen perlu mengetahui pula.

Jadi, saran saya adalah: Angkat dengan bos, yakinkan dia ini bom yang berdetak, dan minta dibuatkan kebijakan resmi.


1

Saya hanya akan mengatakan kepada mereka bahwa sebagian besar dari ini adalah peringatan yang tidak penting dan sangat penting bagi kita untuk menghapusnya dari daftar. Sehingga kita tidak akan melewatkan peringatan signifikan nyata di tengah orang banyak, seperti yang terjadi!


Persis. Sangat penting untuk menyingkirkan mereka karena mereka tidak signifikan.
MatrixFrog

0

Anda akan membutuhkan banyak kesabaran dalam mencoba meyakinkan mereka, menghapus peringatan ketika melakukan pemrograman pasangan akan membantu orang lain mengambil kebiasaan itu. Sarankan mereka untuk membaca Java Efektif 'Item 24: Menghilangkan peringatan yang tidak dicentang' (untuk koleksi umum). Dan mungkin mengabaikan banyak contoh ketika orang tidak mengikuti saran Anda;)


0

Pendekatan yang efektif di sini adalah dengan menyiapkan server Integrasi Berkelanjutan yang secara otomatis membangun proyek dan menjalankan tes setiap kali ada orang yang memeriksa kode. Jika Anda belum menggunakan ini, Anda harus melakukannya, dan tidak terlalu sulit untuk meyakinkan orang lain tentang manfaat melakukannya.

  1. Meyakinkan manajemen / tim tentang manfaat melakukan hal ini (mencegah pembangunan yang buruk dari penyebaran, menemukan bug lebih awal, terutama yang secara tidak sengaja memengaruhi bagian lain dari perangkat lunak, mempertahankan pengujian regresi, pengujian awal dan sering, dll.)

  2. Instal server CI dengan semua tes lulus (jika Anda tidak memiliki tes, tulis tes cepat yang lulus, sehingga semua orang melihat bahwa itu hijau).

  3. Siapkan email atau pemberitahuan lain tentang status setiap bangunan. Penting di sini untuk memasukkan siapa yang memeriksa dalam kode, apa perubahannya (atau tautan ke komit), dan status + output dari build. Ini adalah langkah penting, karena menyebabkan visibilitas di seluruh tim baik untuk kesuksesan maupun untuk kegagalan.

  4. Perbarui server CI untuk membuat tes gagal jika ada peringatan. Ini akan dilihat oleh pimpinan dan pimpinan tim sebagai peningkatan disiplin tim secara sistematis, dan tidak ada yang mau bertanggung jawab atas semua kegagalan email yang keluar.

  5. (opsional) dapatkan baik dan culun dengan ini, dengan menambahkan beberapa dashboard terlihat, lampu lava, lampu berkedip, dll untuk menunjukkan status membangun dan siapa yang merusak / memperbaikinya.

Dengan menggunakan situs kami, Anda mengakui telah membaca dan memahami Kebijakan Cookie dan Kebijakan Privasi kami.
Licensed under cc by-sa 3.0 with attribution required.