[a.....e]

Jak wolisz se?

A: mieć 3 funkcje z 1 parametrem:

getDocumentsForRevenue(Department dep)
getDocumentsForCosts(Department dep)
getDocumentsForTaxes(Department dep)

czy 1 funkcję ale 2 parametry:
getDocumentsFor(DocType.REVENUE, Department dep)
getDocumentsFor(DocType.COSTS, Department dep)
getDocumentsFor(DocType.TAXES, Department dep)

#programowanie #java

[Saly]

raczej A choć to zależy. Użyłbym B jeśli element zmienny jest prosty do ogarnięcia. Dobrym przykładem mogłoby być image.ExportAs(ImageType.PNG) zamiast image.ExportAsPNG(): co prawda logika konwersji z generycznego obrazka do PNG jest pewnie cholernie skomplikowana to koncepcyjnie jest to proste do ogarnięcia: wynikiem ma być ten sam obrazek tylko w formacie PNG co łatwo zweryfikować

W twoim wypadku pewnie wolałbym A jeśli jest tam inna logika a B, jeśli nie np. element

[HeavyFuel]

@aczutuse: zależy od tego czy rodzajów tych dokumentów będzie więcej, ale jeśli już istnieje enumerator to raczej tylko opcja 2 ma sens

[a.....e]

@HeavyFuel: będą 3 rodzaje, nie więcej

[Kaczus2B]

@aczutuse: jeśli kod będzie różnił się minimalnie to 2, jeśli będą to zupełnie rożne kody to 1.

[a.....e]

@Kaczus2B: minimalnie, będą się różnić tylko predicate

np.
void getDocumentsForRevenue(...) {
Predicate p = ...

[DoubleAxxis]

@aczutuse 2 która woła 1 ( ͡º ͜ʖ͡º)

[Krisu122]

@aczutuse: to w takim razie oczywiste że odp 2 wywalasz switcha do funkcji.
w 1szym przypadku będziesz musiał mieć gdzieś switcha żeby wybrał odpowiednią funkcję. W 2gim masz to schowane w funkcji.
Dzięki temu też nei będziesz duplikował kodu. Chyba że płacą od linijki :>

Jeśli w każdej funkcji byłaby totalnie inna logika to wtedy ok żeby mieć 3 różne.

[a.....e]

@Krisu122: nieee żadnego switcha. Polimorfizm zamiast switcha.

[a.....e]

@Krisu122: linijkowo wychodzi tak samo akurat

[Krisu122]

@aczutuse: to powiem stare porzekadło - rób jak uważasz i uważaj jak robisz :>
A tak imho to nie ma co się zbytnio spuszczać - ważne żeby działało i przynosiło wartośc biznesową.