PHP.SU

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

PHP.SU Портал

Главная

Помощь

Поиск

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

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

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

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

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

Как вы реализуете вывод ошибок

Форумы портала PHP.SU » PHP » Пользовательские функции (Модераторы: valenok, ALEN)

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

Описание: Ваша реализация вывода ошибок по тому или иному поводу

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

n1k86	Отправлено: 28 Января, 2012 - 14:42:52
Частый гость Покинул форум Сообщений всего: 160 Дата рег-ции: Март 2011 Помог: 0 раз(а)	Ребят, всем привет. Сложилась такая ситуация, что я всё не могу никак подойти к тому, как же лучше всё-таки делать ошибочки при тех или иных ситуациях. Вот остановлся на таком варианте: PHP: скопировать код в буфер обмена $err = array(); //Брум-бум-бум-бум масив $posts = mysql_result(mysql_query("SELECT COUNT(id) FROM news"), 0); $_GET['edit'] < $posts \|\| $_GET['edit'] > $posts ? $err[] = 'Новости с таким ID не существует.' : NULL; //Проверяем, есть ли такая новость. f(!empty($_POST['title']) && !empty($_POST['text'])) { mb_strlen($_POST['title']) < 3 \|\| mb_strlen($_POST['title']) > 50 ? $err[] = 'Заголовок должен быть не меньше 3 символов и не больше 50 символов.' : NULL; //Проверяем на кол-во знаков. mb_strlen($_POST['text']) < 50 \|\| mb_strlen($_POST['text']) > 2000 ? $err[] = 'Текст новости должен быть не меньше 50 символов и не больше 2000 символов.' : NULL; //Проверяем на кол-во знаков. } //Если ошибок нет. if(count($err) == 0) { $output .= '<div class="sys_bg"><div class="system succes">Новость успешно изменена!</div></div>'; } else { //Если есть - выводим. foreach($err AS $error) { $output .= '<div class="sys_bg"><div class="system error">'.$error.'</div></div>'; } } Но в этом варианте есть недостаток, что если например много отправляешь по форме, и потом их надо занести в базу, то если где-то вот проходит ошибка, то можно сказать весь скрипт ступориться. А если ты начинаешь мудрить с проверкой для каждого раза, типа count($err) == 0 то продолжаем, то получается, если честно, бред. Подскажите, как лучше это всё реализовать, как вы это реализовываете.

LIME	Отправлено: 28 Января, 2012 - 16:37:01
Активный участник Покинул форум Сообщений всего: 10732 Дата рег-ции: Нояб. 2010 Помог: 322 раз(а)	n1k86 пишет: то если где-то вот проходит ошибка, то можно сказать весь скрипт ступориться имеется ввиду необходимость снова заполнять форму?

n1k86	Отправлено: 28 Января, 2012 - 16:40:31
Частый гость Покинул форум Сообщений всего: 160 Дата рег-ции: Март 2011 Помог: 0 раз(а)	Нет. Смотри, вот у меня есть страница. В ней есть форма с полями: имя, фамилия, возраст. Дальше я вот беру обрабатываю всё, заношу как бы эти ошибки в массив $error, если они будут. Ну вдруг он введет имя меньшего размера или в Фамилию не укажет. Так вот. В моём коде получается, что если вдруг будет ошибка хоть где-то, и count($error) покажет значение не равное 0 (т.е. будут ошибки), то скрипт остальные данные не хочет обрабатывать. Т.е. сразу тормозит и показывает, вот мол, вы в имени ошиблись. Хотя возможно, что фамилия и возраст то были указаны верно, и их можно было и сохранить, и написать, мол: !Ваше имя короче 3 символов. Фамилия успешно сохранена. Возвраст успешно сохранён. Ну вот так, как-то....думаю сейчас понятно объяснил?

n1k86	Отправлено: 28 Января, 2012 - 18:40:52
Частый гость Покинул форум Сообщений всего: 160 Дата рег-ции: Март 2011 Помог: 0 раз(а)	Добился нужного мне результата с помощью того, что при проверке на ошибки каждого ПОЛЯ из формы, я создаю массив, а потом его убиваю. И вот так для каждого поля...это вообще нормально такой подход??

LIME	Отправлено: 28 Января, 2012 - 20:04:36
Активный участник Покинул форум Сообщений всего: 10732 Дата рег-ции: Нояб. 2010 Помог: 322 раз(а)	n1k86 ну я так и думал вобщем проще помещать в форму при выводе значения из GET если их нет то форма пуста если ошибка в каком-то значении то вместе с ошибкой unset($_GET['это_значение']) следовательно у вас появится форма с правильно заполнеными полями и пустотой в полях с ошибкой а в базу всеже заносить только при отсутствии ошибок все сразу

n1k86	Отправлено: 28 Января, 2012 - 20:07:09
Частый гость Покинул форум Сообщений всего: 160 Дата рег-ции: Март 2011 Помог: 0 раз(а)	По реализации это сложнее. Так я просто делаю как у меня было, но каждую проверочку инпута обрамляю в так сказать массив. (Союзаю -> Проверяю -> Убиваю)*n-раз. Т.е. сколько принимаю - столько и проверяю. По сути своего добился...спасибо большое!) Я тебе Спасибо нажал;)

LIME	Отправлено: 28 Января, 2012 - 20:17:38
Активный участник Покинул форум Сообщений всего: 10732 Дата рег-ции: Нояб. 2010 Помог: 322 раз(а)	n1k86 пишет: Я тебе Спасибо нажал;) видимо не за что значит отработаю спасибку )) во пример PHP: скопировать код в буфер обмена <?PHP if (isset($_GET['name']) && $_GET['name']=='ошибка') { $err[]='Ошибка номер раз'; unset($_GET['name']); } if (isset($_GET['age']) && $_GET['age']=='ошибка') { $err[]='Ошибка номер два'; unset($_GET['age']); } foreach ($err as $error) echo $error.'<br/>'; ?> <form> <input type="text" value="<?=$_GET['name']?>"> <input type="text" value="<?=$_GET['age']?>"> </form> вот что тут сложного в реализации наоборот меньше гимороя

n1k86	Отправлено: 28 Января, 2012 - 20:20:12
Частый гость Покинул форум Сообщений всего: 160 Дата рег-ции: Март 2011 Помог: 0 раз(а)	Есть за что. Ты один, кто ответил. Не по ленился) Ну вот смотри код мой(весь файл): Спойлер (Отобразить) CODE (htmlphp): скопировать код в буфер обмена <? $tit['title'] = 'Управление регистрацией'; //Заголовкок. //Подключаем настройки. include_once '../sys/bdconnect.php'; include_once '../sys/functions.php'; include_once '../sys/online.php'; include_once '../sys/settings.php'; include_once '../head.php'; if (isset($_SESSION['hash'])) { if($_SESSION['adm'] >= 2) { if(isset($_POST['submit'])) { $err = array(); empty($_POST['reg_status']) ? $err[] = 'Не был указан статус модуля регистрации.' : NULL; !is_numeric($_POST['del_noconf']) ? $err[] = 'Статус модуля регистрации должен быть указан цифрой!' : NULL; if(count($err) == 0) { mysql_query("UPDATE settings SET reg_status='".nohack($_POST['reg_status'])."' WHERE id='1'"); $output .= '<div class="sys_bg"><div class="system succes">Статус модуля регистрации успешно сохранён!</div></div>'; } else { foreach($err AS $error) { $output .= '<div class="sys_bg"><div class="system error">'.$error.'</div></div>'; } } unset($err); $err = array(); empty($_POST['conf_status']) ? $err[] = 'Не был указан статус активации аккаунта.' : NULL; !is_numeric($_POST['conf_status']) ? $err[] = 'Статус активации аккаунта должен быть указан цифрой!' : NULL; if(count($err) == 0) { mysql_query("UPDATE settings SET conf_status='".nohack($_POST['conf_status'])."' WHERE id='1'"); $output .= '<div class="sys_bg"><div class="system succes">Статус активации аккаунта успешно сохранён!</div></div>'; } else { foreach($err AS $error) { $output .= '<div class="sys_bg"><div class="system error">'.$error.'</div></div>'; } } unset($err); $err = array(); empty($_POST['del_noconf']) ? $err[] = 'Не был выбран промежуток времени, после которого просходит удаление неактивированных аккаунтов.' : NULL; !is_numeric($_POST['del_noconf']) ? $err[] = 'Промежуток для удаления неактивированных аккаунт должен быть задан цифрой!' : NULL; if(count($err) == 0) { mysql_query("UPDATE settings SET del_noconf='".nohack($_POST['del_noconf'])."' WHERE id='1'"); $output .= '<div class="sys_bg"><div class="system succes">Промежуток для удаления неактивированных аккаунтов сохранен!</div></div>'; } else { foreach($err AS $error) { $output .= '<div class="sys_bg"><div class="system error">'.$error.'</div></div>'; } } unset($err); $output .= '<div class="nav"><img src="/style/icons/quadre.gif" /> <a href="/adm_panel/index.php">В админ-панель</a><br /><img src="/style/icons/quadre.gif" /> <a href="/users/usermenu.php">В профиль</a></div>'; include_once '../foot.php'; header('Refresh: 2; URL=/adm_panel/index.php'); exit(); } $output .= ' <div class="text"> <form method="post"> Статус регистрации: <select name="reg_status">'; $settings['reg_status'] == 1 ? $text_status = 'Включена' : $text_status = 'Выключена'; switch ($text_status) { case 'Включена': $output .= '<option value="1">Включена</option> <option value="2">Выключена</option>'; break; case 'Выключена': $output .= '<option value="2">Выключена</option> <option value="1">Включена</option>'; break; } $output .= '</select><hr /> Статус активация аккаунта по E-Mail: <select name="conf_status">'; $settings['conf_status'] == 1 ? $text_status = 'Включена' : $text_status = 'Выключена'; switch ($text_status) { case 'Включена': $output .= '<option value="1">Включена</option> <option value="2">Выключена</option>'; break; case 'Выключена': $output .= '<option value="2">Выключена</option> <option value="1">Включена</option>'; break; } $output .= '</select><hr /> Удаление неактивированных аккаунтов через: <select name="del_noconf"> <option value="'.$settings['del_noconf'].'">'.$settings['del_noconf'].' '.GetZapisi($settings['del_noconf'], array("день", "дня", "дней")).'</option>'; for($value = 3; $value <= 24; $value = $value + 3) { $value != $settings['del_noconf'] ? $output .= '<option value="'.$value.'" >'.$value.' '.GetZapisi($value, array("день", "дня", "дней")).'</option>' : NULL; } $output .= '</select> <input type="submit" name="submit" value="Сохранить"> </form> </div> <div class="nav"><img src="/style/icons/quadre.gif" /> <a href="/adm_panel/index.php">В админ-панель</a><br /><img src="/style/icons/quadre.gif" /> <a href="/users/usermenu.php">В профиль</a></div>'; include_once '../foot.php'; } else { header('Location: /index.php'); } } else { header('Location: /aut.php'); } ?> Куда проще, ведь правда? Кстати, если есть замечания по коду какие-нибудь, т.к. я думаю ты опытнее меня куда больше, подскажи, пожалуйста!)

sKaa	Отправлено: 29 Января, 2012 - 00:07:48
Частый посетитель Покинул форум Сообщений всего: 979 Дата рег-ции: Окт. 2011 Откуда: Россия г. Нижний Новгород Помог: 25 раз(а) [+]	n1k86 пишет: Ну вот смотри код мой(весь файл): Жесть ваще)))))

n1k86	Отправлено: 29 Января, 2012 - 06:52:44
Частый гость Покинул форум Сообщений всего: 160 Дата рег-ции: Март 2011 Помог: 0 раз(а)	sKaa пишет: n1k86 пишет: Ну вот смотри код мой(весь файл): Жесть ваще))))) Ээээ...ну я как бы только учусь....поэтому, может ты лучше подскажешь что-то полезное, а не эмоции выражать будешь???)))

Slesher	Отправлено: 29 Января, 2012 - 14:36:46
Гость Покинул форум Сообщений всего: 69 Дата рег-ции: Дек. 2011 Помог: 0 раз(а)	весь код не смотрел что в последнем варианте не смотрел, но могу посоветовать использовать класс Exception, а также try{}, catch {}. можно много автоматизации и упрощения ними сделать.

n1k86	Отправлено: 29 Января, 2012 - 14:38:12
Частый гость Покинул форум Сообщений всего: 160 Дата рег-ции: Март 2011 Помог: 0 раз(а)	В принципи вопрос исчерпан, хотя мне предлагал друг сделать тру и кэтч. Но это же для ооп, так ведь? Или нет?

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

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

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

« Пользовательские функции »

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