クラスで使用する「参照渡し」のやり方

Question

クラスを使い始めて間もないので、ご教授願います。
PHPでMySQLからSELECT文でデータを持ってくる記述です。

関数での参照渡しは使うことが出来るので、同じように考え、下記のように記述しました。ただ、loadメソッドの最後に
「return $data;」
と書くと、一応意図する通りに実行できるのですが、これはなぜですか？

「参照渡し」ならばreturnで値を返す必要が無いのでは？と思うのですが、この一行を削ると"$article->load(&$data)"は”FALSE”になるようです。
アンパサンド「＆」の位置が間違っているのでしょうか？


class XXX{
function load(&$data){
$query = "SELECT * FROM　テーブル名";
if(!$result = @mysql_query($query, データベースへの接続)){
echo "失敗";
}
while(@$line = mysql_fetch_array($result, MYSQL_ASSOC)){
array_push($data, $line);
}
return $data;
}
}

// クラスの生成
$obj = new XXX;
// 配列の設定
$data = array();

if($obj ->load($data)){
print_r ($data);
}else{
echo "失敗";
}

naktak · Accepted Answer

戻り値に$dataとするのは不必要ですね。

$obj ->load($data)の結果（つまりSQLが実行できたかできなかったか）を
知りたいだけならば、戻り値はtrue/falseで十分なはずです。

参照渡しとなっている$dataは、あくまでSQLで取得したレコード、
戻り値は、あくまで実行できたかできなかったかの区別。
であると考えれば分かり易いでしょうか？

最後にreturn $data;を追記すると動作する所以は、＃２の方が
回答されている通りです。

wp_ · Answer

>if($obj->load($data)){
これはXXXクラスのload()が返す値をtrue/falseでチェックしています。
returnが無い場合はnullとして判別されるので、判別式が論理否定となりifの中に入っていきません。

function test($arg) {
　return $arg;
}
function testNUll() {
}

if(test(true)) {
　echo "trueだよ";
}
if(test(false)) {
　// この中には入ってこない
}
if(testNull()) {
　// この中にも入ってこない
}


もしload()を使ってデータを読み出すのであれば

$obj->load(&$data);
if($data){
...

とせねばなりません。
関数の成否ではなくデータそのものをチェックすると考えればよいでしょう。

関数の成否を検査するのであれば、関数の中身は全ての動作において場合分けし
それぞれのパターンにおいてreturnを記述するのが良いでしょう。
そうしないとバグが発生やすくなります。

himajin100000 · Answer

<?php
/* それは多分参照渡し関係ない。単純に混乱しているだけ */
class XXX{
function load($a){ //要するに戻り値void。PHPで同評価されるか正式な書は調べてないが


}
}

$obj = new XXX;
if($obj->load(5)){ //戻り値に対して真偽判定しているのであって，5に対する真偽判定ではない
print "TRUE";
}else{
print "FALSE";
}

?>

クラスで使用する「参照渡し」のやり方

戻り値に$dataとするのは不必要ですね。

>if($obj->load($data)){

<?php

この回答への補足

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング