[lms] propozyjca optymalizacji kodu

16 Lip 2004


      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

[lms] propozyjca optymalizacji kodu

lukas