画像のファイル名をデータベースに保存する方法

Question

ファイル名をデータベースに保存する方法

アップロード画面からファイルを特定のディレクトリに移動して、同時にファイル名をデータベースに保存したいのですが、
画像は指定したディレクトリに保存はされますが、ファイル名がデータベースに保存できておりません。

ご教授いただければ幸いです。

※現段階ではバリデーションは無くて大丈夫です。

 <form action="upimg.php" method="POST" enctype="multipart/form-data">
        <input type="file" name="image">
        <input type="submit" value="アップロード">
    </form>

$file = $_FILES['image']['name'];

try{
    $dsn = "sqlite:C:/xampp/htdocs/image.db";
}catch(Exception $e){
    echo "データベースに接続できませんでした。";
    exit();
}

if(!empty($_FILES['image']['tmp_name'])){
    move_uploaded_file($_FILES['image']['tmp_name'], "./upload/$file");
    $dsn->query("INSERT INTO image(path) VALUES('$file')");
}else{
    echo "画像が選択されていません。";
}

データベースは以下の構成です。
SQLite
id(int) , path(text)

エラー表示

Fatal error: Uncaught Error: Call to a member function query() on string in C:\xampp\htdocs\image\upimg.php:14 Stack trace: #0 {main} thrown in C:\xampp\htdocs\image\upimg.php on line 14

よろしくお願いいたします。

Comments

[m.ts10806]

「うまくいきません」だと何も伝わらないので実際に起きている現象を具体的に記載してください

[takepan1973]

質問するときに「うまくいきません」はやめるようにしましょう。
うまく行かないから質問されているのは分かりきっていることです。
どう「うまくいかない」のかを書いてください。

sqliteに詳しくないのですが、$dsnがただの文字列で、
$dsn->queryはどう考えても正しく動作するように見えません。
sqliteについて、何を参考にされましたか？

[rinrin1137]

画像は指定したディレクトリに保存できているのですが、ファイル名がデータベースに保存できておりません。m(_ _)m

[takepan1973]

エラー表示の追加を確認しました。
やはり、$dsn->queryのところが問題ですよね。
sqliteについて使い方が間違っていると思いますので、そのところを再確認してください。

[hermit19901127]

…どこでディレクトリ名を保存してるのかわからないですが、ディレクトリ取得と、ファイル名取得の関数は分かれている印象です…

[rinrin1137]

失礼しました。
new PDOが完全に抜け落ちてました・・・。
無事にファイル名を保存することができました。
ありがとうございました！

Answer 1

<?php

$file = $_FILES['image']['name'];

try{
    $dsn = new PDO("sqlite:C:/xampp/htdocs/image.db");
}catch(Exception $e){
    echo "データベースに接続できませんでした。";
    exit();
}

if(!empty($_FILES['image']['tmp_name'])){
    move_uploaded_file($_FILES['image']['tmp_name'], "./upload/$file");
    $dsn->query("INSERT INTO image(path) VALUES('$file')");
}else{
    echo "画像が選択されていません。";
}

?>

$dsn = new PDO("sqlite:C:/xampp/htdocs/image.db");
を見直して記述ミスを修正したところ無事に保存ができました。
ありがとうございました。

Answer 2

現状、変数$dsnには単なる文字列です。
文字列なのでアロー演算子によりプロパティやメソッドも存在しないので提示のエラーになっています。

下記の記事のようにきちんとPDOによりインスタンス化してください。
PHP + SQLite3 超入門したメモ

上記の記事でもう1つ参考にしてほしいのは
try-catchを接続クローズ（要はPDOの用が済むまで）保持していることです。
異常が起きるのは接続時よりも実行時のほうが多いです（接続は一度できてしまえばほとんど問題は起きないので）
それぞれの実行結果だけでは原因まで拾えないことも多いです。
SQLはPHPからすると外部の仕組みなので細かいところまでPHPの関数では拾えないからです。
そのようなときにExceptionを捕捉することにより、SQLのエラーまで拾えるようになります。

Comments

[rinrin1137]

ありがとうございます。
基本的な部分の記述をミスしておりました。
無事に解決いたしました。m(_ _)m

[m.ts10806]

大事な追記をしています。
そちらもご一読ください

[rinrin1137]

ありがとうございます。

if(!empty($_FILES['image']['tmp_name'])){
move_uploaded_file($_FILES['image']['tmp_name'], "./upload/$file");
$dsn->query("INSERT INTO image(path) VALUES('$file')");
}else{
echo "画像が選択されていません。";
}

上記をtry{}の中に記述した方がいいとのことでしょうか？

[m.ts10806]

はい。接続時より実行時の方が異常は起きやすいです。そこを検知するためには中にいれないといけません。
PHPマニュアルやQiitaの評価の高い記事(特にPHPでデータベースを扱う関係の記事で必ずといっていいほど参照される高評価2000超えの記事があります)