[PHP] Dlaczego pętla nie działa? POMOCY[rozwiązany] Opcje

[miccom]

Witam serdecznie.
Dlaczego moja petla nie wykonuje zleconych zadań?
Czy ktoś może mi pomóc ?
Proszę o pomoc i podpowiedzi.
Chcę wyciągnąć dane z tabeli koordy, i zapisać je do tabeli miasta, ale nie mogą się powtarzać właśnie w tabeli miasta i zablokowane.
Więc najpierw sprawdzam czy wylosowanych rekordów nie ma w tabeli miasta, potem czy nie ma ich w tabeli zablokowane.
I jeśli nie ma takich rekordów w tabeli miasta ani w tabeli zablokowane- to zapisuję te wylosowane rekordy do tabeli miasta.
Gdzie robię błąd?

[PHP] pobierz, plaintext 
<?php
 
include('config.php');
 
$mapastart=1;
$mapakoniec=30;
 
$mapa=rand($mapastart,$mapakoniec);
$pole=rand(1,100);
$wyjdz = true;
while($wyjdz) {
   $wynik=mysql_query("SELECT * FROM miasta WHERE  mapa='$mapa' AND koord='$pole' ");
   $cnt = mysql_num_rows($wynik);
       if( $cnt == 0 ) {
            $wynik=mysql_query("SELECT * FROM zablokowane WHERE  mapa='$mapa' AND koord='$pole' ");
            $cnt = mysql_num_rows($wynik);
        }
 
   if( $cnt == 0 ) {
       $wynik=mysql_query("SELECT * FROM koordy WHERE  nr_mapy='$mapa' AND numer='$pole' AND rodzaj != 'morze' ");
       $cnt = mysql_num_rows($wynik);
       if( $cnt == 1 ) $wyjdz = true;
    }
     if( !$wyjdz ) {
       $mapa=rand($mapastart,$mapakoniec);
       $pole=rand(1,100);
     }
}
 
     $rows3=mysql_fetch_array($wynik);
     $koordx=$rows3['koordx'];    
     $koordy=$rows3['koordy'];
     $koordxduzy=$rows3['koordxduzy'];    
     $koordyduzy=$rows3['koordyduzy'];
 
   //$wynik1=mysql_query("INSERT INTO miasta (koord, koordx, koordy, mapa, koordxduzy, koordyduzy) VALUES ( '$pole','$koordx','$koordy','$mapa', '$koordxduzy', '$koordyduzy')");
   //
echo "
wylosowałem koordx= $koordx, i koordy=$koordy";
?>
[PHP] pobierz, plaintext 

Ten post edytował miccom 5.10.2009, 20:40:31

[jasin]

A skrypt wychodzi w ogóle z pętli WHILE? Jest

[PHP] pobierz, plaintext 
while($wyjdz)
[PHP] pobierz, plaintext 

i $wyjdz ciągle ma wartość TRUE więc pętla ciągle się wykonuje.

Ten post edytował jasin 5.10.2009, 09:34:12

[miccom]

No ok. Ale jak to zrobić.
Czy jak nie zainicjuję wartości $wyjdz, to pętla się wykona?

[wookieb]

Czy jak nie zainicjuję wartości $wyjdz, to pętla się wykona?

Nie.

[PHP] pobierz, plaintext 
 if( $cnt == 1 ) $wyjdz = false;
[PHP] pobierz, plaintext 

[miccom]

Nie.

[PHP] pobierz, plaintext 
 if( $cnt == 1 ) $wyjdz = false;
[PHP] pobierz, plaintext 

No ale to $wyjdz które wskazałeś dopiero ma zatrzymać wykonywanie pętli. Ale ona nawet nie zacznie działać.

[phpion]

Może zamiast pętli while użyj do-while. Różnica między nimi jest taka, że do-while wykona się conajmniej raz, podczas gdy while może się nie wykonać ani razu.

[wookieb]

Podstawy logiki i czytania manuala...
While wykonuje się dopoki ma w swoim "argumencie" wartość true. Wiec jak chcesz zatrzymac wykonywanie petli skoro CAŁY CZAS zmienna $wyjdz na wartosc true?
Dlatego przeciez napisalem, żebys w tamtej linijce ustawił wartość na FALSE i po klopocie.

Ale ona nawet nie zacznie działać.

No jak nie? Przecież masz przed petal $wyjdz = true;
Czy ja mówilem zebys to zmienial? Przetestuj a dopiero pisz posty.

[zegarek84]

No ale to $wyjdz które wskazałeś dopiero ma zatrzymać wykonywanie pętli. Ale ona nawet nie zacznie działać.

od razu za:
while($wyjdz) {
daj echo ++$i."\r\n<br />";

pętla się wykonuje - nie wiem jaki masz dokładnie zamiar ale na pewno masz złą logikę wewnątrz pętli
wyświetlaj sobie przez echo krokami zmienne tam gdzie są ok i tam gdzie się psuj ą żebyś znalazł błąd - tam gdzie niepotrzebne echo daj komentarz...

[miccom]

ok. Dałem tak jak mówiliście:

[PHP] pobierz, plaintext 
<?php
 
include('config.php');
 
$mapastart=1;
$mapakoniec=30;=rand($mapastart,$mapakoniec);//zmienić drugie jrdrn na 90
$pole=rand(1,100);
$wyjdz = false;
while($wyjdz) {
	echo ++$i."\r\n<br />";
	$zapytanie="SELECT * FROM miasta  WHERE  mapa='$mapa' AND koord='$pole' "; 
	$wynik=mysql_query($zapytanie);
	$cnt = mysql_num_rows($wynik);
	if( $cnt == 0 ) {
		$zapytanie="SELECT * FROM zablokowane WHERE  mapa='$mapa' AND koord='$pole' "; 
		$wynik=mysql_query($zapytanie);
		$cnt = mysql_num_rows($wynik);
	}
 
	if( $cnt == 0 ) {
		$zapytanie="SELECT * FROM koordy WHERE  nr_mapy='$mapa' AND numer='$pole' AND rodzaj != 'morze' "; 
		$wynik=mysql_query($zapytanie);
		$cnt = mysql_num_rows($wynik);
		if( $cnt == 1 ) $wyjdz = true ;
	}	
	if( !$wyjdz ) {
		$mapa=rand($mapastart,$mapakoniec);//zmienić drugie jrdrn na 90
		$pole=rand(1,100);
	}
 
 
}
 
 
	$rows3=mysql_fetch_array($wynik);
	$koordx=$rows3['koordx'];  
	$koordy=$rows3['koordy'];
	$koordxduzy=$rows3['koordxduzy'];  
	$koordyduzy=$rows3['koordyduzy'];
	$rodzaj=$rows3['rodzaj'];
	$obrazek_miasta="mapy/miasto_p.gif";    
 
echo "wyjdz=$wyjdz<br />
zapytanie1=$zapytanie<br />
zapytanie2=$zapytanie1<br />
zapytanie3=$zapytanie2<br />
wylosowałem koordx= $koordx, i koordy=$koordy";
?>
[PHP] pobierz, plaintext 

I otrzymuję coś takiego:
Array ( )
Notice: Undefined variable: wynik in /losowanie.php on line 50

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /losowanie.php on line 50

Notice: Undefined variable: zapytanie in /losowanie.php on line 63

Notice: Undefined variable: zapytanie1 in /losowanie.php on line 64

Notice: Undefined variable: zapytanie2 in /losowanie.php on line 65
wyjdz=
zapytanie1=
zapytanie2=
zapytanie3=
wylosowałem koordx= , i koordy=
Linia 47 to linia $rows3=mysql_fetch_array($wynik);, więc mniemam ze się w ogóle pętla nie wykonuje. Chcę wyświetlić wyniki dla zapytań- to nie mogę bo ich nie ma. Tak myślę.

Ten post edytował miccom 5.10.2009, 14:16:48

[wookieb]

To sprawdź/przeczytaj jeszcze raz jak ci mówiłem. Ale lepiej byłoby gdybyś powiedział co ty robisz tymi zapytaniami. Co chcesz uzyskać, jakie są struktury tabel bo coś czuje, że jest dużo prawdopodobieństwo załątwienia wszystkiego jednym zapytaniem. Dobrze by było gdybyś dał jeszcze mała ilość przykładowych danych ( w formie zapytań INSERT)

Ten post edytował wookieb 5.10.2009, 14:23:21

[miccom]

ok. Zmieniłem.
Ale ja i tak mówię: ten $wyjdz ma zatrzymać pętlę! Stąd zmieniam jej wartość na false, aby przestała się wykonywać, A na początku mam true, aby się zaczęła pętla iterować.

Teraz po twojej zmianie mam pętlę nie kończącą się

[HTML] pobierz, plaintext 
Array ( ) 
Notice: Undefined variable: i in /losowanie.php on line 22
1 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 2 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 3 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 4 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 5 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 6 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 7 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 8 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 9 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 10 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 11 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 12 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 13 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 14 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 15 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 16 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 17 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 18 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 19 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 20 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 21 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 22 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 23 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 24 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 25 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 26 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 27 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 28 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 29 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 30 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 31 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 32 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 33 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 34 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 35 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 36 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 37 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 38 
SELECT * FROM miasta WHERE mapa='9' AND koord='29' SELECT * FROM jednostki WHERE mapa='9' AND koord='29' SELECT * FROM koordy WHERE nr_mapy='9' AND numer='29' AND rodzaj != 'morze' 39 
[HTML] pobierz, plaintext 

I takich rekordów mam 240000 bo skrypt po 30sek jest wyłączany.
Fatal error: Maximum execution time of 30 seconds exceeded in /home/bazawesel/domains/wichry-wojny.pl/public_html/strona/losowanie.php on line 39

Ten post edytował miccom 5.10.2009, 14:29:43

[zegarek84]

dopiero pisałeś, że pętla newet nie startuje ;p - nie czytasz ze zrozumieniem - pisałem, że logikę z tymi ifami masz pomieszaną a nie znam struktury bazy i założenia aplikacji (przy okazji masz 100% pewności że będzie tylko 1 wynik wyszukiwania w bazie??)

[miccom]

[SQL] pobierz, plaintext 
Struktura tabeli dla  `miasta`
--
 
CREATE TABLE IF NOT EXISTS `miasta` (
  `miasto` varchar(100) collate utf8_unicode_ci NOT NULL,
  `user` varchar(100) collate utf8_unicode_ci NOT NULL,
  `koord` int(11) NOT NULL,
  `koordx` int(11) NOT NULL,
  `koordy` int(11) NOT NULL,
  `mapa` int(11) NOT NULL,
  `nacja` varchar(100) collate utf8_unicode_ci NOT NULL,
  `obrazek_miasta` varchar(100) collate utf8_unicode_ci NOT NULL,
  `koordxduzy` int(11) NOT NULL,
  `koordyduzy` int(11) NOT NULL,
  `swiat` int(11) NOT NULL
) ENGINE=MyISAM DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;
 
 
 
Struktura tabeli dla  `zablokowane`
--
 
CREATE TABLE IF NOT EXISTS `zablokowane` (
  `miasto` varchar(100) collate utf8_unicode_ci NOT NULL,
  `user` varchar(100) collate utf8_unicode_ci NOT NULL,
  `koord` int(11) NOT NULL,
  `koordx` int(11) NOT NULL,
  `koordy` int(11) NOT NULL,
  `mapa` int(11) NOT NULL,
  `nacja` varchar(100) collate utf8_unicode_ci NOT NULL,
  `obrazek_miasta` varchar(100) collate utf8_unicode_ci NOT NULL,
  `koordxduzy` int(11) NOT NULL,
  `koordyduzy` int(11) NOT NULL,
  `swiat` int(11) NOT NULL
) ENGINE=MyISAM DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;
[SQL] pobierz, plaintext 

dopiero pisałeś, że pętla newet nie startuje ;p - nie czytasz ze zrozumieniem - pisałem, że logikę z tymi ifami masz pomieszaną a nie znam struktury bazy i założenia aplikacji (przy okazji masz 100% pewności że będzie tylko 1 wynik wyszukiwania w bazie??)

losuję dwie liczby, pole i mapa.
Wrzucam je do pętli, i jeśli takiego zestawu nie ma w tabeli (miasta) i 2 tabeli (zablokowane)- to wrzucam do tabeli (miasta) wylosowane rekordy.
Ale jeśli będą już takie rekordy w 1 lub 2 tabeli- to losuję ponownie liczby pole i mapa( za pomoca rand())- i wykonuję ponownie tą pętlę sprawdzając powyższe warunki. Aż do skutku.
Ale jakoś mi nie idzie

Witam.
Zmodyfikowałem pętelkę- i teraz jest ok.

[PHP] pobierz, plaintext 
 
$mapastart=1;
$mapakoniec=30;
 
$mapa1=rand($mapastart,$mapakoniec);//
$pole1=rand(1,100);
$wyjdz = true;
while($wyjdz) {
	$zapytanie="SELECT * FROM miasta  WHERE  mapa='$mapa1' AND koord='$pole1' "; 
	$wynik=mysql_query($zapytanie);
	$cnt = mysql_num_rows($wynik);
	if( $cnt == 0 ) {
		$zapytanie1="SELECT * FROM zablokowane WHERE  mapa='$mapa1' AND koord='$pole1' "; 
		$wynik1=mysql_query($zapytanie1);
		$cnt = mysql_num_rows($wynik1);
	}
 
	if( $cnt == 0 ) {
		$zapytanie2="SELECT * FROM koordy WHERE  nr_mapy='$mapa1' AND numer='$pole1' AND rodzaj != 'morze' "; 
		$wynik2=mysql_query($zapytanie2);
		$cnt = mysql_num_rows($wynik2);
		if( $cnt == 1 ) $wyjdz = false ;
	}	
	if( $wyjdz ) {  
		$mapa=rand($mapastart,$mapakoniec);
		$pole=rand(1,100);
	}
 
 
 
}
[PHP] pobierz, plaintext 

Ten post edytował miccom 5.10.2009, 14:40:33