PHP OOP ::すべてのクラスにアクセス可能なグローバル設定値を保持する

Question

私は、Factoryクラスを使ってオブジェクトのインスタンス化を管理し、コンストラクタに任意の依存オブジェクト（Dependency Injection）を渡しています。

function createBasket() { 
    //pass in dependent objects 
    $apiCon = $this->createAPIConnector($this); 
    $basket = new Basket($this, $apiCon); 
    return $basket; 
} 

私は、よりシンプルで堅牢なユニットテストを可能にするために、（工場以外）の任意のクラス内の「新しい」キーワードを使用しないようにしようとしています、明確なコードなど

さて、私の質問はに関し、多くのクラスが必要とする設定値

$apiURL = 'http://some.api.com'; 
$apiKey = 'abcde12345'; 
$adminEmail = 'someone@site.com'; 

これらの値は、アプリケーションインスタンスごとに一定です。私は現在、シングルトンクラスConfigでそれらを持っていると、彼らは従うことによって、任意のクラス内から到達可能である：

$cfg = Config::getInstance(); 
$address = $cfg->apiURL; 

しかし、このクラスのインスタンスはまだそれを呼び出す任意のクラス内の依存関係ですので、私は合格を考える必要がありますこれをクラスコンストラクタに

function createBasket() { 
    //pass in dependent objects 
    $apiCon = $this->createAPIConnector($this); 
    $cfg = Config::getInstance(); 
    //pass singleton config object to constructor 
    $basket = new Basket($this, $apiCon, $cfg); 
    return $basket; 
} 

...または多分むしろ、コンストラクタ経由よりも、setメソッドを経由して、それらを渡す：

function createBasket() { 
    //pass in dependent objects 
    $apiCon = $this->createAPIConnector($this); 
    $basket = new Basket($this, $apiCon); 
    //pass singleton config object to setter 
    $basket.setConfig(Config::getInstance()); 
    return $basket; 
} 

最良の方法上の任意のガイダンスははるかに高く評価されるだろう。

ありがとう、ジェームズ

Answer 1

これは、グローバルに利用可能なシングルトン設定オブジェクトを持つという目的を破るようです。全体のポイントは、あなたが作るすべてのクラスへのパラメータとしてそれを渡す必要を避けることです。

Answer 2

私はいつも定数として作った。私はこれが "最高の"解決策であるかどうかはわかりませんが、これを見て何度も使用しました。そのconst（静的なvarsの場合も同じですが、私はなぜ他のものより優れているのか聞くことに興味があります）は、あなたがニッキーになりたいならばおそらくオーバーヘッドを減らすクラスをインスタンス化する必要はありません...あなたは、単一のアレイまたはファイル内のすべてを設定し、それを定義し、ファイルを必要とし、それを使用して、私は同様の方法を使用し

SomeClass::MYCONST //to get your config info 

Answer 3

ような何かをする必要がある場合で、その後

class SomeClass 
{ 
    const MYCONS = "APIKEY or Whateva"; 
} 

それに応じて：

$config = array(
    'MYCONST_1'=>'myValue', 
    'USER'=>'username', 
    'PASSWORD'=>'y3ahr1ght' 
); 

foreach($config as $const=>$value){ 
    define($const,$value); 
}