Forum PHP.pl > mysql zawiodlo przy duzej ilosci userow

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: mysql zawiodlo przy duzej ilosci userow

Forum PHP.pl > Forum > Bazy danych > MySQL

inwob

30.09.2010, 16:12:59

Witam!
Napisałem skrypt który automatycznie rozdaje konta lecz zawiódł przy dużej ilości chętnych.
Wygladal on mniej wiecej tak

[PHP] pobierz, plaintext 
$selee = mysql_query("znajdz wszystkich graczy spejniajacych kryteria");
if(jesli znalazl cos) {
while ($sel = mysql_fetch_assoc($selee)) { // sprawdz az znajdzie wolne konto
$seleee = mysql_query('Sprawd czy konta juz ktos nie dostal tego konta');
if (@mysql_num_rows($seleee)) {
break;
}
}
if (@mysql_num_rows($seleee)) {
mysql_query("ustaw wlasciciela konta!");
}
}
[PHP] pobierz, plaintext

Miałem kilka przypadków że kilka osób dostało to samo konto (uzyli skryptu w tym samym czasie)
Jak temu zapobiec?

wookieb

30.09.2010, 16:26:16

Użyć transakcji. Jak? google->Mysql transaction

vtuner

30.09.2010, 20:24:33

Cytat(inwob @ 30.09.2010, 17:12:59 )

Napisałem skrypt który automatycznie rozdaje konta lecz zawiódł przy dużej ilości chętnych.
Wygladal on mniej wiecej tak

[PHP] pobierz, plaintext 
(...)
while ($sel = mysql_fetch_assoc($selee)) { // sprawdz az znajdzie wolne konto
$seleee = mysql_query('Sprawd czy konta juz ktos nie dostal tego konta');
if (@mysql_num_rows($seleee)) {
break;
}
}
(...)
[PHP] pobierz, plaintext

Zawiódł, bo nie umiesz pisać. Nigdy nie wykonuj zapytań w pętli.

Cytat(inwob @ 30.09.2010, 17:12:59 )

Jak temu zapobiec?

Ja często pisząc "zwykłym" PHP stosuje takie rozwiązanie.

[PHP] pobierz, plaintext 
$arrAccounts = array();
$seleee = mysql_query('Sprawdz kto dostal konta');
if (mysql_num_rows($seleee) > 0) {
  while ($row = mysql_fetch_assoc($seleee)) {
     $arrAccounts[] = $row['identyfikator_uzytkownika'];
  }
}
 
 
$seleee = mysql_query("znajdz wszystkich graczy spejniajacych kryteria");
if(mysql_num_rows($seleee) > 0) {
  while ($sel = mysql_fetch_assoc($seleee)) { // sprawdz az znajdzie wolne konto
    if(in_array($sel['identyfikator_uzytkownika'], $arrAcounts)) {
      break;
    }
  }
  mysql_query("ustaw wlasciciela konta!");
}
[PHP] pobierz, plaintext

Rozwiązanie może nie jest super poprawne, ale na pewno tak szybko nie zawiedzie

yevaud

30.09.2010, 21:27:46

problem o ktorym mowisz pojawia sie nie dlatego ze mysql Cie zawiodl, tylko dlatego ze nie przewidziales tego ze wielu uzytkownikow moze wykonywac ten kod jednoczesnie

jak wspomnial kolega wyzej, najlatwiej jest uzyc transakcji, ale mozesz tez zalatwic to "zaklepujac" jakos zasob dla siebie za pomoca jakiejs operacji atomowej tzn. jak widzisz ze zasob jest wolny(w Twoim wypadku to chyba login), rzucasz insertem ktory go dla Ciebie "zaklepuje", jesli insert z unikalnym polem sie udal tzn. nikt go nie zaklepal przed Toba, to znaczy ze login jest Twoj.

a tak btw naloz indeks unique na login to nie bedziesz rozdawal tych samych kont ludziom, nawet jak cos w kodzie bedzie spieprzone

btw2. rozwiazanie kolegi vtuner tez bedzie sie wykladalo dokladnie tak jak Twoje, tylko moze nie tak czesto

inwob

2.10.2010, 13:33:07

A ja zrobiłem tak:

[PHP] pobierz, plaintext 
$selee = mysql_query("znajdz wszystkich graczy spejniajacych kryteria");
if(jesli znalazl cos) {
while ($sel = mysql_fetch_assoc($selee)) { // sprawdz az znajdzie wolne konto
$seleee = mysql_query('Sprawd czy konta juz ktos nie dostal tego konta');
if (@mysql_num_rows($seleee)) {
$znalazblad = 0;
mysql_query("ustaw wlasciciela konta tam gdzie nie ma wlasciciela!") and or $znalazblad = 1;
if(!(empty($znalazblad))) {
break;
}
}
}
if (@mysql_num_rows($seleee)) {
}
}
[PHP] pobierz, plaintext

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.