-
Path: news-archive.icm.edu.pl!agh.edu.pl!news.agh.edu.pl!news.onet.pl!newsgate.m10r2.
onet.pl!niusy.onet.pl
From: m...@t...pl
Newsgroups: pl.comp.programming
Subject: Re: kwestia estetyczna
Date: Sun, 07 Aug 2011 12:02:07 +0200
Organization: Onet.pl
Lines: 70
Sender: n...@n...onet.pl
Message-ID: <4...@n...onet.pl>
References: <4...@n...onet.pl>
NNTP-Posting-Host: newsgate.m10r2.onet.pl
Mime-Version: 1.0
Content-Type: text/plain; charset="iso-8859-2"
Content-Transfer-Encoding: 8bit
X-Trace: newsgate.onet.pl 1312711328 21311 213.180.150.14 (7 Aug 2011 10:02:08 GMT)
X-Complaints-To: n...@o...pl
NNTP-Posting-Date: Sun, 7 Aug 2011 10:02:08 +0000 (UTC)
Content-Disposition: inline
X-Mailer: http://niusy.onet.pl
X-Forwarded-For: 89.229.34.123, 10.174.28.60
X-User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:5.0) Gecko/20100101 Firefox/5.0
Xref: news-archive.icm.edu.pl pl.comp.programming:191727
[ ukryj nagłówki ]
> tak naprawde jak sie zastanowic to duze zaglebienia ifow
> nie zdarzaja sie czesto - nawet 'if w ifie' to raczej ewenement,
Jesli napiszemy tak:
if( warunek1 ) {
if( warunek2 ) {
if( warunek3 ) {
if( warunek4) {
if( warunek5 ) {
operacje_1();
}
}
operacje_2();
}
}
}
to zaglebienie ifow jest duze. Analiza po pierwszym spojrzeniu
na taki kod jest utrudniona - latwo pomylic sie w liczeniu
klamerek.
Jesli napiszemy tak:
if( !warunek1 ) return;
if( !waruenk2 ) return
// reszta kodu
to od razu mamy pewnosci ze warunek1 i warunek2 nie
ma nic wsplolnego z operacje_2(). Analiza
wersji z return jest prostsza.
> musze kiedys zwrocic na to wiecej uwagi
> (mozna by moze nawet powiedzec ze same ify zdarzaja sie srednio
> czesto - w sensie ifow nie ma wcale az tak duzo )
Wlasnie siedze nad takim projektem w ktorym ze wzgledu na
ogromna ilosc testow i ifow w "kazdej" klasie zostaly do
tego celu wydzielone 3 wirtualne metody. One niemal nic
innego nie robia tylko:
tmp = dane_tymczasowe()
if( ! jakis_test( tmp ) ) {
jakies_logi();
return kod_bledu;
}
Druga procedure na okolo 1-2tys wierszy mam taka:
a = dane_a1()
b = dane_b1()
p1 = funckcja_nieliniowa( a , b )
a = dane_a2()
b = dane_b2()
p2 = funckcja_nieliniowa( a , b )
..........................
return p1 * p2 * ... * pn;
Przy czym ponad 50% tego kodu to komentarze.
Nie da sie tego podzielic na male funkcje z wyraznymi
korzysciami.
Pozdrawiam
--
Wysłano z serwisu OnetNiusy: http://niusy.onet.pl
Następne wpisy z tego wątku
- 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
- 07.08.11 21:35 A.L.
- 08.08.11 08:03 m...@t...pl
- 08.08.11 13:27 A.L.
- 08.08.11 14:56 Adam Przybyla
- 08.08.11 16:30 Paweł Kierski
Najnowsze wątki z tej grupy
- Can you activate BMW 48V 10Ah Li-Ion battery, connecting to CAN-USB laptop interface ?
- We Wrocławiu ruszyła Odra 5, pierwszy w Polsce komputer kwantowy z nadprzewodzącymi kubitami
- Ada-Europe - AEiC 2025 early registration deadline imminent
- John Carmack twierdzi, że gdyby gry były optymalizowane, to wystarczyły by stare kompy
- Ada-Europe Int.Conf. Reliable Software Technologies, AEiC 2025
- Linuks od wer. 6.15 przestanie wspierać procesory 486 i będzie wymagać min. Pentium
- ,,Polski przemysł jest w stanie agonalnym" - podkreślił dobitnie, wskazując na brak zamówień.
- Rewolucja w debugowaniu!!! SI analizuje zrzuty pamięci systemu M$ Windows!!!
- Brednie w wiki - hasło Dehomag
- Perfidne ataki krakerów z KRLD na skrypciarzy JS i Pajton
- Instytut IDEAS może zacząć działać: "Ma to być unikalny w europejskiej skali ośrodek badań nad sztuczną inteligencją."
- Instytut IDEAS może zacząć działać: "Ma to być unikalny w europejskiej skali ośrodek badań nad sztuczną inteligencją."
- Instytut IDEAS może zacząć działać: "Ma to być unikalny w europejskiej skali ośrodek badań nad sztuczną inteligencją."
- U nas propagują modę na SI, a w Chinach naukowcy SI po kolei umierają w wieku 40-50lat
- C++. Podróż Po Języku - komentarz
Najnowsze wątki
- 2025-07-14 granice
- 2025-07-14 Awaria VM?
- 2025-07-14 Gdańsk => Programista Kotlin <=
- 2025-07-14 Warszawa => Junior Rekruter <=
- 2025-07-14 Warszawa => Specjalista rekrutacji IT <=
- 2025-07-14 Wkłady do zniczy...
- 2025-07-14 Warszawa => Specjalista ds. Sprzętu Komputerowego <=
- 2025-07-14 Re: PO chroniło i chroni policyjnych bandziorów [zawiasy za katowanie obywatela (Poznań czerwiec 2012)]
- 2025-07-14 Warszawa => International Freight Forwarder <=
- 2025-07-14 Warszawa => Recruiter 360 <=
- 2025-07-14 Re: Rz?Âd ZAKAZUJE magazyn?Â?w energii ?!! Nowe prawo od 14 lipca to SZOK! ??Â
- 2025-07-14 Warszawa => Sales Assistant <=
- 2025-07-13 Fałszywe alerty
- 2025-07-12 dlaczego gadacie z tym debilem
- 2025-07-13 Unia Europejska przygotowuje nowy podatek