コンストラクタから値を返すのは悪いですか？

Question

次の単純なクラスコンストラクタを考えてみましょう。 。（私は明らかに参照されているすべてのメソッドを含めておりませんのでご注意

// Initialize User class. 
public function __construct($user_id = NULL) 
{ 
    // If user is loaded (and a user ID is provided) 
    if ($user_id) 
    { 
     // If user is authorized. 
     if ($this->authorized($user_id)) 
     { 
      // Load user information. 
      $this->info = $this->load($user_id); 
     } 
     else 
     { 
      // Return an empty (nonexistent) user. 
      return NULL; 
     } 
    } 

    // If user is loaded (and no user ID is provided) 
    else 
    { 
     // Create a new user. 
     $new_user = create_user(); 

     // Return the new user's ID. 
     return $new_user; 
    } 
} 

私の質問はこれです：？ここで間違った値を返す私の方法がある私の友人は、コンストラクタは、常にNO MATTER WHATオブジェクトを返すべきではないと主張しています。しかし、私がここにレイアウトしたやり方ははるかに簡単で、作業がはるかに簡単です（新しいユーザーを作成する場合は、バットからIDを取得します）。それが間違っている場合は、すぐに）

を彼女/彼の情報へのアクセスを持っていることは、なぜ？なぜこの悪い？

Answer 1

あなたは、単に動作しない何をしようとして、コンストラクタはあなたがnullを返すようにしようとした場合でも、とにかくUserの新しいインスタンスを返します。 。

たとえば、この：

class User { 
    function __construct() { 
    return null; 
    } 
} 
var_dump(new User()); 

が印刷されます：

object(User)#1 (0) { 
} 

http://codepad.org/0IdJydkY

Answer 2

あなたの__construct（）関数shouldnでありますすべての値を返すわけではなく、常に自動的にオブジェクトを返します。これは、特定のものを開始するために使用する必要があります。

コードを別の機能にすることをおすすめします。

この上

詳細はここで読むことができます：Echo Return construct method;

Answer 3

をあなたはユーザーを作成するためにあなたのクラスに静的メソッドを追加することができ、またはnull

を返すために、

public static function createUser() { 
    // do your checks 
    // if valid return instance 
    // return null; 
} 

$user = User::createUser(); 

注：あなたのauthorized()メソッドを静的にする必要があるかもしれません - あなたのクラスの残りの部分に依存します。