[PHP] DO-WHILE, Problem z funkcją. Opcje

[Tomplus]

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.

Ten post edytował Tomplus 25.10.2010, 15:53:07

[CuteOne]

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 

Ten post edytował CuteOne 25.10.2010, 14:54:03

[markonix]

[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

Ten post edytował markonix 25.10.2010, 14:57:22

[Tomplus]

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 ?

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 

Ten post edytował Tomplus 25.10.2010, 15:12:35

[markonix]

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

[Tomplus]

Po zakończonym IF miałeś średnik

[markonix]

Fiu, fiu. Racja.

Ale problem rozwiązany chyba to oznacz odpowiednio temat

[Tomplus]

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

A pytanie dot. funkcji mojej i CuteOne.

[markonix]

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]

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.

Ten post edytował Tomplus 25.10.2010, 15:52:25

[markonix]

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.