Для чего нужны некоторые строчки в обработчике для писем?

Question

[Senseich]

Всем привет! В интернете ходит такой популярный обработчик для отправки писем, вот его код:

<?php

$method = $_SERVER['REQUEST_METHOD'];

//Script Foreach
$c = true;
if ( $method === 'POST' ) {

	$project_name = trim($_POST["project_name"]);
	$admin_email  = trim($_POST["admin_email"]);
	$form_subject = trim($_POST["form_subject"]);

	foreach ( $_POST as $key => $value ) {
		if ( $value != "" && $key != "project_name" && $key != "admin_email" && $key != "form_subject" ) {
			$message .= "
			" . ( ($c = !$c) ? '<tr>':'<tr style="background-color: #f8f8f8;">' ) . "
			<td style='padding: 10px; border: #e9e9e9 1px solid;'><b>$key</b></td>
			<td style='padding: 10px; border: #e9e9e9 1px solid;'>$value</td>
		</tr>
		";
	}
}
} else if ( $method === 'GET' ) {

	$project_name = trim($_GET["project_name"]);
	$admin_email  = trim($_GET["admin_email"]);
	$form_subject = trim($_GET["form_subject"]);

	foreach ( $_GET as $key => $value ) {
		if ( $value != "" && $key != "project_name" && $key != "admin_email" && $key != "form_subject" ) {
			$message .= "
			" . ( ($c = !$c) ? '<tr>':'<tr style="background-color: #f8f8f8;">' ) . "
			<td style='padding: 10px; border: #e9e9e9 1px solid;'><b>$key</b></td>
			<td style='padding: 10px; border: #e9e9e9 1px solid;'>$value</td>
		</tr>
		";
	}
}
}

$message = "<table style='width: 100%;'>$message</table>";

function adopt($text) {
	return '=?UTF-8?B?'.base64_encode($text).'?=';
}

$headers = "MIME-Version: 1.0" . PHP_EOL .
"Content-Type: text/html; charset=utf-8" . PHP_EOL .
'From: '.adopt($project_name).' <'.$admin_email.'>' . PHP_EOL .
'Reply-To: '.$admin_email.'' . PHP_EOL;

mail($admin_email, adopt($form_subject), $message, $headers );

Возможно я "загнался", но для чего эта проверка

$value != ""&& $key != "project_name" && $key != "admin_email" && $key != "form_subject"

Я понимаю $value != "" проверяет на пустоту, а дальше то что происходит?

Answer 1

[alex-1917]

ппц, вы где-то нарыли Г-код и просите объяснить, в чем его Г?)))

вам совсем наплевать на золотое время своих коллег?

следующей вашей темой наверное будет Почему метод mail не отправляет письма

Юзайте smtp! https://github.com/Ipatov/SendMailSmtpClass

Comments

[Senseich]

alex-1917, ну как не странно, но сам обработчик работает нормально. Я просо хотел разобраться в этом г-коде ) , а на счет smtp, спс, поюзаю.

Answer 2

[Кирилл Левунин]

$value != ""&& $key != "project_name" && $key != "admin_email" && $key != "form_subject"

&& - это значит И (А ТАКЖЕ). Равнозначно AND, просто короче и более читаемо
Далее идет проверки на поля project_name, admin_email, form_subject. То есть информация оп этим полям не должна попасть в таблицу

$message .= "
      " . ( ($c = !$c) ? '<tr>':'<tr style="background-color: #f8f8f8;">' ) . "
      <td style='padding: 10px; border: #e9e9e9 1px solid;'><b>$key</b></td>
      <td style='padding: 10px; border: #e9e9e9 1px solid;'>$value</td>
    </tr>
    ";

То есть подразумевается, что мы не фига не знаем что там за форма. Просто знаем что там есть три поля
project_name - название проекта
admin_email - почта администратора. То есть ваша наверно.
form_subject - Тема формы.

То есть это информацию выведена в другом месте (в теме письма и заголовках и собственно куда посылаем и от кого) и тут идет пропуск ее а все остальное в табличку пишется что бы потом отослать.

Ни знаю что популярного в этом скрипте. Я бы рекомендовал почитать про почту. Тема на самом деле обширная довольно.

Для php есть отличная библиотека (популярный скрипт гулящий по сети) для почты https://swiftmailer.symfony.com/

Comments

[Senseich]

Кирилл Левунин, ну я знаю что такое логический оператор означает. Но тут же ключ, в любом случае будет project_name, admin_email или form_subject. И раз ключ не должен им равняться, то что же тогда сюда попадает?

<td style='padding: 10px; border: #e9e9e9 1px solid;'><b>$key</b></td>

И нафига в пост почта админа, её не легче в обработчике прописать так. Получается она должна быть в форме. обычно через форму получают почту посетителя.

[Stalker_RED]

Senseich,

Но тут же ключ, в любом случае будет project_name, admin_email или form_subject.

С чего вы взяли? В запросе можно любые параметры указать.

И да, этот "обработчик" - яркий представитель говнокода, который лучше выбросить и не использовать никогда.

[Senseich]

Stalker_RED, а вот здесь так и должно быть?:

'From: '.adopt($project_name).' <'.$admin_email.'>' . PHP_EOL .
'Reply-To: '.$admin_email.'' . PHP_EOL;

Мы получается от самого себя отправляем себе

[Олег]

Правильные вопросы у Вас. Выкиньте этот говнокод.
Через него только спам удобно рассылать.