[php] dostaję warning, dlaczego? Opcje

[john_doe]

witajcie,

plik prostej klasy

[PHP] pobierz, plaintext 
<?php
    
class totoLotto
{
    public $wylosowana = array();
    var $tmp;
    
    public function setTmp()
    {
        $this -> tmp = $tmp;
    }
 
    public function setWylosowane()
    {
        $this -> wylosowana = $wylosowana;
    }
    
    public function losujLiczby()
    {        
        $i = 0;
        while( $i <= 5 )
        {
            $this -> tmp = rand(1,10);
            if( !in_array( $this -> tmp, $this -> wylosowana ) )
            {
                $this -> wylosowana[$i] = $this -> tmp;
                $i++;
            }
        }
    }
    
    public function wypiszWylosowaneLiczby()
    {
        for($i = 0 ; $i <= 5 ; $i++)
        {
            print $i . ' - ' . $this -> wylosowana[$i] . '<br />';
        }    
    }
    
}
 
?>
[PHP] pobierz, plaintext 

oraz index

[PHP] pobierz, plaintext 
<?php
include('_class.php');
 
$kcLotto = new totoLotto;
 
$kcLotto -> setWylosowane();
$kcLotto -> losujLiczby();
$kcLotto -> wypiszWylosowaneLiczby();
 
 
 
?>
[PHP] pobierz, plaintext 

Warning: in_array() [function.in-array]: Wrong datatype for second argument in C:wampwwwoopkcLotto_class.php on line 24

proszę ogólnie o opinie na temat tego kodu. Jestem samoukiem. Chciałbym dobrze zacząć.
Ogólnie to coś działa ale najpierw wywala mi warninga (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

pozdrawiam

Ten post edytował john_doe 6.01.2009, 21:51:36

[wrzasq]

Potrzeby sam sobie określasz. Ale to, co ty ciągle robisz, już ci wytknął mike. Taka metoda jaką ty przed chwilą napisałeś jest bez sensu (może się przyzwyczailes do globali?). Ty tych zmiennych po prostu nie masz, więc co chcesz przypisać?

[PHP] pobierz, plaintext 
<?php
public function __construct($zmienna, $wylosowana)
{
    $this->zmienna = $zmienna;
    $this->wylosowana = $wylosowana;
}
?>
[PHP] pobierz, plaintext 

Tak jeśli już musisz (aczkolwiek w tym wypadku używanie setTmp() i setWylosowane() będzie chyba odpowiedniejsze niż robienie wszystkiego w konstruktorze.

O :: sobie poczytaj w manualu, nosz proszę, podstaw to jednak sam powinieneś chociaż próbować poszukać.