Haruskah kurung kurawal muncul di jalurnya sendiri? [Tutup]

Haruskah kurung kurawal berada di jalurnya sendiri atau tidak? Apa yang Anda pikirkan?

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

atau seharusnya begitu

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

atau bahkan

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

Harap bersikap konstruktif! Jelaskan mengapa, berbagi pengalaman, dukung dengan fakta dan referensi.

Ketika saya masih mahasiswa saya biasa memasang kurung kurawal pada baris yang sama, sehingga ada lebih sedikit garis, dan kode dicetak pada lebih sedikit halaman. Melihat satu karakter braket yang dicetak sebagai satu-satunya hal dalam satu baris mengganggu. (lingkungan, pemborosan kertas)

Tetapi ketika mengkode aplikasi besar, memungkinkan beberapa baris dengan hanya kawat gigi di dalamnya terjangkau, mengingat perasaan 'pengelompokan' yang diberikannya.

Gaya apa pun yang Anda pilih, konsistenlah sehingga tidak menjadi overhead bagi otak Anda sendiri untuk memproses banyak gaya dalam potongan kode terkait . Dalam skenario yang berbeda (seperti di atas) saya akan mengatakan tidak apa-apa untuk menggunakan gaya yang berbeda, lebih mudah untuk 'mengalihkan konteks' di tingkat tinggi.

Anda seharusnya tidak pernah melakukan metode ke-3.

Skimping pada kawat gigi mungkin menyelamatkan Anda beberapa kali penekanan tombol, tetapi pembuat kode berikutnya yang datang, menambahkan sesuatu ke klausa Anda yang lain tanpa memperhatikan blok yang hilang kawat gigi akan ada untuk banyak rasa sakit.

Tulis kode Anda untuk orang lain.

Untuk waktu yang lama saya berpendapat bahwa mereka bernilai sama, atau sangat dekat dengan sama sehingga kemungkinan mendapatkan dengan membuat pilihan yang tepat jauh, jauh, di bawah biaya berdebat tentang hal itu.

Menjadi konsisten itu penting . Jadi saya katakan, mari kita melempar koin dan mulai menulis kode.

Saya telah melihat programmer menolak perubahan seperti ini sebelumnya. Lupakan saja! Saya telah berganti karier berkali-kali. Saya bahkan menggunakan gaya yang berbeda di C # saya daripada di PowerShell saya.

Beberapa tahun yang lalu saya mengerjakan sebuah tim (~ 20 pengembang) yang memutuskan untuk meminta input, dan kemudian membuat keputusan, dan kemudian menegakkannya di semua basis kode. Kami punya 1 minggu untuk memutuskan.

Banyak erangan & memutar mata. Banyak "Saya suka cara saya, karena itu lebih baik" tetapi tidak ada substansi.

Ketika kami mempelajari poin-poin penting dari pertanyaan, seseorang bertanya bagaimana menangani masalah ini dengan gaya brace-on-the-same-line:

void MyFunction(
    int parameterOne,
    int parameterTwo) {
    int localOne,
    int localTwo
}

Perhatikan bahwa tidak segera jelas di mana daftar parameter berakhir, dan tubuh dimulai. Dibandingkan dengan:

void MyFunction(
    int parameterOne,
    int parameterTwo) 
{
    int localOne,
    int localTwo
}

Kami melakukan beberapa bacaan tentang bagaimana orang-orang di seluruh dunia telah mengatasi masalah ini, dan menemukan pola menambahkan garis kosong setelah penjepit terbuka:

void MyFunction(
    int parameterOne,
    int parameterTwo) {

    int localOne,
    int localTwo
}

Jika Anda akan membuat jeda visual, Anda sebaiknya melakukannya dengan penjepit. Kemudian jeda visual Anda menjadi konsisten juga.

Sunting : Dua alternatif untuk solusi 'garis kosong ekstra' saat menggunakan K&R:

1 / Indent argumen fungsi berbeda dari fungsi tubuh

2 / Letakkan argumen pertama pada baris yang sama dengan nama fungsi dan sejajarkan argumen lebih lanjut pada baris baru dengan argumen pertama

Contoh:

1 /

void MyFunction(
        int parameterOne,
        int parameterTwo) {
    int localOne,
    int localTwo
}

2 /

void MyFunction(int parameterOne,
                int parameterTwo) {
    int localOne,
    int localTwo
}

/Sunting

Saya masih berpendapat bahwa konsistensi lebih penting daripada pertimbangan lain, tetapi jika kita tidak memiliki preseden yang mapan , maka brace-on-next-line adalah cara untuk pergi.

Aturan utamanya adalah:

1. Ikuti standar pengkodean proyek yang ada.
2. Jika tidak ada standar pengkodean dan Anda mengedit basis kode yang ada yang dimiliki oleh orang lain - konsisten dengan gaya kode yang ada, tidak peduli seberapa banyak Anda suka / tidak suka.
3. Jika Anda bekerja pada proyek lapangan hijau - diskusikan dengan anggota tim lain, dan datang ke konsensus tentang standar pengkodean formal atau informal.
4. Jika Anda bekerja pada proyek lapangan hijau sebagai satu-satunya pengembang - buat keputusan sendiri, dan kemudian konsisten.

Bahkan jika Anda tidak memiliki kendala eksternal pada Anda, yang terbaik adalah (IMO) untuk mencari pedoman pengkodean atau gaya gaya (banyak digunakan) yang ada, dan coba dan ikuti itu. Jika Anda menggulung gaya Anda sendiri, ada kemungkinan besar Anda akan menyesalinya dalam beberapa tahun.

Akhirnya, gaya yang diimplementasikan / diimplementasikan menggunakan style checker dan pemformat kode yang ada lebih baik daripada yang perlu "ditegakkan" secara manual.

Manfaat dari metode pertama adalah lebih kompak secara vertikal, sehingga Anda dapat memasukkan lebih banyak kode di layar Anda, dan itulah sebabnya saya lebih suka. Satu-satunya argumen yang saya dengar mendukung metode kedua adalah membuatnya lebih mudah untuk memasangkan kurung buka dan tutup, tetapi sebagian besar IDE memiliki pintasan keyboard untuk itu, dan itu sebenarnya pernyataan salah - alih-alih memasangkan braket pembuka ke penutup braket Anda dapat memasangkan braket penutup dengan ekspresi "awal blok" (jika, selain itu, untuk, sementara) pada tingkat indentasi yang sama, sehingga mudah untuk menentukan di mana awal blok.

Saya tidak melihat alasan untuk menyia-nyiakan seluruh baris hanya untuk braket ketika sebelumnya untuk / sementara / jika konstruksi sudah secara visual menunjukkan awal blok.

Yang mengatakan, saya percaya bahwa braket penutup harus berada di jalurnya sendiri karena kita perlu sesuatu untuk menunjukkan ujung blok dan struktur lekukannya dengan cara yang terlihat.

aku lebih memilih

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

lebih

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

karena garis you.postAnswer();itu jauh lebih mudah dibaca dan ditemukan pada pandangan pertama. Dengan cara kedua, itu akan tercampur dengan garis di atasnya ( you.hasAnswer()) membuat mata saya harus lebih fokus untuk membacanya.

Saya lebih suka metode pertama. Kawat gigi sama sekali tidak layak terpisah.

Masalahnya, kawat gigi itu tidak penting. Itu hanya sampah sintaksis , yang sama sekali tidak perlu untuk memahami apa tujuan kode, tujuan itu dan cara penerapannya. Mereka hanya merupakan penghormatan kepada bahasa seperti C gaya lama di mana pengelompokan visual dari operator tidak mungkin karena ruang layar yang rendah tersedia.

Ada bahasa (Python, Haskell, Ruby) yang OK tanpa kawat sama sekali. Ini hanya mengonfirmasi bahwa kawat gigi adalah sampah, dan seharusnya tidak layak menerima saluran untuk mereka bila memungkinkan:

if (you.hasAnswer()){
    you.postAnswer();
}else{
    you.doSomething();
}

Gunakan Python dan hindari argumen sepenuhnya.

Posisi kurung kurawal seharusnya

meta data

dapat dikonfigurasi dalam IDE oleh programmer. Dengan begitu, kawat gigi sial di semua kode, terlepas dari penulis, terlihat sama.

Tergantung.

Jika saya mengkode dalam Javascript atau jQuery, saya menggunakan formulir pertama:

jQuery(function($) { 
    if ($ instanceOf jQuery) { 
        alert("$ is the jQuery object!"); 
    } 
}); 

Tetapi jika saya mengkode dalam C #, saya menggunakan bentuk kedua, karena itu adalah cara kanonik untuk melakukannya dalam C #.

public int CalculateAge(DateTime birthDate, DateTime now) 
{ 
    int age = now.Year - birthDate.Year; 
    if (now.Month < birthDate.Month 
        || (now.Month == birthDate.Month && now.Day < birthDate.Day)) 
        age--; 
    return age; 
} 

Perhatikan bahwa contoh Anda dapat ditulis

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

dalam C #.

Saya lebih suka yang pertama karena lebih sulit bagi saya untuk melihat kesalahan dalam contoh ini.

if (value > maximum);
{
    dosomething();
}

daripada dalam contoh ini

if (value > maximum); {
    dosomething();
}

The ; {hanya tampak lebih salah bagi saya daripada garis berakhir dengan ;jadi aku lebih mungkin untuk menyadarinya.

Saya lebih suka varian sedikit 1)

if (you.hasAnswer()) {
    you.postAnswer();
} // note the break here
else {
    you.doSomething();
}

Mengapa?

• Saya pikir selalu menempatkan kawat gigi pada jalurnya sendiri mengurangi keterbacaan. Saya hanya bisa memasukkan sejumlah kode sumber pada layar saya. Gaya braket 2) membuat algoritma heave dengan banyak loop bersarang dan persyaratan lama sekali.

• Namun, saya ingin elsememulai pada jalur baru karena ifdan elsemilik bersama, secara visual. Jika ada braket di depan else, itu jauh lebih sulit untuk menemukan milik apa.

• 3) mendiskualifikasi dirinya sendiri. Kita semua tahu hal buruk apa yang bisa terjadi jika Anda meninggalkan tanda kurung dan melupakannya.