Article View: pl.comp.lang.delphi
Article #345876Re: Zeos Delphi XE2
From: Roman Tyczka
Date: Wed, 12 Jul 2017 13:27
Date: Wed, 12 Jul 2017 13:27
140 lines
6559 bytes
6559 bytes
On Wed, 12 Jul 2017 13:13:34 +0200, wloochacz wrote: >>> Wiesz, że dość mocno się czepiasz? >>> Po prostu. >> >> W którym miejscu? > Odpowiem Ci twoim sposobem - w każdym. > I dodam coś więcej; chodzi o to jak o tym piszesz, a nie o czym piszesz. > Jesteś niepotrzebnie napastliwy, arogancki i wychodzi na to, że > pozjadałeś wszystkie rozumy. No popatrzcie kto to mówi, największy grupowy pyskacz od zarania tej grupy :-) Boisz się, że stracisz pozycję? :-) Dobre! +1 > Też tak miałem, ale na dłuższą metę, przynosi to więcej szkody niż > pożytku. Dla Ciebie i innych, dla wszystkich. > Ale oczywiście zrobisz co zechcesz :) Ok, będę się od Ciebie uczył łagodności, empatii, kultury i savoir vivru - obiecuję! :-) >>>> 1. Spacje wokół przypisania zmiennych, Zmienna := Wartość; >>> Ctrl+D >>> >>>> 2. Spacje po przecinkach, dwukropkach itd. >>> Ctrl+D >> >> Po co podajesz mi ten skrót? > Ponieważ da się to zrobić przy pomocy automatycznego formatowania kodu, > dostępnego domyślnie właśnie przez CTRL+D. Ale nieś ten kaganek oświaty tam gdzie jest mrok (czyt. konsul), a nie do mnie. >> Oświecę Cię, znam go. Oświecę Cię drugi raz: >> wątek dotyczy kodu jaki wkleił OP, a ja odpisuje mu na pytanie dlaczego ten >> kod uważamy za syfiarsko napisany. W czym jeszcze mogę Cię oświecić? > Hmm... naprawdę mam trudność w przyznaniu się w czym jeszcze mógłbyś > mnie oświecić. Aczkolwiek jak tylko coś takiego się znajdzie, od razu > poproszę o oświecenie. > A za aktualne oświecenie serdecznie dziękuję. No problem ;-) >>>> 3. Nazwy parametrów metod oznaczamy np. małą literą a z przodu, np: >>>> function TMyClass.Process(aParam : TSomeType); >>> Chyba Ty oznaczasz. >>> Albo Wy. >>> Ale My nie. >> >> No to bawcie się w tym bagnie, ale go nie publikujcie. Po prostu. > Jesteś jakiś fanatyk czy co? Nie, jedynie wysoce sobie cenię czytelność kodu. > Na przykład dla mnie bagnem, syfem, kiła i mogiłą może być oznaczenia > małą literą prefiksu argumentu funkcji. Nieważne tak naprawdę czy małą czy dużą, ważne żeby było to odróżnione. Tak jak consty, fieldy itd. > Tylko ja, w odróżnieniu od Ciebie, uważam że to jest czysto subiektywna > opinia i szkoda prądu na jej argumentowanie. Popatrz jak pięknie się różnimy. >>>> 7. Pola klas powinny mieć prefiks f (wszystkie! i do tego nie raz F a raz >>>> f, tylko tak samo!) >>> Ja mam F, nie f - jak w źródłach producenta... >>> I jak widzę fDoopa, to mnie to boli, właśnie tam... ;-) >> >> Dlaczego zatem nie piszesz LZmiennaLokalna? > Bo jestem leniwy i nie chce mi się wciskać Shifta. > A drugie primo, ponieważ prefiksy z małej litery mam zarezerwowane dla > zmiennych lokalnych. Jest to jakaś konwencja i dzięki temu zmienia drastycznie sytuację w stisunku do braku konwencji, a szczególnie braku konsekwencji czyli pisania za każdym razem inaczej, a to OP właśnie nagminnie czyni. >>>> 8. Poczytaj co robi const w parametrach metod i zastanów się dlaczego >>>> powinieneś go używać >>> I dlaczego nie zawsze i wszędzie jest to potrzebne ;-) >> >> Ale lepiej tego nadużywać niż nie używać, bo nadużycie nie szkodzi, a brak >> ...w skrajnych przypadkach bardzo. > Zgoda. > Jednakże zawsze stoję na stanowisku, że ZAWSZE trzeba wiedzieć i > rozumieć co się robi i jak to działa. > Nie wytłumaczyłeś dlaczego należy używać const, tylko zachowałeś się jak > fachowiec-hydraulik - niby coś wiesz, ale nie powiesz, tylko opierdolisz > innego fachowca, że zjebał i jest na pewno źle. > Ale dlaczego źle i co to da in plus, tego hydraulik już nie powie, bo > uważa, że wtedy każdy może zostać hydraulikiem. I cala magia i prestiż > znika... > > Osobiście tego typu zachowanie uważam za wyjątkowo słabe... Ten wątek tego nie dotyczył. Zwróciłem uwagę, że to bardzo ważne i poleciłem doczytać szczegóły, wg mnie takie zachowanie jest zupełnie na miejscu i zupełnie wystarczające dla osoby zainteresowanej tematem. A jeśli ktoś ma to gdzieś to nawet wykład będący offtopem do niego nie dotrze. >> A mi owszem. Lepiej odcinają się od reszty nazwy i nie zaburzają jej >> czytania, np. >> >> fLCDSize >> >> FLCDSize >> >> aAddValue >> >> AAddValue > Jak pisałem, to subiektywna opinia i szkoda prądu. > > Dodam coś jeszcze - u mnie w kodzie praktycznie nie ma odwołania do > zmiennych prywatnych, które utrzymują stan właściwości (bo w 99% > przypadków używam ich do właściwości). > Zatem ich występowanie w kodzie jest w dwóch miejscach; deklaracja i > ciało gettera danej właściwości. > A skoro nie mieszam używania zmiennej prywatnej klasy, ze zmienną > lokalną metody, to nie mam takich potrzeb. A co z lazy getterami? >> Może dlatego, że kto inny zaczynał pisać Springa a kto inny go teraz >> prowadzi? > I właśnie dlatego tym bardziej powinni stosować się do własnych zasad > opisanych w Style Guide. Takie dokumenty tworzy się głównie dla > zespołów, a nie dla jednoosobowych guru. > A więc jeśli takie zasady istnieją, to powinni się do nich stosować, bo > sami wprowadzają, jak to określasz, syf w kodzie. Racja. -- pozdrawiam Roman Tyczka
Technical Details
Message-ID:
<1r57wfnl3x3a3$.dlg@tyczka.com>Path:
mbox-mo1.pugleaf.net!news-archive.icm.edu.pl!news.icm.edu.pl!newsfeed2.atman.pl!newsfeed.atman.pl!.POSTED!not-for-mailReferences:
<5950f43c$0$15190$65785112@news.neostrada.pl> <5950ff29$0$640$65785112@news.neostrada.pl> <5952c2f7$0$15208$65785112@news.neostrada.pl> <ok1vsm$gb2$1@portraits.wsisiz.edu.pl> <d1oytca0xw8i$.dlg@tyczka.com> <ok23gn$k5m$1@portraits.wsisiz.edu.pl> <n6gl583g0iv1$.dlg@tyczka.com> <5964eeb7$0$15204$65785112@news.neostrada.pl> <1bjy8iyp86buw.dlg@tyczka.com> <59660461$0$15187$65785112@news.neostrada.pl>Show Full Headers
Complete Article Headers
Path: mbox-mo1.pugleaf.net!news-archive.icm.edu.pl!news.icm.edu.pl!newsfeed2.atman.pl!newsfeed.atman.pl!.POSTED!not-for-mail From: Roman Tyczka <noemail@because.no> Newsgroups: pl.comp.lang.delphi Subject: Re: Zeos Delphi XE2 Date: Wed, 12 Jul 2017 15:27:34 +0200 Organization: ATMAN - ATM S.A. Lines: 140 Sender: romek@tyczka.no.found Message-ID: <1r57wfnl3x3a3$.dlg@tyczka.com> References: <5950f43c$0$15190$65785112@news.neostrada.pl> <5950ff29$0$640$65785112@news.neostrada.pl> <5952c2f7$0$15208$65785112@news.neostrada.pl> <ok1vsm$gb2$1@portraits.wsisiz.edu.pl> <d1oytca0xw8i$.dlg@tyczka.com> <ok23gn$k5m$1@portraits.wsisiz.edu.pl> <n6gl583g0iv1$.dlg@tyczka.com> <5964eeb7$0$15204$65785112@news.neostrada.pl> <1bjy8iyp86buw.dlg@tyczka.com> <59660461$0$15187$65785112@news.neostrada.pl> NNTP-Posting-Host: 94.75.90.137 Mime-Version: 1.0 X-Trace: node1.news.atman.pl 1499866055 571 94.75.90.137 (12 Jul 2017 13:27:35 GMT) X-Complaints-To: usenet@atman.pl NNTP-Posting-Date: Wed, 12 Jul 2017 13:27:35 +0000 (UTC) User-Agent: 40tude_Dialog/2.0.15.84 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit