DBから取得したデータを文字列に変換

Question

DBから取得したデータを文字列に変換

・前提
1.phpとmysqlを使い、記事掲載ページを作っています。
2.異なるデータベースを使用しています。

・やりたいこと
異なるDBのデータと、使用しているDBのデータ紐づけ、条件をつけて表示したいです。

実現したいことの例)
DB:AとDB:Bがあったとします。
DB:Aには記事の情報が入っています。
具体的には、記事ID,記事Title, 記事Body.....のような感じです。
DB:Bには、Bookmarkした記事IDが入っています。
具体的には、2,4,8,22...のように入っています。
そこで、DB:Bから取得したデータ(Bookmarkした記事ID)を一度文字列にし、
DB:Aのデータと一致すれば、Bokmarkした記事を取得できる。

といった感じにしたいです。

しかし、DB:BからBookmarkした記事IDは取得できるのですが
どうやら文字列に変換できないようです。

こちらがそのエラーメッセージです。

発生している問題・エラーメッセージ

Notice: Array to string conversion in ......on line 80 Array ( )

該当のソースコード1

SQL文の関数です。

 function from_B(){ //BはDB名です。
$sql = <<<SQL
select bookmarked_id
from bookmarke_table
SQL;

return query($sql); // 本来のqueryはもっとちゃんとしたものを使っています。
}

function from_A($data){ //AはDB名です。
$sql = <<<SQL
select *
from A.all_article where article_id in ("+$data+")  --AはDB名です。
SQL;

return query($sql);
}

該当のソースコード2

PHP側でデータを取得する関数です。

 function from getbookedarticle(){
$get_first = $this->from_B() //bookmarkした記事のIDを取得
$implode = implode(",",$get_first) //配列を文字列にする
$get_second = $this->from_A($implode);//bookmarkした記事IDと記事のIDが一致したデータを取得
echo $get_second;
}

補足情報（FW/ツールのバージョンなど）

現在、テスト用にDB:Bに入っているBookmarkした記事の数は1つだけです。
そのため、文字列へ変換し、連結するimplode関数で大丈夫なのでしょうか？
また、print_rなどでDB:Bから取得した値は、多次元配列になっております。

Array ( [0] => Array ( [article_id] => 4 ) ) 
のような感じです。

DB:A、DB:B各々のIDはintです。
詳しい情報を提供できなく、大変申し訳ないのですが、
よろしくお願いいたします。

Comments

[Orlofsky]

理解しにくいからテーブルの定義は CREATE TABLE文で載せては？

[退会済みユーザー]

既に既存してあるサービスで、既存のものを使うしかありません。大変申し訳ないのですが、ご理解よろしくお願いいたします。

[ttyp03]

前置きとか不要。要は二次元配列をimplodeするには？ってことですよね？

[退会済みユーザー]

できるだけ詳しく書こうと思ったのですが、そうですね。おっしゃる通りです。

Answer 1

<?php

$array = [
    0 => ['article_id' => 4],
    1 => ['article_id' => 6],
];

$result = [];
foreach ($array as $arr_item) {
    $result[] = $arr_item['article_id'];
}

echo implode(',', $result);  // '4,6'

こんなんでいかがでしょうか。

Comments

[退会済みユーザー]

ご回答ありがとうございます。
試して見ます！

[退会済みユーザー]

もしも前提になるデータ構造が違うのであればこのサンプルコードは無意味になるので、その点のチェックをお願いします。

Answer 2

implodeではなく、
foreachを使い、配列の値だけを取得するようにすると、うまく行きました。

    foreach($get_first as $marked){
      foreach($marked as $m){
                 echo $m;
      }
    }
    

この方法だと、まだまだ初心者感が出ていると思いますが正常に動作しました。

みなさんご協力ありがとうございました！

Answer 3

まず前提としてDBとDBのリレーション的な流れで話をされていますが
テーブルとテーブルでよろしいでしょうか？

内容的にはfrom_A($data)で渡している$dataが配列なのでは？
またquery()という関数に$sqlを投げていますが,$dataを直接埋め込むような
処理は適切ではありません。
もし状況が許すのであればプレースホルダーを利用して
セキュリティやエスケープ作業の負荷をシステム側に任せたほうがよいでしょう

Comments

[退会済みユーザー]

ご回答ありがとうございます。
おっしゃる通り、$dataが配列ということは承知しております。
引数の$dataに値を渡す以前に、取得したデータを文字列にし、$dataには文字列が渡されるようにしたいと思っております。
まだまだ初心者でなので、その実装方法がわからずにいます。
勉強不足で、申し訳ございません。
queryに関して、セキュリティーが弱い件に関しては
上位のクラスにセキュリティーなどの処理を実装しております。
御気遣いありがとうございます。

[退会済みユーザー]

> 上位のクラスにセキュリティーなどの処理を実装しております。
これ、事故起こす発想ですよ＾＾；

[退会済みユーザー]

そうなのですね、、、
もう一度確認して見ます。
私もまだまだですね。ありがとうございます。

[yambejp]

> queryに関して、セキュリティーが弱い件に関しては
> 上位のクラスにセキュリティーなどの処理を実装しております。

処理とデータが別れていない以上、上位のクラスもないも
セキュリティを高めることはほぼ不可能です
処理とデータを分けて提案する形にしないと意味がありません
（それがすなわちプレイスホルダ）

[退会済みユーザー]

ご返信ありがとうございます。
プレイスホルダに関しても、ざっくりとしか理解しきれてなかったので
勉強します。
ありがとうございます！