|
|
|
| Добрый день, уважаемые форумчане.
Может кто-нибудь, что-нибудь сказать по поводу этого кода:
<?
//получаем данные из аякс формы и формируем письмо на мыло
if($_POST['spls']=='' && $_POST['phone']=='' && $_POST['aress']==''){
die();
}
if($_POST['komns']==1){
$comts = "1";
}
elseif($_POST['komns']==2){
$comts = "2";
}
elseif($_POST['komns']==3){
$comts = "3";
}
elseif($_POST['komns']==4){
$comts = "4 и более";
}
else{
$comts = "";
}
if($_POST['cutys']==1){
$typeray = 'Москва';
}
elseif($_POST['cutys']==2){
$typeray = 'МО';
}
else{
$typeray = '';
}
$mess="Площадь: ".$_POST['spls']."\r\n";
$mess.="Телефон: ".$_POST['phone']."\r\n";
$mess.="Адрес: ".$_POST['aress']."\r\n";
$mess.="Комнат: ".$comts."\r\n";
$mess.="Район: ".$typeray."\r\n";
if(mail('usermail@mail.ru','Заявку на оценку',$mess)){
echo 1;
}
|
| |
|
|
|
|
|
|
|
для: tvv123456
(06.04.2016 в 13:15)
| | if($_POST['komns']==1){
$comts = "1";
}
elseif($_POST['komns']==2){
$comts = "2";
}
elseif($_POST['komns']==3){
$comts = "3";
}
elseif($_POST['komns']==4){
$comts = "4 и более";
}
else{
$comts = "";
}
Это никчемное бесполезное занятие и к проверке никакого отношения не имеющее. Проверка, это согласно условиям - либо обязательно заполнено, либо в пределах диапазона, только цифры или строковые значения, и т.д, и т.п. В данном же случае вы просто значение одной переменной (а $_POST['komns'] такая же переменная как $comts) приравниваете другой, а зачем?
И почтовые отправления так не оформляют
mail('usermail@mail.ru','Заявку на оценку',$mess)
это можно делать для тестовой отправки и на англицком, на русском не факт еще, что читатель прочтет именно 'Заявку на оценку'. | |
|
|
|
|
|
|
|
для: confirm
(06.04.2016 в 15:15)
| | confirm, спасибо вам. Был бы еще благодарен, если бы сказали, что отправлять данные от пользователя на почту без какой-либо обработки не совсем безопасно. Но вы единственный кто откликнулся :) Может еще кто скажет, что? Как вам именование переменных? | |
|
|
|
|
|
|
|
для: tvv123456
(06.04.2016 в 13:15)
| | Не стоит обращаться напрямую к суперглобальному массиву $_POST вместо этого используйте функции фильтрации filter_input(), filter_array(), is_*(), и другие | |
|
|
|