[inny]Własny framework z auto wstrzykiwaniem Opcje

[primosz67]

Witam,

Hobbistycznie napisałem własny framework. Czy ktoś mógłby się wypowiedzieć co o nim sądzi.

https://github.com/primosz67/tahona-spark-mvc

[nospor]

Skoro piszesz teraz cos od podstaw, to wypadaloby isc z duchem czasu i pisac to juz w php7 a nie stac w starociu.

Unikaj zbednych zagniezdzen
Nie

[PHP] pobierz, plaintext 
 public function getRootNamespace() {
        if ($this->hasMultipleNamespaces()) {
            return $this->namespace[0];
        } else {
            return $this->namespace;
        }
    }
[PHP] pobierz, plaintext 

a:

[PHP] pobierz, plaintext 
 public function getRootNamespace() {
        if ($this->hasMultipleNamespaces()) {
            return $this->namespace[0];
        }
 
        return $this->namespace;
    }
[PHP] pobierz, plaintext 

Unikaj zbednych linii kodu
Nie

[PHP] pobierz, plaintext 
 public function getNamespaces() {
        if ($this->hasMultipleNamespaces()) {
            return $this->namespace;
        } else {
            return array($this->namespace);
        }
    }
[PHP] pobierz, plaintext 

a:

[PHP] pobierz, plaintext 
 public function getNamespaces() {
        return (array)return $this->namespace;
    }
[PHP] pobierz, plaintext 

ps: przenosze

edit:
zaprzyjaznij sie z PSR
zaprzyjaznij sie z UNIT TESTS - teraz to podstawa


return isset($obj) && false === is_null($obj);
czemu poprostu nie:
return !is_null($obj);
?

return is_null($obj) || false === isset($obj);
czemu poprostu nie
return is_null($obj);
?

[PHP] pobierz, plaintext 
 
    public static function getYearsBetween(DateTime $from, DateTime $to) {
        $da = new DateTime();
        $interval = date_diff($from, $to);
        return $interval->y;
    }
[PHP] pobierz, plaintext 

Po co tworzysz obiekt $da, ktory nigdzie nie jest wykorzystywany? Polecam dobry edytor do pracy, ktore wylapuje takie bledy.
Rowniez przydatny moze byc statyczny analizer, ala PHAN

isExist()
a moze poprostu exist()?

Po co tworzysz kilka funkcji, ktore robia dokladnie to samo?

[PHP] pobierz, plaintext 
 public static function isExist($directoryPath) {
        return file_exists($directoryPath) && is_dir($directoryPath);
 }
 
public static function isDirExist($dir) {
        return is_dir($dir);
    }
 
[PHP] pobierz, plaintext 

Na bakier tez jestes z warunkow logicznych

[PHP] pobierz, plaintext 
 public static function div($a, $b) {
        if (empty($a) || empty($b) || $b == 0 || $a == 0) {
            return 0;
        }
        return $a / $b;
    }
[PHP] pobierz, plaintext 

Przeciez to:
if (empty($a) || empty($b) || $b == 0 || $a == 0)
to jest poprostu to
if (empty($a) || empty($b))

Poza tym jak juz tworzysz funkcje do dzielenia to od kiedy 2/0 daje 0?

Jaki sens maja takie funkcje?

[PHP] pobierz, plaintext 
 public static function upper() {
        return function ($x) {
            return strtoupper($x);
        };
    }
 
public static function trim($string) {
        return trim($string);
    }
[PHP] pobierz, plaintext 

I cala masa innych podobnych?

Dalej juz mi sie nie chce. Idee moze i miales sluszna, ale wykladasz sie na drobnostkach. Jesli pisales to tylko dla nauki to i moze ok. Dla innych celow srednio sie nadaje

[emp]

Unikaj zbednych linii kodu
Nie

[PHP] pobierz, plaintext 
 public function getNamespaces() {
        if ($this->hasMultipleNamespaces()) {
            return $this->namespace;
        } else {
            return array($this->namespace);
        }
    }
[PHP] pobierz, plaintext 

a:

[PHP] pobierz, plaintext 
 public function getNamespaces() {
        return (array)return $this->namespace;
    }
[PHP] pobierz, plaintext 

No tu mnie zaskoczyłeś że wolisz taki nieczytelny "syf" z dwoma returnami w jednej instrukcji. Ja tam stanowczo wole więcej linii kodu jeśli tylko objaśnia to lepiej sam kod i powoduje że kod staje się czytelniejszy. Nie wiem czy dalej cie lubię. Musze to teraz przemyśleć