空欄でｸﾘｯｸしたとき、エラーを表示させたい

前提・実現したいこと

phpとmysqlでtodolistを作りっており、
空欄でｸﾘｯｸするとエラーをechoさせたいのですが、どこに書くべきかわからないため、
もしよかったら教えてください。

phpとhtmlの記載順がよくわかっておらず、phpから書いて、htmlを書く、
など決まりがあるのかも含めて教えていただけば幸いです。

コード

お手数をおかけしますがどうぞよろしくお願いいたします。

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

syntax error, unexpected '{' in /home/vagrant/todos/Todolist.php on line 12

該当のソースコード

php
1<?php
2    try{
3    $pdo = new PDO("mysql:host=localhost;dbname=mydb;charset=utf8","root","");
4    $pdo->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
5    if(isset($_POST['submit'])){
6        $name=$_POST['name'];
7        $sth=$pdo->prepare("INSERT INTO todos(name)VALUES(:name)");
8        $sth->bindValue(':name',$name,PDO::PARAM_STR);
9        $sth->execute();
10    }
11    elseif(isset($_POST['delete'])){
12        $id=$_POST['id'];
13        $sth=$pdo->prepare("delete from todos where id=:id");
14        $sth->bindValue(':id',$id,PDO::PARAM_INT);
15        $sth->execute();
16    }
17    }catch(PDOException $e){  
18        echo($e->getMessage());         
19    }
20?>

html
1<!DOCTYPE HTML>
2<html lang="ja">
3<head>
4    <title>Todo List</title>    
5</head>
6
7<body class="container">
8    <h1>Todo List</h1>
9    <form method="post" action="">
10        <input type="text" name="name" value="">
11        <input type="submit" name="submit" value="Add">
12    </form>    
13    <h2>Current Todos</h2>
14    <table class="table table-striped">
15        <therad>
16        <th>Task</th>
17        <th></th>
18        </therad>
19        <tbody>
20<?php
21    $sth = $pdo->prepare("SELECT * FROM todos ORDER BY id DESC");
22    $sth->execute();  
23    
24    foreach($sth as $row) {
25?>
26            <tr>
27                <td><?= htmlspecialchars($row['name']) ?></td>
28                <td>
29                    <form method="POST">
30                        <button type="submit" name="delete">Delete</button>
31                        <input type="hidden" name="id" value="<?= $row['id'] ?>">
32                        <input type="hidden" name="delete" value="true">
33                    </form>
34                </td>
35            </tr>
36<?php
37    }
38?>
39        </tbody>
40    </table>
41<?php
42if(isset($_POST['submit'])&&(empty($_POST['name'])){
43    echo "you should fill out";
44}
45?>
46</body>
47</html>

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

PHP 5.6.40　、mysql 5.6.47 ,windows 10

Orlofsky

2020/06/04 14:28

半角カタカナを使うのは止めては？

Orlofsky

2020/06/04 14:29

コードは https://teratail.com/help/question-tips#questionTips3-7 の [コード] に修正してください。

kawa00

2020/06/06 11:08

回答いただきありがとうございます。体調を崩してしまい、返信が遅くなり申し訳ありません。アドバイスいただき、ありがとうございます。今後気を付けて投稿いたします。貴重なアドバイスありがとうございました。

kei344

2020/06/06 11:22

ここは「質問への追記・修正の依頼」です。質問本文は編集することが可能なので、その依頼をされています。

kawa00

2020/06/06 11:49

回答いただきありがとうございます。そうなのですね、申し訳ありませんでした。修正いたします。ありがとうございました。

Orlofsky

2020/06/06 18:08

半角カタカナに拘ったり、英数字を全角文字で書いたり、字下げが雑だったり、チームのコーディング・ルールを無視して嫌われるパターン。

kai0310

2020/06/07 12:41

ですね。せめてロジックはわけてもらいたいです。

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

回答2件

ベストアンサー

以下の部分の意図がわかりません。

PHP
1if(isset($_POST['submit'])){

いくつか方法があるかと思いますが、HTMLのバリデーションの場合はrequired属性をつけてあげれば良いかと思います。

PHPで行うのであれば、空文字かどうかで判定してあげる方法が理解しやすいかと思います。

PHP
1// nameの場合
2if($name == ''){
3  // 処理...
4}

まずは簡単にから文字かどうか判定して真であればechoしてあげるコードを記述してイメージしてみてください。

PHP
1<?php 
2if(empty($_POS['name'])){
3  echo 'you should fill out';
4}

asuchi0819さんの回答でもありますが、issetとemptyどちらでも判定することは可能です。
この場合意味が変わりますのでそちらも注意してください。

投稿2020/06/05 01:34

編集2020/06/06 15:09

kai0310

総合スコア2070

kawa00

2020/06/06 11:05

回答いただきありがとうございます。体調を崩してしまい、返信が遅くなり申し訳ありません。 if(isset($_POST['submit'])){　は、submitボタンをクリックしたとき、という意味で書きました。空欄状態で、submitボタンをクリックしたとき、you should~が表示されるようにしたかったのですが、 try部分に書くのか、html部分に書くのかよくわかっておりませんでした。 phpに以下を書いたところ、エラーは出ず、ｱｸｾｽ時点でyou should~が表示されていました。 if($name==''){ echo "you should fill out"; } requireは、どのように書けばよいか理解不足で、できませんでした。せっかくアドバイスいただいたのに、申し訳ありません。

kai0310

2020/06/06 11:20

> submitボタンをクリックしたとき、you should~が表示されるようにしたかったのですが、意図がわからないのですが、submitを押された時すなわちpostリクエストがされた時。というコードをかけば良いのではないでしょうか？ if ($_SERVER['REQUEST_METHOD'] == 'POST' && isset($_POST['user']) { // 処理... }

kai0310

2020/06/06 11:26

> 体調を崩してしまい、返信が遅くなり申し訳ありません。お大事にどうぞ

kawa00

2020/06/06 11:48

早急にご回答いただきありがとうございます。下記を書いてみたところ、エラーなどはなかったです。 todolistに未記載(空欄)のままクリックしたとき、画面にエラーyou should~を表示させたい、という目的となります。(ここの文言は表示されませんでした) if($_SERVER['REQUEST_METHOD']=='POST'&&isset($_POST['user'])){ echo "you should fill out"; } わかりづらく申し訳ありません。

kai0310

2020/06/06 15:09

追記しました。

kawa00

2020/06/07 08:17

ご回答ありがとうございます。下記のように試してみましたが、最初からyou should~が表示されており、todolistに未記載(空欄)のままクリックしたとき、画面にエラーyou should~を表示させたい、という風にはなりませんでした。理解不足で申し訳ありません。 if(empty($_POST['name'])){ echo "you should fill out"; }

kai0310

2020/06/07 12:40

先日説明した通りリクエストされた時にifにしないと表示されますよ

kawa00

2020/06/07 13:25

たくさん返信いただき、本当にありがとうございます。下記のようにtryの中に記載したところ、初めて空欄でｸﾘｯｸの状態でのみ、you shoud~が表示されました。ありがとうございました。 if($_SERVER['REQUEST_METHOD']=='POST'&&isset($_POST['submit'])){ if(empty($_POST['name'])){ echo "you should fill out"; } }

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