DBの値が一致したら更新

Question

前提・実現したいこと

PHP初心者です。ちょっと説明が下手ですが、他サイトの指定classの<h1>要素の値を取得をして、DBの値と一致したら特定のカラムのみ全件更新させたいと思っています。

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

プログラムは出来ていますが、全件に更新させたいのですが、なぜかDBの最後のレコードしか更新されません。
サイトのスクレイピングにphpQuery-onefile.phpを利用しています。

該当のソースコード

<?php

try {
    $pdo = new PDO('mysql:host=localhost;dbname=##;charset=utf8', '##', '##',
array(PDO::ATTR_EMULATE_PREPARES => false));
} catch (PDOException $e) {
    exit('データベース接続失敗。'.$e->getMessage());
}

$stmt = $pdo->query('SELECT * FROM model');
while ($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
    $ttitle = $row['url'];
}

require_once 'phpQuery-onefile.php';

$html = file_get_contents($ttitle);

$doc = phpQuery::newDocument($html);

$test = $doc['.test2 h1']->text();

try {
    // 静的プレースホルダを指定
    $pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);

    // DBエラー発生時は例外を投げる設定
    $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

    $stmt = $pdo->prepare('UPDATE model SET status=:status WHERE url=:url');

    //トランザクション処理
    $pdo->beginTransaction();

    try {
        $stmt->bindParam(':status', $test);
        $stmt->bindParam(':url', $ttitle);
        $stmt->execute();

        //コミット
        $pdo->commit();
    } catch (PDOException $e) {
        //ロールバック
        $pdo->rollback();
        throw $e; //
    }
    // 接続を閉じる
    $pdo = null;
} catch (PDOException $e) {
    // UTF8に文字エンコーディングを変換します
    echo mb_convert_encoding($e->getMessage(), 'UTF-8', 'SJIS-win');
    die();
}

試したこと

どうしても最後のレコードしか反映されない状態で悩んでいます。
ご教授をお願いします。

Answer 1

インデントが整理されていなくて非常に見づらいのでそこは調整していただきたいところですが、

while($row = $stmt -> fetch(PDO::FETCH_ASSOC)) {
 $ttitle = $row["url"]; }

上記、= で$ttitleに代入しているので常に最後の情報しか保持していませんよ。
while内でupdateも行うか配列で保持して配列ループ内でupdateされては？

ループ中の情報を別の配列変数に受ける　ミニマムコード

$a = []; // []はarray()の省略。
for($i=0;$i<100;$i++){
    $a[] = $i;  //$a[]のように[]で受けることで0番目から順番に情報を詰め込む
}

foreach($a as $b){
    echo $b;
}

Comments

[web_tanaka]

回答ありがとうございます。
while($row = $stmt -> fetch(PDO::FETCH_ASSOC)) {
}
この中にupdateするように組み込むということでしょうか？

[m.ts10806]

問題の原因は分かったんですよね？
あとは扱いたい情報を全て利用するように軌道修正するだけです。
回答にも書いたようにやり方は1つではありません。
要件や状況次第ではもっとシンプルに同様の効果をあげられる方法も存在するはずです。

[web_tanaka]

まだ全部は理解できていない状態です。
$ttitleに代入しているので常に最後の情報しか保持されないことは理解できましたけど、配列することによって改善できるということでしょうか？
例えばですけど、クエリした結果を配列に入れてからupdateするような形でしょうか？
それとも、updateをループさせないといけないでしょうか？

[m.ts10806]

今のつくりを踏襲するのであればUPDATEのSQLは1つずつ発行しているので
ループさせないといけないですね。
もちろん工夫すれば1回のSQLで実行するのは不可能ではないですが、
いずれにしても格納した配列のループは必要です。

[m.ts10806]

一番簡単なのは回答の1つ目にあるwhileの中でupdateもしてしまうことです。

[web_tanaka]

回答ありがとうございます。ようやく理解できました。
ですが、今の状態のwhile($row = $stmt -> fetch(PDO::FETCH_ASSOC)) {
$ttitle = $row["url"]; }は一応全件取得できてる状態なのでこちらの文はループできているという認識でいいのでしょうか？

UPDATEのsqlにはループ文がないのでループされてないのは理解できてます。それが原因で一つのレコードしか反映されてない状態でよろしいでしょうか？

[m.ts10806]

> $ttitle = $row["url"]; }は一応全件取得できてる状態なのでこちらの文はループできているという認識でいいのでしょうか？

おそらくほしい情報は全件取得できていてループはされていますが、保持が常に最新の値のみとなっています。UPDATEの方にループがない以前の問題です。

下記echoの結果で「99」が出力されるのと同じ状態です。
$a = "";
for($i=0;$i<100;$i++){
$a = $i;
}
echo $a;

0～98は$iには入ってきていて$aに「都度代入」されるため、値が上書きされ続けていき、最後の値しか残らないわけです。

回答の繰り返しとなりますが、
上書きされ続けるのを回避するには、「上書きされる前に使ってしまう」か「配列として全情報を受け取っておいて都度使う」のどちらかです。

やりとりをしていて感じた事でもしかしたら失礼に当たるかもしれないですが、
後者の「配列として受け取る」のコードの書き方が分からないということでしょうか・・？

[web_tanaka]

度々回答ありがとうございます。
下記echoの結果で「99」が出力の件はすごく理解できました。

調べてみると配列にはarrayなどを使っていますが、まさにその通りです。
恥ずかしながらも配列として受け取る方法がまだ分かってない状態です。
何故、一つのレコードしか更新されないのかとしか疑問に思っていなかったですが、mts10806様のおかげで今ようやく理解できたって感じです。ですがまだ実現はできていません。試行錯誤中でございます。

[m.ts10806]

>配列として受け取る方法がまだ分かってない状態です。

私の提示したミニマムコードを例にするとこんな感じ

$a = [];
for($i=0;$i<100;$i++){
$a[] = $i;
}

foreach($a as $b){
echo $b;
}

冗長ですがあくまで例として。

ちなみに最初のfor内でechoをするのが「上書きされる前に使ってしまう」です。
ミニマムコードだと配列で受け取った後にさらにループするのはかなり冗長に見えてしまうかもしれませんが、
要件次第でその方が効率よく処理できたり、可読性も上がったり、メンテナンスもしやくすくなったりします。
そういった意味でも「やり方は1つではない」ということになります。
全体のコードのバランスや要件を考えたらもっと簡略化できる可能性もあります。
「絶対にこうしなきゃいけない」というルールはないので、状況に応じて柔軟にコードを変化させると良いです。

[web_tanaka]

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

$stmt = $pdo->query("SELECT url FROM model");
while($row = $stmt -> fetchAll(PDO::FETCH_ASSOC)) {
$arr = array (
$row['url']
);

このように配列としてやりましたが、そのあとの処理がどうもうまくいかないです。
var_export($row);でURLが[0],[1]などに入っているのは確認できています。

require_once("function/phpQuery-onefile.php");
$html = file_get_contents($row);
$doc = phpQuery::newDocument($html);
$test = $doc[".show_deny h1"]->text();

$html = file_get_contents($row);
ここに配列を渡したいのですが記述方法がまったく分からない状態です。
これらを使ってループ文の組み方が分からないといった方がいいかもしれません。
良ければ引き続きご教授をしてもらえると嬉しい限りです。

[m.ts10806]

配列にするのは受ける方ですよ。
私のミニマムコード参考にしてます？

$a[] = $i;
と
$a = [$i];

では全く違いますよ。
ひとまず質問本文にやってみたところを含めた改修コード全文を追記してください。

[m.ts10806]

ループの組み方というより配列の組み方と言ったほうが正しいですね。
ひとまず簡単な方を試してください。「上書きされる前に使ってしまう」
・whileの閉じカッコをコード末尾に移動
・require_once("phpQuery-onefile.php");をwhileよりも前に移動
・$pdo = null;をwhileの閉じカッコよりも後に移動
でとりあえずできると思います。

ただ、コードはこのままでは読みにくいのでインデントはきちんと整理してください。

[web_tanaka]

度々回答ありがとうございます。
なるほどです。閉じカッコも気を付けないとループが途中で終わってしまうのですね。

ご教授ありがとうございます。
ループがどういったものなのか簡単なサンプルを参考にしながら実行結果などを確認してますので頑張ってみたいと思います。

[m.ts10806]

関数やクラスもそうですが「スコープ」というのは常に気にしてください。
変数もスコープ外では使えないこともあります。
http://php.net/manual/ja/language.variables.scope.php
どのような機能を調べるにしてもまず最初はPHPマニュアルを見てください。
機能紹介だけではなく上記のようなスコープのことや注意点なども詳しく書かれていますし、User Contributed Notesでは例文も投稿されています。

[web_tanaka]

まったく聞いたことない言葉ばかりなので本当勉強になります。
URLまでありがとうございます。参考にさせてもらいます。
今週頑張って実行できるようにしますのでまた報告させてもらいます。その時はお世話になるかもしれませんがよろしくお願いします。