Зачем изобретать велосипед когда ес вот такая вещь: http://php.net/manual/en/filter.filters.validate.php

PHP_Fox
Почему некорректно? Я недавно выдался проверять, всё работает как надо.

пробовал фильтер, в раздных версиях php тупит иногда, пропускает)

if(!preg_match('/^[-a-z0-9_.]{1,50}@[-a-z0-9_.]{1,50}\.[a-z]{2,6}$/i', $email))

Ошибочка
if(!preg_match('/^[-a-z0-9_.]{1,50}@[-a-z0-9_.]{1,50}\.[a-z]{2,6}$/i', $email))
Ты не заэкранировал точку. Если ты ее не заэкранируешь то там будет принят любой текст

pak
Ты бы хоть говорил, что именно пропускает! А то как в стоге сена искать ТВОЮ ошибку....

neadekvat всмысли пробится?

inpost
ну вот к примеру у меня пропускает:

$e = 'te#st-@bk.ru';
var_dump(filter_var($e, FILTER_VALIDATE_EMAIL));

pak
И? Я не вижу тут ошибки нарушения стандартов. Может быть конкретный бесплатный почтовик не разрешает некоторые символы, но дифис на конце ставить можно. Решетка входит в допустимые для e-mail адресов символы.
Тут стандарты не нарушены, всё ок!

я сравнивал с mail ru, значит у них свои стандарты)

Гугл не зарегистрирует подобный e-mail, но это его личные тараканы.
Вот по правилам. Под рукой нет мануала официального, но его читал и проверял, всё как надо работает. Но есть ссылка из гугла, где ты можешь убедиться, что ты не допустил ошибки в e-mail адресе: http://www.tftf.ru/stati/internet/email/

Гугл, мейл.ру, яндекс... у каждого свои собственные тараканы. Ну захотелось им запретить часть символов без повода... что тут скажешь, но они не идеальны, а стандарты и правильность адресов зарегистрировано и написано там.
Максимум что можно, так это проверять на существование такого e-mail адреса. А вот корректность - есть стандарты!

inpost
благодарю, не знал про таких тараканов)

inpost, стандарты - это круто, конечно, я их тоже читал. Но все-таки надо исходить из реального положения дел. Я вот не могу вспомнить сервисы, где можно такие адреса зарегистрировать:

te#st-@bk.ru

Соответственно, скорее всего, в адресе ошибка или ложь.

Вот только ради вас. Создал:

inpost, эм.. и?

neadekvat
И то, что такой аккаунт существует и был создан! Ты сказал: "Я вот не могу вспомнить сервисы, где можно такие адреса зарегистрировать. Соответственно, скорее всего, в адресе ошибка или ложь. "

Какая ошибка, если я тебе показываю, что у меня такой почтовый ящик?!

Мы что обсуждаем? Как на личном сайте фильтровать e-mail. То есть если ты создашь на своём сайте фильтр неверный, то не пропустит мои ящики, а должен пропустить. Чья ошибка? ПРОГРАММИСТА, то есть твоя. Ты должен пропускать существующие ящики для регистрации, потому что они существуют и это право человека, пользуется ли он чужим gmail, или своим серверным.

В жизни не встречал адреса с #, и еще ни один человек мне не предъявил жалобу.
Зато я не пропустил кучу ошибок, связанных со случайно зажатым шифтом или не переключенным языком.
Я работаю для большинства, а не для всех. И у большинства нет спец. символов в адресе. Ты лично можешь создать себе адрес с этой херней и.. и нигде не зарегистрироваться. Кто виноват? Программисты, которые создавали сайты, но не ты, что пошел против тенденции, выпятив вперед свой стандарт.

Если завтра все прыгнут в окно, то ты прыгнешь тоже. Вот твой ответ! Ты учишься быть профессионалом, или лишь бы кем? Или сюда ты пришел давать знания, а не получать их?

neadekvat
Программисты ПХП - говнокодеры.
Ты - Бог!.

Недавно мне на одном сайте пришлось отказаться от filter_var() не по причине, что он пропускает лишнего, а по причине, что он не пропускает некоторые действующие адреса. Было несколько жалоб. Сейчас не могу вспомнить подробностей, постараюсь. Но факт есть факт.

twin
Я надеюсь, что этот баг был известен ПХП программистам и зарепортирован, поэтому в новых версиях бага нет. К тому же с момента твоего старого сообщения прошел целый год, уже и 5.4. успела выйти.

Ну, будем надеяться. Функция то действительно удобная, если баги пофиксили.

neadekvat
Вполне нормальное сравнение. ПХП разрабы, Google, Twitter - все слушают стандарты(!!!), а ты один, кто не уважает стандарты. При этом ты не хочешь признать своей собственной вины. Но ничего страшного, ведь за счёт неграмотности одних возрастает цена грамотным программистам, таким, как те, кто создают ПХП, гугл, твиттер и т.д. И да, я не считаю себя достаточно грамотным, но я к этому стремлюсь.

neadekvat
http://phpforum.ru/index.php?showtopic=670...5&#entry2064526
Вот глянь. Они сами не создают, но при этом для восстановления пароля поддерживают стандарты.

Ты будешь 70-летним стариком, и я всё равно скажу, что что-то ты не знаешь. Такова жизнь, век живи - век учись. А ты сейчас на стадии, когда считаешь себя умным, а это и есть утопия знаний, потому что когда ты скажешь себе, что ты глупый, тогда ты начнёшь узнавать больше и становиться лучше.
Я не говорю о твоей общей квалификации программиста, а говорю о тебе в данном вопросе. Ты глуп в данном вопросе, хотя в остальных вопросах я могу сделать предположение, что глупее я. Это общая относительность, не бывает умных во всём и глупых во всем. Поэтому не надо обижаться.

neadekvat
Кто тебе сказал, что я не смог вспомнить стандарт? Не сочиняй свои глупости.

PHP_Fox
Филькина грамота, кто-то сказал, кто-то подхватил! Заметь, Николай (twin) как раз использовал filter_var() и отказался не от того, что он что-то там пропускает, а от того, что в нём были баги. А прошел уже год, и возможно завтра Николай снова почитает багфиксы и вернётся к filter_var. А ты поверишь каким-то советам псевдо-кодеров, которые не пропускают решетку или тире, а когда-нибудь тебе покажут, что гугл и твиттер же пропускают, и придерживаются стандартов...

Да и регулярки работают медленней, чем стандартные функции.

Так, ну вроде filter_var() работает корректно, и соблюдает все стандарты, а стандарты и правила я уважаю, ровно как и разработчиков PHP. Если они сказали, что так правильно и пользуйтесь функцией filter_var() значит так я и буду поступать. С этим разобрался! Если конечно ни кто не чего не хочет добавить!?

Вопрос к профи! Ребят смотрите вот такой проверки хватит, все я правильно сделал и можно ли эти данные смело записывать в базу без каких либо еще операций.

$email = trim($_POST['email']);

if(empty($email) and $email !== '0'){
	echo 'Вы не заполнили поле E-mail';
}elseif(!filter_var($email, FILTER_VALIDATE_EMAIL)){
	echo 'Вы ввели не корректный E-mail';
}

И e-mail то полюбому нужно тримить? В отличии от пароля!

И еще вопрос, стоит ли вообще делать такую проверку на пустоту поля или достаточно сделать так?

if($email == ''){
	echo 'Вы не заполнили поле E-mail';
}elseif(!filter_var($email, FILTER_VALIDATE_EMAIL)){
	echo 'Вы ввели не корректный E-mail';
}

Просто почему в первом варианте я использую функцию $empty() просто увидел в других скриптах, что ее используют вместо == '' хотя по сути ведь тоже самое. А вот к этому

and $email !== '0'

я выводу пришел в процессе тестирования, ведь 0 это для пользователя не пустая строка, а для функции пустая ))

Ну вот как-то так, прошу вашего мнения!

И пишем в базу

$db->query("INSERT INTO `users` (`email`) VALUES ('$email')");

Всели тип топ? Или что-то посоветуете, порекомендуете?

Тут нотис будет:

$email = trim($_POST['email']);

А так сойдет)

Не совсем. Такая проверка исключает конкретизацию ошибки.

Обработка данных в одном месте, вывод данных в другом, максимально возможное разделение логики и представления очень важно при построении веб приложений.
К тому же ошибку надо выводить рядом с полем email, а не где попало в начале страницы. Поэтому ошибки необходимо записывать в переменные, а сами переменные уже выводить на экран в нужном месте.

inpost

ну вот видишь, как можно говорить про норм или нет, когда у тебя вот как всё сложно! Вдруг там не просто норм, но и круто!

Если делаешь шедевр, делай значит по правилам))) Функция твоя называется "валидатор", может это для тебя откровение... Так вот, использовать внутри функции суперглобальные массивы - моветон. Функция должна принимать аргументы и возвращать результат.

twin

Тогда налей мне кофе погуще. Гадать не на чем. Только что было это:

Я не про это. Я про то, что у тебя функция валидатора. И ты праямо внутри неё обращаешься к суперглобальному массивы $_POST. По идее нужно передать в неё аргументы. А инициализировать их снаружи.

$email = @$_POST['email'];

function walidator($email)
{
    $email = trim($email);
..................
}

Что-то какой-то форум пустой ((( общаются 2-3 человека, и одни и те же люди, да и те не особо разговорчивые (((( От части сам с собой общаюсь ((((

90% ответа в правильно заданном вопросе. Ты уже несколько страниц толчешь воду в ступе. Какой интерес отвечать на одно и то же.

Задавай конкретные вопросы, тогда появится интерес на них отвечать.

PHP_Fox
Влом читать 5 страниц. Если ответ уже давали, а его наверняка давали, то покажи где..... а вопрос у меня такой - а чем filter_var не подходит?

m4a1fox
Ну ты вообще не чего не прочитал, тебе в лом? А теперь спрашиваешь, чем не подходит, а если бы прочитал, то уже знал бы ответ на свой вопрос, уже подходит, пару лет назад не подходил, были там баги ))
И я не на все вопросы получил ответы. И это форум а не вопрос ответ, мне интересно пообщаться и все обсудить, что почему так а не иначе и т.д а не тупа, вопрос, ответ.

PHP_FoxOk! МОлодчина!

Потому что все начинает повторяться.

Michael
Еще один человек который не читал всю тему и делает замечания ))

if(empty($email) and $email !== '0'){

if($email == ''){

if(isset($_POST['login'], $_POST['password'], $_POST['repeat_password'], $_POST['email'])){

PHP_Fox
Сейчас скажу основные 2 ошибки, но это лишь ИМХО!
1. Тебе тримом в любом случае необходимо будет обрабатывать данные, как бы ты не хотел, поэтому можно весь POST массив обработать ещё на уровне ядра.
2. Массивы, включая суперглобальный массив POST - структурированные данные на группы. Допустим входящие данные = $_POST, внутренние настройки = массив $options, и в таком духе. (Класс со свойствами в виде настроек - расширенный массив). Получается мы имеем уже логическую структуру и данные связанные между собой логической цепочкой. Внутренние переменные $login - это уже индивидуальная переменная, чем на мой взгляд и плоха.

А возмущение - ты даёшь какие-то маленькие куски, которые сами по себе являются говнокодом. Возможно внутри структуры они преобразуются в профессиональный код, но для этого надо видеть весь код, всю структуру, а не маленькие кусочки. Невозможно сказать, хорошо ли это или плохо!

PHP_Fox
Я просто не хотел показывать весь код потому что он еще не законьчен это лишь наброски и работает все пока в одном файле и многое еще не дописанно. Он в процессе разработки. Думаю сейчас выложу и начнутся сыпаться, да тут это ты не здел, то не здела, ну конечно не сделал он не дописан.
Вот код, но сразу говорю, я начал писать по старому своему методу, тоесть грубо ограничивал данные введенные пользователем, конечно теперь тут все измениться в плане ограничений.

	if(isset($_POST['login'], $_POST['password'], $_POST['repeat_password'], $_POST['email'])){
		
		$login = trim($_POST['login']);
		$password = trim($_POST['password']);
		$repeat_password = trim($_POST['repeat_password']);
		$email = trim($_POST['email']);
		
		if($login == ''){
			$es_id['219'] = true;
		}elseif(preg_match('|^[-a-z0-9_.]+$|i', $login)){
			if(!preg_match('|^[-a-z0-9_.]{3,20}$|i', $login)){
				$es_id['221'] = true;
			}
		}else{
			$es_id['220'] = true;
		}
		
		if($password == ''){
			$es_id['222'] = true;
		}elseif(preg_match('|^[-a-z0-9_.]+$|i', $password)){
			if(!preg_match('|^[-a-z0-9_.]{6,20}$|i', $password)){
				$es_id['224'] = true;
			}
		}else{
			$es_id['223'] = true;
		}
		
		if($repeat_password == ''){
			$es_id['225'] = true;
		}elseif(preg_match('|^[-a-z0-9_.]+$|i', $repeat_password)){
			if(!preg_match('|^[-a-z0-9_.]{6,20}$|i', $repeat_password)){
				$es_id['227'] = true;
			}
		}else{
			$es_id['226'] = true;
		}
		
		if(mb_strtolower($password) == mb_strtolower($login)){
			$es_id['231'] = true;
		}elseif($password !== $repeat_password){
			$es_id['228'] = true;
		}

		if($email == ''){
			$es_id['229'] = true;
		}elseif(!filter_var($email, FILTER_VALIDATE_EMAIL)){
			$es_id['230'] = true;
		}
		
		if(!isset($es_id)){
			$password = $password_hash->get_hash($password);
			$db->query("INSERT INTO `users` (`login`, `password`, `email`) VALUES ('$login', '$password', '$email')");
			$db->close();
		}else{
			es_reports($es_id);
		}
	}

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

	<form action='' method='post'>
		<div class='input'>
			<label class='label' for='login'>Логин:</label>
			<input class='input_box' id='login' maxlength='20' type='text' name='login'>
		</div>
		<div class='input'>
			<label class='label' for='password'>Пароль:</label>
			<input class='input_box' id='password' maxlength='20' type='password' name='password'>
		</div>
		<div class='input'>
			<label class='label' for='repeat_password'>Повторите пороль:</label>
			<input class='input_box' id='repeat_password' maxlength='20' type='password' name='repeat_password'>
		</div>
		<div class='input'>
			<label class='label' for='email'>Ваш E-Mail:</label>
			<input class='input_box' id='email' maxlength='200' type='text' name='email'>
		</div>
		<div class='submit'>
			<input class='submit_button' type='submit' value='Зарегистрироваться'>
		</div>
	</form>

Ты сам то запускал? Попробуй сам сначала, в процессе теста многие вопросы сами отпадут. С иссетм по крайней мере сразу.

Вот это зачем?

		if($repeat_password == ''){
			$es_id['225'] = true;
		}elseif(preg_match('|^[-a-z0-9_.]+$|i', $repeat_password)){
			if(!preg_match('|^[-a-z0-9_.]{6,20}$|i', $repeat_password)){
				$es_id['227'] = true;
			}
		}else{
			$es_id['226'] = true;
		}

какая разница, что ввел юзер в поле подтверждения. Если не совпало - гуляй лесом. А основной уже проверили.

А за это

if(!preg_match('|^[-a-z0-9_.]{6,20}$|i', $password))

после прошлой темы на стопицот страниц, руки надо рвать по саму майку.

twin

HTML: необходимо свойства писать в двойных кавычках.
В какой кодировке работаешь? Что значит модификатор u для регулярок?
Зачем регулярки дублируешь?
Инфекция sql: Введи email: "la'la@mail.ru" , увидишь ошибку.

twin
И я все там тестил, все отлично работает, может и не доглядел, так укажи конкретно где, только не говори мне снова про пароли, я там уже сделал для себя выводы и применю их в этом коде, а сейчас он такой как и был 3 дня назад, до того как я эти темы открывал, я лишь поправил вот это, и то не потому что не знал что так лучше а просто хотел что бы вы это подтвердили

if($password == ''){

И вот это, я знал об этой функции просто когда я начинал программировать она еще была сырой и все об этом трубили

}elseif(!filter_var($email, FILTER_VALIDATE_EMAIL)){

inpost

inpost

inpost

Если в ЮТФ работаешь, почему не добавляешь u?
Это стандарты HTML, именно двойная кавычка. Одинарная - допущения лишь.

И вообще, ты пришел слушать советы, или защищать свой код? Тебе уже десяток ошибок сказали, а ты каждую из них пытаешься защитить.

А инфекция потому что нет mysql_real_escape_string.

PHP_Fox
Одинарные кавычки как раз и есть ошибка, в стандартах говорится про две кавычки именно!

И ты пытаешься оправдываться. Ты написал уязвимый для sql-inj код, значит уже он плохой.

Хитрый какой.

if (все установлено) {
  что то делаем
}

ты сделал.
А надо еще else тоже. Зачем? Чтобы человек ранее введенные данные заново с нуля не вводил, их надо подставить в элементы формы в атрибут value. (понятно что кроме полей пароля). И вот тут ты увидишь, что нужно из каждого все равно достать данные. ;)

Michael
ты сделал.

PHP_Fox
Ты пытаешь оправдать свой код, а это говорит уже о присутствии гордости в твоей речи. Гордыня - грех, и от неё надо избавляться. Если указали на ошибку, то лучше сказать: "спасибо большое, исправлю". Ты пытаешься объяснить, зачем ты в том или ином месте допустил ошибки.
Снова же, обиженный ребёнок? Видите ли не увидели эксперты форума в твоём коде гениальности, видим банальные ошибки новичков и указали тебе на них. В ответ: "да вы только обижать горазд". Какой ответ ты ждал?! "чувак, это самый клёвый код, который я видел?" - да это гордыня, опять же.

И хотя бы 1 сообщение покажи, аля "ляпнуть", вот мне любопытно, кто же у нас на форуме не по теме что-то сказал?

Ты слишком горд и самовлюблён. Мы же тебя опустили, но опустили не в грязь, а на землю. Если ты пишешь говнокод, это ещё не значит, что человек, который тыкает тебя пальцем в говнокод не пишет его сам. Поэтому фраза: "он крут, а ты лох" - не верна с самого начала. Правильнее: "ты лох, несмотря на то, лох ли я или нет, и, возможно, я тоже лох и ты такой же лох как и мы".

К нам на форум люди приходят учиться и слушать замечания. У тебя 1 год обучения (как ты писал), у меня 3.5 года, из них все 3.5 года жесткой практики и работы именно программистом, до этого несколько лет верстальщиком подрабатывал, когда учился в студенческие годы и школьные. Николай в ПХП пришел на год-полтора раньше меня, итого у него 4.5-5 лет практики. Михаила я тоже помню давно, Киллер - то в общем отдельный разговор, за ним десятки лет практики. Неадекват давно бы стал экспертом, если бы вёл себя более культурно и адекватнее, потому чего его опыт тоже очень большой.

И самое ГЛАВНОЕ. 3 дня прошло, а ты ничего не сделал. Дам ссылочку, но без обид, это то, каким должен быть программист: пиши-код-блять.рф (баг на форуме с ссылками, поэтому просто скопируй в браузер его!).

Есть такая старая интересная штука - обращение хакеров. Много букв. но постарайся вникнуть. Ключевые моменты я выделил:

inpost

PHP_Fox
Никогда не понимал людей, которые программировать ни для кого не хотят и надеются, что программирование является хобби. Без реальных проектов программирование твоё будет выглядеть как копание в песочнице младенцем.
И к тебе подойдет цитата из уст Николая: "Такие люди убивают время — они берут, ничего не давая взамен, они отнимают время, которое мы могли бы посвятить другому вопросу, более интересному, и другому человеку, более достойному ответа". Ты балуешься, а не программируешь, и меня печалит, что я столь много времени посвятил тебе.