Forum PHP.pl

dobry wieczór potrzebuje napisać skrypt który w formularzu (typ select) będzie w losowej kolejności wyświetlać dni tygodnia. niestety już na początku pojawiają się problemy (IMG:style_emoticons/default/questionmark.gif) z góry mówię pierwszy raz próbuję napisać własną 'funkcje'..

[PHP] pobierz, plaintext 
function losowa_liczba()
{
$losowa_liczba = rand(1,7);
return $losowa_liczba;
}
array($tablica);
 
losowa_liczba();
 
if(!in_array(losowa_liczba(), $tablica)) 
{
$tablica = implode(',',losowa_liczba());
}
[PHP] pobierz, plaintext 

błąd jest w liniach z in array i implode.

Błędy ->

Warning: in_array() [function.in-array]: Wrong datatype for second argument in E:\VertrigoServ\www\test.php on line 26

Warning: implode() [function.implode]: Invalid arguments passed in E:\VertrigoServ\www\test.php on line 28

Ten post edytował QoooQ 24.09.2010, 20:44:01

Przetłumacz sobie na j. polski: drugi argument ma zły typ (linia 26) no i zły argument w funkcji implode(). Nie bardzo rozumiem co Ty kombinujesz z in_array. Przecież wylosowałeś już "dzień" za pomocą rand() i zwracasz go w funkcji losowa_liczba.

I jeszcze jedno: po co Ci funkcja, która zwraca losową liczbę? Przecież wystarczy Ci rand(). Chyba, że to tak dla sportu.

Ten post edytował Daiquiri 24.09.2010, 20:56:30

to moze w ramach pomocy Twoj kod z kometarzami:

[PHP] pobierz, plaintext 
<?php
 
//funkcja zwraca losiwa liczbe calkowita z zakresu od 1 do 7
function losowa_liczba()
{
	$losowa_liczba = rand(1,7);
	return $losowa_liczba;
}
 
var_dump($tablica);
array($tablica); 	//tworzysz tablice, ktorej zawartoscia jest NULL bo zmienna $tablica nie zostala zainicjowana
					//nigdzie nie zapisujesz wyniku operacji. linijka nizej zapamietuje to co zrobiles
$twojaTablica = array($tablica);
var_dump($twojaTablica); //jak widac utworzyles tablice z jednym elementem NULL
 
losowa_liczba(); 	//wywolujesz funkcje ktora zwraca losowa liczbe ale nigdzie nie zapamietujesz ani nawet go nie wyswietlasz
					//ponizej przyklad:
$wylosowanaLiczba = losowa_liczba(); //zapisanie wyniku dzialania funkcji do zmiennej
var_dump($wylosowanaLiczba); //wypisanie zapamietanej linijke wczesniej liczby
 
 
if(!in_array(losowa_liczba(), $tablica)) //nie dziala bo zmienna $tablica nie jest nawet tablica tylko NULLem co mozna wyczytac wprost z warning: in_array() expects parameter 2 to be array, null given - drugi parametr ma byc tablica, a jest null
{
	$tablica = implode(',',losowa_liczba()); 	//tutaj juz nie mam pojecia co chciales osiagnac, ale musisz sobie zdawac sprawe ze kazde wywolanie funkcji losowa_liczba() zwraca nowa losowa liczbe. jesli chcesz wylosowac raz musisz zapisac wynik dzialania funckji do zmiennej i uzywac tej zmiennej tak jak pokazalem wyzej
												//po drugie implode laczy w jeden string (tutaj oddzielony przecinkami) elementy tablicy podany jako 2 argument (Ty podajesz liczbe) co jest sygnalizowane przez warning:  [function.implode]: Invalid arguments passed - podano nieprawidlowe parametry
}
 
//jesli dobrze rozumiem to chcesz osiagnac w wyniku tablice ktora posiada liczby od 1 do 7 w nieuporzadkowwnaj kolejnosci. nie latwiej zrobic to tak?:
$tablica = array(1,2,3,4,5,6,7); //albo w forze
shuffle($tablica); //wymieszaj elementy tablicy
var_dump($tablica); //wynik to np. array(7) { [0]=> int(4) [1]=> int(5) [2]=> int(2) [3]=> int(6) [4]=> int(1) [5]=> int(3) [6]=> int(7) } //jak potrzebujesz to w stringu to teraz mozesz potraktowac ta tablice funkcja implode()
[PHP] pobierz, plaintext 

Podsumowujac: jak zabierasz sie za nowy temat przeczytaj rozdzial ksiazki/manuala albo jakis tutorial z nim zwiazany
komunikaty (warningi noticey, errory) daja duzo cennych informacji. var_dumpuj dane przed wywolaniem funkcji zeby sie przekonac czy jest w nich to co rzeczywiscie chcesz.
jesli info dotyczy konkretnej funkcji przeczytaj manuala wraz z przykladami zwracajac uwage na parametry, ich typy, zwracana wartosc, czy jest referencja itp.
bez debugowania kodu nie liczylbym na to ze napisze cos nowego i bedzie od razu dzialac

Błąd 1) drugi przekazany parametr złego typu
Błąd 2) właściewie, to samo.
Polecam przede wszystkim odrobinę nauki angielskiego, bo dzięki jego znajomości nie będziesz miał pytań tego typu.
Poza tym, co mają robić te dwie linijki:

[PHP] pobierz, plaintext 
array($tablica);
losowa_liczba();
[PHP] pobierz, plaintext 

bo w tej chwili, to raczej nic pożytecznego. I skąd w array($tablica); zmienne $tablica?

hmm wielkie dzięki .. (IMG:style_emoticons/default/smile.gif)
użyłem array(tablica) poniewaz chcialem zdeklarować że zmienna w ogóle istnieje , nie jestem pewien ale gdybym tego nie użył to wywaliło by NOTICE

hmm czy takie rozwiązanie może być ?

[PHP] pobierz, plaintext 
$tablica = array(1,2,3,4,5,6,7);
shuffle($tablica);
//print_r($tablica);
//var_dump($tablica);
 
$suma = count($tablica)-1;
 
for($l = 0;$l <= $suma;++$l)
{
 
	switch($tablica[$l])
{
case 1:
$dzien = 'Wtorek';
break;
 
case 2:
$dzien = 'Środa';
break;
 
case 3:
$dzien = 'Czwartek';
break;
 
case 4:
$dzien = 'Piątek';
break;
 
case 5:
$dzien = 'Sobota';
break;
 
case 6:
$dzien = 'Niedziela';
break;
 
default:
$dzien = 'Poniedziałek';
break;
}
 
echo $tablica[$l],'/',$dzien,'<br/>';
 
}
[PHP] pobierz, plaintext 

jesli chcesz funkcje co losuje ci dzien tygodnia slownie to prosze:



//edit: rand(0, 6) a nie rand(0, 7), pomylka w kodzie (IMG:style_emoticons/default/winksmiley.jpg)

Ten post edytował DowNlOaD_ 24.09.2010, 21:26:32

Dwie pierwsze linijki są kluczowe, ale już nawet zamiast liczb możesz dać dni.

[PHP] pobierz, plaintext 
$tablica = array('Poniedziałek', 'Wtorek', 'Środa', 'Czwartek', 'Piątek', 'Sobota', 'Niedziela');
shuffle($tablica);
echo implode('<br />', $tablica);
[PHP] pobierz, plaintext 

To działa prawie tak samo jak Twoje, tyle że nie dopisuje liczby ze slashem na początku linijek. Możesz je dodać bezpośrednio do tablicy:

[PHP] pobierz, plaintext 
$tablica = array('1/Poniedziałek', '2/Wtorek', '3/Środa', '4/Czwartek', '5/Piątek', '6/Sobota', '7/Niedziela');
[PHP] pobierz, plaintext 

EDIT, @UP:
Zamiast rand() używaj mt_rand(). Wg dokumentacji jest czterokrotnie szybsze, a rezultaty są takie same.

Ten post edytował franki01 24.09.2010, 21:21:59

a nie latwiej tak?

[PHP] pobierz, plaintext 
<?php
$tablica = array(1 => 'poniedzialek',2 => 'wtorek',3 => 'sroda',4 => 'czwartek',5 => 'piatek',6 => 'sobota',7 => 'niedziela');
shuffle($tablica);
 
foreach($tablica as $dzien)
{
	echo $dzien.'<br>';
}
[PHP] pobierz, plaintext 

Poswiec temu co napisalismy troche wiecej czasu niz 10 minut (IMG:style_emoticons/default/winksmiley.jpg)

A nie lepiej:

[PHP] pobierz, plaintext 
$days=array('poniedziałek','wtorek',...);
shuffle($days);
echo implode('<br>',$days);
[PHP] pobierz, plaintext 

Efekt taki sam:)

@up:
To się nazywa szybki odzew:)

Ten post edytował bastard13 24.09.2010, 21:23:04

Napisałem wyżej to samo. Otwórz oczy! (IMG:style_emoticons/default/smile.gif)

EDIT:
Nie nadążam z tym co tu się dzieje (IMG:style_emoticons/default/tongue.gif)

Ten post edytował franki01 24.09.2010, 21:24:15

no tak tylko że ja potrzebuje tez nr klucza..
próbowałem tak:

[PHP] pobierz, plaintext 
<?php
 
if(isset($_POST['dzien']))
{
$dzien = abs($_POST['dzien']+1);
 
echo $dzien,'/',date('w');
 
if($dzien == date('w'))
{
echo '<span>poprawny</span>';
}
else 
{
echo '<span>źle</span';
}
 
}
 
$tablica = array(1 => 'poniedzialek',2 => 'wtorek',3 => 'sroda',4 => 'czwartek',5 => 'piatek',6 => 'sobota',7 => 'niedziela');
shuffle($tablica);
 
 
 
echo 'Jaki dziś mamy dzień tygodnia ?<form action="" method="POST" onsubmit="return toWin(this)"><select name="dzien">';
 
foreach($tablica as $nr_dzien => $nazwa_dzien)
{
echo '<option value="'.$nr_dzien.'">'.$nazwa_dzien.'('.$nr_dzien.')</option>';
}
echo '</select>&nbsp;<INPUT type="submit" value="Potwierdź"></form>';
?>
[PHP] pobierz, plaintext 

ale zauważyłem że coś nie działa i wyszło że problem nr_dzien zawiera liczbe ale nie klucz tablicy tylko tak jak by przy każdym wywołaniu forecha zwieksza o +1 ,co zrobić aby w option value wstawić nr klucza ?

[PHP] pobierz, plaintext 
$tablica = array(1,2,3,4,5,6,7);
shuffle($tablica);
$dni = array(1 => 'Poniedziałek', 'Wtorek', 'Środa', 'Czwartek', 'Piątek', 'Sobota', 'Niedziela');
foreach($tablica as $k => $v)
{
  if($k > 0) echo '<br />';
  echo $v . '/' . $dni[$v]; # format: 1/Poniedziałek
}
[PHP] pobierz, plaintext 

o super działa , wielkie dzięki (IMG:style_emoticons/default/smile.gif)
tylko jeszcze takie pytanko.. skrypt ma słuzyć autoryzacji , tzn co 45min jego zadaniem jest pojawianie się , wadą tego jest to że użytkownik może sobie w innej zakładce ustawić autoodświeżanie (już po wysłaniu forma) więc cały co 45 min automatycznie będzie miał autoryzowane .. jakiś pomysł jak zrobić aby takie rozwiązanie nie przeszło (IMG:style_emoticons/default/questionmark.gif)

hmm wpadłem na coś takiego:

[PHP] pobierz, plaintext 
<?php
 
if(isset($_POST['dzien_'.$_SESSION['ll'].'']))
{
 
$dzien = abs($_POST['dzien_'.$_SESSION['ll'].'']);
 
if($dzien == date('w'))
{
echo '<span>poprawny</span>';
}
else 
{
echo '<span>źle</span';
}
 
}
 
$tablica = array(1,2,3,4,5,6,7);
shuffle($tablica);
$dni = array(1 => 'Poniedziałek', 'Wtorek', 'Środa', 'Czwartek', 'Piątek', 'Sobota', 'Niedziela');
 
$losowa = rand(1,100);
$_SESSION['ll'] = $losowa;
 
echo 'Jaki dziś mamy dzień tygodnia ?
<form action="" method="POST" onsubmit="return toWin(this)">
<select name="dzien_'.$losowa.'">';
 
foreach($tablica as $k => $v)
{
echo '<option value="'.$v.'">'.$dni[$v].'</option>';
}
echo '</select>&nbsp;<INPUT type="submit" value="Potwierdź"></form>';
 
?>
[PHP] pobierz, plaintext 

działac działa czyli po odświeżeniu strony nic się nie dzieje (taki post nie istnieje ;]) ale czy to jest poprawnie , nie będzie komplikacji.. (chodzi mi dokładnie o to:

[PHP] pobierz, plaintext 
if(isset($_POST['dzien_'.$_SESSION['ll'].'']))
$dzien = abs($_POST['dzien_'.$_SESSION['ll'].'']);
[PHP] pobierz, plaintext