csvをインポートして、そのデータをupdateする方法について

Question

Deprecated: mysql_connect(): The mysql extension is deprecated and will be removed in the future: use mysqli or PDO instead in ...
以下のコードを実行したら上記のエラーメッセージが出ました。調べてみたら"mysql"はもう古いので、"mysqli" か "PDO"にしなさいということだと思うのですが、どうしたらいいのかわかりません。
ご教授のほどよろしくお願い致します。
あと、"tb_name"のカラムの"code"と一致する"xxx.csv"の"code($data[0])"の"price($data[5])"をupdateしたいのですが、以下のコードでよろしいでしょうか？

<?php

// データベース接続
define("DB_SERVER", "xxx"); //サーバ名
define("DB_USER", "xxx"); //ユーザー名
define("DB_PASS", "xxx"); //パスワード
define("DB_NAME", "xxx");	 //データベース名

$db_conn = mysql_connect(DB_SERVER, DB_USER, DB_PASS);
if (!$db_conn) {
	die("DB connection failed");
}

mysql_query("SET NAMES utf8",$db_conn);

$db_current = mysql_select_db(DB_NAME, $db_conn);
if (!$db_current) {
	die ("Can't use currentdb");
}

// ファイルを読み込む
if(!$rows=file('xxx.csv')) {
        die("File not found");
}

// ファイルを一行づつ処理
foreach($rows as $row) {

	$row = ereg_replace("\r|\n","",$row);
	$data = explode( ",", $row );		//カンマ区切りの場合
	$query = "SELECT code FROM tb_name WHERE code = '$data[0]';";
	$result = mysql_query($query);
	if (!$result) {
		die("Select query failed : code = $data[0]");
	}

	if(mysql_num_rows($result)!=0) {

		$query =  "UPDATE tb_name SET price = '$data[5]' WHERE code = '$data[0]';";
		$result = mysql_query($query);
		if (!$result) {
			die("Update query failed : YJ_code = $data[0]");
		}
		
	}

}

mysql_close($db_conn);

?>

Answer 1

データが無ければ無視されるのでデータがあるか確認してからupdateしなくてもいいのでは？
（データがない場合とデータが同じで更新されなかった場合をわけたい主旨だとは思いますが）
また変数をSQL文に直接つなげるのはどうかと思います

CSVを読み込むにはfgetcsv()をご利用なさるとよいかと

もし１件１件の検証が必要ないならforeachで回すと都度SQLを発行するため効率が悪いです
まとめて流し込むような仕組みに変えたほうが実用的です

補足

create table tbl (id int not null unique key,data int);
insert into tbl values(1,100),(2,200),(3,300);

上記tblに対して、更新データid=2→data=20、id=3→data=300、id=5→data=50を適用するとき
id=2は更新され、id=3はデータが同じなので更新されず、id=5は存在しないので更新されません。
普通にやると以下のように３つのupdate文が必要です

update tbl set data=20 where id=2;
update tbl set data=300 where id=3;
update tbl set data=50 where id=5;

しかSQL文は発行すればするほど無駄がおおく、処理が遅くなります。
今回のケースですと以下のようなSQL文1つでできます。

update tbl set data=COALESCE(ELT(FIELD(id,2,3,5),20,300,50),data);

fgetcsvの使い方

仮にtest.csvがこうだとします

2,a,b,c,d,20,e,f
3,a,b,c,d,300,e,f
5,a,b,c,d,50,e,f

これを配列として受けるのであればこうします

if (($fp = fopen("test.csv", "r")) !== FALSE) {
  while (($row = fgetcsv($fp, 1000, ",")) !== FALSE) {
    $rows[]=$row;
  }
  fclose($fp);
}
print_r($rows);

その上で使うのは各列の0番目と5番目の要素なのでそこだけ抜け出してSQL文に渡します

$id=[];
$val=[];
if (($fp = fopen("x.csv", "r")) !== FALSE) {
  while (($row = fgetcsv($fp, 1000, ",")) !== FALSE) {
    $id[]=$row[0];
    $val[]=$row[5];
  }
  fclose($fp);
}
$prepare_param=implode(",",array_fill(0,count($id),"?"));
$sql ="update tbl set data=COALESCE(ELT(";
$sql.="FIELD(id,".$prepare_param.")";
$sql.=",".$prepare_param."),data)";
print $sql;

$params =[str_repeat( 'i', count($id)*2)];
$params=array_merge($params,$id,$val);
print_r($params);

$stmt = $mysqli->prepare($sql);
call_user_func_array([$stmt, "bind_param"], $params);
$stmt->execute();

Comments

[alkeran]

効率が悪いというのは、それぞれの"code"の"price"をアップデートすると、大量のデータだと時間がかかるということでしょうか？
詳細を記載せずに申し訳ありませんでしたが、そもそもやりたいことは、元々、”tb_name”のデータベースの中には、商品名とコード(code)と価格(空欄:0)があり、アップロードするcsvデータ(xxx.csv)の価格(price)を上書きしたいのです。コードが一致したら。

[yambejp]

効率について補足をつけておきました。

[alkeran]

csvデータは複数個あり、各社商品の価格を上書きしたいため、更新されないものがあってもよいと考えております。今回のコードは自身で考えたものではないため、詳細を教えて頂ければと思うのですが、変数"$data[num]"はcsvデータの1列目のデータということでしょうか？

[yambejp]

csvを配列に抜き出した後、mysqlに渡すまでの処理を追記しときました

[alkeran]

Warning: Parameter 2 to mysqli_stmt::bind_param() expected to be a reference, value given in ...
コードを書いていただきありがとうございました。試してみたら上記のエラーが出ました。あと、ブラウザに以下が表示されましたが、データベースの価格は更新されませんでした。
update tb_name set price=COALESCE(ELT(FIELD(id,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?),?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?),price)Array ( [0] => iiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii [1] => 1234565789 [2] => 987654321 [3] => 2468101214 [4] =>...

Answer 2

古いというか、PHP 5.5で非推奨となり、PHP7で削除されました。 PHPマニュアル：mysql_connect

• mysqlからmysqliへの書き換え方法
• mysqliを使ってデータベースへ接続

記事にもありますが、phpマニュアルを参考にして書き換えると良いです。
基本的には同じですが引数に与える情報や順番が変わってきます。

Answer 3

とりあえず、下記の記事を読んでみては如何でしょうか。
[PHP] mysqli使い方まとめ（MySQL接続～SELECT実行まで）

Comments

[alkeran]

ありがとうございます。勉強になります。
自分なりに解釈して以下のコードに書き換えました。エラーは出なくなったのですが、updateが上手にできませんでした。

<?php
// mysqliクラスのオブジェクトを作成
$mysqli = new mysqli('xxx', 'xxx', 'xxx', 'xxx');
if ($mysqli->connect_error) {
echo $mysqli->connect_error;
exit();
} else {
$mysqli->set_charset("utf8");
}

// ファイルを読み込む
if(!$rows=file('xxx.csv')) {
die("File not found");
}

// ファイルを一行づつ処理
foreach($rows as $row) {

$row = preg_replace("/\r/", "/\n/","",$row);
$data = explode( ",", $row ); //カンマ区切りの場合
$sql = "SELECT code FROM tb_name WHERE code = '$data[0]';";
$result = $mysqli->query($sql);
if (!$result) {
die("Select query failed : YJ_code = $data[0]");
}

if(mysql_num_rows($result)!=0) {

$sql = "UPDATE tb_name SET price = '$data[5]' WHERE code = '$data[0]';";
$result = $mysqli->query($sql);
if (!$result) {
die("Update query failed : code = $data[0]");
}

}

}

$mysqli->close();
?>