Forum PHP.pl > [PHP] DO-WHILE

Pomoc - Szukaj - Użytkownicy - Kalendarz

Tomplus

25.10.2010, 14:38:04

Witam.

Mam następujący skrypt:

[PHP] pobierz, plaintext 
do if( !in_array($rep[rand(1,32)], $repr) ) $los = $rep[$rand];
while($los);
echo "Wylosowana: [ $los ]";
[PHP] pobierz, plaintext

a tutaj tablice.

[PHP] pobierz, plaintext 
//tablica wygenerowana ręcznie
$rep = array ('','Polska',"RPA","Meksyk","Urugwaj","Francja", "Argentyna", "Nigeria", "Korea Południowa","Grecja", "Anglia", "USA", "Irlandia Północna", "Algeria", "Niemcy", "Australia", "Serbia", "Ghana", "Holandia", "Dania", "Japonia", "Kamerun", "Włochy", "Paragwaj", "Nowa  Zelandia", "Wybrzeże Kości Słoniowej", "Portugalia", "Brazylia", "Korea Północna", "Honduras", "Chile", "Hiszpania", "Szwajcaria");
//tablica wygenerowana z bazy danych
$repr = array ('','','','','','','','','','','','','','','','','','','','','','','Grecja','','','','','','','','','','','','','','Polska','','','','','','','','','','','','Dania','','','','','','','','','','','','','','','','','','','','','');
[PHP] pobierz, plaintext

Wynik powinien być prosty:
Wylosowana [ XXXXX ] ale w zmiennej $los nic się nie pokazuje.

Jednak robiąc to samo BEZ do-while działa tak jak trzeba, ale nie spełnia mojego zadania, aby funkcja losowała dopóki nie wylosuje kraju którego nie ma w tablicy $repr.

W nast. przykładzie nie powinno być problemu z wylosowaniem takiego kraju.

CuteOne

25.10.2010, 14:53:44

Sprawdź

[PHP] pobierz, plaintext 
	do {
		$rand = rand(1,32);
		if( !in_array($rep[$rand], $repr) ) 
			$los = $rep[$rand];
		else
			echo $rep[$rand]."<br>";
	}
	while(empty($los));
	echo "Wylosowana: [ $los ]";
[PHP] pobierz, plaintext

markonix

25.10.2010, 14:55:54

[PHP] pobierz, plaintext 
      while(!$cos) {
        $index = rand(1,32);  
        $zmienna = $rep[$index];
        if(!in_array($zmienna,$repr);
        {
          echo 'Znalazłem kraj:'  . $zmienna;
          $cos = true;
        }
      }
[PHP] pobierz, plaintext

Może o to chodzi?

Edit: spóźnienie

Tomplus

25.10.2010, 15:11:36

Cytat(CuteOne @ 25.10.2010, 15:53:44 )

Sprawdź

[PHP] pobierz, plaintext 
	do {
		$rand = rand(1,32);
		if( !in_array($rep[$rand], $repr) ) 
			$los = $rep[$rand];
		else
			echo $rep[$rand]."<br>";
	}
	while(empty($los));
	echo "Wylosowana: [ $los ]";
[PHP] pobierz, plaintext

Powiedz mi dlaczego Twój skrypt działa, a mój nie, skoro prócz funkcji empty() są praktycznie identyczne ?

Cytat(markonix @ 25.10.2010, 15:55:54 )

Może o to chodzi?

Edit: spóźnienie

Trochę błędów w Twoim poscie, ale po poprawie, skrypt dziala.

[PHP] pobierz, plaintext 
while(!$cos)
{
	$index = rand(1,32);  
	$zmienna = $rep[$index];
	if(!in_array($zmienna,$repr))
	{
		echo 'Znalazłem kraj:'  . $zmienna;
		$cos = true;
	}
}
[PHP] pobierz, plaintext

markonix

25.10.2010, 15:16:57

To z ciekawości spytam teraz JA co miałem źle (tzn. co poprawiłeś)?

Tomplus

25.10.2010, 15:22:39

Po zakończonym IF miałeś średnik

markonix

25.10.2010, 15:25:13

Fiu, fiu. Racja.

Ale problem rozwiązany chyba to oznacz odpowiednio temat

Tomplus

25.10.2010, 15:32:04

Problem będzie rozwiązany gdy dostanę odpowiedź na moje pytanie.

A pytanie dot. funkcji mojej i CuteOne.

markonix

25.10.2010, 15:41:08

Hm.. Jest tam np. zmienna $rand bez żadnej deklaracji wartości.
Poza tym dla lepszej analizy porobiłbym tam parę tabulatorów i enterów - lepiej się czyta.
Klamerka obejmująca zawartość pętli też nie zaszkodzi.

Tomplus

25.10.2010, 15:51:03

Własnie ja mam taką zasadę: aby nie używać zbędnych znaków.
Klamry są potrzebne jeżeli jest więcej niż jedna rzecz do obsłużenia i automatycznie pojawiają się entery i tabulatory.

Ale z tym $rand masz racje ... tutaj jest pies pogrzebany. Wielkie dzięki

Drobiazgi zawsze szybciej zobaczy ktoś inny niż my sami.

markonix

25.10.2010, 16:02:34

Dla mnie to są niezbędne znaki

Dodam tylko, że ta druga tablica nie musi być wypełniona tymi pustymi ciągami.
Wystarczą 3 elementy - 3 kraje, in_array zadziała tak samo.
W komentarzu napisałeś, że ta tablica jest ręcznie napisana dlatego trochę mnie dziwi jej konstrukcja.

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.