Cytat
Nie myślisz chyba o konstruktorze
Myślę o wszystkim. Każdej jednej rzeczy, z nazewnicstwiem zmiennych włącznie.
Jeśli jest to Twój kod, to musisz naprawdę wziąść się do pracy. Jest taki termin jak
kod spagetti. Ten skrypt do niego należy.
1. Trzymasz kod HTML w stałych. Żadna klasa nie powinna zawierać kodu HTML, chyba że jest to specjalna klasa służąca do wyświetlania dnaych w takiej formie. W programowaniu obiektowym chodzi miedzy innymi o to aby odzielić warstwę prezentacji(kod html) od logiki biznesowej(operacji na danych wykonywanych przez php).
Polecam ten artyukł2. Nazewnicstwo zmiennych
...
public $css_1024;
public $css_800;
public $css_768;
public $css_685;
public $css_600;
public $css_568;
public $css_480;
public $css_380;
public $css_320;
public $css_240;
public $css_w3c_re;
public $css_w3c_ver;
public $css_1;
public $css_2;
public $css_3;
public $css_4;
public $css_5;
...
Nazwy tych zmiennych nic mi nie mówią o ich przeznaczeniu. Mają coś wspólnego z css.
Ponadto w tej klasie jest ok. 60 właściwości(
!), nie licząc stałych.
3. Łamanie zasad
KISS oraz DRY4. Ten kod nie ma nic wspólnego z zasadmi
SOLID 5. Kod nie zawiera testów. Ale w sumie nie wiem czy taki kod da się testować...
To po prostu jest kod programowania strukturalego objęty w klasę. Widać nie rozumiesz na czym polega programowanie obiektowe. Klasy traktuje się jako moduły, osobna klasa pobiera dane, osobna je wyświetla... Osobna klasa pobiera dane x a osobna y itd. musisz bardziej zagłębić się w programownaie obiektowe.
Jeśli mi nie wieżysz, zapytaj się innych programistów na tym forum, dlaczego ten kod jest zły.