Özel ve Korumalı - Görünürlük İyi Uygulama Endişesi [kapalı]

Aradım ve teorik farkı biliyorum.

• public - Herhangi bir sınıf / işlev yönteme / özelliğe erişebilir.
• korumalı - Yönteme / özelliğe yalnızca bu sınıf ve tüm alt sınıflar erişebilir.
• private - Yönteme / özelliğe yalnızca bu sınıf erişebilir. Kalıtımsal bile olmayacak.

Her şey yolunda ve iyi, soru şu, aralarındaki pratik fark nedir? Ne zaman privateve ne zaman kullanırsınız protected? Bunun üzerinde standart veya kabul edilebilir iyi bir uygulama var mı?

Şimdiye kadar, kalıtım ve polimorfizm kavramını korumak için publicdışarıdan erişilmesi gereken herhangi bir şey (inşaatçılar ve ana sınıf işlevselliği gibi) ve protectediç yöntemler (mantık, yardımcı yöntemler vb.) İçin kullanıyorum. Ben doğru yolda mıyım?

(Bu soru benim için değil, aynı zamanda böyle bir soru görmedim gelecekteki referans için olduğunu unutmayın SO).

Hayır, doğru yolda değilsiniz. İyi bir kural: her şeyi mümkün olduğunca özel yapmak. Bu, sınıfınızı daha kapsüllenmiş hale getirir ve sınıfınızı kullanarak kodu etkilemeden sınıfın içini değiştirmenize olanak tanır.

Sınıfınızı kalıtsal olacak şekilde tasarlarsanız, o zaman geçersiz kılınabilecek ve alt sınıflardan erişilebilecek şeyleri dikkatlice seçin ve bunu erişilebilir kılmak, ancak geçersiz kılmak istemiyorsanız, korumalı hale getirin (ve son olarak Java'dan bahsedin). Ancak, sınıfınızın alt sınıflarını kabul ettiğinizde ve korumalı bir alan veya yöntem olduğunda, bu alanın veya yöntemin sınıfın genel API'sının bir parçası olduğunu ve daha sonra alt sınıfları bozmadan değiştirilemeyeceğini unutmayın.

Devralınması amaçlanmayan bir sınıfın final yapılması gerekir (Java dilinde). Birim testi uğruna bazı erişim kurallarını (özelden korumalı, finalden finale kadar) gevşetebilir, ancak daha sonra bunu belgeleyebilir ve yöntem korunmasına rağmen, geçersiz kılınmaması gerektiğini açıkça belirtebilirsiniz.

Öncelikle burada yöntem erişiminden bahsettiğimi söyleyerek ve biraz daha az ölçüde, sınıfları üye erişimi değil , finalleri işaretleyerek bunu önceden yazayım .

Eski bilgelik

"yapmamak için iyi bir nedeniniz yoksa özel olarak işaretleyin"

açık kaynak geliştirici kütüphane alanı ve VCS / bağımlılık mgmt hakim önce yazıldığı günlerde mantıklı. Github, Maven vb. sayesinde hiper işbirlikçi oldu. O zamanlar bir kütüphanenin nasıl kullanılabileceğini kısıtlayarak para kazanacaktı. Muhtemelen kariyerimin ilk 8 veya 9 yılını bu "en iyi uygulamaya" kesinlikle bağlı kalarak geçirdim.

Bugün bunun kötü bir tavsiye olduğuna inanıyorum. Bazen özel bir yöntemi veya bir sınıf finalini işaretlemek için makul bir argüman vardır, ancak son derece nadirdir ve o zaman bile muhtemelen hiçbir şey geliştirmez.

Sen hiç:

• Hayal kırıklığına uğramış, miras ve birkaç kod satırı ile düzeltilmiş bir hataya sahip olan bir kütüphane vb. Sahibim.
• Bir kütüphane, yazarlar tarafından hayal edilenden biraz farklı bir kullanım durumu için kullanmak istedi, ancak özel / son yöntemler ve sınıflar nedeniyle bunu yapamadı? Sahibim.
• Hayal kırıklığına uğramış, şaşırmış veya genişletilebilirliğinde aşırı derecede izin verilen bir kütüphane vb. Bende yok.

Bunlar, varsayılan olarak özel işaretleme yöntemleri için duyduğum en büyük üç rasyonelleştirmedir:

Rasyonalizasyon # 1: Güvenli değil ve belirli bir yöntemi geçersiz kılmak için hiçbir neden yok

Yazdığım belirli bir yöntemi geçersiz kılmanın gerekip gerekmediği konusunda kaç kez yanlış olduğumu sayamıyorum. Birkaç popüler açık kaynak kütüphanesinde çalıştıktan sonra, şeyleri özel olarak işaretlemenin gerçek maliyetini zor yoldan öğrendim. Genellikle öngörülemeyen sorunlara veya kullanım durumlarına tek pratik çözümü ortadan kaldırır. Tersine, ben ettik asla API güvenliği ile ilgili nedenlerle yerine özel korumalı bir işaretleme yöntemi mesleki gelişim 16+ yıl içinde pişman. Bir geliştirici bir sınıfı genişletmeyi ve bir yöntemi geçersiz kılmayı seçtiğinde, bilinçli olarak "Ne yaptığımı biliyorum" diyorlar. ve verimlilik için bu yeterli olmalı. dönem. Tehlikeli ise, Javadocs sınıfında / yönteminde not edin, sadece körü körüne kapıyı kapatmayın.

Varsayılan olarak korunan markalama yöntemleri, modern SW geliştirmedeki ana sorunlardan biri için bir azaltmadır: hayal gücünün başarısızlığı.

Rasyonalizasyon # 2: Genel API / Javadoc'ları temiz tutar

Bu daha mantıklı ve hedef kitleye bağlı olarak yapılması gereken doğru şey bile olabilir, ancak API'yi "temiz" tutmanın maliyetinin gerçekte ne olduğunu düşünmeye değer: genişletilebilirlik. Yukarıda belirtilen nedenlerden dolayı, muhtemelen her durumda varsayılan olarak korunan şeyleri işaretlemek daha mantıklıdır.

Rasyonalizasyon # 3: Yazılımım ticari olarak kullanılıyor ve kullanımını kısıtlamam gerekiyor.

Bu da mantıklı, ancak bir tüketici olarak her seferinde daha az kısıtlayıcı olan rakiplerle (önemli bir kalite farkının olmadığı varsayılarak) giderdim.

Asla asla Deme

Asla yöntemleri özel markalama demiyorum. Ben daha iyi bir kural "iyi bir neden olmadığı sürece yöntemleri korumak yapmak" olduğunu söylüyorum.

Bu tavsiye, kütüphaneler veya modüllere ayrılmış daha büyük ölçekli projeler üzerinde çalışanlar için en uygunudur. Daha küçük veya daha fazla monolitik projeler için, zaten tüm kodu kontrol ettiğiniz için çok önemli değildir ve ihtiyacınız olduğunda / gerektiğinde kodunuzun erişim seviyesini değiştirmek kolaydır. O zaman bile yine de aynı tavsiyeyi veririm :-)

Özel alanları kötüye kullanmayı bırakın !!!

Buradaki yorumlar, özel alanların kullanılmasına büyük ölçüde destekleyici görünmektedir. O zaman söyleyecek farklı bir şeyim var.

Özel alanlar prensipte iyi mi? Evet. Ama altın bir kural olduğunu söylemek kesinlikle yanlış olduğundan emin değilseniz her şeyi özel yapmaktır ! Biriyle karşılaşana kadar sorunu göremezsiniz. Bence, emin değilseniz alanları korumalı olarak işaretlemelisiniz .

Bir sınıfı genişletmek istediğiniz iki durum vardır:

• Temel sınıfa ekstra işlevsellik eklemek istiyorsunuz
• Geçerli paketin dışındaki mevcut sınıfı değiştirmek istiyorsunuz (belki bazı kitaplıklarda)

İlk durumda özel alanlarla ilgili yanlış bir şey yok. İnsanların özel alanları kötüye kullanması, bokta değişiklik yapamayacağınızı öğrendiğinizde onu çok sinirlendiriyor.

Otomobilleri modelleyen basit bir kütüphane düşünün:

class Car {
    private screw;
    public assembleCar() {
       screw.install();
    };
    private putScrewsTogether() {
       ...
    };
}

Kütüphane yazarı düşündü: kütüphanem kullanıcılarının assembleCar()doğru uygulama detaylarına erişmeleri için bir neden yok mu? Vidayı özel olarak işaretleyelim.

Yazar yanlış. assembleCar()Tüm sınıfı paketinize kopyalamadan yalnızca yöntemi değiştirmek isterseniz, şansınız kalmaz . Kendi screwalanınızı yeniden yazmalısınız . Diyelim ki bu araba bir düzine vida kullanıyor ve her biri farklı özel yöntemlerde bazı gereksiz başlatma kodları içeriyor ve bu vidaların hepsi özel olarak işaretlenmiş. Bu noktada, emmeye başlar.

Evet, benimle , kütüphane yazarının daha iyi kod yazmış olabileceğini ve böylece özel alanlarda yanlış bir şey olmadığını iddia edebilirsiniz . Özel alanın OOP ile ilgili bir sorun olduğunu iddia etmiyorum . İnsanlar bunları kullanırken bir sorundur.

Hikayenin ahlaki, bir kütüphane yazıyorsanız, kullanıcılarınızın belirli bir alana erişmek isteyip istemediğini asla bilemezsiniz. Emin değilseniz, protecteddaha sonra herkesin daha mutlu olması için işaretleyin . En azından özel alanı kötüye kullanmayın .

Nick'in cevabını çok destekliyorum.

Bir süre önce her sınıfı mümkün olduğunca kilitlemekten bahseden bir makale okudum. Bir yoksa nihai ve gizli yap şeyi hemen dış dünyaya bazı veri veya işlevini göstermek gereğini. Kapsamı daha sonra daha izin verilecek şekilde genişletmek her zaman kolaydır, ancak tam tersi değildir. Öncelikle final, aralarından seçim yapmayı privateve protectedçok daha kolay hale getirecek mümkün olduğunca çok şey yapmayı düşünün .

1. Hemen alt sınıfa ayırmanız gerekmedikçe tüm sınıfları kesin yapın.
2. Hemen alt sınıfa ayırmanız ve geçersiz kılmanız gerekmedikçe tüm yöntemleri sonlandırın.
3. Her ne kadar çoğu zaman garip olan yöntemin gövdesi içinde değiştirmeniz gerekmedikçe tüm yöntem parametrelerini sonlandırın.

Eğer son bir sınıftan ayrılırsanız, o zaman dünya tarafından kesinlikle bir şeye ihtiyaç duyulmadıkça her şeyi özel yapın.

Alt sınıf (lar) içeren bir sınıftan ayrılırsanız, her özelliği ve yöntemi dikkatle inceleyin. Öncelikle söz konusu özelliği / yöntemi alt sınıflara göstermek isteyip istemediğinizi düşünün. Bunu yaparsanız, bir alt sınıfın, geçersiz kılma sürecinde özellik değerini veya yöntem uygulamasını bozduysa nesnenize zarar verip vermeyeceğini düşünün. Mümkünse ve sınıfınızın özelliğini / yöntemini alt sınıflardan bile (ironik, biliyorum) korumak istiyorsanız , o zaman özel yapın. Aksi takdirde korunmasını sağlayın.

Feragatname: Java'da fazla program yapmıyorum :)

Ne zaman privateve ne zaman kullanırsınız protected?

Özel Kalıtım , IS-A ilişkisi yerine ilişki açısından uygulandığı düşünülebilir . Basitçe söylemek gerekirse, devralınan sınıfın dış arabiriminin miras alınan sınıfla hiçbir (görünür) ilişkisi yoktur, Kalıtım yalnızca Base sınıfının sağladığı benzer bir işlevi uygulamak için kullanır .private

Özel Kalıtım'ın aksine, Korumalı kalıtım , IS-A türetme sınıfının Base sınıfından türetildiği ve türetilmiş üyelerin sadece türetilmiş sınıfa erişimini kısıtlamak istediği sınırlı bir Kalıtım biçimidir .

Peki, ödeme faturaları ödemelerin faturalandırılmasını idare ederse kapsülleme ile ilgilidir, o zaman ürün sınıfında neden tüm faturalandırma sürecine ihtiyaç duyulur, yani ödeme yöntemi nereye ödeme yapmalı .. öylesine sadece diğer sınıflar ve nesneler için kullanılanlara izin vermek diğer sınıfların da kullanacağı kişiler için halka açık olandan başka bir şey değildir, sadece sınıfları genişletmek için bu sınır için korunur. Madara uchiha gibi özel "limboo"görebildiğiniz gibi (sadece tek sınıf sınıf).