> Witam,

>

> Czy taka konstrukcja narusza jakieś zasady/sty dobrego projektowania lub

> jeszcze innego wzorca projektowego? Chodzi mi o drabinkę if..else

>

> if (preserveR)

> {

>    if (oldW >= oldH && !fit)

>    {

>        if (!onlyG || width < oldW)

>        {

>            newW = width;

>            newH = (oldH * newW) / oldW;

>        }

>    }

>    else if (!fit)

>    {

>        if (!onlyG || height < oldH)

>        {

>            newH = height;

>            newW = (oldW * newH) / oldH;

>        }

>    }

>    else

>    {

>        //...

>    }

> }

> else

> {

>    newW = width;

>    newH = height;

> }

>

trudno powiedziec (przynajmniej mi trudno powiedziec) trzebaby sie
zastanowic czy jest jakis sposob by robic takie rzeczy lepiej - osobiscie
nie przychodzi mi do glowy teraz jakis lepszy sposob (ani nie mam tez sily sie
zastanawiac) i owiedzialbym ze jest raczej ok
- co do nazewnictwa to takie 'krotkie' konwencje nazewnicze maja w sobie
cos fajnie technicznego, algebraicznego (sklaniaja by patrzec na ten kod
bardziej jak na wzory czy rownania) ale obecny trend (do ktorego sam niejako
tez zostalem przekonany - choc nie wiem czy sie kiedys nie zbuntuje albo
co by poprobowac pisania 'krotkimi' nazwami) z tego co wiem sklania sie
raczej po temu by uzywac dlugich nazw (wogole nie uzywac skrotow itp) czyli
nie 'newH = height' tylko 'newHeight = height' - taki kod czyta sie bardziej
jak tekst a nie jak wzory
Z poczatku zauwazylem rozbicie ifow ktore mozna by skompensowac, teraz jak
patrze to widze ze moze warto by chodzic w dokladnie odwrotna strone tj
porozdzielac je by staly sie bardziej 'wyrazalne'; jak mi przejdzie ten lekki
bol glowy to moze pozniej pomysle chwile nad lekko poprawiona wersja i zapostuje



--
Wysłano z serwisu OnetNiusy: http://niusy.onet.pl