-
Data: 2011-08-07 15:06:15
Temat: Re: kwestia estetyczna
Od: p...@p...onet.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.
>
no ja powiedzialbym akurat raczej zgadzam sie z tym co ty tu
mowisz tak ze mnie nie musisz co do tego przekonywac -
zarazem nie pale sie zeby sie w to wglebiac bo tych spostrzezen
dokonalem sobie juz jakis czas temu;
choc mysle ze tez pare rzeczy o ifach mozna sobie jeszcze ustalic,
np przyjac robocza zasade by unikac czy przyjrzec sie przypadkow
'zagniezdzonych' ifow czy 'dlugich' ifow (jak mowie zdarza sie
to raczej rzadko) lub ifom wogole
--
Wysłano z serwisu OnetNiusy: http://niusy.onet.pl
Następne wpisy z tego wątku
- 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
- 08.08.11 17:40 A.L.
- 08.08.11 17:41 A.L.
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 <=