Помогите разобраться с формой (PHP).

Автор tabits, 25-06-2016, 17:06:46

« назад - далее »

tabitsTopic starter

Знающие люди, помогите разобраться с формой обратной связи.

Это HTML.
<div class="registration-form">
<form enctype='multipart/form-data' method='post' action='http://magic-of-massage.com/mail.php'>

<input name='admin_mode' type='hidden' value='' />
<input name='catalogue' type='hidden' value='1' />
<input name='cc' type='hidden' value='5' />
<input name='sub' type='hidden' value='23' />
<input name='posting' type='hidden' value='1' />
<input name='isNaked' type='hidden' value='1' />

<input name="f_Name" type="text" placeholder="Ваше имя*" class="required" value="" />
<input name="f_Phone" type="text" placeholder="Телефон*" class="required" value="" />
<input name="f_Email" type="text" placeholder="E-mail*" class="required" value="" />
<input name="f_Residence" type="text" placeholder="Место жительства*" class="required" value="" />
<textarea name="f_Experience" placeholder="Сообщение"></textarea>
<input type="submit" name="send" value="Отправить" />
<p class="required">*-обязательно заполнить!</p>
<div class="done-mes"></div>
</form>
</div>
</div>


Это PHP.

<?php
    session_start
(oid);
    if (isset(
$_POST["send"])) {
    
 $f_Name htmlspecialchars ($_POST["f_Name"]);
    
 $f_Phone htmlspecialchars ($_POST["f_Phone"]);
         
$f_Email htmlspecialchars ($_POST["f_Email"]);
         
$f_Residence htmlspecialchars ($_POST["f_Residence"]);
         
$f_Experience htmlspecialchars ($_POST["f_Experience"]);
         
$_SESSION["f_Name"] = $f_Name;
         
$_SESSION["f_Phone"] = $f_Phone;
         
$_SESSION["f_Email"] = $f_Email;
         
$_SESSION["f_Residence"] = $f_Residence;
         
$_SESSION["f_Experience"] = $f_Experience;
    }
?>


Заранее спасибо!
  •  


Старый

Ну и?
У вас форма в HTML + открытие сессий в PHP.
Мы должны удивиться?  :)
Вопрос в чём? И дайте обработку формы.  :)


tabitsTopic starter

Вопрос в том что не работает. Что такое обработчик?
  •  

Старый

Обработчик это та часть кода, которая обрабатывает данные пришедшие из формы.
У вас, судя по приложенному коду, идёт экранирование символов переданных POST-запросом и их переименование. Потом вы пытаетесь эти переменные снова наполнить данными из сессии.
1. Сессия открыта, но данных там нет.
2. В начале переменные наполнены POST-запросом, а потом очищены, так как вы удаляете то, что было вначале, перезаписываете их. (В сессии нет данных == пустые переменные).
3. Как скрипт должен работать, если в коде нет действий? Они не прописаны.

После экранирования должны быть проверки (длинны, символов, соответствия написания мыла и т.д.).
И потом уже отправка письма, и если нужно, запись инфы в базу или файл.
В вашем коде этого нет.

tabitsTopic starter

#4
Вы имеете в виду код PHP? Проверки обязательны?
  •  


Старый

Да, PHP.
Проверки делаются для проверки соответствия полей. Пардон за тафтологию.  :)

Чтобы были обязательные поля, телефон содержал только цифры, мыло имело вид mail@mail.ru, вам не могли отправить пустое, очень короткое или очень длинное сообщение и т.д.

tabitsTopic starter

А так?
<?php
#  В данном файле ничего менять не нужно, однако если у вас есть трудности с возвратом на указанную
#  в настройках страницу, то отредактируйте строку 100 как указано в коментарии.
error_reporting(0);
require("./config.inc");
require("./header.inc");
# Функции
  function formtohtml ($str) {
    if (get_magic_quotes_gpc()) {
    $str = stripslashes($str);
    }
  $str = trim($str);
  $str = htmlspecialchars ($str, ENT_QUOTES);
  $str = str_replace("|", "/", $str);
  $str = str_replace("\r", "", $str);
  $str = str_replace("\n", "<br>", $str);
  $str = eregi_replace("(<br>*){2,}","<br><br>",$str);
  $str = eregi_replace("[ ]{2,}", " ", $str);
  return $str;
  }
  function htmltoform ($str) {
  $str = str_replace("&amp;", "&", $str);
  $str = str_replace("&quot;", "\"", $str);
  $str = str_replace("'", "'", $str);
  $str = str_replace("&lt;", "<", $str);
  $str = str_replace("&gt;", ">", $str);
  $str = str_replace("<br>", "\r\n", $str);
  return $str;
  }
# Отправка
  if (!empty($_GET['a']) and $_GET['a'] == "m") {
  $var['fields'] = array("name"=>"имя", "email"=>"e-mail", "subject"=>"заголовок", "message"=>"текст сообщения");
  $notice['error'] = array();
    foreach ($var['fields'] as $key => $value) {
    if (empty($_POST[$key]) || (($_POST[$key] = formtohtml($_POST[$key])) == "0")) $notice['error'][] = $value;
    }
    if (empty($notice['error'])) {
      if (preg_match("/^([a-z,0-9,_,\-,\.])+\@([a-z,0-9,_,\-])+(\.([a-z,0-9])+)+$/",$_POST['email'])) {
      $t['c'] = file($config['recip_f']);
      $t['n'] = sizeof($t['c']);
        for ($i=0;$i<$t['n'];$i++) {
        $t['c'][$i] = explode("|",trim($t['c'][$i]));
        }
        if (!empty($_POST['recip']) and !empty($t['c'][$_POST['recip']-1])) {
        $var['recip_n'] = $t['c'][$_POST['recip']-1][1];
        $var['recip_e'] = $t['c'][$_POST['recip']-1][0];
          if (function_exists("imap_binary")) {
            function mail_convert($str) {
            $str = trim(imap_binary(addcslashes($str, "\"!@\\!@(!@)")));
            return $str;
            }
          $var['subj'] = "=?Windows-1251?B?".trim(imap_binary(htmltoform($_POST['subject'])))."?=";
          $var['header']  = "From: =?Windows-1251?B?".mail_convert(htmltoform($_POST['name']))."?= <".$_POST['email'].">\r\n";
          $var['header'] .= "MIME-Version: 1.0\r\n";
          $var['header'] .= "Content-Transfer-Encoding: 8bit\r\n";
          $var['header'] .= "Content-Type: text/plain; charset=\"Windows-1251\"\r\n";
          $var['header'] .= "X-Mailer: PHP v.".phpversion();
          $var['message']  = htmltoform($_POST['message']);
          if (!empty($config['features'])) $var['message'] .= "\r\n\r\n---------------------------------------\r\nIP адрес отправителя: ".$_SERVER['REMOTE_ADDR'];
          # Пытаемся послать сообщение
            if (mail("=?Windows-1251?B?".mail_convert(htmltoform($var['recip_n']))."?= <".$var['recip_e'].">",$var['subj'],$var['message'],$var['header'])) {
            $notice['ok'] = "Сообщение успешно отправлено";
            }
            else {
            $notice['error'] = "Системная ошибка - сбой функции &quot;mail&quot;";
            }
          }
          else {
          $notice['error'] = "Ошибка программы - недоступна функция &quot;imap_binary&quot;";
          }
        }
        else {
        $notice['error'] = "Ошибка настоек программы - неверный получатель";
        }
      }
      else {
      $notice['error'] = "Пожалуйста, проверьте правильность e-mail";
      }
    }
    else {
    $notice['error'] = "Пожалуйста, введите ".implode(", ",$notice['error']);
    }
  }
echo "<table border=\"0\" cellpadding=\"3\" cellspacing=\"0\">\r\n";
  if (empty($notice['ok'])) {
  echo "<form name=\"SM\" method=\"POST\" action=\"".$_SERVER['PHP_SELF']."?a=m\">\r\n";
  }
  else {
  // ПРИ ВОЗНИКНОВЕНИИ ОШИБКИ "METHOD POST NOT ALLOWED" ИЗМЕНИТЬ "POST" на "GET"
  echo "<form name=\"GT\" method=\"POST\" action=\"".$config['url_path']."\">\r\n";
  }
# Сообщение о завершении / ошибке
  if (!empty($notice)) {
  echo "<tr><td align=\"center\">";
    if (!empty($notice['ok'])) {
    echo "<p><b><font color=\"#008000\">".$notice['ok']."!</font></b></p>";
    }
    elseif (!empty($notice['error'])) {
    echo "<p><b><font color=\"#800000\">".$notice['error']."!</font></b></p>";
    }
  echo "</td></tr>\r\n";
  }
# Вывод формы
  if (empty($notice['ok'])) {
  echo "<tr><td align=\"center\">";
  require("./form.inc");
  echo "</td></tr>\r\n";
  }
echo "<tr><td align=\"center\"><table border=\"0\" cellpadding=\"5\" cellspacing=\"0\"><tr><td>";
# Кнопки
  if (empty($notice['ok'])) {
  echo "<input type=\"submit\" value=\"Отправить\"></td>\r\n";
  echo "<td><input type=\"reset\" value=\"Сбросить\">\r\n";
  }
  else {
  echo "<input type=\"submit\" value=\"".$config['url_name']."\">\r\n";
  }
echo "</td></tr></table></td></tr>\r\n";
  if (!empty($config['copyshow'])) {
  echo "<tr><td align=\"center\"><font style=\"font-size: 10px;\">PHPSendMail v.2.1.2 © 2004</font></td></tr>\r\n";
  }
echo "</form></table>\r\n";
# Футер
require("./footer.inc");
?>
  •  

Старый

Хм... Мудрёно написано.  :)
А как форма у вас не работает? Совсем? Что-нить говорит? Ошибки выдаются?

Если тишина полная, то в строке error_reporting(0); поставьте "1". Вот так: error_reporting(1);
И посмотрите, какие ошибки выдаются.
У вас там какой-то дизайн вписан, поэтому можете не увидеть сразу ошибки. Нажмите Ctrl+U и посмотрите в коде. Если выдаются ошибки, то там точно увидите.

Покажите, на что скрипт будет ругаться.


tabitsTopic starter

Посмотрю, сейчас основная проблема в том что письма на приходят.
  •  


tabitsTopic starter

А где лучше расположить: в отдельном файле PHP или в теле HTML?
  •