Почему не работает условие else if?

Question

[santavits]

Почему не работают условия else if

Вот сам код

$x = "мой текст";
$search="http";
$dub = mysql_query("SELECT (text) FROM `biznes` WHERE 1", $db);
$ret = mysql_fetch_array($dub);
if (strstr($x, $search)) {
    echo "Рекламный пост, пропускаем";
	mysql_query("INSERT INTO `biznes`(reklama) VALUES ('1')");
}else if ($ret == $x){
	mysql_query("INSERT INTO `biznes`(reklama) VALUES ('1')");
	}else{
	mysql_query("INSERT INTO `biznes`(foto,text) VALUES ('/images/$imge','$x')");
	echo "Занесли пост в базу";
}

Условия таковы
Значением $ret получаем выборку с базы данных, со всеми строками где нужно сверить текст
И так вот само условие
Если в тексте есть http, то делаем запись в базу данных с текстом "Рекламный пост, пропускаем"
Если нет, то проверяем, есть ли в базе текст который мы хотим добавить сейчас
Если не того, не того нет, то добавляем текст который у нас в значение $x

Ну вот, а получается так
Если в тексте есть http, то делаем запись в базу данных с текстом "Рекламный пост, пропускаем"
Если не того, не того нет, то добавляем текст который у нас в значение $x

т.е второе условие оно просто напросто пропускает.

Comments

[abroabr]

business

[Алексей]

abroabr,
business это бизнес по-английски. А у автора бизнес по-русски. И это очень большая разница.

[santavits]

Алексей, название таблицы тут думаю роли вообще никакой не играет

[Алексей]

santavits, Кто знает, кто знает. Как говорил Врунгель: "Как вы яхту назовете, так она и поплывет" :)

[Adamos]

Полностью менять вопрос после ответов - очень дурная практика.
Примерно как ваш говнокод.
По текущему вопросу - почитайте документацию по тем функциям, которые используете. Особенно - их возвращаемые значения. Вы строку на голубом глазу сравниваете с массивом.
А лучше - познакомьтесь с PDO, пока в ваш запрос в $imge не пришло ../../../../../../etc/passwd, например. И забудьте уже mysql_ - функции навсегда.

Answer 1

[Александр Синицын]

А почему просто не сделать колонку unique? Зачем плодить в таблице пустые записи, если можно поставить 1 в поле reklama у единственного сообщения при ошибке добавления дубля... или увеличить на 1 - тогда будет счетчик спама )

Answer 2

[kot2566]

Примерно так:

$sql =mysql_query ("SELECT 'text' FROM 'biznes' WHERE 'text'='text')";
if(mysql_num_rows($sql) < 0)
{  

mysql_query("INSERT INTO `biznes`(foto,text) VALUES ('/images/$imge','$x')");
}else {
    mysql_query("INSERT INTO `biznes`(reklama) VALUES ('1')"); 
}

Comments

[Adamos]

В этом тексте прекрасно все.
И ochen' russkie названия полей с таблицей.
И использование mysql_ (а местами - mysqli_ !!!) %)
И подстановка переменных прямо в запрос, без всяких там плейсхолдеров. Причем одна из них - путь, по которому потом скрипт полезет "за картинкой".
И наворачивание трех запросов там, где должен быть один INSERT ... ON DUPLICATE KEY UPDATE
И особенно - то, что автор уверенно отметил его решением.
Верной дорогой идете, товарищи!

[kot2566]

Adamos, вы правы.

[santavits]

Свой код всё таки решил проверить, он и оказался верным, всё работает.
Может кому пригодится

$dub = mysql_query("SELECT (text) FROM `biznes` WHERE 1");
$ret = mysql_fetch_array($dub);
if ($ret == $x) {
mysql_query("INSERT INTO `biznes`(foto,text) VALUES ('/images/$imge','$x')");
}else {
    mysql_query("INSERT INTO `biznes`(reklama) VALUES ('1')");
  }

[Adamos]

kot2566, боюсь, приведение всех запросов к mysql_ (который давно deprecated) особенно картинку не улучшило :)

[kot2566]

Adamos, да, верно. Просто подвел под пример автора.

[santavits]

kot2566, Столкнулся с новой проблемой

Answer 3

[Евгений Воробьев]

elseif пишется слитно