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