Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770
Форумы портала PHP.SU :: Версия для печати :: Несколько операторов && подряд [2]
Форумы портала PHP.SU » » Вопросы новичков » Несколько операторов && подряд

Страниц (2): « 1 [2]
 

16. DlTA - 04 Декабря, 2011 - 19:09:56 - перейти к сообщению
Oleh пишет:
&& $email=''

это не условие а факт !!
(Добавление)
тут будет true
17. Panoptik - 04 Декабря, 2011 - 19:13:13 - перейти к сообщению
вообще если поэтапно просматривать все переменные, то тоже получается неплохо, просто нужно собирать всю информацию в переменную а потом выводить
PHP:
скопировать код в буфер обмена
  1.  
  2. if(!$login) {
  3.   $msg = "Вы не ввели логин!\n";
  4.   //редирект на повторное заполнение поля
  5. } else if(!$mail) {
  6.   $msg = "Вы не ввели e-mail!";
  7.   //редирект на повторное заполнение поля
  8. } else if(!$psw) {
  9.   $msg = "Пароль не может быть пустым!";
  10.   //редирект на повторное заполнение поля
  11. } else {
  12.   $msg = "Спасибо за регистрацию!";
  13.   //редирект на другую страницу
  14. }
  15. //print $msg; //там где это нужно вывести
  16.  
18. Champion - 04 Декабря, 2011 - 19:13:15 - перейти к сообщению
Oleh пишет:
if($log=='' && $psw=='' && $email='')
Если вы это копировали, то ошибка прям тут. Тут происходит присваивание, а результат такого присваивания - false.
Oleh пишет:
А то если я сначала буду
А если на форме хотя бы 5 обязательных полей?
PHP:
скопировать код в буфер обмена
  1. $errors = array();
  2. if (empty($email)) $errors[] = 'Пустой емейл';
  3. if (empty($login)) $errors[] = 'Пустой логин';
  4. if (empty($password)) $errors[] = 'Пустой пароль';
  5.  
  6. if (!empty($errors)) {
  7. // были ошибки
  8. } else {
  9. // не было ошибок
  10. }

(Добавление)
DlTA пишет:
тут будет true
Будет пустая строка - false)
19. Самогонщик - 04 Декабря, 2011 - 19:16:15 - перейти к сообщению
Oleh пишет:
if($a=='' && $b=='' && $c=='')
exit("Fields $a, $b and $c are empty!");
А ведь сначала правильно написал. Всё таки копирование оригинального кода является хорошей рекомендацией.
20. DlTA - 04 Декабря, 2011 - 19:18:23 - перейти к сообщению
Champion пишет:
DlTA пишет:
тут будет true
Будет пустая строка - false)
а да.
21. Oleh - 04 Декабря, 2011 - 19:25:39 - перейти к сообщению
Bio man пишет:
отладьте. var_dump($log, $psw, $email);

возможно $email имеет какое то значение


Сделал var_dump().
На стр. '1.php' все поля оставляю пустыми, нажимаю 'Submit'.
Стр. '2.php' выводит:

string(0) "" string(0) "" string(0) ""
Fields "Login" and "Password" must be filled to register!
Back to the page with the registration form

Т.е. первое условие не работает. Работает второе.
(Добавление)
Champion пишет:
Oleh пишет:
if($log=='' && $psw=='' && $email='')
Если вы это копировали, то ошибка прям тут. Тут происходит присваивание, а результат такого присваивания - false.
Oleh пишет:
А то если я сначала буду
А если на форме хотя бы 5 обязательных полей?


Точно ! Ошибка элементарная ! Опять невнимательность ! Исправил на '==' и всё заработало.
На счёт обязательности полей, то ты правильно написал. Я, кстати, такой свой громоздкий и ненужный код исправил на почти такой, как ты написал, только с проверкой других условий ниже на той же странице. А вот с проверкой на пустоту - написал много чего ненужного. Просто мне не нужно было использовать функцию exit(), а писать print() и флаг устанавливать в '1'. А уже потом проверит на '1' флаг и вывести гиперссылку на страницу с регистрацией.
Спасибо.
В следующий раз буду 10 проверять код прежде чем публиковать вопрос : )

 

Powered by ExBB FM 1.0 RC1