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

pozdrawiam

Przeczytaj najpierw co zwraca funkcja rand() a czego w argumentach oczekuje in_array().

Przenoszę na Przedszkole.

taki testowy kod działa

[PHP] pobierz, plaintext 
<?php
 
$tab = array(1,2,3);
 
$losowa = rand(1,3);
 
if(in_array($losowa, $tab))
    echo 'znalazlem ' . $losowa;
else echo 'nie znalazlem';
 
?>
[PHP] pobierz, plaintext 

in_array zwraca wartość "boolową"

in_array() zakłada, że przekażesz element tablicy i tablcię. I w teście faktycznie tak robisz.
A w poprzednim przykładzie? Sprawdzasz istnienie elementu $this->tmp w tablicy $this->wylosowana. Tylko czy na pewno to jest tablica? Nie.

Mike zobacz na początku klasy mam
public $wylosowana = array();
co tu jest nie tak?

A co masz później? W wywołaniu.
$kcLotto->setWylosowane();

Poza tym metody totoLotto::setTmp() i totoLotto::setWylosowane() są pozbawione sensu.
Obie wywalą komunikat Notice: Undefined variable. W pierwszym przypadku korzystasz ze zmiennej $tmp, której nie ma a w drugim jest to $wylosowana. Zmienne te powinny przyjść do metod jako argumenty.

dziekuje Mike. racja. Pływam w OOP ale podoba mi sie


w sumie na końcu dodałem to wywołanie. Chyba tego do końca nie rozumiem. ubzdurałem sobie, że każdą zmienna zwykła czy tablicową muszę zainisjować poprzez właśnie taką funkcję "set".

czyli moge robić tak: ?
np.
public $zmienna;

a potem odwoływać się do niej $this -> zmienna;

myślałem że funkcja set pozwoli mi dopiero na takie odwoływanie się bez $
może to głupie ale ucze się OOP.

jeszcze raz dzięki



pozdrawiam.

Uczenie się to zachowanie dalekie od głupiego. Nikt nie rodzi się z umiejętnością programowania obiektowego.
Zresztą każdy uczy się całe życie. Ja dla przykładu siedzę teraz i też się uczę i pewnie popełniam błedy porównywalne do Twoich.

Przenoszę z powrotem na OOP. Bo jednak faktycznie tu masz problemy.

Mike a czy byłby sens gdybym zmienne tmp i wylosowana powołał do życia w konstruktorze?
function __construct()
{
$this -> zmienna = $zmienna;
$this -> wylosowana = $wylosowana;
}
czy w ogóle nie ma takiej potrzeby...

Możesz mi tylko jeszcze zobrazować o co chodzi z tymi ::
?

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ć.