проверка формы обратной связи

[apexis]

Один спамер бомбит мой ящик через форму обратной связи на сайте. В каждом письме вводит телефон 123456. Я задумал написать проверку на введение этого телефона, но она не работает.

<div>Ваш телефон</div>
    <input type="text" name="phone" class="formfields" value="'.@$_POST['phone'].'" />
	<br /><span style="font-size: 11px; color: gray; margin-left: 147px;">Например: 0501111211</span><br /> ';
  if(@$_POST['submit'] and !@$_POST['phone'] or @$_POST['phone'] == '123456' ){
echo '<span style="color:red;">Не указан телефон</span><br />';

в чем ошибка?

Edited November 25, 2012 by apexis

[Radiocity]

Добавьте капчу в форму обратной связи и бомбить не будут. Если этот "спамер" живой, вычислите его IP и блокируйте его по адресу.

[Alex Shnayder]

Этот кусок выглядит рабочим, при вводе "123456" должна выводиться ошибка. Но подход в целом неверный, да и качество кода оставляет желать лучшего.

[Николя223]

  if(@$_POST['submit'] and ( !@$_POST['phone'] or @$_POST['phone'] == '123456') ){
echo '<span style="color:red;">Не указан телефон</span><br />';}

может так? да и вообще это не поможет. действительно баньте по ай-пи... либо - обманите его. фильтруйте письмо с этим номером (пусть он думает что письма доходят) ...

интересно... почему всегда один и тот же номер... бот?

[apexis]

Сделал проверку ФИО на проверку введения трех слов. Боты обычно одно вводят. Думаю такая защита получше и проще. Если банить по ip то можно сразу несколько людей забанить, которые сидят на одном ip.

[krasnyi]

Сделал проверку ФИО на проверку введения трех слов. Боты обычно одно вводят. Думаю такая защита получше и проще. Если банить по ip то можно сразу несколько людей забанить, которые сидят на одном ip.

Все это не очень хорошо - банить по номеру "123456" и банить по ФИО глупо -

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

Ставьте каптчу - один раз сделать и проблема решена.

Или хотя бы ответ на вопрос, если не каптчу.