Tek Hat İfadeleri ve İyi Uygulamalar

Son zamanlarda, birçoğunuzun kaşlarını çatabileceğini bildiğim, ancak sonunda, tek bir (bazen) tekrarlayan yöntemin yapısından ziyade küresel kod yapısını izlememe yardımcı olan bir alışkanlık kazandım: bir sayı gruplama ifadeleri tek bir satırda, şöyle:

textBox1.Text = "Something!"; textBox2.Text = "Another thing!"; textBox3.Text = "Yet another thing!";

aksine

textBox1.Text = "Something!";
textBox2.Text = "Another thing!";
textBox3.Text = "Yet another thing!";

Genel kod "güzelliği" korumak ve program yapısını kolayca izlememe yardımcı olmak için tekrarlayan görevler için bunu yapmak için kullanıyorum, ama iyi bir uygulama olmayabilir. Aslında çok kullanıyorum, bu yüzden bu konudaki düşüncelerin neler olduğunu bilmek istiyorum. Ayrıca, kodumu korumak zorunda kalacak herhangi birinin bu yaklaşımla ilgili sorunları olduğunu düşünüyor musunuz?

Gerçekten okunabilirliğin hem sizin için hem de kesinlikle kodu okuyan herkes için büyük zarar göreceğini düşünüyorum. İlk kez yazdığınızda her şey anlamlıdır çünkü aktif olarak zihninizde. Hangi değişkenlerin ve fonksiyonların nerede olduğunu görmek için kodu tararken farklıdır ... kendi kodunuzu tarama yeteneğinizi yok ediyorsunuz. Yani hayır-hayır büyük, ve bad ötesinde başkasının eğer hiç kodunuzu okumak zorundadır.

Ayrıca, kodu nasıl okuduğunuzu da düşünün. Her zaman yukarıdan aşağıya, aşağı doğru kayar. Yönteminiz bununla örtüşmez ve hatta kod okumada mümkün olan en çirkin sorunlardan birini sunar; yatay kaydırma . Bunun okuma kodunu ne kadar zorlayabileceğini asla küçümsemeyin. Asla yatay kaydırma yapmazsınız, asla yatay kaydırma yapmazsınız, neredeyse her bağlamda son derece doğal değildir.

Ayrıca, sorununuz tekrarlayan kod girişi ise ... Ctrl-C'yi unutmayın. Örnek kodunuzdan, bunları manuel olarak yazmak daha verimli olabilir, ancak bir satır satırını birkaç kez kopyalamanız gerekiyorsa, satır bir artı yeni bir satır kopyalamak kadar etkili gibi görünüyor, yapıştırın x kez ve değişiklikleri yapmak, bir yazım hatası yapmak daha az olasıdır.

Oh, ve yazım hataları! Kodunuzun okunabilirliğine bu şekilde zarar vermek, 50 değişken bildiriminden hangisini yanlış ayarladığınızı bulmanın bir kabus olmasını sağlayabilir. Çoğu derleyici satır VE sütun numaralarında hata verir, ancak bir satırda hata bulmak sütun bulmaktan çok daha kolaydır.

Satır başına bir ifade, yan yana farklılıkta nelerin değiştiğini görmeyi de kolaylaştırır.

Örnek bunu göstermese de, bir satırda birden çok ifadeyi gruplandırmayla ilgili başka bir sorun var. Tek bir satırdaki beş ifadeden biri istisna atarsa ​​ne olur?

Yığın izlemeniz "N satırındaki EBlah" diyecektir ... ve şimdi bu beş ifadeden hangisinin istisnayı attığını bilmiyorsunuz.

(Aynı şey, herhangi bir türden aşırı uzun bir açıklama ile olur.)

Satır başına bir deyim, yaygın olarak kullanılan bir kodlama stilidir. Sonuç olarak, gelecekte kodunuza bakan çoğu geliştirici muhtemelen satır başına birden fazla ifade gördüğünde sizi uyandıracaktır. Bir şeyi bir şekilde görmeye alışkın olduğunuzda, başka bir şekilde görmek korkutucu olabilir.

Bu nedenle nadir durumlar dışında buna karşı tavsiyede bulunuyorum.

Bunu en son 25 yıl önce düşük saat hızlarında çalışan küçük mikrolar üzerinde yorumlanmış dilleri kullanarak yaptım, burada her alan veya satırbaşı geri dönüşü performans artışı sağladı.

Şimdi bunun düşüncesinde wince (iyi bir nedenden dolayı yapıldıysa).

Ne yazık ki bu tür kodların okunması zordur ve bu nedenle sürdürülmesi zordur.

Sözdizimsel olarak, bununla ilgili yanlış bir şey yoktur. Bu gerçekten ekibinizin kodlama stiline bağlıdır.

Gördüğüm kodun çoğu (standart c ++ başlıkları içindeki kod dahil) bu şekilde yapıldığından, ilk yönteminizle giderdim.

textBox1.Text = "Something!";
textBox2.Text = "Another thing!";
textBox3.Text = "Yet another thing!";

Bu gerçekten alışılmadık bir kodlama tarzı.

Bunun yerine kodun mantıksal bölümlerini sınırlamak için boş satırlar kullanmanızı öneririm.

Sağa çok ileri gitmek, birden fazla satır kadar sorun yaratabilir.

Düzinelerce alanla bazı sql ifadeleriyle uğraşmak zorunda kaldım. Normalde, her satıra bir tane koyardım, ancak birkaç kez 3 veya 4'ü arka arkaya birleştirdim. Bu, geliştirme sırasında birkaç kez yukarı ve aşağı kaydırmanız gerektiğinde iyi bir fikir gibi görünüyor.

Bu koda dönen pişmanım. Ek satırlara sahip olmak o kadar çok sorun yaratmaz gibi görünüyor, bu yüzden genellikle temizliyorum.

Ayrıca, kodumu korumak zorunda kalacak herhangi birinin bu yaklaşımla ilgili sorunları olduğunu düşünüyor musunuz?

Bir dakika boyunca ellerini başının üstüne koyduktan sonra, tüm okunamayan kodları her satırda bir ifadeye otomatik olarak ayırmak için en sevdiği IDE Regex işlevlerini kullanacak.

Gösterdiğiniz örneğe hızlı bir bakış, ikinci yaklaşımın ne kadar okunabilir olduğunu anlamak için yeterlidir.

Gözlerinizin sonsuza kadar yatay olarak hareket etmesine gerek kalmadan dikey sayfa akışını takip etmek çok daha kolaydır.

Örneğinize bakın: kodun tamamen Textfarklı textBoxnesnelerin özelliği ile ilgili olduğunu ve değer olarak dize içerdiğini hemen bilirsiniz . Oldukça basit.

Şahsen böyle bir tarz kullanmazdım. Özetlemek

Artıları

• kaydırmak için daha az kod satırı
• şu ifadeyi ifade etmek üzere anlamsal olarak gruplamak için kullanılabilir: "çok sayıda atama öğesi". ANCAK sizi çok rahatsız ederse, böyle bir bloğu her zaman bir işleve yeniden aktarabilirsiniz.

Eksileri

• okunması zor, kodu yatay olarak okumak genellikle daha kolaydır (gözden geçirme, göz hareketi yok, ...)
• diff kolayca birleşme de dahil olmak üzere bir kabus haline gelir
• değiştirmek daha zor (kopyala yapıştır, yorum yapmak, ...)
• hata ayıklama, bireysel ifadeler yerine satırlarda çalıştıkları için birçok IDE'de bir sorun olabilir.