コンタクトフォームについて

Question

inputタグtextareaタグの名前とメールアドレスとお問い合わせの３項目を入力しないと送信できないようにしたいのですが、今のコードの状態だと何も値を打ち込まない状態で送信ボタンを押してもメールが送れてしまいます。やってみたことはsend.phpとゆうファイルの所にmail関数、isset関数を定義してissetの方に値があればtrue、なければfalthにしたいのですがうまく行きません。解決方法教えてもらえますでしょうか。

コード
<form action="send.php" method="POST">
        <div class="form-content pt-5">
          <div class="form-left ">
            <div class="form-group">
              <input type="text" class="form-control" name="name" placeholder="名前">
            </div>
            <div class="form-group mt-5">
              <input type="text" class="form-control" name="email" placeholder="メールアドレス">
            </div>
          </div>
          <div class="form-right">
            <div class="form-group">
              <textarea class="form-control" name="message" placeholder="お問い合わせ" rows="10"></textarea>
            </div>
          </div>
        </div>
        <p><input type="submit" name="submit" value="上記の内容で送信する" class="text-center" id="button"></p>
      </form>

コード<?php

if (isset($_POST['submit'])) {
    $name = $_POST['name'];
    $emailFrom = $_POST['email'];
    $message = $_POST['message'];

    $mailTo = "[e-mail]";
    $headers = "From: ".$emailFrom;
    $txt ="You have received an e-mail from " .$name.".\n\n".$message;

    if (mail($mailTo, $headers , $txt) && isset($mailTo, $headers , $txt)) {
        echo "<h1>お問い合わせありがとうございます"." ".$name."様。折り返し<br>連絡いたしますので、今しばらくお待ちください。</h1>";

    } else {
        echo "<h1 class='text-danger'>送信できませんでした。全項目ご入力ください。</h1>";

    }

Answer 1

if (mail($mailTo, $headers , $txt) && isset($mailTo, $headers , $txt))
この処理順は
mail($mailTo, $headers , $txt)を行ってからisset($mailTo, $headers , $txt)を行うので、逆にしてみたらいかがでしょう。

＆issetは空文字でもtrueになるのでemptyのほうがいいかも

Comments

[rururu3]

$name = $_POST['name'];
$emailFrom = $_POST['email'];
$message = $_POST['message'];
これらの変数が入っているかを見ないといけないですね…(isset($mailTo, $headers , $txt)の各変数は何かしらの文字列入ってるので意味がない)

[fuyukixxx]

返信ありがとうございます。mailとisset逆にしましたが結果変わらず。emptyの方は引数を３つにするとエラーががでます。issetの変数を
if (isset($name, $emailFrom, $message) && mail($mailTo, $headers , $txt))
でもメールが項目の方打たなくても送られてしまいます。

[rururu3]

回答の方でissetの変数までちゃんと見てませんでしたコメントで記載したとおり
$name, $emailFrom, $messageは処理で何かしらの文字列入ってるのは確実なので絶対にtrueですので無意味です。

$_POST['name'];　$_POST['email'];　$_POST['message'];が入ってるかをempty関数を使って調べてください。
https://www.php.net/manual/ja/function.empty.php
をみてempty関数を理解していただければ＆emptyの方は引数を３つにするとエラーががでる理由もわかるかと