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

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/topic.php on line 737
Форумы портала PHP.SU :: Где/как лучше валидировать данные [2]

PHP.SU

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

PHP.SU Портал

Главная

Помощь

Поиск

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

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

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

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

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

Где/как лучше валидировать данные

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

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

Без описания

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

Мелкий	Отправлено: 29 Мая, 2012 - 20:44:52
Активный участник Покинул форум Сообщений всего: 11926 Дата рег-ции: Июль 2009 Откуда: Россия, Санкт-Петербург Помог: 618 раз(а)	etoYA пишет: Если б я знал что это такое)). Включая обычные замыкания $func = function($arg) {//}; и create_function**, так и вызов обычных функций просто по имени, переданным переменной. http://ru.php.net/manual/ru/lang...pes.callable.php ----- PostgreSQL DBA

etoYA	Отправлено: 29 Мая, 2012 - 20:46:45
Участник Покинул форум Сообщений всего: 1859 Дата рег-ции: Июль 2011 Откуда: Крым Помог: 21 раз(а)	Если следовать тому, как написал caballero, то это нужно делать в контроллере. Ну или в отдельной модели, вот я сделал в контроллере. before (Отобразить) PHP: скопировать код в буфер обмена if (isset($_POST['enter'])){ if (is_null($this->login)){ $this->error = true; $this->errstr .= "Введите логин\r\n"; } if (is_null($this->pass)){ $this->error = true; $this->errstr .= "Введите пароль\r\n"; } if ($this->error === false){ // авторизация } $view->error = $this->errstr; } after (Отобразить) PHP: скопировать код в буфер обмена if (isset($_POST['enter'])){ if (is_null($this->login)){ parent::Error(self::USER_EMPTY_LOGIN); } if (is_null($this->pass)){ parent::Error(self::USER_EMPTY_PASS); } if (!parent::hasError()){ // авторизация } else{ foreach (parent::getErrors() as $error){ $view->error .= $error; } } } Как не крути, огромное спасибо sKaa! Этот вариант очень удобен). Так как не надо переписывать кучу текста (типа "Введите логин"). (Добавление) Мелкий, спасибо, уже почитал)). Правда на этом форуме). (Отредактировано автором: 29 Мая, 2012 - 20:53:57)

etoYA	Отправлено: 30 Мая, 2012 - 01:43:51
Участник Покинул форум Сообщений всего: 1859 Дата рег-ции: Июль 2011 Откуда: Крым Помог: 21 раз(а)	Вынес константы в отдельный файл, в общем вот что получилось)). UserController (Отобразить) PHP: скопировать код в буфер обмена public function actionLogin(){ $view = new View; $user = new User; $view->set(array( 'msg' => null, 'page' => 'login', 'inc' => 'forms/login.phtml' )); if (isset($_POST['enter'])){ if (is_null($this->login)){ parent::Error(USER_EMPTY_LOGIN); } if (is_null($this->pass)){ parent::Error(USER_EMPTY_PASS); } elseif (sizeof($user->login($this->login, md5($this->pass))) == 0){ parent::Error(USER_INVALID_PASS_OR_LOGIN); } if (!parent::hasError()){ $_SESSION['user'] = $user->login($this->login, md5($this->pass)); $view->msg = sprintf(USER_WELLCOME, $this->login); } else{ foreach (parent::getErrors() as $error){ $view->msg .= "$error\r\n"; } } } FrontController::setLayout($view->render('index')); } User (model) (Отобразить) PHP: скопировать код в буфер обмена public function checkUser($login){ $r = mysqli()->query("SELECT `login` FROM `users` WHERE `login` = '$login' LIMIT 1"); if (!is_object($r)) throw new ErrorController('db'); return $r->num_rows; } public function login($login, $pass){ $r = mysqli()->query("SELECT `u`.`id`, `u`.`login`, `u`.`group`, `g`.`name` AS 'group', `g`.`actions` FROM `users` u INNER JOIN `groups` g ON `u`.`group` = `g`.`id` && `login` = '$login' && `password` = '$pass'"); if (!is_object($r)) throw new ErrorController('db'); return $r->fetch_assoc(); } Const.php (Отобразить) PHP: скопировать код в буфер обмена <?PHP define('USER_EMPTY_LOGIN', 'Введите логин'); define('USER_INCORRECT_LOGIN', 'Логин должен быть 2-16 символов и содержать латинские буквы и цифры. Разрешенные знаки ._-'); define('USER_EMPTY_PASS', 'Введите пароль'); define('USER_INCORRECT_PASS', 'Пароль должен быть 6-16 символов и содержать латинские буквы и цифры'); define('USER_PASS_DO_NOT_MATCH', 'Введенные пароли не совпадают'); define('USER_EMPTY_EMAIL', 'Введите е-мейл'); define('USER_INCORRECT_EMAIL', 'Введен не корректный емейл адресс'); define('USER_ALREADY_EXISTS', 'Пользователь с таким логином уже существует'); define('USER_INVALID_PASS_OR_LOGIN', 'Неправильный логин или пароль'); define('USER_WELLCOME', 'Добро пожаловать %s'); define('USER_SUCCESSFULL_REGISTRATION', 'Регистрация прошла успешно'); ?> P.S Только методы для авторизации (Отредактировано автором: 30 Мая, 2012 - 01:44:49)

DelphinPRO	Отправлено: 30 Мая, 2012 - 21:05:39
Активный участник Покинул форум Сообщений всего: 7187 Дата рег-ции: Февр. 2012 Помог: 353 раз(а)	вот текстовые константы-то лучше в ini-файлах держать. их удобнее править при переводе на другой язык ;) ----- Чем больше узнаю, тем больше я не знаю.

etoYA	Отправлено: 30 Мая, 2012 - 22:42:18
Участник Покинул форум Сообщений всего: 1859 Дата рег-ции: Июль 2011 Откуда: Крым Помог: 21 раз(а)	DelphinPRO, как вариант проходит. Спасибо) (Добавление) DelphinPRO, как то так? PHP: скопировать код в буфер обмена foreach (parse_ini_file(ROOT. '/application/ini/constants.ini') as $k=>$v){ define(strtoupper($k), $v); } CODE (htmlphp): скопировать код в буфер обмена USER_EMPTY_LOGIN = Введите логин; USER_INCORRECT_LOGIN = Логин должен быть 2-16 символов и содержать латинские буквы и цифры. Разрешенные знаки ._-; USER_EMPTY_PASS = Введите пароль; USER_INCORRECT_PASS = Пароль должен быть 6-16 символов и содержать латинские буквы и цифры; ETC.....

sKaa	Отправлено: 31 Мая, 2012 - 05:25:26
Частый посетитель Покинул форум Сообщений всего: 979 Дата рег-ции: Окт. 2011 Откуда: Россия г. Нижний Новгород Помог: 25 раз(а) [+]	etoYA, да сам define очень медленная херовина, просто если ты держишь в define'ах всякие там конфиги или языки, это ужас как медленно будет работать когда их у тебя накопится по бане... Я собственно по этому и показал.. PHP: скопировать код в буфер обмена class classname { const QWE = '', ASD = '', ZXC = ''; } Ведь к ним тоже можно обратиться из любого места кода если класс существует. PHP: скопировать код в буфер обмена classsname::QWE; Ну, а так, DelphinPRO абсолютно прав! На хабре давным давно статья была, про скорости всякие define и прочей мерзости где люди умудряются конфиги хранить, ini файлы лидировали по скорости работы с большим отрывом, а define чуть ли не самым медленным оказался ) Таки вот она, статья : http://habrahabr.ru/post/112402/ (Отредактировано автором: 31 Мая, 2012 - 05:57:11)

etoYA	Отправлено: 07 Июня, 2012 - 15:39:08
Участник Покинул форум Сообщений всего: 1859 Дата рег-ции: Июль 2011 Откуда: Крым Помог: 21 раз(а)	sKaa, хм... Выходит, что это не фонтан? PHP: скопировать код в буфер обмена foreach (parse_ini_file(ROOT. '/application/ini/constants.ini') as $k=>$v){ define(strtoupper($k), $v); }

sKaa	Отправлено: 07 Июня, 2012 - 15:56:17
Частый посетитель Покинул форум Сообщений всего: 979 Дата рег-ции: Окт. 2011 Откуда: Россия г. Нижний Новгород Помог: 25 раз(а) [+]	etoYA, выходит да. Но если уж есть такое желание юзать ини и ещё, чтоб ООП на каждом шагу могу подсказать вариант с классом Language, который при инициализации загрузит нужный ru\en\uk\kz .ини, распарсит его внутрь приватного свойства класса в виде массива, а доступ будет через магический __get.. Получиться что-то типо : PHP: скопировать код в буфер обмена $Language = Language::getInstance('ru'); echo $Language->HELLO_MESSAGE; // Привет Можно долго размышлять как и что лучше, я сам хз даже как, просто как вариант можно и так.

caballero	Отправлено: 07 Июня, 2012 - 16:05:54
Активный участник Покинул форум Сообщений всего: 5998 Дата рег-ции: Сент. 2011 Откуда: Харьков Помог: 126 раз(а)	и к чему этот изврат обычные файл с константами const подключаемые инклудом для нужного языка ----- Бесплатная система складского учета с открытым кодом https://zippy[dot]com[dot]ua/zstore

DelphinPRO	Отправлено: 07 Июня, 2012 - 21:19:22
Активный участник Покинул форум Сообщений всего: 7187 Дата рег-ции: Февр. 2012 Помог: 353 раз(а)	etoYA нет, без дефайнов, просто массив с константами получите и используйте как удобно вариант caballero тоже неплох - инклудишь и сразу получаешь массив. но не факт что он быстрее ----- Чем больше узнаю, тем больше я не знаю.

caballero	Отправлено: 07 Июня, 2012 - 21:39:26
Активный участник Покинул форум Сообщений всего: 5998 Дата рег-ции: Сент. 2011 Откуда: Харьков Помог: 126 раз(а)	на массив память выделяется а константы сразу подставляются копилятором ----- Бесплатная система складского учета с открытым кодом https://zippy[dot]com[dot]ua/zstore

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

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

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

« Объектно-ориентированное программирование »

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

Powered by ExBB FM 1.0 RC1. InvisionExBB