🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Bootstrap

BootstrapはウェブサイトデザインやUIのWebアプリケーションを素早く 作成する可能なCSSフレームワークです。 Twitter風のデザインを作成することができます。

Q&A

1回答

1240閲覧

mysqlのupdate setに関する謎のエラーです

Kazutoshi-gif

総合スコア0

MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Bootstrap

BootstrapはウェブサイトデザインやUIのWebアプリケーションを素早く 作成する可能なCSSフレームワークです。 Twitter風のデザインを作成することができます。

0グッド

0クリップ

投稿2020/12/01 14:00

編集2020/12/01 14:41

前提・実現したいこと

プログラミング初心者です。
phpをmysqlにつなげて商品のecサイトを作っています。商品の情報更新システムに関するedit機能に問題が起きました。oopのオブジェクト指向の形でアクションファイルに$_POSTで各データの情報をセットしてクラスファイルにpubli functionsでupdateinsertでmysqlへ情報を更新したいです。しかし、今よくわからないエラーが出て困っています。

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

どこの個所がエラーかも書いていなくてどこを直したらよいか分かりません。
以下がエラーメッセージです
updateItem($itemID,$cost,$title, $author, $publisher, $materials,$information,$imageName);

ビューファイル <div class="col-4 float-left"> <form action="../actions/editItem.php" method="post" enctype="multipart/form-data"> <input type="hidden" name="id" value="<?= $selectItemList['id'] ?>"> <img src="../img/<?= $selectItemList['img'] ?>" alt="<?= $selectItemList['img'] ?>" class="card-img-top" alt="text with cd"> </div> <div class="custom-file small col-3"> <label for="choosePhoto" class="custom-file-label">Choose Photo</label> <input type="file" name="img" id="choosePhoto" class="custom-file-input"required> </div> <div class="container text-right"> <div class="number" name="cost"> <h1 class="h4"><input type="number"name="cost" id="" class="col-3" placeholder="cost" value="<?= $selectItemList['cost'] ?>"required></h1> </div> </div> <div class="container col-8 float-right"> <div class="font-weight-bolder"> <h3 class="h5"><input type="text"name="title" id="" class="form-control" placeholder="title"value="<?= $selectItemList['title'] ?>"required></h3> </div> <div class="mb-2"> <input type="text"name="author" id="" class="form-control" placeholder="author"value="<?= $selectItemList['author'] ?>"required> </div> <div class="mb-2"> <input type="text"name="publisher" id="" class="form-control" placeholder="publisher"value="<?= $selectItemList['publisher'] ?>"required> </div> <div class="mb-2"> <input type="text"name="materials" id="" class="form-control" placeholder="materials"value="<?= $selectItemList['materials'] ?>"required> </div> </div> <div class="container"> <textarea type="text"name="information" id="" class="form-control" placeholder="information"><?= $selectItemList['information'] ?></textarea> </div> <div class="container text-center"> <button class="btn btn-outline-info rounded-pill w-25 mt-3"type="submit" href="admin.php"> Save Changes </button> </div> </form>
アクションファイル include "../classes/admin.php"; $itemID = $_POST['id']; $cost = $_POST['cost']; $title = $_POST['title']; $author = $_POST['author']; $publisher = $_POST['publisher']; $materials = $_POST['materials']; $information = $_POST['information']; $imageName = $_FILES['image']['name']; $admin = new Admin; $admin->updateItem($itemID,$cost,$title, $author, $publisher, $materials,$information,$imageName);
クラスファイル public function updateItem($itemID,$cost,$title,$author,$publisher,$materials,$information,$imageName){ $sql="UPDATE items SET cost='$cost',title='$title',author='$author',publisher='$publisher',materials='$materials',information='$information',img='$imageName' WHERE id='$itemID'"; if($this->conn->query($sql)){ $destination = "../img/" . basename($imageName); if(move_uploaded_file($_FILES['image']['tmp_name'], $destination)){ header("location:../views/admin.php"); exit; }else{ die("Error moving photo."); } }else{ die("Error updating Item:".$this->conn->error); } }

試したこと

色々やりましたが分かりません。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

phper.k

2020/12/01 14:10

エラーが出ているならそのエアラーメッセージを質問文に書いてください。
Kazutoshi-gif

2020/12/01 14:15

説明不足で申し訳ございません。そのエラーメッセージが以下の文です。 updateItem($itemID,$cost,$title, $author, $publisher, $materials,$information,$imageName);
phper.k

2020/12/01 14:18 編集

あと、ファイルごとに ``` ...``` は分けましょう。複数のファイルの中身を一緒にしては、見づらい上に回答をミスリードしてしまいます。
m.ts10806

2020/12/02 00:26

質問は編集できます。
m.ts10806

2020/12/02 00:28

あと、SQLインジェクション可能な作りになってますし、$_POSTはスーパーグローバル変数なので引数で渡さなくても取れます。 なんか、いろいろとメチャクチャです。
guest

回答1

0

試したところ、ブラウザにError moving photo.と表示されました。

パラメータ名が、ビューとphpで噛み合ってないのが原因だと思います。
ビューはimgで、php側はimageになってます。

html

1<input type="file" name="img" id="choosePhoto" class="custom-file-input" required>

php

1// アクションファイル 2$imageName = $_FILES['image']['name']; 3 4// Adminクラス 5if(move_uploaded_file($_FILES['image']['tmp_name'], $destination)){

投稿2020/12/01 14:53

gpsoft

総合スコア1323

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Kazutoshi-gif

2020/12/01 15:17

回答ありがとうございます。 htmlの方をimageとして書いたのですがまだ直らないです。 エラーメッセージも同じのままです。
gpsoft

2020/12/01 15:27

コードが断片的にしか提示されてないので、 私の方で、いろいろ想像して補いながら再現環境を作りました。 パラメータ名を修正すれば、ファイルのアップロードも DBの更新も成功しましたよ。 提示されてない部分に、問題があるのかもしれませんね。 <?php の書き忘れとか、誤記、 ディレクトリ階層(classesとかimgとか)のズレ などを再確認してみてはいかがでしょうか。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.36%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問