- lms - lists.lms.org.pl

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

Re: Bez MACa :(
by Tomasz Chilinski 15 Lip '04

15 Lip '04

On Wed, 14 Jul 2004 21:30:18 +0200, A.L.E.C wrote > nbtscan jest potrzebne przy skanowaniu, tutaj dane pobierane są z > /proc/net/arp (zobacz funkcję GetMACs() w LMS.class.php) lub > zdalnie. Więcej w pliku modules/choosemac.php. Więc jeśl ktoś ma restricted proc filesystem i serwer www działających na prawach np. usera apache, to GetMACs() nic nie zwróci :) Można oczywiście załatwić to jakimś skryptem pomocniczym wywoływanym przez sudo. > Aleksander 'A.L.E.C' Machniak http://alec.pl gg-2275252 Pozdrawiam Tomasz Chiliński

1 0

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

15 Lip '04

ŁJM> To mu się przydziela drugi IP. Limity można robić naraz na dwa adresy ŁJM> IP wspólne. Nigdy nie będzie możliwości przypisania jednego IP do ŁJM> 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ń. -------------------- Łukasz Wojciechowski

1 0

Re: Zmiana w nodeedit.php
by Łukasz Jarosław Mozer 15 Lip '04

15 Lip '04

Witam. W liście z dnia 14 lipca 2004 (19:55:43) można przeczytać: > Takze podpisuje sie pod tym pytaniem. To rozwiazanie jest o tyle dobre, > ze czesto "syn student ma laptopa" i przyjezdza na weekendy. 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ł. -- Łukasz Jarosław Mozer http://www.baseciq.org mailto: lukasz(a)rulez.pl

1 0

Re: Drobny test mysql vs postgres
by A.L.E.C 15 Lip '04

15 Lip '04

Rafal Ramocki wrote: > Oh zesz w morde jeza :P ta baza ktora w mysql'u zajmowala mi 9 MB w > postgresie zezarla ponad 100 a ze sie nazbieralo jeszcze starych > pakietow z apt na partycji /var ...... Dopiero teraz zauwazylem.... :/ poczytaj o poleceniu VACUUM FULL -- Pozdrawiam Aleksander 'A.L.E.C' Machniak http://alec.pl gg-2275252 Lan Management System Developer http://lms.rulez.pl

1 0

Re: Drobny test mysql vs postgres
by Rafal Ramocki 15 Lip '04

15 Lip '04

On Wed, 14 Jul 2004 12:00:16 +0200, A.L.E.C wrote > Rafal Ramocki wrote: > > Warning: pg_query(): Query failed: ERROR: could not extend > > relation "stats_pkey": Brak miejsca na urządzeniu > > HINT: Check free disk space. > > . in /home/daemons/apache- > > > ssl/vhosts/adm.interblock.pl/html/lms/lib/LMSDB_driver_postgres.class.ph > p > > on line 82 Oh zesz w morde jeza :P ta baza ktora w mysql'u zajmowala mi 9 MB w postgresie zezarla ponad 100 a ze sie nazbieralo jeszcze starych pakietow z apt na partycji /var ...... Dopiero teraz zauwazylem.... :/ > napisz, czy napewno masz wolne miejsce na dysku? jaka wersja postgresa? > jaki system, jądro?postaram się dowiedzieć o co chodzi, bo to juz nie > jest wina lmsa. > > > No i to chyba wydaje mi sie ze nawet wielokrotnie szybsze. Nom > > pakowanie bazy danych do postgresa zajmuje jakies 15 min, nie > > sprawdzalem ile do mysql'a ale jeszcze to dzisaj z czystej ciekawosci > > sprawdze. To tyle wiecej bede bawil sie moze wieczorem. > > przeczytaj resztę postów wtym wątku > > -- > Pozdrawiam > Aleksander 'A.L.E.C' Machniak http://alec.pl gg-2275252 > Lan Management System Developer http://lms.rulez.pl >

1 0