- lms - lists.lms.org.pl

Re: propozyjca optymalizacji kodu
by Przemek SAKO Galczewski 16 Lip '04

16 Lip '04

Hello Dawid, DW> Wydaje mi się, że wystarczyłoby zamienić kolejność warunków w pierwszym DW> przykładzie, aby Twój cel został osiągnięty ("jedno zapytanie mniej"). DW> Z tego, co się orientuję, parser PHP nie sprawdza logiki dalszych warunków, DW> jeśli któryś z wcześniejszych jest false. Z tego typu praktykami to bym uwazal, to ze nie sprawdza nie znaczy ze nastepna wersja nie zacznie. W dokumentacji nie znalazlem niczego na ten temat, a to oznacza, ze moze sprawdzac w dowolnej kolejnosci (jak np c/c++). -- Best regards, Przemek mailto:sako@infinity.waw.pl

1 0

KP - dowód wpłaty i adres korespondencyjny
by Bartosz 16 Lip '04

16 Lip '04

Czy może ktoś dorobić odpłatnie KP – czyli dowód wpłaty, oraz dodatkowe pole na adres do korespondencji?? I ile ta przyjemność może kosztować ☺

1 0

Re: propozyjca optymalizacji kodu
by lukas 16 Lip '04

16 Lip '04

DW> Nie wiem, czy potrzebnie... :)) DW> <?php DW> if (condition1() && condition2()) { DW> echo "Oba warunki zostały spełnione."; DW> } DW> function condition1() { DW> echo "Sprawdzam warunek pierwszy...\n<br>"; DW> return false; DW> } DW> function condition2() { DW> echo "Sprawdzam warunek drugi...\n<br>"; DW> return true; DW> } Rzeczywiście masz rację. Ale zmiana wg mnie jest konieczna, ale tylko taka: elseif($LMS->GetNodeIDByMAC($nodedata['mac']) && $LMS->CONFIG['phpui']['allow_mac_sharing'] == FALSE) na: elseif($LMS->CONFIG['phpui']['allow_mac_sharing'] == FALSE && $LMS->GetNodeIDByMAC($nodedata['mac'])) Swoją drogą nie wiedziałem o tym, że PHP myśli w ten sposób :) Miłe zaskoczenie. No i taka sama zmiana (kolejność warunków) przydałaby się w nodeedit.php. -------------------- Łukasz Wojciechowski

1 0

Re: propozyjca optymalizacji kodu
by Dawid Wójcicki 16 Lip '04

16 Lip '04

> > Dla ludzi, którzy mają 'allow_mac_sharing' włączone oznaczałoby to > > jedno zapytanie mniej. > zgadza się, poprawione Nie wiem, czy potrzebnie... :)) <?php if (condition1() && condition2()) { echo "Oba warunki zostały spełnione."; } function condition1() { echo "Sprawdzam warunek pierwszy...\n<br>"; return false; } function condition2() { echo "Sprawdzam warunek drugi...\n<br>"; return true; } ?>

1 0

Re: propozyjca optymalizacji kodu
by A.L.E.C 16 Lip '04

16 Lip '04

lukas wrote: > Dla ludzi, którzy mają 'allow_mac_sharing' włączone oznaczałoby to > jedno zapytanie mniej. zgadza się, poprawione > Być może znalazłoby się więcej podobnych miejsc w kodzie. czekamy na zgłaszanie uwag. -- Pozdrawiam Aleksander 'A.L.E.C' Machniak http://alec.pl gg-2275252 Lan Management System Developer http://lms.alec.pl

1 0

Re: propozyjca optymalizacji kodu
by Dawid Wójcicki 16 Lip '04

16 Lip '04

> Witam > > Podam przykład w odniesieniu do nodeadd.php, ale to samo występuje w > nodeedit.php i pewnie w jeszcze kilku innych miejscach. > > Wg mnie i mając na uwadze optymalizację zamieniłbym poniższe 2 > linijki: > > elseif($LMS->GetNodeIDByMAC($nodedata['mac']) && $LMS->CONFIG['phpui']['allow_mac_sharing'] == FALSE) > $error['mac'] = "Podany MAC jest już w bazie!"; > > na takie 5: > elseif($LMS->CONFIG['phpui']['allow_mac_sharing'] == FALSE) > { > if($LMS->GetNodeIDByMAC($nodedata['mac'])) > $error['mac'] = "Podany MAC jest już w bazie!"; > } > > Dla ludzi, którzy mają 'allow_mac_sharing' włączone oznaczałoby to > jedno zapytanie mniej. > > Być może znalazłoby się więcej podobnych miejsc w kodzie. > > -------------------- > Łukasz Wojciechowski Wydaje mi się, że wystarczyłoby zamienić kolejność warunków w pierwszym przykładzie, aby Twój cel został osiągnięty ("jedno zapytanie mniej"). Z tego, co się orientuję, parser PHP nie sprawdza logiki dalszych warunków, jeśli któryś z wcześniejszych jest false. Pozdrawiam seredcznie, Dawid Wójcicki

1 0

propozyjca optymalizacji kodu
by lukas 16 Lip '04

16 Lip '04

Witam Podam przykład w odniesieniu do nodeadd.php, ale to samo występuje w nodeedit.php i pewnie w jeszcze kilku innych miejscach. Wg mnie i mając na uwadze optymalizację zamieniłbym poniższe 2 linijki: elseif($LMS->GetNodeIDByMAC($nodedata['mac']) && $LMS->CONFIG['phpui']['allow_mac_sharing'] == FALSE) $error['mac'] = "Podany MAC jest już w bazie!"; na takie 5: elseif($LMS->CONFIG['phpui']['allow_mac_sharing'] == FALSE) { if($LMS->GetNodeIDByMAC($nodedata['mac'])) $error['mac'] = "Podany MAC jest już w bazie!"; } Dla ludzi, którzy mają 'allow_mac_sharing' włączone oznaczałoby to jedno zapytanie mniej. Być może znalazłoby się więcej podobnych miejsc w kodzie. -------------------- Łukasz Wojciechowski

1 0

Re: nikt nie chce zarobic? Re: Re: korekta faktur + koszta - zaplace ... - odpowie ktos sensownie?
by tomder 15 Lip '04

15 Lip '04

zrobilo sie lekko offtopicowo .... panowie za duzo filozoficznych filmow i ksiazek .... ja zwyczajnie pytam - Wy zwyczajnie odpowiedzcie .... tak - nie .... ALEC sie zainteresowal - i ciesze sie - dogadamy sie pewnie nie pytam czy lubicie ogorki czy ogrodnika corki .... ale czy ktos sie podejmie konkretnej rzeczy zysk dla wszystkich uzytkownikow a jeden frajer placi a Wy zaczynacie toczyc jakies dyskusje nad wyzszoscia Swiat Bozego Narodzenia nad Swietami Wielkiej Nocy .... ciesze sie z Wami z Waszych zajec i tego ze dostajecie za to pieniadze ktore Was zadawalaja .... ;-) i nie musicie sie przepracowywyac - dziekujcie Bogu bo moglibyscie pracowac w 'biedronce' na sali .... skoro mianujecie sie developerami tego produktu i nie chce Wam sie czyms zajac ... to sorry ale to dziecinanda i boje sie o przyszlosc ... ;-) nie mam zalu ani pretensji .... ale toczycie glupawe spory i dyskusje kto moze wyjsc z pracy punktualnie a kto nie i dlaczego a nie zajmujecie sie tym do czego jest ta grupa stworzona .... w ciagu ostatnich dni to kolejna tego typu tyrada .... jak ktos chce sie zajac dodaniem opisanych przeze mnie opcji i pomoc ALECowi w zrozumieniu problemu i wykonaniu to zapraszam ale tylko rzeczowo ciao! na oftopici .... EOT

1 0

Re: Zmiana w nodeedit.php
by lukas 15 Lip '04

15 Lip '04

ALEC> oczywiście sam zdecydujesz co modyfikować, czy LMSa, czy 'używane ALEC> obecnie rozwiązania'. Pamiętaj tylko, żeby nie liczyć na łatwą ALEC> aktualizację LMSa, gdyż ta zmiana nie jest wcale taka banalna. Rozmiary "środowiska", do którego mam wdrożyć LMS'a są nie małe i niechcę ingerować już na początku w przyjęte w nim rozwiązania, dlatego zdecydowałem się na pewne zmiany w LMS'ie - myślę, że z czasem przyjęte rozwiązania zostaną zmienione w ten sposób aby z LMS'a dało się wycisnąć maksymalnie dużo. Zdaje sobię sprawę z późniejszych trudności, które pojawią się przy aktualizacjach, ale na szczęście CVS będzie bardzo pomocny. -------------------- Łukasz Wojciechowski

1 0

Re: Zmiana w nodeedit.php
by A.L.E.C 15 Lip '04

15 Lip '04

lukas wrote: >> To mu się przydziela drugi IP. Limity można robić naraz na dwa adresy >> IP wspólne. Nigdy nie będzie możliwości przypisania jednego IP do >> dwóch komputerów. Zawsze będę to bojkotował. > A co więcej wiesz o systemie/systemach, z którymi LMS ma współpracować > ? No właśnie - niewiele :). A ja muszę i chcę dostosować LMS'a do > używanych obecnie rozwiązań a > obecnie przyjęte rozwiązania przyjmują możliwość istnienia dwóch par > IP/MAC o takim samym adresie IP ale różniących się MAC'iem. > > Więc pozostają mi tylko samodzielne modyfikacje kodu LMS'a. > > Wystarczy, że dołożę sobie założenie, że ten sam adres IP można > zdefiniować o obrębie jednego użytkownika - banalne do > zaimplementowania i nie będzie burzyć aż tak przyjętych oryginalnie > założeń. oczywiście sam zdecydujesz co modyfikować, czy LMSa, czy 'używane obecnie rozwiązania'. Pamiętaj tylko, żeby nie liczyć na łatwą aktualizację LMSa, gdyż ta zmiana nie jest wcale taka banalna. -- Pozdrawiam Aleksander 'A.L.E.C' Machniak http://alec.pl gg-2275252 Lan Management System Developer http://lms.rulez.pl

1 0