Re: propozyjca optymalizacji kodu
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
uczestnicy (1)
-
lukas