[PHP]funkcja obliczajaca wiek Opcje

[pela222]

Cześć,

mam taki plik:

[PHP] pobierz, plaintext 
class Person
{
    /**
     * @var bool
     */
    private $dob = '';
    private $age = '';
 
   /**
     * @param string $dob
     */
    public function setDob($dob)
    {
        $this->dob = $dob;
    }
 
    /**
     * @return string
     */
    public function getDob()
    {
        return $this->dob;
    }
}
[PHP] pobierz, plaintext 

Chialem zbudować funkcje dla potrzeb oblczenia wieku osoby na podstawie daty urodznia. Kiedyś używałem czegoś na kształt poniżej ale nie potrafie tego użyć w strukturze funkcji / klas.

[PHP] pobierz, plaintext 
 public function setAge($age ){
        $now = new DateTime(); //dzisiejsza data
	//echo $now->format('Y-m-d');
	$date = new DateTime($dob); //format daty dla potrzeb obliczenia wieku osoby na podstawie dob
	//echo $dob1->format('Y-m-d');
        $this->age = $now->diff($date); //oblicz wiek...odwołąnie do wieku w formacie echo '.$age->y.'
    }
 
 
    public function getAge(){
        return $this->age;
    }
[PHP] pobierz, plaintext 

Dziekuje z góry za pomoc jak to zrobić.

[Crozin]

1. W komentarzu podajesz, że Person::$dob jest typu bool po czym przypisujesz domyślnie pustego stringa? PHP na takie coś pozwala, ale wyzbądź się takich paskudnych praktyk.
2. Metoda nazywa się setAge() więc raczej powinna przyjąć wiek jako liczbę, a nie datę urodzenia (setDateOfBirth()).
3. Przecież na dobrą sprawę ten kod jest po prostu do przekopiowania w klasę (pomijając dwa powyższe punkty).

[bobek358]

Po pierwsze zobacz co zwróci $now->diff($date); - będzie to tablica w której różnica ilości dni będzie w kluczu: days.

[PHP] pobierz, plaintext 
$interval = $datetime1->diff($datetime2);
echo $interval->format('%R%a days');
[PHP] pobierz, plaintext 

i tutaj błąd:

[PHP] pobierz, plaintext 
$date = new DateTime($this -> dob);
[PHP] pobierz, plaintext 

[nospor]

Po pierwsze zobacz co zwróci $now->diff($date); - będzie to tablica w której różnica ilości dni będzie w kluczu: days.

@bobek.... przeciez autor w komentarzu wyraźnie napisal jak sie dobrac do tego diff.... czytamy czytamy i jeszcze raz czytamy, potem sie wymądrzamy

[aniolekx]

dodam swoje 3 grosze ;p

data urodzenia powinna byc przekarzana w kontruktorze, a setter dla niej usuniety. W kontruktorze powinno by takze wywolanie dla metody liczacej wiek.

[bobek358]

@bobek.... przeciez autor w komentarzu wyraźnie napisal jak sie dobrac do tego diff.... czytamy czytamy i jeszcze raz czytamy, potem sie wymądrzamy

Racja, nie doczytałem

[pela222]

Hej,
dziękuje ze szybką odpowiedz. Niestety z tego co widzę to wszyscy trzej już wiecie jakie jest rozwiązanie a ja dalej nic nie rozumiem. Doceniam uwagi typu:

data urodzenia powinna byc przekarzana w kontruktorze, a setter dla niej usuniety

,ale ponieważ siadam do php bardzoooo sporadycznie to niestety nic mi to nie mówi.
Podpowiedz

3. Przecież na dobrą sprawę ten kod jest po prostu do przekopiowania w klasę

jest bardzo czytelna ale oczywiście że:

[PHP] pobierz, plaintext 
class Person
{
        private $now = new DateTime(); //dzisiejsza data //echo $now->format('Y-m-d');
	private $date = new DateTime($dob);
}
[PHP] pobierz, plaintext 

jest nieprawidłowe...wiec i ta oczywista oczywistość dla was nie jest oczywista dla Przedszkolaka.
Oczywiście mógłybym zacząć czytać manual php i po 5-8 godzinach dojść za pomocą waszych uwag do rozwiązania i przy okazji nauczyć się wielu innych rzeczy związanych z php.
Czy jest szansa na poprawny kod z komentarzem co sie gdzie dzieje.
Dziękuje za waszą pomoc i zrozumienie.

[ctom]

a nie łatwiej skorzystać z czegoś co już istnieje ? np. biblioteki Carbon

potem tylko :

[PHP] pobierz, plaintext 
    $person =  \Carbon\Carbon::createFromDate(1999,2,30);
    $now =  \Carbon\Carbon::now();
 
    $val = $now->diffInYears($person);
[PHP] pobierz, plaintext 

[pela222]

a nie łatwiej skorzystać z czegoś co już istnieje ? np. biblioteki Carbon

potem tylko :

[PHP] pobierz, plaintext 
    $person =  \Carbon\Carbon::createFromDate(1999,2,30);
    $now =  \Carbon\Carbon::now();
 
    $val = $now->diffInYears($person);
[PHP] pobierz, plaintext 

Wolałbym uniknąć strzelania z armaty do wróbla. Potrzebuje zrobić to dla dwóch zmiennych.
Jeżeli jest chętny na wskazanie mi rozwiązania bedę wdzięczny.
Pozdrawiam.

[bobek358]

Taki szybki kod:

[PHP] pobierz, plaintext 
<?php
 
$object = new Person('1987-01-01');
echo $object->getAge();
 
class Person
{
 
    private $dob = '';
    private $age = 0;
 
    public function __construct($dob)
    {
        $this->dob = $dob;
        $this->setAge();
    }
 
    public function getDob()
    {
        return $this->dob;
    }
 
    public function getAge()
    {
        return $this->age;
    }
 
    public function setAge()
    {
        $now = new DateTime();
        $date = new DateTime($this->dob);
        $interval = $now->diff($date);
        $this->age = $interval->y;
    }
 
}
[PHP] pobierz, plaintext 

[pela222]

Dziękuje wszystkim za uwagi. Ostateczne rozwiązanie wyglada tak:

[PHP] pobierz, plaintext 
class Person
{
    private $dob = '';
    private $age = '';
 
 
     /**
     * @param $dob
     */
    public function setDob($dob)
    {
        $this->dob = $dob;
        $this->setAge(); //to dodałem aby obliczyć wiek
    }
    /**
     * @return
     */
    public function getDob()
    {
        return $this->dob;
    }
 
    /**
     * @param string $age
     */
    public function setAge(){
        $now = new DateTime(); //dzisiejsza data
	//echo $now->format('Y-m-d');
	$date = new DateTime($this->dob); //format daty dla potrzeb obliczenia wieku osoby na podstawie dob
	//echo $dob1->format('Y-m-d');
        $interval = $now->diff($date);
        //EDIT nr2
        $this->years = $interval->y;  //oblicz wiek...odwołąnie do wieku w formacie echo '.$age->y.'
        $this->months = $interval->m;
        $this->days = $interval->d;
        $this->age = $this->years .' Year(s)';
        //kiedy miesiace a kiedy lata
        if($this->years === 0) {
            $this->age = $this->months .' Month(s)'; 
            if($this->months === 0) {
                $this->age = $this->days .' Day(s)'; 
            }
        }
    }
    /**
    * @return string
    */
    public function getAge(){
        //EDIT nr1
        if (($this->age > 21 && ($this->age % 10 == "2" || $this->age % 10 == "3" || $this->age % 10 == "4")) || $this->age < 5) { 
        $koncowka = "lata"; 
        } else { 
        $koncowka = "lat"; 
        }  
    return $this->age . ' ' . $koncowka;
    }
}
[PHP] pobierz, plaintext 

Możliwe że bede chciał znaleść udoskonalenie tego obliczania.
1) Dla osoby gdzie wiek jest <0 żeby policzyć miesiące
2) wpisanie lat / lata np. 31 lat, 32 lata etc (gdzieś takie coś widziałem wiec może jutro poszukam i zobacze czy ogarniam temat

===EDIT nr1====
Punkt nr 2 już znalazłem i dodałem Happy hours

===EDIT nr2===
Zrobiłęm punkt nr1 czyli obliczanie miesiecy dla osoby o mniej niz 1 rok życia. I jest ok.

PYTANIE
Ale mam jeden problem. Jeśli mam osobę bez daty urodzenia to mi oblicza że ma 2015 lat. Gdzie jest problem.
Jak zwykle dziękuje za podpowiedzi.
Pozdrawiam.

Ten post edytował pela222 5.08.2015, 00:56:25

[nospor]

Ale mam jeden problem. Jeśli mam osobę bez daty urodzenia to mi oblicza że ma 2015 lat. Gdzie jest problem.

Jak to mówią: między krzeslem a monitorem
Tak ciezko sie domyslic, ze gdy nie ma daty urodzenia, to nie wykonywac zadnych obliczen? No wiesz, masz uzyc IF.