[kiler129]

I weź tu użyj gotowej biblioteki....

for ($i = 0, $len = strlen($payload); $i < $len; $i++)

Pomyślałem: zrobię pull request, będę dobry ... jak zobaczyłem resztę kodu to mi się odechciało.

#programowanie #optymalizacjakodu #januszeprogramowania

[0.....c]

@kiler129: współczuwam pehapa.

[kiler129]

@0x2bc: Nie zaczynaj wojny ;]

Nie język czyni kod, w każdym da się napisać dobry kod i w każdym da się napisać g---o

[noisy]

@kiler129: mimo wszystko, zrób Pull Requesta.

[kiler129]

@noisy: Żeby to poprawić musiałbym poprawić 30 innych rzeczy które mnie drażnią. Dodajmy do tego, że lib nie jest stabilny ... w wolnej chwili może ;)

[dupasmoka]

@kiler129: Hmmm... ale, ze jak niby lepiej? Czy w ten sposob czasem nie przeliczasz raz dlugosci payloadu i go zapamietuesz?

[kiler129]

@dupasmoka: Faktycznie, na szybko wyglądało jak liczenie za każdym razem [tak to jest jak się pisze po nocach].

Co nie zmienia faktu, że cały kod wygląda jak akademicka kupa a sam lib nie działa poprawnie ;<

Swego czasu spędziłem z nim 3 noce aby go odpalić ... działał ale po ~48h się coś gdzieś wysypywało i przestawał działać poprawnie. Wtedy napisałem własny ( ͡° ͜ʖ ͡°)

[dupasmoka]

@kiler129: Googlnalem. Wartosc strlen() jest obliczana raz, ale narzut na wywolanie funkcji nie jest pomijalny w stosunku do czytania zmiennej. http://stackoverflow.com/questions/14835246/why-is-strlen-slower-than-use-of-variable-when-string-size-does-not-matter

Petla napisana w ten sposob powinna wykonywac sie szybciej, ale bez zmierzenia tego nie jestem w stanie zajac stanowiska co lepsze (czytelnosc vs szybkosc).

[kiler129]

@dupasmoka: Ja pierwszy raz się spotykam z taką konstrukcją - zazwyczaj liczy się po prostu nad forem dla czytelności kodu.

Wywoływanie za każdym razem strlen to zabójstwo.

[dupasmoka]

@kiler129: Liczenie nad forem zmniejsza lokalnosc tej zmiennej. Dobra praktyka jest definiowanie zmiennych jak najblizej uzycia.

Co prawda moim zdaniem zmniejsza to czytelnosc, ale jezeli wzrost wydajnosci jest zauwazalny, to warto.

[dupasmoka]

@kiler129: Co do wykresu, to z samej legendy wykresu nie jestem w stanie wyobrazic sobie implementacji tych forow. Inline calc. to przy definiowaniu zmiennych? Czy w miejscu warunku stopu? Czym/gdzie generowany jest wykres? Przydatne bylo by narzedzie do szybkiego porownywania kawalkow kodu, jakies onlineowe czy cos.

[kiler129]

@dupasmoka: To, czy policzy się go w forze (a powinien tutaj policzyć raz) czy nad nim to nie ma znaczenia w tym przypadku akurat.

Zmniejszanie zasięgu zmiennej to fajny frazes, ale czasami idiotycznie stosowany - np. tutaj moim zdaniem. W tym przypadku można pominąć zużycie pamięci w realnych scenariuszach.

Pewne przyzwyczajenia istnieją, i jeżeli język bazuje na innym to wypadałoby ich nie zmieniać. Przykład najpopularniejszy: zmienne i stałe w klasach

[dupasmoka]

@kiler129: Zgadzam sie, ze powinno pisac sie zrozumiale i czytelnie dla wszystkich i wedlug standardu (i to tez granicach rozsadku i nie przyjmowania wszystkiego za jedyne sluszne rozwiazanie), ale idac dalej, moze ten czlowiek/grupa ludzi ma swoj coding standard w tym projekcie :)

[kiler129]

@dupasmoka: Własny coding style jest super ale zazwyczaj wygląda to tak "nie chce nam się używać standardu [czyt. PSR] to wymyślimy swój" (⌐ ͡■ ͜ʖ ͡■)

Co do kodu to implementacja for z

[dupasmoka]

@kiler129: To dorzuc jeszcze for($i=0, $c=count($a); $i<$c; $i++) i porownaj. Podejrzewam, ze wynik bedzie zblizony do precalc (a moze nawet lepszy?) ;)

[kiler129]

@dupasmoka: Jutro zmodyfikuję scenariusz, dziś ledwo żyje i kładę się właśnie spać ;)

Wynik jednak powinien być identyczny jak precalc - tutaj niczym się to nie różni niż scope.

[dupasmoka]

@kiler129: To czekam na wyniki. Pozdrawiam :)

[dupasmoka]

@kiler129: No wlasnie - tylko i az.

[dupasmoka]

@kiler129: I jak?

[kiler129]

@dupasmoka: Nie miałem czasu wcześniej zrobić testów, ale wyszło tak jak się spodziewałem ;)

Gdybyś chciał spojrzeć na wygląda całego testu to spakowałem wyniki + wygenerowany kod: https://www.dropbox.com/s/3j7t1tggvsfqd1z/1420126057.zip?dl=1