🚀 go-pugleaf

RetroBBS NetNews Server

Inspired by RockSolid Light RIP Retro Guy

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

Re: Szybkość odczytu stanu itemu w menu

#347087
From: Roman Tyczka
Date: Tue, 08 Oct 2019 06:52
36 lines
2564 bytes
On Mon, 7 Oct 2019 23:18:14 +0200, J-23 wrote:

>>> 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

Profiler właśnie po to, bo wskaże co i gdzie warto optymalizować.

> 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ć.

Nie wiem skąd takie wnioski wysnuwasz. Wystarczy, że kod biznesowy zależny
jest od stanu jakiejś flagi reprezentowanej przez menu i ten kod w trakcie
obliczeń używa tej flagi do wybrania ścieżki wykonania. Gdzie tu widzisz
odrysowywanie to ja nie mam pojęcia.

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

Stary, legacy system napisać od nowa? Chyba przesadziłeś.
 
-- 
pozdrawiam
Roman Tyczka
Technical Details
Message-ID: <5wrbfqr53zkb$.dlg@tyczka.com>
Path: mbox-mo1.pugleaf.net!news-archive.icm.edu.pl!news.icm.edu.pl!newsfeed.pionier.net.pl!goblin2!goblin1!goblin.stu.neva.ru!newsfeed.neostrada.pl!unt-exc-02.news.neostrada.pl!unt-spo-a-01.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> <5d9bab99$0$17365$65785112@news.neostrada.pl>

Show Full Headers
Complete Article Headers
Path: mbox-mo1.pugleaf.net!news-archive.icm.edu.pl!news.icm.edu.pl!newsfeed.pionier.net.pl!goblin2!goblin1!goblin.stu.neva.ru!newsfeed.neostrada.pl!unt-exc-02.news.neostrada.pl!unt-spo-a-01.news.neostrada.pl!news.neostrada.pl.POSTED!not-for-mail
From: Roman Tyczka <noemail@because.no>
Subject: Re: Szybkość odczytu stanu itemu w menu
Newsgroups: pl.comp.lang.delphi
User-Agent: 40tude_Dialog/2.0.15.84
MIME-Version: 1.0
Sender: romek@tyczka.no.found
References: <5d91f9c2$0$541$65785112@news.neostrada.pl> <18dcwoy3zraou.dlg@tyczka.com> <5d9a65e4$0$524$65785112@news.neostrada.pl> <yuaojqihdgu7$.dlg@tyczka.com> <5d9bab99$0$17365$65785112@news.neostrada.pl>
Date: Tue, 8 Oct 2019 08:52:26 +0200
Message-ID: <5wrbfqr53zkb$.dlg@tyczka.com>
Lines: 36
Organization: Telekomunikacja Polska
NNTP-Posting-Host: 94.75.90.137
X-Trace: 1570517546 unt-rea-b-01.news.neostrada.pl 510 94.75.90.137:11858
X-Complaints-To: abuse@news.neostrada.pl
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit