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]

ale to powinno być po prostu super dobrze.

Z jednej strony nalegasz na super dobrze, a z drugiej strony piszesz metode ktora zwraca albo ID albo komunikat bledu, wszystko jako tekst i czort wie czy to co zwrocilo to id czy moze jednak error i sie upierasz ze tak ma byc (IMG:style_emoticons/default/wink.gif)

Singleton u ciebie bedzie OK bo caly kod nadal nie jest super dobrze.

No ale jak sie upierasz to w FrameworkClass w metoddzie model() masz
return new $modelNamespace();
tam wiec do constructora masz przekazac $database
A skad FrameworkClass ma miec $database? No to widze kontroller dziedziczy po FC wiec w RoutClass masz przekazac do kontrolera w konstruktorze $database.
A skad RoutClass ma miec $database? No widze ze w index.php tworzysz obiekt RoutClass wiec tam masz przekazac jeszcze $database

I tak z kazda sciezka gdzie $database jest wykorzystywane

[PHP] pobierz, plaintext 
 
$database = new DatabaseClass();
new RoutClass($database);
[PHP] pobierz, plaintext 

Albo zaimplementuj Registry, wloz raz $database do registry i mozesz pominac pare krokow.
Albo zastosuj singleton (IMG:style_emoticons/default/wink.gif)

edit: wlasnie sobie uswiadomilem ze i tak ta cala sciezka z $database co ci teraz podalem jest o kant 4 liter skoro u ciebie kazdy model dziedzicy po DatabaseClass a ta w konstruktorze na dzien dobry tworzy nowe polaczenie z baza
Dlatego nic juz niekombinuj, tylko utworz ten singleton na klasie DatabaseCLass i problem z glowy
Tylko zeby zrobic z tego Singleton, to wywal to dziedziczenie po PDO co widze dodales ostatnio w klasie DatabaseClass rob normalnie new PDO itp. Oczywiscie okraszone juz Singletonem


edit2:
A tu masz pokaze ci jeszcze jak sie pozbywac tych nieszczesnych zagniezdzen
Aktualnie masz taka metode (przeczytaj komentarze co tam wlozylem)

[PHP] pobierz, plaintext 
    public function queryExecute(string $query, ?array $params = [], bool $reference = false): bool
    {
        try {
            $this->result = $this->prepare($query);
 
            if (empty($params)) {
                return $this->result->execute();
            } else {
                // PO CO TEN ELSE? Przeciez IF mial RETURN wiec ELSE NIEPOTRZEBNY
                $first = array_key_first($params);
 
                if (is_string($first)) {
                    // PO CO ten dlugi kod w tym IF? Nie lepiej zrobic negacje i w IF tylko wlozysz jeden prosty return?
                    foreach ($params as $key => &$value) {
                        is_int($value) ? $type = PDO::PARAM_INT : $type = PDO::PARAM_STR;
 
                        if (!$reference) {
                            $this->result->bindValue($key, $value, $type);
                        } else {
                            $this->result->bindParam($key, $value, $type);
                        }
                    }
 
                    return $this->result->execute();
                }
 
                return $this->result->execute($params);
            }
        } catch (PDOException $exception) {
            throw new DbmException($exception->getMessage(), $exception->errorInfo[1]);
        }
    }
[PHP] pobierz, plaintext 

A tu po poprawkach, Zniknely dwa zbedne zagniezdzenia i od razu czytelniej.

[PHP] pobierz, plaintext 
    public function queryExecute(string $query, ?array $params = [], bool $reference = false): bool
    {
        try {
            $this->result = $this->prepare($query);
 
            if (empty($params)) {
                return $this->result->execute();
            }
 
            $first = array_key_first($params);
 
            if (!is_string($first)) {
                return $this->result->execute($params);
            }
 
            foreach ($params as $key => &$value) {
                is_int($value) ? $type = PDO::PARAM_INT : $type = PDO::PARAM_STR;
 
                if (!$reference) {
                    $this->result->bindValue($key, $value, $type);
                } else {
                    $this->result->bindParam($key, $value, $type);
                }
            }
 
 
            return $this->result->execute();
 
        } catch (PDOException $exception) {
            throw new DbmException($exception->getMessage(), $exception->errorInfo[1]);
        }
    }
[PHP] pobierz, plaintext