-
Data: 2011-08-07 10:02:07
Temat: Re: kwestia estetyczna
Od: m...@t...pl szukaj wiadomości tego autora
[ pokaż wszystkie 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
- 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 <=