MYSQL 画像表示

###前提・実現したいこと
選択した画像をコピーしてフォルダにいれる
###ヘディングのテキスト

PHP
1<?php
2  require 'common.php';
3  $error = $name =$type =$comment = $price =$pic = '';
4  $pdo = connect();
5  if (@$_POST['submit']) {
6    $name = $_POST['nam'];
7    $type = $_POST['typ'];
8    $comment = $_POST['comme'];
9    $price = $_POST['pri'];
10    $pic = $_POST['pice'];
11    if (!$name) $error .= '商品名がありません。<br>';
12    if (!$type) $error .= '商品の種類がありません。<br>';
13    if (!$comment) $error .= '商品説明がありません。<br>';
14    if (!$price) $error .= '価格がありません。<br>';
15    if (preg_match('/\D/', $price)) $error .= '価格が不正です。<br>';
16    if (!$pic){
17      $error .= 'ファイルを選択してください。<br>';
18    } else {
19      $image = $_FILES['pic']['tmp_name'];
20    }
21    if (!$error) {
22      if(move_uploaded_file($_FILES['pic']['tmp_name'], "images/$image.jpg")){
23        $pdo->query("INSERT INTO goods(name,type,comment,price,image) VALUES('$name','$type','$comment','$price','$image')");
24        header('Location: index.php');
25        exit();
26      } else {
27        $error .='ファイルをアップロードできません。<br>';
28      }
29    }
30  }
31  require 't_insert.php';
32?>

###現在の問題点
上手くif(move_uploaded_file($_FILES['pic']['tmp_name'], "images/$image.jpg"))文が機能せずエラー文としてelseになってしまう

m.ts10806

2017/05/31 02:15

エラーが出ているのでしたらエラー文も添付してください。

abcdef

2017/05/31 02:41

エラーは表示されませんが「ファイルをアップロードできません。」のelse文が帰ります。

m.ts10806

2017/05/31 02:44

了解です。今の「現在の問題点」だとPHPのエラーが出ているのか自分で出力されているエラーメッセージが出力されているのか分かりにくかったための確認でした。

行動規範の内容に同意します

回答2件

「images/$image.jpg」の部分で
move_uploaded_file() は「アップロードされたファイルを新しい位置に移動する」ものなので実行しているプログラムファイルから見て「images」ディレクトリが存在しないのだと思います。
サーバー内のディレクトリを指定する場合はDIRなどを利用してプログラムファイルからのパスを正しくアップロードしたいパスに指定した方が良いでしょう。

また、tmp_nameは全く意味のないランダムの文字列がランダムに振られたファイルです。
保存したい情報に紐付けた関連する名前をプログラム側でつけたほうが良いでしょう。
（今回のケースだと商品コードなど）

投稿2017/05/31 02:22

m.ts10806

総合スコア80850