phpでCSVファイルを複数行データベース登録したい。

Question

CSVファイルで以下のようなものがあったとします。

foo1,foo2,foo3
bar1,bar2,bar3

これをデータベースに登録したくて、以下のようなコードを書いてみたんですが

<?php
$file = new SplFileObject('test.csv');
$file->setFlags(SplFileObject::READ_CSV);

$dsn='mysql:dbname=test_csv;host=localhost';
$user='root';
$password='';
$dbh=new PDO($dsn,$user,$password);
$dbh->query('SET NAMES utf8');

foreach ( $file as $line)
{
  $sql='INSERT INTO csvtest(data1,data2,data3)VALUES(?,?,?)';
  $stmt=$dbh->prepare($sql);
  $data[]=$line[0];
  $data[]=$line[1];
  $data[]=$line[2];
  $stmt->execute($data);
  $dbh=null; 
}
    
?>

上手くいきませんでした。たぶん考え方がそもそも違うのかもしれません。どなたか助言よろしくお願いします。

なお、DBはMYSQLで以下のように設定してます。

ID data1 data2 data3

Answer 1

以下のようにしたらうまく動くことを確認しました。
基本的なやり方はあっていると思います。

<?php
$file = new SplFileObject('test.csv');
$file->setFlags(SplFileObject::READ_CSV);

$dsn='mysql:dbname=test_csv;host=localhost';
$user='root';
$password='';
$dbh=new PDO($dsn,$user,$password);
$dbh->query('SET NAMES utf8');

foreach ( $file as $line)
{
  $sql='INSERT INTO csvtest(data1,data2,data3)VALUES(?,?,?)';
  $stmt=$dbh->prepare($sql);
  $data = [];
  $data[]=$line[0];
  $data[]=$line[1];
  $data[]=$line[2];
  $stmt->execute($data);
}
$dbh=null; 
?>

$dataは初期化が必要
$dbhにnullを代入するのはすべての操作が終わった後にする

Comments

[退会済みユーザー]

すみません。$dataを初期化することにはどういった意図があるんでしょうか？

[popobot]

$data[] = XXX は繰り返すとどんどん配列にデータが増えてしまうので初期化が必要です。
初期化しないとループの2回目で[foo1,foo2,foo3,bar1,bar2,bar3]という配列になります。

ループの途中にprint_r($data);というのを入れてると配列の中が見れるのでやってみてください。

[退会済みユーザー]

言われてみれば確かにそうでした！わかりやすい回答ありがとうございました

[kunai]

横からすみません。
ついでに、せっかくprepareしてるんだから、
$sql='INSERT INTO csvtest(data1,data2,data3)VALUES(?,?,?)';
$stmt=$dbh->prepare($sql);
の部分はforeachの外(前)に出しておいた方がいいと思います。