DbM Framework - Autorska aplikacja frameworka opartego na wzorcu MVC Opcje

[Malinaa]

Witajcie,
miałem chwilę i usiadłem do autorskiego frameworka opartego na wzorcu MVC.
Chciałbym go dopracować, utworzyć wersje stabilną.

Kod frameworka jest dostępny pod adresem:
https://github.com/artimman/dbmframework

Zakończyłem pierwszy test aplikacji na serwerze zdalnym - powodzeniem.
Wydaje się, że już jest całkiem Ok i ciekawe, czy ktoś się ze mną zgodzi?

[nospor]

$this->translation = json_encode($this->translation());
$trans = json_decode($this->translation, true);

Czyli robisz json_encode raz, tylko po to, by potem za kazdym razem w metodzie trans petryliard razy robic json_decode. I teraz wytlumacz mi prosze, bo naprawde nie kumam, czemu nie mozesz tej tablicy przechowywac jako tablce, tylko musisz ja kodowac a potem dekodowac za kazdym razem. Jaka logika temu przyswiecala? Bo jakas na pewno musiala prawda?

$html .= '<img class="dbm-img-profile rounded-circle" src="' . $this->path() . 'images/avatar/' . $avatar . '">' . "\n"; // PHP_EOL powinno być tu lepsze, ponieważ wykona to samo w każdym systemie

Przeciez tu generujesz kod html... tu w ogole nie trzeba nowej linii od tego zacznijmy. A nawet jesli by byla to co z tego ze serwer ci wygeneruje nowa linie wg systemu serwera, skoro klient ktory dostanie kod html moze byc na milionie innych systemow. wal \n i sobie glowy nie zawracaj

file_put_contents($cachedFile, '<?php class_exists(\'' . __CLASS__ . '\') or exit; ?>' . PHP_EOL . $code);

To samo tutaj, to sa pliki cache, co ci zalezy czy bedzie tam \n czy PHP_EOL? TO nic nie zalatwia

Przy okazji będąc w class Translation poprawiłem else itp. o czym wcześniej pisałeś, chociaż czy klasa jest już optymalna?

Nie wiem gdzie to poprawiles, na githubie zmian nie widze