Sposób na GLOBALNY konfig aplikacji, Co o tym sądzicie? Opcje

[LowiczakPL]

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 

[vokiel]

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.

Ten post edytował vokiel 14.05.2016, 11:19:55

[LowiczakPL]

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.

Ten post edytował LowiczakPL 14.05.2016, 14:25:02

[com]

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ć.

[LowiczakPL]

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.

Ten post edytował LowiczakPL 24.05.2016, 20:13:38

[!*!]

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...

Ten post edytował !*! 25.05.2016, 10:08:07

[LowiczakPL]

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 

Ten post edytował LowiczakPL 17.06.2016, 19:22:37