Pętla while + mysql Opcje

[Majcon]

witam mam taki kod

[PHP] pobierz, plaintext 
 
$staff = 50;
$tekst = 'test';
do{ 
	$pobranie_numerow_z_users = $db->query("SELECT `number` FROM `users` WHERE `zgoda` =  '0' AND `globalban` =  '0'");
	$la = $pobranie_numerow_z_users->num_rows;
	while($pnu = $pobranie_numerow_z_users->fetch_assoc())
	{
	 $pobranie_numerow_z_whois = $db->query("SELECT * FROM `whois` WHERE `ranga` >=  {$staff} AND `number` = '".$pnu['number']."'");
	$nr = array();
while($pnw = $pobranie_numerow_z_whois->fetch_assoc())
	{
	 $nr[] = $pnw['number'];
	}}
	$tekst->allsay($co, $tekst, $nr);
	 $q = $db->query("UPDATE `users` SET `wyslano` = '1' WHERE `globalban` = '0' AND `wyslano` = '0' AND `zgoda` = '0'");
	 	sleep(1);
	} while($la > 0 && $la <= 1000); 
[PHP] pobierz, plaintext 

tylko problem jest taki że pętla się nie zatrzymuje i wysyła cały czas wiadomość

[netmare]

Może spróbuj zastanowić się co chciałeś napisać w tym kodzie. Może go sformatuj żeby innym się łatwiej czytało. I może spróbuj poczytać o złączeniach, żeby zrobić sobie jedno zapytanie.

[Majcon]

chodzi o to żeby pobrać paczkę 1000 numerów z tabeli users następnie sprawdzić który numer posiada rangę wtym przypadku 50 i pobrać te numery następnie wywołać funkcję allsay i wszystko byłoby ok tylko że właśnie max 1000 numerów więc pętla while

[werek]

[PHP] pobierz, plaintext 
$pobranie_numerow_z_users = $db->query("SELECT `number` FROM `users` WHERE `zgoda` =  '0' AND `globalban` =  '0'");
 
$la = $pobranie_numerow_z_users->num_rows;
[PHP] pobierz, plaintext 

w tym kodzie pobierasz numery i sprawdzasz ile ich jest, ustawiajac warunek $la, a warunkiem wyjscia z całej pętli jest sytuacja kiedy ich liczba bedzie mniejsza od zera bądź większa od tysiąca

[PHP] pobierz, plaintext 
while($la > 0 && $la <= 1000);
[PHP] pobierz, plaintext 

natomiast nie robisz nic co sprawiłoby że pierwszy select miałby zwrócić mniej bądź wiecej numerów tj. updateujesz pola:

[PHP] pobierz, plaintext 
$q = $db->query("UPDATE `users` SET `wyslano` = '1' WHERE `globalban` = '0' AND `wyslano` = '0' AND `zgoda` = '0'");
[PHP] pobierz, plaintext 

ale nie zmieniasz nic co mogłoby wpłynąć na wynik warunku wyjścia z pętli, myslę że źle wziąłeś się za pętle która miała się przeiterować przez wszystkie numery. poniżej moja wersja twojego kodu przy założeniach które podałeś (nie znam struktury twoich danych ale mysle ze będzie ok)

[PHP] pobierz, plaintext 
$staff = 50;
$tekst = 'test';
 
$pobranie_numerow_z_users = $db->query("SELECT * FROM `users` WHERE `zgoda` =  '0' AND `globalban` =  '0'");
$la = $pobranie_numerow_z_users->num_rows;
 
//sprawdzenie zakresu
if ($la > 0 && $la <= 1000) {
	//iteracja przez wszystkie wyniki
	while($pnu = $pobranie_numerow_z_users->fetch_assoc())
	{
		$pobranie_numerow_z_whois = $db->query("SELECT * FROM `whois` WHERE `ranga` >=  {$staff} AND `number` = '".$pnu['number']."'");
		$nr = array();
		while($pnw = $pobranie_numerow_z_whois->fetch_assoc())
		{
			$nr[] = $pnw['number'];
		}
		$tekst->allsay($co, $tekst, $nr);
		$q = $db->query("UPDATE `users` SET `wyslano` = '1' WHERE `globalban` = '0' AND `wyslano` = '0' AND `zgoda` = '0'");
	}
	//to jest w sumie niepotrzebne, po co sleep na 1 sekunde?
	//zakładając że wysyłasz maile to i tak wysyłasz do potencjalnie różnych nadawców
	//chyba że twój serwer jest uznany za spamerski, ale w takim wypadku
	//i tak wiadomość nie dotrze
	sleep(1);
}
[PHP] pobierz, plaintext 

ps. tak jak kolega już wspomniał, popracuj nad formatowaniem bo za rok, dwa będziesz rwał włosy z głowy próbujac doszukać się zależności w kodzie

Ten post edytował werek 23.02.2013, 11:27:25

[Majcon]

Pierwsza sprawa to nie do maila
Druga żeby wysłać wiadomość do wszystkich czyli do 2050 osób muszę wysłać po 1000 bo inaczej nie pójdzie
więc pobieram numery z users zliczam i wykonuje dalszy kod i jeżeli jest więcej od 0 i mniej niż 1000 wykonuje to wszystko jeszcze raz i dopiero gdy jest 0 przepuszcza i idzie dalej ale chyba już znalazłem błąd

[pr0woKator]

Dodaj na końcu zapytania LIMIT

[Majcon]

Właśnie o to chodziło

[PHP] pobierz, plaintext 
 
$te = "test";
$staff = 50;
	do{ 
	$pobranie_numerow_z_users = $db->query("SELECT `number` FROM `users` WHERE `globalban` = '0' AND `wyslano` = '0' AND `zgoda` = '0' ORDER BY `id` ASC LIMIT 1");
	$la = $pobranie_numerow_z_users->num_rows;
	while($pnu = $pobranie_numerow_z_users->fetch_assoc()) {	
	 $pobranie_numerow_z_whois = $db->query("SELECT `number` FROM `whois` WHERE `ranga`  >=  '{$staff}' AND `number` = '".$pnu['number']."'");
	$nr = array();
while($pnw = $pobranie_numerow_z_whois->fetch_assoc()){
	 $nr[] = $pnw['number'];
	}}
	$tekst->allsay$te , $nr);
	 $q = $db->query("UPDATE `users` SET `wyslano` = '1' WHERE `globalban` = '0' AND `wyslano` = '0' AND `zgoda` = '0' ORDER BY `id` ASC LIMIT 1");
	} while($la > 0 && $la <= 1); 
[PHP] pobierz, plaintext 

tylko teraz strasznie obciąża to serwer bo musi każdy numer sprawdzić może można jakoś to przerobić