-
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-16 Co w KANADZIE wolno komercyjnie (na razie się nie czepili?)
- 2025-03-16 silnik-chwilówka
- 2025-03-16 Prokurator Wrzosek "Bezstronna" nie przyczynia się do śmierci (dowodnie) - oświadcza bodnatura [Dwie Kacze Wieże]
- 2025-03-15 kraje nieprzyjazne samochodom
- 2025-03-15 parking Auchan
- 2025-03-15 Art. 19.1 ustawy o ochronie praw autorskich
- 2025-03-15 przegląd za mną
- 2025-03-15 Na co komu okna
- 2025-03-15 Mój elektryk
- 2025-03-15 Fejk muzyczny czy nie fejk
- 2025-03-15 China-Kraków => Senior PHP Symfony Developer <=
- 2025-03-15 Wrocław => Konsultant wdrożeniowy Comarch XL (Logistyka, WMS, Produk
- 2025-03-15 Błonie => Analityk Systemów Informatycznych (TMS SPEED) <=
- 2025-03-15 Warszawa => Senior Frontend Developer (React + React Native) <=
- 2025-03-15 Warszawa => Java Full Stack Developer (Angular2+ experience) <=