[PHP]PHP OOP, Początki Opcje

[Codeleft]

Cześć wszystkim,
zaczynam przechodzić na obiektowe pisanie skryptów. Z racji tego, że jestem typowym samoukiem nie do końca jestem pewien czy rozumiem pisanie w klasach.
Obiło mi się o uszy, że można pisać w klasach ale nie rozumieć obiektowości.

Poniżej podsyłam klasę kalendarza, która wczoraj napisałem. Można ją zaimplementować praktycznie bez grzebania w Htmlu.

[PHP] pobierz, plaintext 
<?php
/*
#Php calendar by Codeleft
#Date: 2021-03-21
#Class
*/
class calendar
{
    private $week_day;
    private $month_day;
    private $month_days;
    private $month;
    private $true_month;
    private $year;
    private $daysArray;
    private $monthsArray;
    private $calendarArray;
    private $first_month_day;
    private $side_blocks;
    private $side_blocks_left;
    private $side_blocks_right;
    private $calendarBody;
    private $calendarHead;
    private $calendar;
    private $dayname;
 
    function __construct()
    {
        //Numeric week day
        $this->week_day = date('w');
        //Numeric day of the month
        $this->month_day = date('d');
        //Week days names array
        $this->daysArray = array('Poniedziałek','Wtorek','Środa','Czwartek','Piątek','Sobota','Niedziela');
        //Numeric Month
        $this->true_month = date('m');
        $this->month = date('m');
        if ($this->month <= 9) {
            $this->month=+$this->month;
        }
        //Total month days num
        $this->month_days = date('t');
        //Months Array
        $this->monthsArray = array('Styczeń','Luty','Marzec','Kwiecień','Maj','Czerwiec','Lipiec','Sierpień','Wrzesień','Październik','Listopad','Grudzień');
        //Year
        $this->year = date('Y');
        //Block Calculations
        $this->first_month_day = date("w",strtotime('now') - (($this->month_day -1) * (24 * 60 * 60)));
        $this->side_blocks = 35 - $this->month_days;
        $this->side_blocks_left = $this->first_month_day -1;
        $this->side_blocks_right = $this->side_blocks - $this->side_blocks_left;
    }
 
    public function calendarData($key)
    {
        $this->calendarArray = array(
            'week_day_num' => $this->week_day,
            'week_day_nam' => $this->daysArray[$this->week_day],
            'month_day_num' => $this->month_day,
            'curr_month_num' => $this->month,
            'curr_month_days_num' => $this->month_days,
            'curr_month_nam' => $this->monthsArray[$this->month-1],
            'curr_year' => $this->year
        );
 
        return $this->calendarArray[$key];
    }
    public function calendarHeader($arrow)
    {
        $this->calendarHead = '<div class="calendar-header">';
        if ($arrow == 'on') {
            $this->calendarHead .= '<div class="left-arrow"><</div>';
        }
        $this->calendarHead .= '<div class="clendar-header-val" data-current-date="'.date('Y-m-d').'">';
        $this->calendarHead .= ' '.$this->calendarData('curr_month_nam');
        $this->calendarHead .= ' '.$this->calendarData('curr_year');
        $this->calendarHead .= '</div>';
        if ($arrow == 'on') {
            $this->calendarHead .= '<div class="right-arrow">></div>';
        }
        $this->calendarHead .= '</div>';
        return $this->calendarHead;
    }
 
    private function calendarWeekDays()
    {
        foreach ($this->daysArray as $day) {
            $this->dayname .= '<div class="day-cont day-names">'.$day.'</div>';
        }
        return $this->dayname;
    }
 
    public function calendarBody()
    {
        //Calendar cont
        $bock_num = 0;
        $this->calendarBody .= '<div class="calendar-body">';
        $this->calendarBody .= $this->calendarWeekDays().'<br />';
        //Left empty boxes
        for ($b = 1 ; $b <= $this->side_blocks_left ; $b++) {
            $this->calendarBody .= '<div class="day-cont outer-cont">-</div>';
            $bock_num++;
        }
        //Filled boxes
        for ($i = 1 ; $i <= $this->month_days ; $i++) {
            $con_date = $this->year.'-'.$this->true_month.'-'.$i;
 
            if (( ( $bock_num%7 ) == 0 ) && ( $bock_num > 0 )) {
                $this->calendarBody .= '<br />';
            }
 
            if ($i == $this->month_day) {
                $this->calendarBody .= '<div class="day-cont current-day" data-date="'.$con_date.'"><b>'.$i.'</b></div>';
            } else if ( (date('w',strtotime($con_date)) == 0) || (date('w',strtotime($con_date)) == 6) ) {
                $this->calendarBody .= '<div class="day-cont weekend-day" data-date="'.$con_date.'">'.$i.'</div>';
            } else {
                $this->calendarBody .= '<div class="day-cont" data-date="'.$con_date.'">'.$i.'</div>';
            }
 
            $bock_num++;
        }
        //Right empty boxes
        for ($b = 1 ; $b <= $this->side_blocks_right ; $b++) {
            $this->calendarBody .= '<div class="day-cont outer-cont">-</div>';
            $bock_num++;
        }
        $this->calendarBody .= '</div>';
 
        return $this->calendarBody;
    }
 
    public function createCalendar($header,$arrow,$body)
    {
        $this->calendar .= '<div class="calendar">';
 
        if ( $header == 'on' ){
            $this->calendar .= $this->calendarHeader($arrow);
        }
        if ( $body == 'on' ){
            $this->calendar .= $this->calendarBody();
        }
 
        $this->calendar .= '</div>';
 
        return $this->calendar;
    }
 
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
/*
#Php calendar by Codeleft
#Date: 2021-03-21
#Css
*/
.calendar{
    display: inline-block;
    padding: 10px;
    background-color: white;
    border: 1px solid #78909c;
    color: #455a64;
    border-radius: 3px;
}
.calendar-header{
    font-family: Arial, Helvetica, sans-serif;
    color: #78909c;
    margin-bottom: 10px;
    border: 1px solid #e6e6e6;
    border-radius: 3px;
    background-color: white;
    padding: 15px;
    text-align: center;
    font-weight: bold;
}
.clendar-header-val{
    display: inline-block;
}
.left-arrow{
    float: left;
    cursor: pointer;
}
.right-arrow{
    float: right;
    cursor: pointer;
}
.calendar-body{
    display: inline-block;
}
.day-cont{
    font-family: Arial, Helvetica, sans-serif;
    display: inline-block;
    width: 100px;
    height: 100px;
    border: 1px solid #e6e6e6;
    color: #999999;
    background-color: white;
    padding: 3px;
    cursor: pointer;
}
.day-cont:hover{
    border: 1px solid #717171;
    color: #5a5a5a; 
}
.current-day{
    background-color: #ff9800;
    color: white;
    border-radius: 10%;
}
.outer-cont{
    border: 1px solid #e6e6e6;
    color: white;
}
.outer-cont:hover{
    cursor: default;
    border: 0;
    color: white;
}
.weekend-day{
    color: #ef5350  !important;
}
.day-names{
    max-height: 20px!important;
    text-align: center;
    font-weight: bold;
    border-radius: 3px;
}
.day-names:hover{
    border: 1px solid #e6e6e6!important;
    color: #8a8a8a!important;
    cursor: default!important;
}
[PHP] pobierz, plaintext 

Chciałbym aby jakaś surowa dusza wytknęła mi błędy i podpowiedziała na co zwrócić uwagę.

Dzięki za konstruktywną krytykę ! (IMG:style_emoticons/default/smile.gif)

[goku800]

Na pierwszy rzut oka jest okej, ale po co przechowujesz elementy HTML w zmiennych? Nie lepiej jest mieć template HTML i w elementach wyświetlać wartości?

[rad11]

Moim zdaniem nie jest okej. Klasa którą utworzyłeś ma dwie odpowiedzialności, powinieneś to co najmniej podzielić. Druga sprawa to większość argumentów które tworzysz w konstruktorze powinny być przekazywane jako parametry wtedy taki obiekt jest bardziej generyczny. Zamiast używania date proponuję używanie DateTime/DateTimeInterface. Dodatkowo trzymaj się jakichś standardów klasy pisze się w formacie: NazwaTwojejKlasy już nie wspominając o nazwach pól . Metody prywatne powinny być pod public i pod protected. Dodawaj dokumentacje i typowanie do metod i pól w klasie tak aby było wiadomo co one maja zwracać.

Ten post edytował rad11 23.03.2021, 09:15:59

[nospor]

Mozesz mi napisac co ten kod

[PHP] pobierz, plaintext 
        $this->month = date('m');
        if ($this->month <= 9) {
            $this->month=+$this->month;
        }
[PHP] pobierz, plaintext 

niby miał robic? Bo na chwile obecna nie ma on zadnego sensu.

[Codeleft]

goku800
Zwróć uwagę, na to, że taplatka by się chyba nie sprawdziła w dwóch różnych miesiącach dlatego "wypluwam" całą konstrukcje kalędarza.

Rad dzięki za odpowiedź!

[HTML] pobierz, plaintext 
Metody prywatne powinny być pod public i pod protected
[HTML] pobierz, plaintext 

• Ustawiam private ponieważ nie chce aby właściwości czy metody były dziedziczone. Takie pierwsze kroki..

[HTML] pobierz, plaintext 
 Druga sprawa to większość argumentów które tworzysz w konstruktorze powinny być przekazywane jako parametry wtedy taki obiekt jest bardziej generyczny.
[HTML] pobierz, plaintext 

• Przekazywać właściwości z konstruktora jako parametry metod a nie bezpośrednio przekazywane do nich? Przepraszam, ne rozumiem pojęcie generyczności w tym kontekście - tutaj wychodzą moje braki. Dziękuje !

[HTML] pobierz, plaintext 
Dodatkowo trzymaj się jakichś standardów klasy pisze się w formacie: NazwaTwojejKlasy już nie wspominając o nazwach pól
[HTML] pobierz, plaintext 

• Czy tutaj masz na myśli PSR? zdecydowanie muszę to nadrobić. Nazwy pól ? tutaj wychodzą moje braki. Dziękuje !

[HTML] pobierz, plaintext 
Dodawaj dokumentacje i typowanie do metod i pól w klasie tak aby było wiadomo co one maja zwracać.
[HTML] pobierz, plaintext 

• Z tym nie będzie najmniejszego problemu.

Ropson, przeglądając forum chyba najbardziej na Ciebie liczyłem (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
 
$this->month = date('m');
            if ($this->month <= 9) {
                $this->month=+$this->month;
           }
[PHP] pobierz, plaintext 

Wywalam początkowe zero z date('m') w przypadku poniżej 10 czyli zamiast 05 dostaje 5 i mogę to wrzucić do metody calendarData() jako klucz tablicy.
Jeśli można zostawić 05 to - tutaj wychodzą moje braki.

Na pewno zastosuje się do waszych uwag.

Od razu by nie zakładać kolejnego tematu nasuwa mi się kolejne pytanie.
Czy każda zmienna/właściwość musi być zdefiniowana na początku klasy i posiadać modyfikator dostępu?
Nie za bardzo mam pomysły jak wykorzystać klasy. Macie może pomysły jak mogę sobie poćwiczyć i zrozumieć zastosowanie bo pisanie klasy samochód o kolorze.... nie za bardzo mnie urządza (IMG:style_emoticons/default/biggrin.gif)

Jeszcze raz dzięki!

[gino]

A co to są za parametry metody: 'no' (i chyba - 'yes')? A czemu nie a i b? Masz typ boolean, true albo false, 0 albo 1.
Osławione i klepane już na tym forum, nie:

Kod

if($a == $b)

tylko:

Kod

if($a === $b)

Szczerze nie sprawdzałem dokładnie, ale czy wszystkie atrybuty są Ci potrzebne, bo w metodzie calendarData można od razu przypisać do klucza odpowiednią wartość bez użycia atrybutu.
Kod html-a w klasie to nieporozumienie, przynajmniej dla mnie. PSR zdecydowanie do poczytania. Co można pisać? Co Ci na myśl przyjdzie.

[Codeleft]

Gino,
odwołując się obiektem to metody klasy w parametrach ustawiasz czy np. nagłówek kalendarza ma być wypisany ? no to 'on' jeśli nie to podaj w parametrze co tam chcesz 'off' (IMG:style_emoticons/default/wink.gif)
Jeśli kod html w klasie to pomyłka to naprawdę nie mam pomysłu jak wykorzystać klasy.

hmm.. aktualnie w moim mini projekcie będę wypisywał taskii na podstawie statusów z bazy ale mogę to zrobić PDO i odpowiednim zapytaniem, nie widzę tutaj potrzeby klas..

[gino]

A kto Ci zabroni napisać własną obsługę baz danych opartą o PDO (tak dla wprawki i do innych celów) ? Jeżeli parametr ma przyjąć jedną z dwóch wartości to true lub false, to intuicyjne.

[nospor]

Wywalam początkowe zero z date('m') w przypadku poniżej 10 czyli zamiast 05 dostaje 5 i mogę to wrzucić do metody calendarData() jako klucz tablicy.
Jeśli można zostawić 05 to - tutaj wychodzą moje braki.

Ach, usuwasz zero haha. Ok. No to zamiast
date('m') i potem cudowania, poprostu uzyj
date('n')
Manual sie klania (IMG:style_emoticons/default/wink.gif)

[Codeleft]

(IMG:style_emoticons/default/wstydnis.gif) Tak jak pisałem -> niedouczenie moje

Podajcie proszę jak najwięcej przykładów klas do nauki.

1) Paginacja
2) Mailer
3) ?
4) ?
5) ?
6) ?
7) ?
8) (IMG:style_emoticons/default/co_jest.gif)

Za to dziękuje bardzo (IMG:style_emoticons/default/guitar.gif)

[viking]

Najpierw zrób może ten kalendarz porządnie i pokaż efekty po przepisaniu.