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 

[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