Где у меня ошибка (php)?

Question

[vlaczlav]

На сайте имеется HTML форма обратной связи для сайта + PHP (send.php).
Не могу понять, где у меня ошибка в коде php
Html код

<form ="send.php" method="post">
<div class="row">
<div class="col-md-12">
<div class="form-group">
<input type="text" class="form-control" name="name_contact_home" id="name_contact_home" placeholder="Имя">
</div>
</div>
<div class="col-md-12">
<div class="form-group">
<input type="text" class="form-control" name="email_contact_home" id="email_contact_home" placeholder="Ваш e-mail">
</div>
</div>
<div class="form-group">
<input type="submit" value="Отправить" class="btn_1 full-width" id="submit-contact-home">
</div>

PHP коде

spoiler

$name = ($_POST['name_contact_home']) ?? '';
$email = ($_POST['email_contact_home']) ?? '';
if ($name && $email) {
$message = 'Имя: '.$name.' Email: '.$email;
mail('моя почта@gmail.com', 'Тема сообщения' , $message)

Comments

[Максим Тимофеев]

1. Обычно текст ошибки явно указывает в чем она и на какой строке.
2. код можно оформлять как код
3. зачем скобки вокруг $_POST['name_contact_home']

[Сослан Хлоев]

Максим Тимофеев, это тернарный оператор, который записан неправильно

[Максим Тимофеев]

Сослан Хлоев, Вы думаете я не знал об этом и решил спросить? Тут есть автор вопроса, он спрашивает. И другие кто или дают ему ответы или задают уточняющие вопросы. Так что Ваш ответ мне - это очень странно.
"зачем скобки вокруг" - это вообще не вопрос, а намек на то что они не нужны там.

[Сослан Хлоев]

Максим Тимофеев, сорян, рефлекторно ответил на вопрос, без обид)

[vlaczlav]

Сама ошибка
$name = ($_POST['name_contact_home']) ?? ''; $email = ($_POST['email_contact_home']) ?? ''; if ($name && $email) { $message = 'Имя: '.$name.' Email: '.$email; mail('моя почта@gmail.com', 'Тема сообщения' , $message)

[Александр]

vlaczlav, нужен не только код, где ошибка, а ещё и сам текст ошибки

[Модератор]

vlaczlav, Не нужно писать комментарии в ответах

Answer 1

[Сослан Хлоев]

Тернарный оператор в первых двух строках неправильно оформлен. Не закрыт if.

if ($_POST['name_contact_home'] && $_POST['email_contact_home']) {
mail('моя почта@gmail.com', 'Тема сообщения' , 'Имя: ' . $_POST['name_contact_home'] . ' Email: ' . $_POST['email_contact_home']);
}

Должно работать. А тут можно почитать про тернарный оператор

Comments

[Кирилл Несмеянов]

А тернарный тут причём, если в коде оператор нулевого слияния (null coalescing)?

При этом код совершенно корректный и лучше даже написан чем в вашем ответе (и меньше ошибок содержит в том числе).

[Сослан Хлоев]

Кирилл Несмеянов, Вы правы. Перепутал с тернарным оператором. А какие ошибки у меня?

[Кирилл Несмеянов]

Сослан Хлоев, нет проверки на существование POST аргументов.

Answer 2

[Игорь]

Если уж проверять есть ли переменные, то хотя бы используйте функцию
isset()

Найдите в сети исходник обычной формы отправки, и посмотрите как устроено, а то это велосипед.
Например на: Github