質問をすることでしか得られない、回答やアドバイスがある。

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

新規登録して質問してみよう
ただいま回答率
85.50%
MySQL

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

PHP

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

Q&A

解決済

2回答

1983閲覧

MySQLの論理削除に関して

onslaught

総合スコア19

MySQL

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

PHP

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

0グッド

1クリップ

投稿2019/04/18 06:35

編集2019/04/18 06:45

教材を元にPHPを学習中なのですが、todoリストの作成にあたって削除ボタンを押すと
押した場所のtodoが削除(論理削除)される機能を作成しています。
現在の時刻をdateメソッドで取得しdeleted_atの値を更新する処理をしたいです。

'UPDATE テーブル名 SET 更新するカラム名 = 更新する値 WHERE 条件';

という理解の元下記UPDATE文を記述してみたのですが、特に押しても反応せず止まっている状態です。
そもそも構文の書き方や理解がおかしいのでしょうか?

追記PHP7.2,MySQL5.7

php

1**index.php** 2 3<?php 4 require_once('functions.php'); 5?> 6 7<!DOCTYPE html> 8<html lang="ja"> 9<head> 10 <meta charset="UTF-8"> 11 <title>Home</title> 12</head> 13<body> 14 welcome hello world 15 <div> 16 <a href="new.php"> 17 <p>新規作成</p> 18 </a> 19 </div> 20 <div> 21 <table> 22 <tr> 23 <th>ID</th> 24 <th>内容</th> 25 <th>更新</th> 26 <th>削除</th> 27 </tr> 28 <?php foreach (getTodoList() as $todo) : ?> 29 <tr> 30 <td><?= $todo['id']; ?></td> 31 <td><?= $todo['todo']; ?></td> 32 <td> 33 <a href="edit.php?id=<?= $todo['id']; ?>">更新</a> 34 </td> 35 <td> 36 <form action="store.php" method="post"> 37 <input type="hidden" name="id" value="<?= $todo['id']; ?>"> 38 <button type="submit">削除</button> 39 </form> 40 </td> 41 </tr> 42 <?php endforeach; ?> 43 </table> 44 </div> 45</body> 46</html>

PHP

1**connection.php** 2 3<?php 4require_once('config.php'); 5 6 7function connectPdo() //正規処理と例外処理の関数 8{ 9 try { 10 return new PDO(DSN, DB_USER, DB_PASSWORD); 11 } catch (PDOException $e) { 12 echo $e -> getMessage(); 13 exit(); 14 } 15} 16 17function createTodoData($todoText) 18{ 19 $dbh = connectPdo(); 20 $sql = 'INSERT INTO todos (todo) VALUES (:todoText)'; 21 $stmt = $dbh->prepare($sql); 22 $stmt->bindValue(':todoText', $todoText, PDO::PARAM_STR); 23 $stmt->execute(); 24} 25 26function getAllRecords() 27{ 28 $dbh = connectPdo(); 29 $sql = 'SELECT * FROM todos WHERE deleted_at IS NULL'; 30 return $dbh->query($sql)->fetchAll(); 31} 32 33function updateTodoData($post) 34{ 35 $dbh = connectPdo(); 36 $sql = 'UPDATE todos SET todo = :todoText WHERE id = :id'; 37 $stmt = $dbh->prepare($sql); 38 $stmt->bindValue(':todoText', $post['todo'], PDO::PARAM_STR); 39 $stmt->bindValue(':id', (int)$post['id'], PDO::PARAM_INT); 40 $stmt->execute(); 41} 42 43function getTodoTextById($id) 44{ 45 $dbh = connectPdo(); 46 $sql = 'SELECT todo FROM todos WHERE deleted_at IS NULL'; 47 return $data['todo']; 48} 49 50function deleteTodoData($id) 51{ 52 $dbh = connectPdo(); 53 $now = date('Y-m-d H:i:s'); 54 $sql = 'UPDATE todos SET deleted_at = $now WHERE deleted_at = NULL'; ←この部分の処理が何も行われない。 55} 56 57?>

PHP

1**config.php** 2 3<?php 4 5define('DSN', 'mysql:dbname=php_lesson;host=localhost;unix_socket=/tmp/mysql.sock'); 6define('DB_USER', 'root'); 7define('DB_PASSWORD', ''); 8 9?>

データベース
データベース

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

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

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

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

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

m.ts10806

2019/04/18 06:40

教材とは何でしょうか?差し支えなければ出典記載してください。またphp、mysqlのバージョンを質問本文に記載してください
yoorwm

2019/04/18 06:47

deleted_at IS NULLと、正しく使ってるのに、問題の箇所でdeleted_at = NULLとしているのはなぜでしょうか? (他にも色々突っ込みどころがありますが。)
退会済みユーザー

退会済みユーザー

2019/04/18 06:50

deleteTodoData でクエリー実行してない
onslaught

2019/04/18 06:51

私が記述した部分以外は教材のコピペになります。 なので私自身はまだDB操作やPHPの書き方などがあまりわかっていない状態です。
guest

回答2

0

deleteTodoDataはSQL文を宣言していますが、実行していません。
updateTodoDataと同様に、プレースホルダ(:honyarara)を用いてSQL文を宣言した後、パラメータの設定、SQLの実行まで行ってください。

投稿2019/04/18 06:45

moredeep

総合スコア1507

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

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

onslaught

2019/04/18 07:09

ありがとうございます。
guest

0

ベストアンサー

SQL側の日付関数で処理するほうがよいでしょう

PHP

1$sql = 'UPDATE todos SET deleted_at = now() WHERE id=?' 2$stmt = $dbh->prepare($sql); 3$stmt->execute([$id]);

どうしてもPHPから渡したいならそこもprepareで

PHP

1$sql = 'UPDATE todos SET deleted_at = ? WHERE id=?' 2$stmt = $dbh->prepare($sql); 3$stmt->execute([$now,$id]);

投稿2019/04/18 06:51

編集2019/04/18 06:52
yambejp

総合スコア114585

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

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

onslaught

2019/04/18 07:08

ありがとうございますクエリの実行等ができていませんでした。 値の ? の部分が調べても出てこないのですが、意味としては指定したidとdeleted_atを選択するという事で合ってますか?
onslaught

2019/04/18 07:15

すみませんプレースホルダというんですね。解決しました。
yambejp

2019/04/18 07:16

ご自身でも提示しているbindValueと同等の処理です 疑問符型の指定をすると?の箇所に配列の値を順に適用していきます
onslaught

2019/04/18 23:59

ありがとうございます。 $dbh = connectPdo(); $now = date('Y-m-d H:i:s'); $sql = 'UPDATE todos SET deleted_at = :now WHERE id = :id'; $stmt = $dbh->prepare($sql); $stmt->bindvalue(':now', $post['deleted_at'], PDO::PARAM_STR); $stmt->bindvalue(':id', (int)$post['id'], PDO::PARAM_INT); $stmt->execute(); このような記述だと実行できないのですが、取得した現在時刻($now)が使われてないということでしょうか?
yambejp

2019/04/19 01:02

$stmt->bindvalue(':now', @now, PDO::PARAM_STR); $stmt->bindvalue(':id', $id, PDO::PARAM_INT); じゃないのですか?
onslaught

2019/04/19 03:43

解決しました。 度々ありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問