Как бороться с неопределёнными переменными?

Question

[Alex_87]

Здравствуйте! Столкнулся с такой проблемой! На локальном сервере, таких ошибок не было. Они выявились когда я залил сайт на хост. Сначала было это...

потом, с добавление этой строчки, ошибки ушли

но не совсем... Дело в том, что в самой форме, они остались

Мне посоветовали изменить код добавив это:

Но я не знаю как правильно добавить и что за два знака вопроса? Тернарный оператор?
Помогите пожалуйста решить. Поддержка на хостинге не может. Уже обращался!

Comments

[Twelfth Doctor]

Код формы и код её PHP разработчика в студию.

[Alex_87]

verdex,
Код обработчика формы:

error_reporting(0);
                 if(!empty($_POST['name'])){
                     $to = '[e-mail]';
                     $tema = 'Новое сообщение';
                     $name = strip_tags(trim($_POST['name']));
                     $email = strip_tags(trim($_POST['email']));
                     $skype = strip_tags(trim($_POST['skype']));
                     $sms = strip_tags(trim($_POST['sms']));
                     $message = "Имя пользователя: ".$name."\r\n";
                     $message .= "Почта: ".$email."\r\n";
                     $message .= "Skype: ".$skype."\r\n";
                     $message .= "Сообщение: ".$sms."\r\n";
                     $error = false;
                     $err = array('email' => '', 'name' => '', 'sms' => '');

                     if (empty($email) || !filter_var($email, FILTER_VALIDATE_EMAIL)){
		          $err['email'] = 'Вы ввели не корректный email';
			  $error = true;
		     }
                     if (empty($name)){
		          $err['name'] = 'Вы не ввели имя';
			  $error = true;
		     }


                    // if(!$error){
                        $tema = "=?utf-8?B?".base64_encode($subject)."?=";
                        $headers = "From: $from\r\nReply-to: $from\r\nContent-type:text/plain; charset=utf-8\r\n";
                        mail($to, $tema, $message, $headers);
                       // if(mail($to, $subject, $sms, $headers)){
                           //  header('Location: /otzivi.php');
                       // }
                    // }
                 }

Код формы:

<div class="popupcontact">
            <div class="wrapper">
                 <div class="popupcontacttitle" id="about">Оставьте заявку сейчас и получите бесплатную консультацию!</div>
                 <form id="form" method="post" action="form.php" class="popupform">
                     <input type="text" placeholder="Ваше имя" id="name" name="name"  value="<?=$_POST['name']?>"  />

                     <input type="e-mail" placeholder="Ваша почта" id="email" name="email"  value="<?=$_POST['email']?>" />

                     <input type="text" placeholder="Ваш Skype"  name="skype"  value="<?=$_POST['skype']?>" />
                     <textarea name="sms" cols="5" rows="10" placeholder="Ваше сообщение" value="<?=$_POST['sms']?>"></textarea>

                     <div>
                         <button type="submit" id="sbm">Оставьте заявку</button>
                     </div>
                 </form>

              </div>
            </div>

[Артем Ефимов]

Alex_87, в форме вы выводите переменные которых еще нет, можно изменить свой код примерно вот так
<?=$_POST['name'] ?? '';?>

[Alex_87]

Артем Ефимов, мне этот синтаксис немного не знаком, с 2 знаками вопроса. Это тернарный оператор, как я понимаю... Вставьте пожалуйста это в код, чтобы было видно...

Answer 1

[Андрей Андреев]

php.net/manual/ru/function.error-reporting.php

Comments

[Alex_87]

Так он у меня почему то не сработал. В коде он есть! Почему не работает?

error_reporting(0);
                 if(!empty($_POST['name'])){
                     $to = '[e-mail]';
                     $tema = 'Новое сообщение';
                     $name = strip_tags(trim($_POST['name']));
                     $email = strip_tags(trim($_POST['email']));
                     $skype = strip_tags(trim($_POST['skype']));
                     $sms = strip_tags(trim($_POST['sms']));
                     $message = "Имя пользователя: ".$name."\r\n";
                     $message .= "Почта: ".$email."\r\n";
                     $message .= "Skype: ".$skype."\r\n";
                     $message .= "Сообщение: ".$sms."\r\n";
                     $error = false;
                     $err = array('email' => '', 'name' => '', 'sms' => '');

                     if (empty($email) || !filter_var($email, FILTER_VALIDATE_EMAIL)){
              $err['email'] = 'Вы ввели не корректный email';
        $error = true;
         }
                     if (empty($name)){
              $err['name'] = 'Вы не ввели имя';
        $error = true;
         }


                    // if(!$error){
                        $tema = "=?utf-8?B?".base64_encode($subject)."?=";
                        $headers = "From: $from\r\nReply-to: $from\r\nContent-type:text/plain; charset=utf-8\r\n";
                        mail($to, $tema, $message, $headers);
                       // if(mail($to, $subject, $sms, $headers)){
                           //  header('Location: /otzivi.php');
                       // }
                    // }
                 }

Answer 2

[Артем Ефимов]

Перед выводом проверяйте переменные на пустоту/существование.
error_reporting(0); - скрывает ошибки, почему не скрыл ошибки на форме, видимо у вас это другой файл.
Но ошибки лучше исправлять чем скрывать.
Как проверить на пустоту или существование, вам гугл подскажет кучей ответов на том же тостере.

Comments

[Alex_87]

Вставьте пожалуйста свои код <?=$_POST['name'] ?? '';?> в мой! Покажите как правильно...

Answer 3

[Fudo Tsukiko]

if(empty($var))

Comments

[Alex_87]

Подскажите пожалуйста, мне посоветовали использовать следующий код
<?=$_POST['name'] ?? '';?>
Как мне его сюда вставить?

error_reporting(0);
                 if(!empty($_POST['name'])){
                     $to = '[e-mail]';
                     $tema = 'Новое сообщение';
                     $name = strip_tags(trim($_POST['name']));
                     $email = strip_tags(trim($_POST['email']));
                     $skype = strip_tags(trim($_POST['skype']));
                     $sms = strip_tags(trim($_POST['sms']));
                     $message = "Имя пользователя: ".$name."\r\n";
                     $message .= "Почта: ".$email."\r\n";
                     $message .= "Skype: ".$skype."\r\n";
                     $message .= "Сообщение: ".$sms."\r\n";
                     $error = false;
                     $err = array('email' => '', 'name' => '', 'sms' => '');

                     if (empty($email) || !filter_var($email, FILTER_VALIDATE_EMAIL)){
              $err['email'] = 'Вы ввели не корректный email';
        $error = true;
         }
                     if (empty($name)){
              $err['name'] = 'Вы не ввели имя';
        $error = true;
         }


                    // if(!$error){
                        $tema = "=?utf-8?B?".base64_encode($subject)."?=";
                        $headers = "From: $from\r\nReply-to: $from\r\nContent-type:text/plain; charset=utf-8\r\n";
                        mail($to, $tema, $message, $headers);
                       // if(mail($to, $subject, $sms, $headers)){
                           //  header('Location: /otzivi.php');
                       // }
                    // }
                 }

[Fudo Tsukiko]

Alex_87,

if (empty($name)){
	$err['name'] = 'Вы не ввели имя';
	$error = true;
	exit;
}elseif(empty($email)){
	$err['name'] = 'Вы не ввели мыло';
	$error = true;
        exit;
}elseif(empty($sms)){
	$err['name'] = 'Вы не ввели номер';
	$error = true;
        exit;
}elseif(empty($skype)){
	$err['name'] = 'Вы не ввели скайп';
	$error = true;
        exit;
}

таким колхозным способом

[Alex_87]

Fudo Tsukiko, Спасибо! Только проблема в том, что поле skype заполнять не обязательно. Я забыл вам об этом сказать. А значит, не получиться поставить условие на это поле и ошибка всё равно будет!

[Fudo Tsukiko]

Alex_87, тогда смотрите! попробуйте так

if (empty($name)){
	$err['name'] = 'Вы не ввели имя';
	$error = true;
	exit;
}elseif(empty($email)){
	$err['name'] = 'Вы не ввели мыло';
	$error = true;
	exit;
}elseif(empty($sms)){
	$err['name'] = 'Вы не ввели номер';
	$error = true;
	exit;
}elseif(empty($skype)){
	$skype = '';
	$error = false;
	exit;
}

[Alex_87]

Fudo Tsukiko, Не сработало! Я заметил одну вещь, не сразу обратил внимание, ошибка появляется уже на этапе загрузки(отображения) страницы в браузере. То есть, не при отправки формы, нажатием кнопки, что странно. Ведь сервер ещё не получил данные методом POST. И обратите внимание на этот код, на value полей. Мне кажется ошибка именно из-за этого:

<div class="popupcontact">
            <div class="wrapper">
                 <div class="popupcontacttitle" id="about">Оставьте заявку сейчас и получите бесплатную консультацию!</div>
                 <form id="form" method="post" action="form.php" class="popupform">
                     <input type="text" placeholder="Ваше имя" id="name" name="name"  value="<?=$_POST['name']?>"  />

                     <input type="e-mail" placeholder="Ваша почта" id="email" name="email"  value="<?=$_POST['email']?>" />

                     <input type="text" placeholder="Ваш Skype"  name="skype"  value="<?=$_POST['skype']?>" />
                     <textarea name="sms" cols="5" rows="10" placeholder="Ваше сообщение" value="<?=$_POST['sms']?>"></textarea>

                     <div>
                         <button type="submit" id="sbm">Оставьте заявку</button>
                     </div>
                 </form>

              </div>
            </div>

[Fudo Tsukiko]

Alex_87, смотри! замени
$skype = strip_tags(trim($_POST['skype']));
на

if(!empty($_POST['skype'])){ $skype = strip_tags(trim($_POST['skype'])); }

[Alex_87]

Fudo Tsukiko, Я убрал всё лишнее, чтобы ничего не мешало, а также убрал error_reporting(0); вот код:

<?php


                     $to = '[e-mail]';
                     $tema = 'Новое сообщение';

                     if(!empty($_POST['name'])){$name = strip_tags(trim($_POST['name']));}
                     if(!empty($_POST['email'])){$email = strip_tags(trim($_POST['email']));}
                     if(!empty($_POST['skype'])){$skype = strip_tags(trim($_POST['skype']));}
                     if(!empty($_POST['sms'])){$sms = strip_tags(trim($_POST['sms']));}
                     $message = "Имя пользователя: ".$name."\r\n";
                     $message .= "Почта: ".$email."\r\n";
                     $message .= "Skype: ".$skype."\r\n";
                     $message .= "Сообщение: ".$sms."\r\n";



                        $tema = "=?utf-8?B?".base64_encode($subject)."?=";
                        $headers = "From: $from\r\nReply-to: $from\r\nContent-type:text/plain; charset=utf-8\r\n";
                        mail($to, $tema, $message, $headers);

И вот результат:

[Fudo Tsukiko]

Alex_87, проделай так со всеми переменами. делай в начале скрипта проверку на пустоту $_GET['skype'] (и тд) если он не пустой ,то делай переменые

[Alex_87]

Fudo Tsukiko,

if($_GET['name'] != ''){
                     $name = strip_tags(trim($_POST['name']));
                    }

Всё равно ошибка...

[Fudo Tsukiko]

Alex_87, я только заметил,что ты юзаешь $_POST в переменных. но проверяешь $_GET. Если в форме актион=пост ,тогда проверяй $_POST! И замени все $_GET на $_POST

[Alex_87]

Fudo Tsukiko, Я не могу всё переписать на GET, поскольку я отправляю форму. А она отправляется, с точки зрения безопасности,только POST! Прошу помочь найти решение, касаемое только метода
POST!

[Fudo Tsukiko]

Alex_87, в скрипте замени все $_GET на $_POST

[Fudo Tsukiko]

Alex_87, а не в форме

[Fudo Tsukiko]

Alex_87 и чем таким пост безопасен?,

[Alex_87]

Fudo Tsukiko, Заменил! Попробовал использовать isset и !empty, но ничего не помогает...

if(isset($_POST['name'])){
                     $name = strip_tags(trim($_POST['name']));
                    }
                     if(!empty($_POST['email'])){
                         $email = strip_tags(trim($_POST['email']));
                     }

Почему??? Счёт пошёл уже на дни. Никто не знает как решить эту проблему. Это что, так сложно?

[Fudo Tsukiko]

Alex_87, есть фб? мне туда удобней

[Fudo Tsukiko]

Alex_87, дай файлы проекта. я попробую чтото сделать

[Alex_87]

Fudo Tsukiko, Дать facebook?

[Alex_87]

Fudo Tsukiko, Файлов много... Сейчас по удаляю всё лишнее и попробую дать!

[Fudo Tsukiko]

Alex_87, ++++

[Fudo Tsukiko]

Alex_87, facebook

[Alex_87]

Fudo Tsukiko, дайте пожалуйста свою почту

[Alex_87]

Fudo Tsukiko, https://www.facebook.com/a.kovalkovsky

[Fudo Tsukiko]

Alex_87, [e-mail]