MVC'de bir denetleyici sınıfında özel, eylemsiz, işlevlere sahip olmak iyi bir uygulama olarak kabul edilir mi?

Bazen denetleyici sınıfındaki eylem işlevleri, Modelden Görünüme veri akışını basitçe kontrol etmek için birçok kod satırı ile çok büyük ve kötü olabilir. Bir noktada bu devasa işlevler, iyi kodun temel ilkelerini tamamen kaybeder, yani sadece bir şey yapmak, küçük, okunabilir ve yönetilebilir olmak vb.

Bu devasa eylem işlevlerini denetleyici sınıfındaki daha küçük özel işlevlere bölmek iyi bir uygulama olarak mı kabul edilir veya bu tür bir optimizasyon ihtiyacı, bunları modele eklememiz gerektiğini mi gösterir?

Eylemde göreceli olmaları için denetleyicide daha küçük işlevlerin özel olması için oy kullanardım, ancak model çok büyük ve hantal olabiliyorken denetleyicinin tercihen basit olması gerektiği yönündeki argümanları duydum; ve hangisinin en çok tercih edilen yöntem olacağını merak ediyordum.

En iyi benzetme olmayabilir, ancak denetleyiciyi bir örümceğin ağı hakkında düşündüğünüz şekilde düşünün. Tek işi, örümceğin (alttaki katmanlar) sindirmesi için sinekleri (istekleri) yakalamaktır. Ağ, daha küçük veya daha büyük sinekleri (modeller) yakalayabilir ve tutabilir. Bir örümceğin web rolü, bu amaçla kullanılabilmesine rağmen, avı sindirmek değildir. Ağ ne kadar ince ve temiz olursa, örümceğin geçimini kolaylaştırır.

MVC uygulamanıza biraz aynı mantığı uygulayabilirsiniz. Açıkladığınız büyük ve kötü işlevler modelin büyük olasılıkla davranışıdır ve modele ait olmalıdır (modelin yalnızca görünümde görüntülenen nesne olmadığını unutmayın). Eğer modelin davranışı değişirse, onu kontrol eden kontrolör değil, değiştirilmesi gereken modeldir.

Ayrıca, bunları denetleyicide özel yöntemler olarak tutmak yalnızca onu dağıtır ve bakımını zorlaştırır. Aynı zamanda kötü bir alışkanlık için de yol açar, çünkü kalkınmaya katılan diğer insanlar da projede daha önce yapıldığını gördükleri için aynı şeyi yapmaya cazip gelecektir.

Verebileceğim en iyi cevap, Robert Martin'in "Temiz Kod" adlı konuyla ilgilenen herkese tavsiye ettiğim büyük kitaptan alıntı yapmaktır:

Fonksiyonların ilk kuralı küçük olmalarıdır. İkinci kural, bundan daha küçük olmalarıdır.

Daha iyi söyleyemem. Aynı kitaptan bir başka büyük alıntı daha geçerlidir:

Fonksiyonlar bir şey yapmalıdır. İyi yapmalılar. Sadece yapmalılar.

Kodunuzu daha fazla işleve böldüğünüzde, bu işlevlere kodunuzun okunabilirliğini büyük ölçüde artırabilecek anlamlı adlar vermek zorunda kalırsınız. Söylemeye gerek yok, sınıf dışında kullanılması amaçlanmayan tüm işlevler özel olmalıdır, böylece kodunuzu kalıtım yoluyla kolayca yeniden kullanabilirsiniz.

Denetleyicinizin artık çok fazla işlevi varsa, muhtemelen çok fazla iş yaptığının bir işaretidir. Sonra birkaç bağımsız parçaya bölebilir veya diğer cevaplarda belirtildiği gibi bazı fonksiyonları modellere taşıyabilirsiniz. Ayrıca, Görünümlerin bazı mantıklara sahip olmasına izin verilen klasik olmayan MVC lezzetini takip ederseniz, bazı işlevlerinizi uygun olduğunda buraya koyabilirsiniz.

MVC'de denetleyicimin mümkün olduğunca "ince" olmasını ve modellerimin mümkün olduğunca aptal olmasını sağlamaya çalışıyorum.

İhtiyaç duyulan mantık ve yardımcı fonksiyonları ayrı ayrı yardımcı sınıflara konur. Testimi de çok daha kolay hale getiriyor (test ediyorsun .. doğru ??: D) Denetleyicileri test etmek çok zordur, test etmek için bir kontrolör örneği oluşturmaya çalıştığınızda HTTP Bağlamı ve sahte hakkında düşünmeniz gerekir. http bu ve bu ve bu bir acı, ama bilerek bir acı. Bir denetleyicinin HTTP ve web'e çok yakından bağlı olması nedeniyle tüm bunlara ihtiyacınız var. Web uygulamanızın giriş noktasıdır.

Mantık ve yardımcı işlevlerin web ile ilgisi yoktur. Tamamen çevre agnostik (ya da olmalı). Bu tek başına size aynı yere ait olmadıklarını söylemelidir. Ayrıca, tüm uygulama mantığınızı web'e veya belirli bir web uygulamasına bu kadar yakın bağlarsanız, onu asla yanınıza alamazsınız.

Biz MVC sitemizi tüm veritabanı varlıkları (mvc modellerimiz, gerçek db varlıkları değil), depolarımız, yardımcı sınıflarımız ve ayrı ayrı dll'lerdeki mantığımızla geliştirdik. Sadece her bir web sitemiz vardı, ama yine de böyle yaptık.

Birkaç ay önce, birkaç saçak sistemimizle ilgili birkaç masaüstü uygulaması oluşturmamız istendi. Test edilen kodlarımızın tümü tekrar kullanılabileceğinden bu kolayca yapılabilir. Kodumuzu web projemize soksaydık ya da kontrolörlerimize koyarsak, bunu asla yapamazdık.

Dmitri Zaitsev ve spaceman büyük cevaplar yanında ben de PHP için de geçerli olup olmadığını bilmiyorum: Otomatik test olanakları olmaması nedeniyle özel yöntemlerden kaçınmaya çalışmalısınız.

Evet, özel yöntemleri test etmek için metaprogramlama veya bağımlılık enjeksiyonu kullanabilirsiniz, ancak kodunuzun okunabilirliği üzerinde büyük bir etkisi olduğundan bunu yapmamalısınız.

KISS prensibini daima hatırlayın: Basit, aptalca tutun.