Boolean mengembalikan set.add () jika bersyarat?

Add operator dari kelas set mengembalikan boolean yang benar jika elemen (yang akan ditambahkan) belum ada di sana, dan false jika tidak. Sedang menulis

if (set.add(entry)) {
    //do some more stuff
}

dianggap gaya yang baik dalam hal menulis kode bersih? Saya ingin tahu karena Anda melakukan dua hal sekaligus. 1) menambahkan elemen dan 2) memeriksa apakah elemen tersebut ada.

Ya itu.

Titik biasa dari operasi mengembalikan nilai Boolean adalah bahwa Anda dapat menggunakannya untuk membuat keputusan, yaitu dalam ifkonstruk. Satu-satunya cara Anda dapat menyadari potensi ini adalah dengan menyimpan nilai kembali dalam variabel dan kemudian segera menggunakannya kembali dalam if, yang hanya konyol dan tentu saja tidak dengan cara apa pun lebih baik daripada menulis if(operation()) { ... }. Jadi silakan saja dan lakukan itu, kami tidak akan menghakimi Anda (untuk itu).

Saya akan mengatakan itu tidak sebersih mungkin, karena memaksa pengelola untuk sudah tahu atau mencari tahu apa nilai kembaliannya. Apakah ini berarti nilai sudah ada, tidak sudah ada, berhasil dimasukkan? Jika Anda tidak sering menggunakannya, Anda tidak akan tahu, dan bahkan jika Anda menggunakannya, itu adalah beban mental yang jauh lebih besar.

Saya lebih suka yang berikut ini:

boolean added = set.add(entry);

if (added) {
    //do some more stuff
}

Ya, sedikit lebih banyak verbose, tetapi kompiler harus menghasilkan bytecode yang persis sama, dan bahkan orang-orang yang tidak menggunakan set Java bertahun-tahun dapat mengikuti logika tanpa melihat apa-apa.

Jika true berarti sukses, maka itu kode yang baik dan jelas.

Ada konvensi luas bahwa suatu fungsi atau metode mengembalikan true (atau sesuatu yang mengevaluasi true) pada kesuksesan. Selama kode Anda mengikuti itu, saya pikir menempatkan metode dalam kondisi baik-baik saja.

Kode seperti ini tidak perlu berantakan dalam pandangan saya:

boolean frobulate_succeeded = thing.frobulate();

if (frobulate_succeeded) {
    ...
}

Rasanya seperti Anda mengulangi sendiri.

Namun, pertanyaannya tidak jelas tentang arti nilai kembali. Anda mengatakan "boolean yang menunjukkan apakah elemen yang ditambahkan sudah ada", yang mungkin menyiratkan bahwa true berarti elemen itu ada (dan menambahkan tidak terjadi). Jika demikian, saya idealnya mengubah perilaku pengembalian metode menjadi lebih konvensional. Jika itu tidak memungkinkan, saya akan menambahkan variabel perantara tambahan yang memungkinkan Anda untuk dengan jelas memberi label hasil pengembalian dalam kode Anda (seperti yang disarankan oleh orang lain).

Saya akan mengatakan itu sangat seperti C. Sebagian besar waktu saya lebih suka memiliki variabel yang dinamai secara deskriptif untuk hasil mutasi, dan tidak ada mutasi yang terjadi dalam suatu ifkondisi.

Kompiler akan menghilangkan variabel ini jika langsung digunakan kembali. Manusia akan lebih mudah membaca sumbernya; bagi saya, ini lebih penting.

Jika seseorang harus memperpanjang kondisi dengan menambahkan and/ orklausa ke kondisi if, mereka mungkin akhirnya tidak menelepon .add()dalam kasus-kasus tertentu karena evaluasi hubungan pendek. Kecuali jika korsleting diantisipasi secara khusus, ini mungkin berakhir sebagai bug.

Kode Anda tampaknya memecah Command Query Separation . Ini dibahas dalam buku Kode Bersih dan video Function Structure . Jadi dari perspektif Clean Code saya pikir itu tidak dianggap gaya yang baik.

“Kode bersih itu sederhana dan langsung. Kode bersih berbunyi seperti prosa yang ditulis dengan baik. Kode bersih tidak pernah mengaburkan niat desainer melainkan penuh dengan abstraksi renyah dan garis kontrol langsung.

- Grady Booch penulis Object Oriented Analysis and Design with Applications ”

Bagi saya maksud kode Anda tidak jelas. Apakah jika keduanya dieksekusi ketika entri ditambahkan berhasil, atau juga ketika sudah ada? Apa yang add()kembali? barang itu? Kode kesalahan?