Bir .cs dosyası içerisinde birden fazla sınıf oluşturmanız önerilir mi yoksa her .cs dosyasının ayrı bir sınıfı mı olmalı?
Örneğin:
public class Items
{
public class Animal
{
}
public class Person
{
}
public class Object
{
}
}
Bunun bir an için iyi mimarinin kötü bir örneği olduğu gerçeğini göz önüne alarak, .cs dosyasında tek bir sınıftan daha fazlasına sahip olmak, kod kokusu mu?
Yanıtlar:
Verdiğiniz örnek aslında benim görüşüme göre iyidir. İç sınıfları ilan ediyorsunuz , bu yüzden onları aynı dosyada tutmak kusursuz . Bunun etrafındaki tek yol, Itemssınıfınızı kısmi bir sınıf yapmak ve onu birden fazla dosyaya bölmektir. Bu kötü uygulamayı düşünürdüm. İç içe geçmiş sınıflar için genel politikam, küçük ve özel olmaları gerektiğidir. Bunun iki istisnası var:
Eğer soruyu biraz farklı söyler ve "Her ad alanı- düzey sınıfını kendi dosyasına yerleştirmeli miyim?" Diye sorarsanız cevabım "evet" olur.
Sınıfları tasarlarken, Tek Sorumluluk İlkesine saygı duyuyoruz. Şekli semantiklerini takip ederse kod okumak çok daha kolay hale gelir, bu nedenle dosyaları sınıfa bölmek mantıklıdır.
Mekanik açıdan, sınıf başına bir dosyaya sahip olmanın çeşitli avantajları vardır. Aynı anda birden fazla sınıfı farklı pencerelerde açabilirsiniz. Bu, özellikle ciddi bir geliştirici iki ekrandan daha az ekranla çalışmadığı için önemlidir. Daha bağlam bilgisine sahip olabilme ön kafamdan ben daha fazla bağlam tutmak anlamına gelir içinde kafamda. (Çoğu IDE aynı dosyayı iki kez açmanıza izin verir, ancak bunu garip buluyorum).
Bir sonraki önemli husus, kaynak kontrolü ve birleşmedir. Sınıflarınızı ayrı tutarak, aynı dosyada değişiklik yapıldığında çok fazla zorluk çekmezsiniz, çünkü ayrı sınıfların değiştirilmesi gerekir.
Vahşice dürüst olmak gerekirse, lütfen bir dosyaya birden fazla kök sınıfı eklemeyin. Son işimde, sadece çoklu sınıflara sahip değil, aynı zamanda birden fazla ad alanına sahip dosyalar vardı ve bu da binlerce kod satırına uzanıyordu. Takip etmeye çalışmak çok zor.
Yakından ilgili olan sınıflar varsa, dosyalarını benzer şekilde adlandırın veya bir alt klasöre koyun.
Sınıf dosyalarının fiziksel olarak ayrılması, her şeyin sonu değil, dikkatlerin ayrılmasına ve daha gevşek bir bağlantıya neden olur.
Öte yandan, örneğiniz birden fazla kök sınıf göstermiyor. Birkaç iç içe sınıf vardır (not: iç içe sınıfları hiçbir şey yapmamaya çalışın ama privatebunu tasarlayabilirsiniz) ve bu tek bir dosyada olması gayet iyi .
Eğer dosyalar oldukça uyumluysa, örneğin çok kısa, benzer şekilde adlandırılmış dosyalara sahip olmanın bir alternatifi olduğunda, o zaman iyi bir fikir olabilir.
Mesela Fluent NHibernate kullanırken, saklanırsam daha kolay olduğunu biliyorum. Entity ve EntityMapbenim araçlar bu konuda söylemek zorunda ne rağmen, tek bir dosyada.
Çoğu durumda, sadece sınıfları bulmayı zorlaştırır. Dikkatli ve dikkatli kullanın.
Basitçe evet, bunu yapmanın iyi bir formu olmadığını ve nedenini söyleyeyim, bir süre sonra çözümünüz büyük olduğunda sınıfların nerede olduğunu unutursunuz, dosya adı artık içeriğin ne olduğunu, dosya adının ne olduğunu temsil etmez AnimalPersonObject.cs bu mümkün değil.
Tabii ki, türlere atlamak için yeniden zıplayan gibi araçlardaki özellikleri kullanarak bu sorunu çözebilirsiniz, ancak dosya başına 1 sınıf (arayüzler dahil), sadece .net'te değil, java'da ve şimdiye kadar gördüğüm herhangi bir kod standardının bel kemiğidir. c ++ ve diğer birçok dilde, çoğu zaman bakım sorunları olmayanlar ve küçük geliştiricileri kodunu kavramakta zorlanırlar.
Neredeyse tüm kod optimizasyon araçları size sınıfları ayrı bir dosyaya taşımayı söyleyecektir, bu yüzden benim için evet bu bir kod kokusu ve onu etkisiz hale getirmek için biraz zorlama gerektiriyor :)
Başka bir konu, aynı dosyadaki çok büyük ve benzer sınıflarla - sınıf bildirimini her zaman göremediğiniz zaman - yanlış sınıfa sınır değerler koyup neden vurulmadıklarını düşünerek sona erebilir ... grrr! :)