-
Path: news-archive.icm.edu.pl!agh.edu.pl!news.agh.edu.pl!news.onet.pl!.POSTED!cat.tac
!not-for-mail
From: Wojciech Muła <w...@p...null.onet.pl.invalid>
Newsgroups: pl.comp.programming
Subject: Re: kwestia estetyczna
Date: Sun, 7 Aug 2011 00:49:00 +0200
Organization: http://0x80.pl
Lines: 68
Message-ID: <2...@c...tac>
References: <j1f3om$2bsb$1@news.mm.pl>
NNTP-Posting-Host: public94241.xdsl.centertel.pl
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
X-Trace: news.onet.pl 1312672251 14611 188.47.240.33 (6 Aug 2011 23:10:51 GMT)
X-Complaints-To: n...@o...pl
NNTP-Posting-Date: Sat, 6 Aug 2011 23:10:51 +0000 (UTC)
X-Newsreader: Claws Mail 3.7.9 (GTK+ 2.24.5; i486-pc-linux-gnu)
Xref: news-archive.icm.edu.pl pl.comp.programming:191722
[ ukryj nagłówki ]On Thu, 4 Aug 2011 23:40:37 +0200 <g...@h...com> wrote:
> 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;
> }
Hm, ja bym to raczej napisał tak:
if (preservedR) {
const bool sH = oldW >= oldH and not fit and (not onlyG or width < oldW);
const bool sW = not fit and (not onlyG or height < oldH);
if (sH) {
newW = width;
newH = (oldH * newW) / oldW;
}
else if (sW) {
newH = height;
newW = (oldW * newH) / oldH;
}
else
...
}
else
...
Kiedyś, przy bardziej "dzikiej" logice sterowania, gdzie warunków było
niewiele, jednak różnych kombinacji sporo, to najpierw kodowałem
każdy warunek na osobnym bicie, a później instrukcją switch
obsługiwałem kombinacje. W stosunku do wersji z if/else było 10x lepiej.
w.
Następne wpisy z tego wątku
- 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
- 07.08.11 15:06 p...@p...onet.pl
- 07.08.11 15:53 A.L.
- 07.08.11 16:51 A.L.
- 07.08.11 18:27 Paweł Kierski
- 07.08.11 18:35 Paweł Kierski
- 07.08.11 18:45 A.L.
- 07.08.11 19:07 Wojciech Muła
- 07.08.11 20:29 A.L.
- 07.08.11 21:22 m...@t...pl
Najnowsze wątki z tej grupy
- 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??
- Re: (PDF) Surgical Pathology of Non-neoplastic Gastrointestinal Diseases by Lizhi Zhang
- CfC 28th Ada-Europe Int. Conf. Reliable Software Technologies
- Młodzi programiści i tajna policja
Najnowsze wątki
- 2024-12-04 Warszawa => Architekt rozwiązań (doświadczenie w obszarze Java, AWS
- 2024-12-04 Czy policjantów należy ROZBROIĆ?
- 2024-12-03 Tymoteusz Sz.
- 2024-12-03 Re: Prezydent ułaskawia: Prezydent USA Biden (D) ułaskawia syna własnego
- 2024-12-03 Re: Tani dodatkowy sim do smartwacha
- 2024-12-03 Wróblewo => Analityk finansowy <=
- 2024-12-03 Praktyczny test GPS...
- 2024-12-02 Tak się sprzedają elektryczne woldzwageny ;-)
- 2024-12-02 Akumulator do Hyundai
- 2024-12-02 Olsztyn => Sales Specialist <=
- 2024-12-02 Poznań => Technical Artist <=
- 2024-12-02 Bieruń => Regionalny Kierownik Sprzedaży (OZE) <=
- 2024-12-02 Kraków => Business Development Manager - Dział Sieci i Bezpieczeńst
- 2024-12-02 Chrzanów => Team Lead / Tribe Lead FrontEnd <=
- 2024-12-02 Białystok => Delphi Programmer <=