PHP 配列について

Question

XXX4.phpでループ処理した配列データをデータベースから取得して、メインのクラスで一覧表示するという処理を行いたく、コーディングしてみて分からなくなった箇所があります

メインのクラスのrequire("XXX4.php");からのコーディングで、implode(",", $l_obj);で配列を文字列として返すようにしてみたんですが、配列の引数ではないとエラー表示されました
Warning: implode(): Invalid arguments passed
$dataは$l_objのList_table(配列)を含んでいるという解釈は間違っているのでしょうか

//メインのクラス
<?php
    require_once("XXX.php");

    if (isset($_POST['name'], $_POST['comment'], $_POST['bbs']) && $_POST['bbs'] == "post") {
        require("XXX1.php");
        $p_obj = new Post();
        $p_obj->name = $_POST['name'];
        $p_obj->comment = $_POST['comment'];
        $p_obj->post();
    } elseif (isset($_POST['bbs']) && $_POST['bbs'] == "update") {
        require("XXX2.php");
        $u_obj = new Update();
        $u_obj->name = $_POST['name'];
        $u_obj->comment = $_POST['comment'];
        $u_obj->chk = $_POST['chk'];
        $u_obj->update();
    } elseif (isset($_POST['bbs']) && $_POST['bbs'] == "delete") {
        require("XXX3.php");
        $d_obj = new Delete();
        $d_obj->chk = $_POST['chk'];
        $d_obj->delete();
    }

    require("XXX4.php");
    $l_obj = new List_table();
    $data = $l_obj->List_table();
    echo implode(",", $l_obj);
?>

//XXX4.php
<?php
    require_once("XXX.php");

    class List_table extends Db {
        public $result;
        public $row;
        public function List_table() {
            $this->connect();
            $sql = "SELECT * FROM XXX ORDER BY time DESC";
			$result= $this->class->query($sql);
			$c = 0;
			$this->result = null;
			while($row = $result->fetch_assoc()) {
				$this->result[$c] = array('name' => $this->$row['name'], 'comment' => $this->$row['comment']);
				$c++;
			}
			$this->close();
       return $this->result;
		}
	}
?>

Answer 1

前にも書いたが、function List_table()にreturnがない。
ちゃんと前の質問を解決してから次に行ってください。

そして、表示したいのはList_table()返り値なんだろうから
echo implode(",", $l_obj);は$dataじゃないの？

Comments

[a-_.]

ご回答ありがとうございます

returnの件すいません、他の箇所の処理に没頭していて忘れてしまっていました

[jm1156]

いや、コンストラクタならreturnなくてもOKです。
ただし古い書き方の場合です。
詳しくはこちら
http://php.net/manual/ja/language.oop5.decon.php

[jm1156]

ちょっとまて、

$this->result = null;

なぜnullに…？

というか、コンストラクタで
DB接続＞SQL投げる＞変数に代入＞DB閉じる
って、処理を全部やってしまうのは仕様としてはよくないと思います。
初心者の書くコードの仕様にとやかく言うほどでもないかもしれませんが…。

[a-_.]

ご回答ありがとうございます

returnを質問文のソースにコーディングして訂正しました
nullの箇所は削除するのを忘れていました

[jm1156]

以前の質問のときにも他の回答者さんに注意を受けたと思いますが、
質問を編集するときは、追記して、元のコードのどこをどう訂正したかを書いてください。
他の人が見たときにわからなくなります。

[toutou]

>nullの箇所は削除するのを忘れていました
こういうの毎回あるけど見直してない証拠では？もらったコードを組み込んでるだけでしょ

Answer 2

List_table()が戻り値を返していない（returnしていない）
そもそも結果を$resultに格納しており、更にpublicなのだから返す必要もない。
なのでimplodeすべき対象なのは、$resultということになります。
該当箇所はこう書き直せるでしょう。

$l_obj = new List_table();
$l_obj->List_table();
echo implode(",", $l_obj->result);

皆さんの回答でコンストラクタの罠に気付いたので、書き直します。

$l_obj = new List_table();
echo implode(",", $l_obj->result);

余談ですが、配列への要素追加は、変数名[] = 値 と書くことができます。
なのでList_table内の処理はこう書くことができます。

～省略～
$this->result = null;
while($row = $result->fetch_assoc()) {
    $this->result[] = array('name' => $this->$row['name'], 'comment' => $this->$row['comment']);
}
$this->close();

Comments

[a-_.]

ご回答ありがとうございます

$l_obj = new List_table();
echo implode(",", $l_obj->result);で実行してみましたが、改善しませんでした

[ttyp03]

他にも原因はありそうですね。
例えば、$rowとか。
$rowはList_tableクラスのメンバですが、whileループ内ではローカル変数の$rowに格納してしまっています。
while($row = $result->fetch_assoc()) {
なので、このあとで $this->$row['name'] と参照しても値は取れていないと思います。
こうしないとダメでしょう。
while($this->row = $result->fetch_assoc()) {

Answer 3

$l_objはList_tableクラスのインスタンスですよね？
それはimplodeするって状況がよくわからなんですが
何をしたいのかもう少し噛砕いて説明してもらえませんか？

たぶん
$data = $l_obj->result;
を参照するということだと思うんですが・・・

#追記
XXX.phpの仕様次第でしょうけど、XXX4.PHPはこんな感じじゃないですか？
ずっと前に書きましたが無駄なクラス分けに見えるので統合し、
機能はfunctionで分ければいいように思えます。

class List_table {
  function __CONSTRUCT(){
    require_once("XXX.php");
    $this->conn=new Db;
    $this->conn->connect();
    $sql = "SELECT * FROM XXX ORDER BY time DESC";
    $result= $this->conn->query($sql);
    $this->result = null;
    while($row = $result->fetch_assoc()) {
      $this->result[] = array('name' => $row['name'], 'comment' => $row['comment']);
    }
    $this->conn->close();
  }
}

これに対して

    require("XXX4.php");
    $l_obj = new List_table;
    $data = $l_obj->result;
    echo implode(",", $l_obj);

とすればいけるような気がします

Comments

[yambejp]

ちなみにXXX.phpをrequireしてextendして使用していますが
XXX.php内でDBへの接続をしているなら、XXXのDbクラスから
データを取ってくる方が効率的ではないでしょうか？

それとコンストラクタを古い書き方をしているのは何か意図があるのでしょうか？

[a-_.]

ご回答ありがとうございます

元々行いたい処理はXXX4.phpで配列を取得した後、ループ処理して、その処理後のデータをメインのクラスに出力するという流れです

そこでXXX4.phpをrequireして、$l_objにList_table()というオブジェクトを代入、$dataは$l_obj(List_tableオブジェクト)の要素を代入、配列要素を含んだオブジェクトを代入した$l_objをimplodeで文字列として返すということです

[a-_.]

コンストラクタについては、「オブジェクト指向型」で調べているとこのようなコーディングで処理するという説明書きがあったので参考にしました

古い書き方というのは初耳です

[jm1156]

よく見たらコンストラクタでしたか…。
じゃぁ
$l_obj = new List_table();
$data = $l_obj->result;
echo implode(",", $data);
ですかね？

[a-_.]

ご回答ありがとうございます

実行してみましたが、いけませんでした
因みにその時のエラーがこちらです
Fatal error: Uncaught Error: Call to undefined method Db::query() in XXX4.php:8 Stack trace: #0 メインのクラス.php(25): List_table->__CONSTRUCT() #1 {main} thrown in XXX4.php on line 8

[yambejp]

エラーになっているのはもとのDbクラスの仕様が想定しているものと違うからでしょうね
XXX.phpも例示されたほうが良いかと思います。

[toutou]

そのエラーって投稿者さんはどんなエラーだと思ってるんですか？そしてそれを確認しましたか？

[jm1156]

Dbにquery()というメソッドが見つかりませんという意味です。
エラーメッセージがでたら、それをネットで検索して調べるようにしましょう。

親クラスのDbクラスはどうなっていますか？

[a-_.]

下記ソースがXXX.phpです
<?php
class Db {
public $class;
public function connect() {
$this->class = new mysqli("XXX", "XXX", "XXX", "XXX");
$this->class->select_db("XXX");
}

public function close() {
$this->class->close();
}
}
?>

[jm1156]

最初に提示されてるコードの、
$result= $this->class->query($sql);
ここの部分なんだけど、
$result= $this->query($sql);
こうなってたりしてないですか？

[a-_.]

$res= $this->class->query($sql);です

[jm1156]

$resはどこから出てきたのでしょうか…。