SQL prepareコールの外部クラス化について

Question

外部からprepareCallのクラスをリクエスト毎にnewすると、
同じ変数に代入した際に返り値が上書きされてしまいます。

どのように記述すれば上書きされず、別オブジェクトとして変数に格納できるでしょうか？

abstract class module implements moduleInterface {
	function prepareCall($sql,$name,$val) {
		$this->call = new Sql($sql,$name,$val);
		return;
	}
}

class index extends module {
	function indexRun() {
		$this->prepareCall(
			"SELECT * FROM tbl_news WHERE delete_flg = 0 ORDER BY date DESC LIMIT 0, 3",
			"news",
			array(
			)
		);
		
		$this->prepareCall(
			"SELECT * FROM tbl_info WHERE delete_flg = 0 ORDER BY date DESC LIMIT 0, 5",
			"info",
			array(
			)
		);
		return;
	}
}

上記のように記述した場合、
最初に記述した$this->call->newsに$this->call->infoが上書きされてしまいます。

どなたかご教授ください。

---

追記
すみません、そういえばSqlクラスを書いてませんでした。

class Sql {
	function Sql($sql, $_name="sql", $value) {
		$this->db = new DbAccess();
		$stmt = $this->db->dbh->prepare($sql);
		
		if(!empty($value)) {
			foreach($value as $key => $val) {
				if(gettype($val)=="string") {
					$stmt->bindParam($key+1,$val,PDO::PARAM_STR);
				} elseif(gettype($val)=="integer") {
					$stmt->bindParam($key+1,$val,PDO::PARAM_INT);
				}
				unset($key,$val);
			}
		}
		
		$this->_result = $stmt->execute();
		$this->_primary = $this->db->dbh->lastInsertId();
		$this->$_name = $stmt->fetchAll(PDO::FETCH_ASSOC);
		return;
	}
}
コード

DbAccessクラスはDBに繋いでるだけです。

Comments

[shi_ue]

すんごい割愛っぷりで全然分かりません。moduleクラスで使っているSqlってクラスはどこから来たんでしょうか？

Answer 1

あ、Sqlクラスは関係なかったみたいです。ごめんなさい。

class index extends module {
    function indexRun() {
        $this->call = new Sql(
            "SELECT * FROM tbl_news WHERE delete_flg = 0 ORDER BY date DESC LIMIT 0, 3",
            "news",
            array(
            )
        );

        $this->call = new Sql(
            "SELECT * FROM tbl_info WHERE delete_flg = 0 ORDER BY date DESC LIMIT 0, 5",
            "info",
            array(
            )
        );
        return;
    }
}
```メソッドをそのまま解釈して書き直してみると上記のようになります。
動作が分かりやすいですね。これで上書きされるのが分かる思います。

全然関係ないんですが、Sqlクラスのデフォルト引数の付け方が間違えています。
[デフォルト引数値](http://php.net/manual/ja/functions.arguments.php#functions.arguments.default)
ここを見てみてください。二番目だけ、とかは付けてはいけません。
期待通りの動作をしなくなります。

Comments

[poooooo]

ありがとうございます。
コード間違いもご指摘ありがとうございます。
知りませんでした。

Answer 2

ひょっとして…

$this->{$_name} = $stmt->fetchAll(PDO::FETCH_ASSOC);