[PHP][OOP], klasa obliczania daty urodzenia Opcje

[ostrylg]

Stworzyłem coś takiego :

[PHP] pobierz, plaintext 
<?php
   $data = date(Y);
 
   class ludek{
      public $imie;
      public $wiek;
 
      public function ustal_date(){ 
         global $data;
         $this -> dataU = $this -> data - $this -> wiek;
      }
 
 
   }
 
   $osoba = new ludek;
   $osoba -> imie = Janek;
 
      $osoba -> wiek = 15;
 
   echo 'Data urodzenia tej osoby to : '.$osoba -> ustal_date(wiek).'<br/>';
 
 
?>
[PHP] pobierz, plaintext 

Jest to moja pierwsza klasa, i mam pytanie do Was, co w tym jest nie tak że nie wyświetla mi roku urodzenia ? Jak to skonstruować ?

Ten post edytował ostrylg 4.03.2009, 03:58:05

[dr_bonzo]

1. co tu robi global?
2. nie deklarujesz $this->data, $this->dataU przed pierwsyzm uzyciem
3. poczytaj jak sie zmienne zapisuje ($wiek a nie wiek)
4. nie zwracasz nic z tej funkcji

[Foxx]

[PHP] pobierz, plaintext 
<?php
$data = date(Y); // to generuje notice
$data = date('Y'); // tak jest dobrze
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$osoba -> imie = Janek; // stringi w php ujmujemy w cudzysłów
$osoba -> imie = 'Janek'; // o tak
?>
[PHP] pobierz, plaintext 

Umieszczasz wiek w polu $osoba->wiek a potem odwołujesz się do zmiennej wiek i do tego zapominasz o znaku $

[PHP] pobierz, plaintext 
<?php
$osoba -> wiek = 15;
echo 'Data urodzenia tej osoby to : '.$osoba -> ustal_date(wiek).'<br/>'; // źle
echo 'Data urodzenia tej osoby to : '.$osoba -> ustal_date($osoba->wiek).'<br/>'; // dobrze
?>
[PHP] pobierz, plaintext 

Proponuję też zrezygnować z robienia globalnej daty i ustalania jej poza klasą. Po tych przeróbkach wygląda to tak:

[PHP] pobierz, plaintext 
<?php
class ludek{
     public $imie;
     public $wiek;
 
     public function ustal_date(){ 
        $data = date('Y');
        return $this->dataU = $data - $this->wiek;
 
     }
 
 
  }
 
  $osoba = new ludek;
  $osoba->imie = 'Janek';
  $osoba->wiek = 15;
 
  echo 'Data urodzenia tej osoby to : '.$osoba -> ustal_date($osoba->wiek).'<br/>';
?>
[PHP] pobierz, plaintext 

// edit - zauważ że dodałem return do metody ustal_date

Ten post edytował Foxx 4.03.2009, 04:51:44