Не корректно работает обратная связь на сайте

@kossstil · Регистрация: 26.09.2010

Author24 — интернет-сервис помощи студентам

HTML5

<form  action="send.php" method="post" >
<p>Введите Ваше имя:<br>
<input type="text" name="name"></p>
<p>Введите Ваш email:<br>
<input type="text" name="email"></p>
<p>Введите Ваше сообщение:<br>
<textarea name="message" rows="7" cols="40”></textarea>
<br>
<br>
<input type="submit" value="Отправить сообщение"></p>
</form>

PHP

<?php
 
if (isset($_POST['name'])) {$name = $_POST['name'];}
if (isset($_POST['email'])) {$email = $_POST['email'];}
if (isset($_POST['message'])) {$message = $_POST['message'];}
 
 
$address = "megapolis.all@yandex.ru";
 
 
$sub = "Обратная связь с megapolisss.okis.ru";
 
 
$mes = "Указанное имя обращения: $name \n Указанный E-mail связи: $email \n Содержание письма: \n$message";
 
 
$send = mail ($address,$sub,$mes,"Content-type:text/plain; charset = windows-1251\r\nFrom:$email");
if ($send == 'true')
{
echo "Сообщение отправлено!";
}
else 
{
echo "Сообщение не отправлено!";
}
?>

что не так? день мучаюсь

@insideone · 19.03.2012, 21:37

Функция mail возвращает boolean.

Я уж точно не помню как преобразует типы данных PHP в данном случае:

PHP

1	if ($send == 'true')

Но корректнее писать все таки:

PHP

1	if ( $send === true )

Если функция mail вернёт true и выполнится "Сообщение отправлено!", но в действительности сообщение не дойдёт это означает что проблемы скорее всего на стороне почтового сервера, возможно настроен он не так, или может адрес в спам-листе, тут уже к системному администратору нужно обращаться.

Если же функция mail вернёт false, то тут тоже возможны проблемы с настройкой сервера, а может и со входящими параметрами, попробуйте не передавать последний - "Content-type:text/plain; charset = windows-1251\r\nFrom:$email"

@gephaest · 19.03.2012, 23:06

insideone, можно еще короче

PHP

if($send) {
echo 'отправлено';
} else {
echo 'не отправлено';

UPD:
или даже так

PHP

1	echo ($send) ? 'отправлено' : не отправлено';

@insideone · 19.03.2012, 23:36

Не по теме:

Оффтоп

gephaest, да ладно?

PHP

if($send) {
echo 'отправлено';
} else {
echo 'не отправлено';

Лучше так:

PHP

if($send)
echo 'отправлено';
else
echo 'не отправлено';

а вместо

PHP

1	echo ($send) ? 'отправлено' : не отправлено';

можно

PHP

1	<?=$send?'':'не'?> отправлено

Померимся ещё? :) Да ну, какая разница :drink: не суть ведь)

Своим постом я имел ввиду следующее - нужно быть внимательным к типу возвращаемому функцией. Иногда это может сойти с рук, однако в случае вроде strpos может быть критичным. Вот и все ;)
А вот что за проблема с почтой ещё предстоит узнать, только вот мне кажется решать её придётся сисадмину, чаще так бывает

@gephaest · 20.03.2012, 00:08

Не по теме:

insideone, не хотел подколоть вас. Не принимайте близко к сердцу. Касаемо возвращаемых значений тут уже надо следить за кодом. Если ошибиться и случайно написать $foo = 'false', то интерпритатор примет это за true :)

@gephaest
	20.03.2012, 00:08 Не корректно работает обратная связь на сайте #5
	Не по теме: insideone, не хотел подколоть вас. Не принимайте близко к сердцу. Касаемо возвращаемых значений тут уже надо следить за кодом. Если ошибиться и случайно написать $foo = 'false', то интерпритатор примет это за true :) 0