Dnia 04-08-2011 o 23:40:37 <g...@h...com> napisał(a):

> Witam,
>
> Czy taka konstrukcja narusza jakieś zasady/sty dobrego projektowania lub
> jeszcze innego wzorca projektowego? Chodzi mi o drabinkę if..else

Jeśli zaprzeczylibyśm sensowności tego rozwiązania, to na dobrą sprawę
należałoby odrzucić również konstrukcję switch/case, która przecież
spełnia praktycznie tą samą rolę. Różnica jest taka, że drabinka if..else
pozwala uwzględnić kilka różnych warunków (a także - na przykład -
odrzucić warunki, które w danym momencie są zbędne), gdy switch z czymś
takim sobie nie radzi.

Poza tym - jaką alternatywę zaproponowałbyś dla takiej drabinki? Wyciąć
else nie możesz, bo często ma ono kluczowe znaczenie. Co najwyżej możesz
każdy warunek pakować do kolejnego bloku w else, tzn.

if (war1)
{

}
else if (war2)
{

}
else S;

na

if (war1)
{

}
else
{
if (war2)
{

}
else S;
}

Wydaje mi się jednak, że na dłuższą metę drugie rozwiązanie jest znacznie
mniej czytelne - szczególnie, gdy warunków masz dużo.

Pozdrawiam -- Spook.

--
Używam klienta poczty Opera Mail: http://www.opera.com/mail/