Witam

mam pytanie co sądzicie o takim sposobie rozwiązania na globalną konfigurację aplikacji

Klasa kontrolera konfiguracji

[PHP] pobierz, plaintext 
<?php
 
namespace Core;
 
final class Config {
 
    public static $config = array();
 
    public static function getValue($module, $var) {        
        if (isset(static::$config[$module][$var])) {
            return static::$config[$module][$var];            
        }
        else {
            include_once('conf/conf.php');
            static::$config = $configs;  
            return static::$config[$module][$var];
        }        
    }
 
}
[PHP] pobierz, plaintext 

plik z konfiguracją

[PHP] pobierz, plaintext 
<?php
 
 switch (ENVIRONMENT) {
        case 'development':
            $configs['app']['url_website']              = 'http://testowa.rr/';
            $configs['app']['debug']                    = 'on';  
            break;
        case 'production':
            $configs['app']['url_website']              = 'http://mojadomena.pl/';
            $configs['app']['debug']                    = 'off';
 
            break;
        default:
            exit('Aplikacja nie działa poprawnie, ustaw tryb pracy.');
}
 
    $configs['app']['url_ogloszenia']                   = $configs['app']['url_website'].'ogloszenia/';
    $configs['app']['router_rejestruj']                 = 'rejestruj.html';
    ...
 
[PHP] pobierz, plaintext 

Użycie konfiguracji w klasach

[PHP] pobierz, plaintext 
<?php
    namespace Controllers;
 
    use Helpers\Url;
    use Core\Config;
 
    public function mojeKonto()
    {
    ...
    Url::redirect(Config::getValue('users','router_mojekonto'));  
   }
 
[PHP] pobierz, plaintext 

Kilka uwag:
- Metoda getValue() nie sprawdza czy klucz istnieje po wczytaniu konfiguracji (blok else).
- Zmieniłbym nazwę getValue() na krótsze get() - wiadomo, że chodzi o wartość dla klucza.
- Rozbijasz parametry na $moule i $var - IMHO zbędne, ale jeśli już tak robisz, to każdy moduł powinien być innym plikiem konfiguracyjnym.
- W pliku konfiguracyjnym nie definiujesz na początku zmiennej $configs.
- W ogóle to zwracaj tablicę bez definiowania nazwy zmiennej (patrz przykład poniżej) i przypisuj ją przez

[PHP] pobierz, plaintext 
static::$config = require_once('conf/conf.php');
[PHP] pobierz, plaintext 

- Proponuję rozszerzyć to o poszukiwanie w ścieżce, tak aby można było pobrać pojedynczy, zagnieżdżony element np:

[PHP] pobierz, plaintext 
Config::get('users.default.account.name');
[PHP] pobierz, plaintext 

config.php

[PHP] pobierz, plaintext 
<?php
return [
  'default' => [
    'account' => [
      'name' => 'test',
      'email' => 'test@example.pl',
    ],
    'other' => [
      'foo' => 'bar',
    ],
  ],
];
[PHP] pobierz, plaintext 

- Ostatnia rzecz, proponuję pójść w kierunku DI zamiast globalnego singletona, dzięki takiemu podejściu testowanie będzie o wiele łatwiejsze.

Podoba mi się poszukiwanie po ścieżce, wizualnie lepiej wygląda, podobało mi się bardzo w Laravelu a nie zaimplemetowałem tego u siebie nie wiem czemu.

Co do Di, nie użyłem w tym przypadku bo łatwiej było mi zaimplementować Static, nigdy nie pisałem testów więc nie widziałem w tym problemu.

Jednak jak ktoś wytknie palce lepiej widać, nazwa zmiennej tablicy faktycznie jest zbędna.

i co najważniejsze nie rób tego tak:

[PHP] pobierz, plaintext 
else {
            include_once('conf/conf.php');
            static::$config = $configs;  
            return static::$config[$module][$var];
        } 
[PHP] pobierz, plaintext 

Bo uzależniłeś teraz Twój config od jedynej słusznej implementacji z tablicą w lokalizacji conf/conf.php, co pewnie nawet nie działa bo będzie szukał tego lokalnie tam gdzie będziesz tego używać.

Tam jest względna ścieżka ale uprościłem link do minimum na potrzeby prezentacji ale fakt niepotrzebnie, ...

... wezmę się za klasę poprawię ją i zaprezentuję po liftingu.

Mi tam od razu nasunęło się 10 pytań "a co jeśli..." ;)

- jak masz kilka plików cfg, to przydałaby się metoda która po prostu wgra te pliki (uwzględniając to, co zrobić gdy dany klucz już istnieje)
- jeśli już masz kilka plików, to mogą być one w różnych formatach np ini, json itd.
- skoro już możesz pobierać wartości, to czasami przydaje się wartość domyślna która zostanie zwrócona gdy nie ma odpowiedniego klucza
- a jeśli już możesz pobierać coś z pliku, to możesz go również generować/nadpisać

To tak jakbyś chciał to rozbudować w przyszłości...

Przysiadłem dziś do klasy i zastosowałem się do wskazówek, jak to wygląda po zmianach Waszym zdaniem.

plik z konfiguracją ogloszenia.php

[PHP] pobierz, plaintext 
<?php
 
 /*
  * Ogłoszenia
  */
 return [
     'base_path_img' => 'dir/users/ogloszenia/',
     'base_url_img' => 'dir/users/ogloszenia/',
     'img' => [
         'path_thumbs' => 'dir/user/thumbs/path/id/',
         'path_big_foto' => 'dir/user/big_foto/path/id/',
         'path_mini_foto' => 'dir/user/mini_foto/path/id/',
     ]
 ];
 
[PHP] pobierz, plaintext 

użycie klasy

[PHP] pobierz, plaintext 
<?php
    namespace Controllers;
 
    use Helpers\Url;
    use Core\Config;
 
    public function mojeKonto()
    {
    ...
    $this->cfg = new \Core\Config();
 
     $this->cfg->get('ogloszenia.img.path_thumbs');
     $this->cfg->get('ogloszenia.base_url_img'); 
   }
[PHP] pobierz, plaintext 

nowa klasa

[PHP] pobierz, plaintext 
<?php
 
 namespace Core;
 
 class Config {
 
     public $config = array();
     private $configs = array();
     private $filePath = 'app/config/';
 
     public function __construct() {
 
     }
 
     public function get($var) {
         if (isset($this->config[$var])) {
             return $this->config[$var];
         } else {
             $this->getValue($var);
             return $this->config[$var];
         }
     }
 
     private function getValue($var) {
         $list = explode('.', $var);
         $key = end($list);
         $array = $this->getFile($list);
 
         if (isset($array[$key])) {
             $this->config[$var] = $array[$key];
         } else {
             foreach ($array as $segment) {
                 if (isset($segment[$key])) {
                     $this->config[$var] = $segment[$key];
                 }
             }
         }
     }
 
     private function getFile($list) {
         if (!isset($this->configs[$list[0]])) {
             $configFile = $this->filePath . $list[0] . '.php';
             if (file_exists($configFile)) {
                 $this->configs[$list[0]] = require_once($configFile);
             }  else {
                 throw new Exception('Cannot read the configuration file: ' . $configFile);
             }
         }
         return $this->configs[$list[0]];
     }
 
 }
 
 
[PHP] pobierz, plaintext