Article View: pl.comp.lang.delphi
Article #345872Re: Zeos Delphi XE2
From: Roman Tyczka
Date: Tue, 11 Jul 2017 21:43
Date: Tue, 11 Jul 2017 21:43
76 lines
3383 bytes
3383 bytes
On Tue, 11 Jul 2017 17:28:54 +0200, wloochacz wrote: >>> Co ty masz jakiś gorszy dzień? >> >> Twój kod tak działa na spokojnego człowieka. > To subiektywna opinia; ja tam do spokojnych się nie zaliczam, a na mnie > tak nie działa ;-) Może lubisz taki bajzel w kodzie, ja wręcz przeciwnie. Zatem, gdy przychodzi mi go czytać to mnie trafia. >> Ok, kilka _powierzchownych_ uwag. > Wiesz, że dość mocno się czepiasz? > Po prostu. W którym miejscu? >> 1. Spacje wokół przypisania zmiennych, Zmienna := Wartość; > Ctrl+D > >> 2. Spacje po przecinkach, dwukropkach itd. > Ctrl+D Po co podajesz mi ten skrót? 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ć? >> 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. >> 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? Może dlatego, żeby nie zaciemniać nazwy zmiennej? Tutaj pola klasy? >> 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. >> To tylko garść porad na początek, zacznij czytać książki i zacznij CZYTAĆ >> OBCY KOD - wtedy się nauczysz pisać swój. > Te małe literki a i f mnie się nie podobają. A mi owszem. Lepiej odcinają się od reszty nazwy i nie zaburzają jej czytania, np. fLCDSize FLCDSize aAddValue AAddValue > Poza tym, Spring4D Style Guide też nie trzyma się własnych wytycznych :P > Wiem, bo pośrednio maczałem w tym swoje brudne paluchy... Może dlatego, że kto inny zaczynał pisać Springa a kto inny go teraz prowadzi? -- pozdrawiam Roman Tyczka
Technical Details
Message-ID:
<1bjy8iyp86buw.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>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: Tue, 11 Jul 2017 23:43:42 +0200 Organization: ATMAN - ATM S.A. Lines: 76 Sender: romek@tyczka.no.found Message-ID: <1bjy8iyp86buw.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> NNTP-Posting-Host: 94.75.90.137 Mime-Version: 1.0 X-Trace: node1.news.atman.pl 1499809422 11368 94.75.90.137 (11 Jul 2017 21:43:42 GMT) X-Complaints-To: usenet@atman.pl NNTP-Posting-Date: Tue, 11 Jul 2017 21:43:42 +0000 (UTC) User-Agent: 40tude_Dialog/2.0.15.84 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit