-
Data: 2011-08-05 08:25:52
Temat: Re: kwestia estetyczna
Od: g...@p...onet.pl szukaj wiadomości tego autora
[ pokaż wszystkie nagłówki ]> 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
Następne wpisy z tego wątku
- 05.08.11 09:42 Wojciech \"Spook\" Sura
- 05.08.11 12:52 Artur Muszyński
- 05.08.11 20:33 g...@p...onet.pl
- 06.08.11 16:09 Karol Y
- 06.08.11 16:34 m...@t...pl
- 06.08.11 17:50 A.L.
- 06.08.11 20:13 m...@t...pl
- 06.08.11 21:24 A.L.
- 06.08.11 22:49 Wojciech Muła
- 06.08.11 23:21 m...@t...pl
- 07.08.11 00:05 A.L.
- 07.08.11 06:15 m...@t...pl
- 07.08.11 08:29 g...@p...onet.pl
- 07.08.11 10:02 m...@t...pl
- 07.08.11 10:03 Wojciech Jaczewski
Najnowsze wątki z tej grupy
- Alg. kompresji LZW
- Popr. 14. Nauka i Praca Programisty C++ w III Rzeczy (pospolitej)
- Arch. Prog. Nieuprzywilejowanych w pełnej wer. na nowej s. WWW energokod.pl
- 7. Raport Totaliztyczny: Sprawa Qt Group wer. 424
- TCL - problem z escape ostatniego \ w nawiasach {}
- Nauka i Praca Programisty C++ w III Rzeczy (pospolitej)
- testy-wyd-sort - Podsumowanie
- Tworzenie Programów Nieuprzywilejowanych Opartych Na Wtyczkach
- Do czego nadaje się QDockWidget z bibl. Qt?
- Bibl. Qt jest sztucznie ograniczona - jest nieprzydatna do celów komercyjnych
- Co sciaga kretynow
- AEiC 2024 - Ada-Europe conference - Deadlines Approaching
- Jakie są dobre zasady programowania programów opartych na wtyczkach?
- sprawdzanie słów kluczowych dot. zła
- Re: W czym sie teraz pisze programy??
Najnowsze wątki
- 2025-03-02 Tusk idzie na rekord deportacji po 1989 [Będzie popyt na prawników]
- 2025-03-01 Obywatel telefonuje 112 lub 986
- 2025-03-01 detektyw (?) Rutkowski działał jako prasa
- 2025-03-01 "Policjant został ujęty obywatelsko..."
- 2025-03-01 zatrzymanie zbyszka maja
- 2025-03-01 Warszawa => Expert Recruiter 360 <=
- 2025-03-01 Chrzanów => NodeJS Developer <=
- 2025-03-01 Warszawa => Gen AI Engineer <=
- 2025-03-01 Wrocław => Konsultant wdrożeniowy Comarch XL/Optima (Księgowość i
- 2025-03-01 Kraków => Technical Team Leader (Clojure, Java) <=
- 2025-03-01 Zrobił TV OLED z TV LCD
- 2025-03-01 Gdynia => Sales Executive / KAM <=
- 2025-03-01 Błonie => Sales Specialist <=
- 2025-03-01 Ryga => Konsultant Wdrożeniowy Comarch XL/Optima (Księgowość i Kad
- 2025-03-01 Żerniki => Dyspozytor Międzynarodowy <=