[niedorzecznybubr]

Którą wersję preferujecie?

if (condition) {
return x;
} else {
return y;
}

czy

if (condition) {
return x;
}
return y;

#programowanie #programista15k #pracait #pytaniedoeksperta

[niedorzecznybubr]

@niedorzecznybubr: wykop ukradł wcięcia ale wiadomo o co chodzi

[szubke]

Którą wersję preferujecie?

@niedorzecznybubr: to nie jest kwestia preferencji, nalezy stosowac wczesne returny :)

[Principi]

@niedorzecznybubr: jak instrukcje krótkie to 2 lub ternary operator a jak długie to 1

[Romska_Palo_Ul_Laputa]

@niedorzecznybubr: OPie, w normalnych językach jest to coś w stylu
return condition ? x : y;

[niedorzecznybubr]

@Romska_Palo_Ul_Laputa: mądralo to jest tylko uproszczony przykład, przed returnami moze być jeszcze jakaś logika

[ly000]

@szubke: w obu przypadkach returny są tak samo "wczesne"

[Goddy]

Szybkie wyjście u góry metody, a poza infem logika. Nic gorszego nie ma niz kupa logiki wewnątrz ifa.

[ly000]

@niedorzecznybubr: imho to jest kwestia estetyki, zależy od intencji autora i tego jak wygląda rzeczywisty kod. else często zwiększa czytelność kodu, nawet jeśli jest zbędny. branche kodu są wtedy bardziej ewidentne i bardziej "odseparowane" od siebie.

[szubke]

w obu przypadkach returny są tak samo "wczesne"

@ly000: Nie, dodaj kolejny warunek do pierwszej wersji w tym samym podejsciu i sprawdz czytelnosc kodu, pierwszy lepszy artykul ktory wyczerpuje ten temat.

imho to jest kwestia estetyki

Nie, to jest kwestia jakosci kodu.

[E.....o]

@niedorzecznybubr: oba rozwiązania są błędne.. zamiast ifa powinieneś użyć klasy abstrakcyjnej i wzorca strategii ( ͡°( ͡° ͜ʖ( ͡° ͜ʖ ͡°)ʖ ͡°) ͡°)

[peoplearestrange]

@niedorzecznybubr: tak jak chadgpt wypluje

[ly000]

Nie, dodaj kolejny warunek do pierwszej wersji w tym samym podejsciu i sprawdz czytelnosc kodu, pierwszy lepszy artykul ktory wyczerpuje ten temat.

@szubke: no a jak babcia będzie miała wąsy to będzie dziadkiem. dlaczego zmieniasz sobie dowolnie treść pytania?

Nie, to jest kwestia jakosci

[szubke]

no a jak babcia będzie miała wąsy to będzie dziadkiem

@ly000: podalem ci konkretny artykul na ten temat, czym jest early return

a jakość kodu może być oczywiście mierzona w sposób obiektywny

nie, early return to jedna z zasad w obrebie clean code

[zlodziej_prywaciarz]

@niedorzecznybubr:

Jak masz proste sprawdzenie albo jedno albo drugie, to jedziesz bez elsa. Ale jeśli sprawdzasz, czy jedno zaprzecza drugiemu to lepiej polecieć z elsem.

Wydajność: Pewnie zależy od zagnieżdżenia - trzebaby rozkminić, czy przypadkiem nie wpływa na przeliczanie wszystkich instrukcji warunków kiedy nie ma jasno powiedziane, którą ścieżką ma

[hellfirehe]

return condition?x:y;

[Nolimit]

@niedorzecznybubr tylko i wyłącznie konstrukcja bez else jest poprawna

[Reynolds]

@ElTalento a wewnątrz wzorca, w impl. te same ify :P

[ZygmnuntIgthorn]

nalezy stosowac wczesne returny :)

@szubke: Chyba że musisz stosować misrę, albo inne standardy, gdzie wymagany jest "single point of return".

[Majkel2008]

@niedorzecznybubr: powszechnie stosowane jest bez else. Spotkałem się nawet z zadaniem w trakcie rozmowy rekrutacyjnej typu "popraw kod", gdzie jedną z oczekiwanych rzeczy było usunięcie takich właśnie else żeby zmniejszyć ilość zagnieżdżania.

[bastianek321]

@niedorzecznybubr: ternary operator, ale kwestia dogadania sie z teamem co preferuja