Bir yöntemin parametre listesinde nesneler mi yoksa nesne tanımlayıcıları mı olmalı?

Ekiplerimiz aşağıdaki tartışmayı yapıyor:

Diyelim ki aşağıdaki iki yöntemimiz var:

public Response Withdraw(int clubId, int terminalId,int cardId, string invoice, decimal amount);

public Response Withdraw(Club club, Terminal terminal,Card card, string invoice, decimal amount);

tel üzerinden gönderilen sadece kimlikleri.

bir taraf ilk yöntemin doğru olduğunu söylüyor, çünkü sadece terminal ve kulüp kimliğine sahibiz ve başka bir şeyimiz olmadığı açık olmalı, bu benim yaklaşımım.

diğer taraf ise ikinci yöntemin doğru olduğunu söylüyor çünkü daha esnek.

Nesne parametresi fikrini biliyoruz, diğer taraf ayrıca object parametresinin nesnelere özellik olarak sahip olması gerektiğini düşünüyor.

Doğru yaklaşım hangisi?

Belki üçüncü bir yaklaşım daha iyidir?

Cevap bağlama bağlıdır.

İstemcinin tüm bu nesneleri zaten kullanması bekleniyorsa , nesne parametrelerini kullanırdım. Aksi takdirde, kodları olması gerekenden daha kıvrımlı görünecektir. (Örneğin club.getId(), örneğin çağrıları olacaktır .)

İstemcinin kimlikleri yalnızca kolayca kullanılabilecekse , ikinci yaklaşım daha iyidir, çünkü istemcinin gerçekten yalnızca kimliğe ihtiyacınız varsa tüm bu nesneleri bir araya getirmesini / yüklemesini istemeyebilirsiniz.

Bir seçenek her iki yöntemi de sağlamaktır , böylece müşteri hangisini kullanacağını seçebilir (bunun API'nizi karıştırmaması nedeniyle)

Genel olarak nesne parametreleri daha genişletilebilir, çünkü gelecekte işi yapmak için başka bir veri parçasına ihtiyacınız varsa, bu ekstra bilgiyi alan başka bir yöntem tanıtmanıza gerek yoktur.

Son olarak, yöntem imzalarınız yöntemin ne yaptığının (sizin durumunuzda tam olarak telin üzerinden geçenlerin) özelliklerine göre belirlenmemelidir. API soyut bir anlam ifade etmelidir, bu yüzden uygulama değişirse vidalanmazsınız.

İlk yaklaşım İlkel Saplantı'nın göstergesidir . İçeri ve dizeleri geçtiğiniz için, programcının hata yapması çok kolaydır (örn. Bir clubId'yi terminalId parametresine iletmek). Bu hataların bulunması zor olacaktır.

İkinci örnekte, bir terminal beklendiğinde bir kulübü geçmek imkansızdır - bu size derleme zamanı hatası verecektir.

Yine de, hala bakardım string invoice. Bir fatura gerçekten bir dize mi? Ne anlama amountgeliyor? Bu daha çok parasal bir değerdir.

Sorunuzda "tel üzerinden gönderilenler sadece kimliklerdir." Bu doğrudur, ancak bu gereksinimin alanınızı çamurlamasına izin vermeyin.

Bu yaklaşım lehine gördüğüm en iyi açıklama Object Calisthenics'in 3. kuralıydı :

Kendi başına bir int sadece bir skalerdir, bu yüzden bir anlamı yoktur. Bir yöntem int olarak parametre olarak alındığında, yöntem adının tüm amacı ifade etme işini yapması gerekir. Aynı yöntem parametre olarak bir Saat alırsa, neler olduğunu görmek çok daha kolaydır. Bunun gibi küçük nesneler, programları bir saat parametresi alan bir yönteme geçirmek mümkün olmadığından programları daha sürdürülebilir hale getirebilir. İlkel bir değişkenle derleyici, anlamsal olarak doğru programlar yazmanıza yardımcı olamaz. Bir nesne ile, hatta küçük bir nesne ile, hem derleyiciye hem de programcıya değerin ne olduğu ve neden kullanıldığına dair ek bilgi veriyorsunuz.

Buna doğru bir cevap yok. Her iki seçenek de iş için uygun olabilir. Neredeyse her şekilde, fatura argümanı kaşımda bir karık kaldırdı, kodu okurken ne olduğuna dair bir fikrim yok.

Bir kimlik gönderirseniz, her iki sistemin de temsil ettiği şeye sıkıca bağlanması gerekir. ClubID kulüpler tablosunda anahtardır. Daha da önemlisi, hem arayan hem de callee Kulüpler tablosunun ne olduğunu ve hangi veritabanının içinde olduğunu kabul etmelidir. Bu kısıtlamayı kabul etmek istemiyorsanız veya yapamıyorsanız, nesneyi bazı ortak açıklamaları kullanarak geçirirsiniz, yerli, serileştirilmiş, xml, ad = değer ne olursa olsun, bir ini dosyası :)

Tanımladığınız gibi "tel üzerinden" size mal olacak. Bundan kaçınmak, sadece tanımlayıcıyı göndererek başka bir yerde size maliyeti. Yani hangisi sizi en az acıtıyor, şimdi (veya daha sonra olabilir ...) iyinin kötüye göstergesidir.