🚀 go-pugleaf

RetroBBS NetNews Server

Inspired by RockSolid Light RIP Retro Guy

Article View: pl.comp.lang.delphi
Article #347085

Re: Szybkość odczytu stanu itemu w menu

#347085
From: J-23
Date: Mon, 07 Oct 2019 21:18
34 lines
2752 bytes
W dniu 07.10.2019 o 09:04, Roman Tyczka pisze:
> On Mon, 7 Oct 2019 00:08:36 +0200, Marek wrote:
> 
>>>>     mam w pętlach tysiące odwołań do stanu "Checked" itemu menu np. "mnuBasicMode".
>>>> Ciekawi mnie, gdybym ten stan zapamiętał (po jego zmianie) w jakiejś zmiennej
>>>> np. BasicMode, czy szybszy byłby odczyt stanu ze zmiennej niż z itemu?
>>>
>>> Długo się zastanawiałem jaki spaghetti code trzeba natrzaskać, żeby mieć
>>> pętle czytające tysiące razy stan kontrolek wizualnych... nie wymyśliłem.
>>> Słyszałeś o separacji UI od logiki biznesowej? O MVVM czy chociaż MVC? A
>>> coś Ci mówi SOLID, KISS czy DRY?
>>
>> Słyszałem, słyszałem. Projekt jest przejęty po kimś i nie opłaca się go rozbijać
>> na np. MVP, jedynie zoptymalizować, stąd to pytanie, na które jak widzę nie znasz
>> odpowiedzi :p
> 
> Odpowiedź jest trywialna, choć nieoczywista. Trzeba sprawdzić kod gettera
> od tego property i ocenić jego złożoność.
> Ale i tak na 95% w złym miejscu szukasz optymalizacji.
> Poza tym do takich rzeczy używa się profilerów, a nie strzelania z tyłka na
> oślep. Z darmowych np. tego:
> https://www.delphitools.info/samplingprofiler/
> 
Jak najbardziej profiler jest dobrą radą, ale tutaj już na oko widać że 
tego kodu nie da sie zoptymalizować :P Powód? z tego co pisał autor 
posta jest odwolanie do Checked gdzie na 100% jest to powiązane gdzieś z 
wyświetlaniem na ekranie więc już samo to powoduje spadek wydajności 
skoro to ma sie x razy odrysować.

Podsumowanie tutaj nie ma co "cerować dziury" tutaj trzeba zakasać 
rękawy i napisać to porządnie.

Pozdrawiam
J-23
Technical Details
Message-ID: <5d9bab99$0$17365$65785112@news.neostrada.pl>
Path: mbox-mo1.pugleaf.net!news-archive.icm.edu.pl!news.icm.edu.pl!newsfeed.pionier.net.pl!goblin1!goblin.stu.neva.ru!newsfeed.neostrada.pl!unt-exc-01.news.neostrada.pl!unt-spo-a-02.news.neostrada.pl!news.neostrada.pl.POSTED!not-for-mail
References: <5d91f9c2$0$541$65785112@news.neostrada.pl> <18dcwoy3zraou.dlg@tyczka.com> <5d9a65e4$0$524$65785112@news.neostrada.pl> <yuaojqihdgu7$.dlg@tyczka.com>

Show Full Headers
Complete Article Headers
Path: mbox-mo1.pugleaf.net!news-archive.icm.edu.pl!news.icm.edu.pl!newsfeed.pionier.net.pl!goblin1!goblin.stu.neva.ru!newsfeed.neostrada.pl!unt-exc-01.news.neostrada.pl!unt-spo-a-02.news.neostrada.pl!news.neostrada.pl.POSTED!not-for-mail
Subject: Re: Szybkość odczytu stanu itemu w menu
Newsgroups: pl.comp.lang.delphi
References: <5d91f9c2$0$541$65785112@news.neostrada.pl> <18dcwoy3zraou.dlg@tyczka.com> <5d9a65e4$0$524$65785112@news.neostrada.pl> <yuaojqihdgu7$.dlg@tyczka.com>
From: J-23 <Baczeklu@poczta.fm>
Date: Mon, 7 Oct 2019 23:18:14 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0
MIME-Version: 1.0
In-Reply-To: <yuaojqihdgu7$.dlg@tyczka.com>
Lines: 34
Message-ID: <5d9bab99$0$17365$65785112@news.neostrada.pl>
Organization: Telekomunikacja Polska
NNTP-Posting-Host: 188.146.227.170
X-Trace: 1570483097 unt-rea-a-01.news.neostrada.pl 17365 188.146.227.170:25097
X-Complaints-To: abuse@news.neostrada.pl
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit