Почему не выполняется условие php?

Question

[Adel Abuhasira]

Пытаюсь организовать счетчик просмотра статьи, но что-то не заладилось...
Логика такая:
- Получаем ip-клиента
- Проверяем, есть ли он в бд
- Если есть - выходим
- Если нету - записываем ip в бд и делаем +1 к просмотрам статьи

Новый айпишник в бд записывается, но при повторном заходе на статью просмотр опять плюсуется, несмотря на то, что такой айпи в бд есть :С

Вот код:

$ip_viewer =  $_SERVER["REMOTE_ADDR"];
        $query_views = mysql_query("SELECT `ip` FROM `ip_viewer` WHERE `ip` = '$ip_viewer'",$db);
          mysql_fetch_array($query_views);
        
          if(isset($query_views['ip'])){
                exit;
          }else{
            $views_update = mysql_query("UPDATE `services` SET `views`=`views`+1 WHERE `services`.`url` = '$url'",$db);
            $views_insert = mysql_query("INSERT INTO `ip_viewer` (ip) VALUES ('$ip_viewer')",$db);
          }

Answer 1

[SharuPoNemnogu]

может стоит посмотреть документацию, там хорошо описано как работает функция, даже с примерами. А вообще она больше не поддерживается.

Comments

[Adel Abuhasira]

Не пойму, как нужно изменить?

[SharuPoNemnogu]

Adel Abuhasira все просто, почитать по ссылке, посмотреть пример, понять где в коде накосячил, переделать

[Adel Abuhasira]

SharuPoNemnogu: мне кажется, дело не в этом. У меня на локалке старая версия (5.2) и все равно не работает, никаких ошибок не выдает.

[Adel Abuhasira]

Все таки, вы не правы.

Я написал код иначе - вместо "mysql_fetch_array" я использовал "if(mysql_num_rows($query_views) < 1)" и это также не привило к ожидаемому результату.

Answer 2

[ThunderCat]

$query_views =mysql_fetch_array($query_views); //будет работать, но это кривое решение

Внимательней читайте код, внимательно читайте доки, ОЧЕНЬ внимательно читайте советы опытных людей. Вам написали - функция mysql_fetch_array депрекатед, это раз, и второе - вы ее неверно используете.
Используйте функции вывода переменных (хотя бы) для отлова багов var_dump, print_r;

Comments

[Adel Abuhasira]

Я написал код иначе - вместо "mysql_fetch_array" я использовал "if(mysql_num_rows($query_views) < 1)" и это также не привило к ожидаемому результату.

[ThunderCat]

Adel Abuhasira: а просто проверить что и куда попадает при ваших манипуляциях - неужели так тяжело додуматься - тот же $nr = mysql_num_rows($query_views); var_dump($nr);

Answer 3

[firm]

Немного поправлю $query_views - дескриптор результата запроса. Он не содержит результирующие данные по запросу, их можно получить так:
$arr_ip = mysql_fetch_array($query_views);