Comments 19
Это же в документации все есть.
UPD:
С телефона не заметил. Все вопросы к RigelGL
Приглашён
21 марта 2019 в 20:01 по приглашению пользователя RigelGL
В общем, все способы отправить письмо в спам.
Спрячьте и не никому этот код не показывайте.
Вы бы ещё показали, как echo() пользоваться… И ладно, если бы синтаксических ошибок в тексте и в самом коде не было.
[сарказм]
Отличная и полезная статья, а главное — актуальная для 2019го.
Реквестую статью про strtr и str_replace.
[/ссрказм]
Функция mail дергает локальный, для сервера на котором работает PHP-скрипт, MTA — а это потенциальный источник проблем:
- локальный MTA может отсутствовать
- локальный MTA может давно и прочно прописаться во всех blacklist-ах какие только бывают и администратор сервера может не торопиться его оттуда вытаскивать
- локальный MTA может упасть и администратор может забить на это на денек другой
- локальный MTA может вызывать у получателей серьезные сомнения относительно того, не СПАМ ли это
- mail, как написано в документации, норовит открывать и закрывать соединение с MTA при каждом вызове, что негативно сказывается на производительности
Мораль — используйте полноценный почтовый сервер и обращайтесь к нему через SMTP (для удобства есть соответствующие библиотеки, например PHPMailer)
Машина времени, однако (без обид). Но вы (автор) как то запоздали на пару лет.
1. Двойные кавычки и лишние переменные
$subject = "тема письма"; 2. Расположение и название файла
$filename = "file.doc";
// название файла
$filepath = "files/file.doc";
// месторасположение файла- Путь до файла лучше указывать с использованием магической переменной __DIR__
- Также лучше работать с абсолютным путем, использую функцию realpath()
- Переменную $filename не нужно задавать, можно получить ее из сформированного пути функцией basename($filepath)
3. Комментарии снизу
Комментарии в коде обычно оставляют либо на той же строке, если это //, либо над строкой, которую планируете прокомментировать.
4. Формирование многострочной переменной
Для улучшения читаемости кода, лучше не конкатинировать много строк, а использовать HEREDOC или NOWDOC:
$mailheaders=<<<HEADERS
MIME-Version: 1.0;
Content-Type: multipart/mixed; boundary="$boundary"
From: $user_email <$user_email>
Reply-To: $user_email
HEADERS;5. Нет корректной обработки ошибок
$multipart .= chunk_split(base64_encode(iconv("utf8", "windows-1251", $message)));if (!$fp)
{
print "Не удается открыть файл22";
exit();
}6. Чтение файлаЕсли понадобилось читать файл, но не обязательно использовать потоки, можно вытащить контент тем же самым file_get_contents.
7. Нет кавычек
$message_part .= \r\n;8. Удаление файла
//удаляем файлы через 60 сек.
if (time_nanosleep(5, 0)) {
unlink($filepath);
}9. Код оформлен не по PSR
P.S. Если что-то пропустил, то дополняйте, коллеги!
Это очень забавный комментарий. В стиле, я не знаю, "Вы тут представили телегу без лошади как средство передвижения. Чтобы сделать её более полезной для общества, вот мои замечания: колеса лучше перекрасить в чуть более розовый цвет, борта лучше сделать не из дерева, а из картона потому что красивше и вот еще на узорчиках лучше листики заменить на цветочки."
Все эти замечания не делают код полезнее. Чтобы сделать его полезнее, его надо переписать, целиком. С использованием современных инструментов. О чем и говорили все предыдущие комментаторы.
Все что написано вверху — это мелкие придирки, половина из которых — это вкусовщина про цветочки. Какие кавычки использовать и применять ли хередок — это исключительно на вкус автора.
При этом построчная конкатенация является стандартом при составлении заголовков, поскольку упрощает отладку, позволяя произвольно комментировать отдельные заголовки. Плюс такой формат гарантирует использование строго определенных символов перевода строки, а не тех, что настроены в редакторе по умолчанию.
Корректная обработка ошибок не имеет ничего общего с добавлением проверок на каждый чих. И уж тем более с выплевыванием сообщений об ошибке прямо в браузер. Пользователю сайта нет ни малейшего интереса читать про файлы на сервере. Такого рода сообщения должны писаться в лог. И РНР с этим великолепно справляется сам, помогать ему не требуется.
Без обид, но чтобы сделать нормальный code review, надо сначала научиться кодить самому. Пока у вас получилось не code review, а cosmetic review. Ваш обзор не делает код лучше, он делает его чуть красивше и более подходящим под ваши субъективные вкусы.
Все эти замечания не делают код полезнее.Никто и не говорил, что код от моих замечаний станет полезнее. Это замечания, которые нужны для того, чтобы прочитавший в своих проектах так не писал.
При этом построчная конкатенация является стандартом при составлении заголовков, поскольку упрощает отладку, позволяя произвольно комментировать отдельные заголовки. Плюс такой формат гарантирует использование строго определенных символов перевода строки, а не тех, что настроены в редакторе по умолчанию.Код нужно писать не так, чтобы программисту было удобно что-то закомментировать в процессе написания, а чтобы потом было проще прочитать и понять.
Без обид, но чтобы сделать нормальный code review, надо сначала научиться кодить самому. Пока у вас получилось не code review, а cosmetic review. Ваш обзор не делает код лучше, он делает его чуть красивее и более подходящим под ваши субъективные вкусы.Какие обиды? Судя по вашим
Отправка почты средствами PHP