Penafian: Saya pendatang baru (ini adalah hari kerja ketiga saya), dan sebagian besar rekan satu tim saya lebih berpengalaman daripada saya.
Ketika saya melihat kode kami, saya melihat beberapa bau kode dan praktik rekayasa yang buruk, seperti berikut:
- Agak penamaan pedoman yang tidak konsisten
- Properti tidak ditandai sebagai hanya dapat dibaca bila memungkinkan
- Kelas besar - Saya memperhatikan kelas utilitas yang terdiri dari ratusan metode ekstensi (untuk banyak jenis). Itu lebih dari 2500 baris!
- Metode besar - Saya mencoba memperbaiki metode yang panjangnya 150 baris.
Dua yang terakhir tampaknya menjadi masalah nyata. Saya ingin meyakinkan rekan tim saya untuk menggunakan kelas dan metode yang lebih kecil. Tetapi haruskah saya melakukan itu? Jika ya, lalu bagaimana?
Tim saya mendapat mentor dari tim utama (kami adalah tim satelit). Haruskah saya pergi ke dia dulu?
UPDATE : Karena beberapa tanggapan bertanya tentang proyek, harap diketahui bahwa ini adalah proyek yang berfungsi. Dan IMHO, kelas besar / metode ukuran itu selalu buruk.
Ngomong-ngomong, aku tidak pernah ingin mengecewakan timku. Itu sebabnya saya bertanya - Haruskah saya melakukan itu, dan jika ya, lalu bagaimana saya melakukannya dengan lembut?
UPDATE : Saya memutuskan untuk melakukan sesuatu berdasarkan jawaban yang diterima: karena saya pendatang baru, jadi saya melihat semuanya dalam "mata segar" Saya akan mencatat semua kode bau yang saya temukan (posisi, mengapa buruk, bagaimana kita bisa melakukannya) lebih baik, ...), tetapi pada saat ini, saya hanya berusaha keras untuk mengumpulkan penghormatan dari tim saya: menulis "kode yang lebih baik", tahu orang-orang, tahu mengapa kami melakukan itu ... Ketika waktunya tepat, saya akan mencoba untuk bertanya kepada tim saya tentang beberapa kebijakan kode baru (pedoman penamaan, kelas yang lebih kecil, metode yang lebih kecil, ...), dan jika mungkin, perbaiki beberapa kode lama. Seharusnya bekerja, IMHO.
Terima kasih.