[MySQL][PHP]Optymalizacja i bezpieczeństwo zapytań Opcje

[vegeta]

Można jakoś zoptymalizować te zapytania? Byłbym wdzięczny za podanie błędów, które mogą zaszkodzić bezpieczeństwu bazy.

[PHP] pobierz, plaintext 
$zarejestrowani = "SELECT COUNT(id) FROM `users`";
$zarej = mysql_fetch_array(mysql_query($zarejestrowani));
$online = "SELECT COUNT(active) FROM `users` WHERE active='1' and last_action > ".(time() - 900)."";
$active = mysql_fetch_array(mysql_query($online));
$ostatni = "SELECT `login` FROM `users` ORDER BY `id` DESC LIMIT 1";
$ost = mysql_fetch_array(mysql_query($ostatni));
$mostactive = "SELECT `mostactive` FROM `inne` WHERE `id`='1'";
$mostact = mysql_fetch_array(mysql_query($mostactive));
[PHP] pobierz, plaintext 

Wyniki wyświetlam później np.

Kod

Online: '.$active['COUNT(active)'].'<br />

Ten post edytował vegeta 2.12.2010, 00:38:47

[mmica]

Przefiltruj, escapuj etc., a najlepiej przejdź na OOP i skorzystaj z PDO i preparowania zapytań. Z PDO masz mniej rzeczy na głowie i możesz spać spokojnie wiedząc, że wszystko jest w porządku i niczego nie zapomniałeś przefiltrować pod kątem niepożądanego kodu. Ale nie masz co się martwić jeżeli nie wstawiasz zmiennych do zapytania (IMG:style_emoticons/default/winksmiley.jpg)

@Mephistofeles: Widzę, że są dwie różne szkoły. Jedni radzą zawsze używać apostrofów, a inni wręcz przeciwnie. Czy brak apostrofów nie ułatwia czasami sql injection? Możliwe, że coś pomieszałem, a jeżeli tak się stało niech mnie ktoś oświeci. W samym SQL nie używać, bo nie ma po co, ale obsługa bazy przez PHP to już inna baja, nie? (biorę pod uwagę, że w tym kodzie ma to małe znaczenie [statyczne zapytania], ale przeczyta to ktoś dopiero zaczynający zabawę z PHP i będzie buble pisał, a nie kod..)

Pozdrawiam,
mmica