Q&A
コードが半端な状態です。
可能なら、HTML部分のコードも全体の流れが分かるような状態で記載してください。
前提・実現したいこと
PHPでいいね機能を作成していて、いいねボタンを押すとそのページが更新されて、
ボタン表記をいいね->いいね解除
表示するいいねの数を+1するという処理を作成しています
DBはいいねテーブルにユーザーIDと投稿IDの組み合わせで登録という処理
投稿テーブルのいいねカウントを+1
発生している問題・エラーメッセージ
いいねボタンを2回押さないと上記の処理ができない
1回目押すといいねテーブルにユーザーIDと投稿IDが追加
2回目押すと投稿テーブルのいいねが+1になる
これをボタン1回で処理したい
該当のソースコード
<!DOCTYPE html> <html> <head> <meta charset="UTF-8"> <title>投稿詳細</title> </head> <body> <?php require 'menu.php';?> <?php $pdo = new PDO('mysql:host=localhost;dbname=fps;charset=utf8','staff','password'); $sql=$pdo->prepare('select * from post_tbl where post_id = ?'); $sql->execute([$_REQUEST['post_id']]); foreach($sql as $row){ echo 'タイトル:'.$row['post_title'].'<br>'; echo '説明文<br>'; echo '<textarea name="post_detail" cols="50" rows="5">'.$row['post_detail'].'</textarea><br>'; echo 'カテゴリー:'.$row['post_category'].'<br>'; echo 'いいね:'.$row['post_like_count'].'<br>';
$sql=$pdo->prepare('select * from favorite_tbl where user_id = ? and post_id = ?'); $sql->execute([$_SESSION['login_id'],$_REQUEST['post_id']]); $result=$sql->fetch(); if(empty($result)){ echo '<form action="" method="post">'; echo '<button type="submit" name="add_like">いいね</button>'; echo '</form>'; if(isset($_POST['add_like'])){ $sql=$pdo->prepare('insert into favorite_tbl values(?,?)'); $sql->execute([$_SESSION['login_id'],$_REQUEST['post_id']]); $sql=$pdo->prepare('update post_tbl set post_like_count = post_like_count + 1 where post_id = ?'); $sql->execute([$_REQUEST['post_id']]); header('refresh'); } }else{ echo '<form action="" method="post">'; echo '<button type="submit" name="del_like">いいね解除</button>'; echo '</form>'; if(isset($_POST['del_like'])){ $sql=$pdo->prepare('delete from favorite_tbl where user_id = ? and post_id = ?'); $sql->execute([$_SESSION['login_id'],$_REQUEST['post_id']]); $sql=$pdo->prepare('update post_tbl set post_like_count = post_like_count - 1 where post_id = ?'); $sql->execute([$_REQUEST['post_id']]); header('refresh'); } } } ?> <?php require 'footer.php';?>
データベース
create table post_tbl( post_id int(10) auto_increment not null primary key, user_id int(10) not null, post_title varchar(50) not null, post_detail text not null, post_category varchar(20) not null, post_date date not null, post_like_count int(200) default 0 ); create table favorite_tbl( user_id int(10) not null, post_id int(10) not null, primary key(user_id,post_id) );
回答2件
0
ベストアンサー
単純にコードを辿るとわかりますが、
「データベースにいいね情報がない時」→「いいねボタンを表示」→「いいねが押されていたらデータベースに追加」
（逆も同じ）
という風になっているため、いいねボタンを押した瞬間は、いいねボタンを表示した上で、データベースに記録されます。
なので、順番的には、
「いいねが押されていたらデータベースに追加」し、「データベースにいいね情報がない時」→「いいねボタンを表示」
という風に書き換えれば意図した動作になるのではないかと思います。
余談ですが、データベースとphpのコード内で、like_tblとfavorite_tblで表記が一致していないので、書き間違えていないかだけ確認してみてください。
投稿2021/01/14 11:25
総合スコア1139
いくつか試してみたのですがうまくいきませんでした
順番の話は分かったのですがそれをどのように変更すればよいのかをもう少し詳しくお聞きしてもよろしいですか
> 順番の話は分かったのですが
ちなみに、ご自身のコード内で「データベースにいいね情報がない時」「いいねボタンを表示」「いいねが押されていたらデータベースに追加」がどれに該当しているか分かりますか？
そこは理解しています
それでは、「いいねが押されていたらデータベースに追加」を、「データベースにいいね情報がない時」の前、つまり今回であれば
echo 'タイトル:'.$row['post_title'].'<br>';
の前もしくは
$sql=$pdo->prepare('select * from favorite_tbl where user_id = ? and post_id = ?');
の前に持ってこれば(最低限の)やりたい動作になるのではないでしょうか？
（なお話の中ではいいねを押す前→押す後、の話のみしているので、いいねが押されている→外す動作も同様に変更する必要があります。）
ありがとうございます
押されている→外す動作のことを忘れていました
お手数かけてすみませんでした
0
直接の回答ではなく、提案です。
画面表示の中に処理ロジックが入り込んでいるため、「どういう順番で実行されるか」が見えにくくなっているように思います。
既に回答にあるように「基本は上から順番」なので、「出力の後にDB処理」では出力についてこないのは当然です。
ここはひとつ、完全に「画面表示」と「DBの処理」を分けてしまっては如何でしょうか。
現在のデータを画面表示
→ボタンを押す
→JavaScriptで非同期通信でリクエスト送信
→該当の投稿にいいね＋１＆現在のいいね数取得（DBの処理）
→現在のいいね数をJavaScriptで書き換え
DB関係の処理（特に更新）をバックグラウンドに任せることでメリハリもつくと思います。
投稿2021/01/14 12:03
総合スコア80875
どうしてもわからなかったらそうしようと思います
このやり方も結局「処理順」「ロジック」が理解できてないとできません。
あくまで「同期処理で出来た前提」です。「分からなかったときの逃げ道」ではないです。
本来は「様々なやり方を把握しておき場面にあった最適なやり方を選択する」うちの選択肢の１つです。
ちょっと厳しいことを言うと、
今の質問者さんにはむしろ私の提案のほうが難しいと思います。
あなたの回答
tips
プレビュー
