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