PHP.SU

Программирование на PHP, MySQL и другие веб-технологии

PHP.SU Портал

Главная

Помощь

Поиск

Поиск Яндекс Вакансии

Пользователи

Здравствуйте, Гость

( Вход · Регистрация · Правила форума )

Забыли пароль?

Проблема с true и false

Форумы портала PHP.SU » PHP » Программирование на PHP » Вопросы новичков (Модераторы: OrmaJever, Саныч, Строитель)

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

Без описания

Поиск в теме | Версия для печати

razorg1991	Отправлено: 13 Сентября, 2014 - 22:33:15
Частый гость Покинул форум Сообщений всего: 131 Дата рег-ции: Сент. 2013 Помог: 0 раз(а)	Пишу проверку существования имени в БД PHP: скопировать код в буфер обмена class DataBase { static function existenceNameListProducts($namelist) { try{ $db = new PDO("sqlite:database/database.db"); $sql = 'SELECT namelist FROM searchlist'; $st = $db->query($sql); $st->setFetchMode(PDO::FETCH_ASSOC); while($row = $st->fetch()) { $name = $row['namelist']; if($name == $namelist) { echo 'Такое имя уже существует!'; return true; break; } else { return false; } } } catch(PDOException $e) { echo $e->getMessage(); } } } Собственно сама проверка PHP: скопировать код в буфер обмена if(DataBase::existenceNameListProducts($namelist) == false ) { DataBase::insertListProducts($namelist, $checklistproduct, $datetime); } Проблема в том, что к примеру существует уже имя "name", и при DataBase::existenceNameListProducts($namelist) == false или true неважно, всегда выполняется DataBase::insertListProducts($namelist, $checklistproduct, $datetime); Почему? Где я накосячил?

OrmaJever	Отправлено: 13 Сентября, 2014 - 22:47:40
Активный участник Покинул форум Сообщений всего: 7540 Дата рег-ции: Янв. 2010 Откуда: Чернигов Помог: 299 раз(а)	а вы попробуйте написать PHP: скопировать код в буфер обмена var_dump( DataBase::existenceNameListProducts($namelist) ); и посмотреть что там ----- Если вы хотя бы 3-4 раза не решите всё выкинуть и начать заново - вы явно что-то делаете не так.

teddy	Отправлено: 13 Сентября, 2014 - 22:52:38
Участник Покинул форум Сообщений всего: 1462 Дата рег-ции: Апр. 2013 Помог: 91 раз(а)	razorg1991 пишет: при DataBase::existenceNameListProducts($namelist) == false или true неважно, всегда выполняется DataBase::insertListProducts($namelist, $checklistproduct, $datetime); Бред. Смотрите внимательнее. Дополнительно: 1. Выводить результат из функции не самое лучшее решение(я про echo). 2. break после return не имеет смысла потому как return и так прервет выполнение цикла 3. Цикл тут даже и не нужен. Можно сделать что то типа SELECT `column` FROM `table` WHERE `column` = :some_param и если запись нашлась значит true иначе false Плюс ко всему зачем создавать объект PDO в методе с выборкой? Получается в каждом методе который будет как то работать с БД Вы будете создавать каждый раз новый объект PDO? (Добавление) Я даже подскажу в чем проблема. Вы выбираете ВСЕ записи в таблице и начинаете цикл. В есть цикле условие if($name == $namelist)else{return false;l} Так вот, на первой же итерации совпадений не бывает(у Вас же записей в более чем одна) и выполнение кода переходит в else, а там сразу возвращается false напрочь убивая цикл. Поэтому условие всегда выполняется. (Отредактировано автором: 13 Сентября, 2014 - 22:53:44)

Поиск в теме | Версия для печати

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

Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)

« Вопросы новичков »

Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.