[X] Помощник

[Pavl_mw Topic Starter]

Добрый день!
Существует счётчик для галереи изображений, но он не совсем правильно работает.
Есть два файла: index.php (в нём находится галерея, выводимая из БД) и page3.php (в нём срабатывает счётчик).
В index.php есть такая строка:

echo "<a href='page3.php?id=".$row['id']."'>$row[name]</a>";

при клике на которую происходит переход на page3.php и срабатывает счётчик.

Проблема в том, что, почему-то, в поле hosts таблицы img кол-во кликов записывается (следственно потом и выводится на экран), только значение одной картинки, а нужно, чтобы выводились значения кол-ва уникальных посетителей всех картинок, несмотря на то, что я прописываю в коде WHERE id = '".$_GET["id"]."', чтобы в поле hosts записывались значения для каждой картинке при клике на неё.
Вот полный код файла page3.php:

<?php
mysql_connect('localhost','root','');
$res = mysql_select_db('images');
//Получаем IP-адрес посетителя и сохраняем текущую дату
$visitor_ip = $_SERVER['REMOTE_ADDR'];
$date = date('Y-m-d');
$id = $_GET['id'];
echo $id;
$current_ip = mysql_query("SELECT ip_id FROM ips2 WHERE ip_address = '$visitor_ip'");
$hosts = $a['hosts'];
if(mysql_num_rows($current_ip) != 1) {
	//Заносим в базу IP-адрес текущего посетителя
	mysql_query("INSERT INTO ips2(ip_address) VALUES('$visitor_ip')");
	//Добавляем в базу +1 уникального посетителя (хост)
	mysql_query("UPDATE img SET hosts = hosts + 1 WHERE id = '".$_GET["id"]."'");	
}
$res = mysql_query("SELECT * FROM img");
$row = mysql_fetch_assoc($res);
echo '<p>Уникальных посетителей: ' . $row['hosts'] . '<br />';
 ?>

Так выглядят две таблицы базы данных images:


Помогите разобраться, в чём здесь проблема?


Сообщение отредактировал Pavl - 2.5.2020, 19:28

[MattCutts_mw]

ты серьезно, еще кто-то пользуется mysql?
какая у тебя версия пыха?
в твоем случае надо использовать либо COUNT либо SUM

"SELECT COUNT(1) FROM img"

при условии, что img у тебя уникальный,тогда данные у тебя будут корректные
а так твои запросы дырявые sql-injection прям из учебника можно провести, жанные надо экранировать, перед записью в базу


Сообщение отредактировал MattCutts - 2.5.2020, 22:35

[miketomlin]

ты серьезно, еще кто-то пользуется mysql?

Вы бы поточнее выражались. Могут не так понять ;)

[Pavl_mw Topic Starter]

MattCutts_mw, 
Использую версию php - 5.5. Есть возможность включить версию 7.0 и выше, но я сейчас изучаю php по видеоурокам, в которых рекомендуется  версия 5.5 - поэтому её и использую.
Прописал строку "SELECT COUNT(1) FROM img" - данные совсем не выводятся.
Прописал экранирование строки $id = mysql_real_escape_string($id); - ничего не изменилось.
Не понимаю, почему счётчик заносит значение в поле hosts, только один раз. Срабатывает один раз, несмотря на то, что я пишу WHERE id = '".$_GET["id"]."'");    При каждом клике по ссылке должно срабатывать, ведь так?

[MattCutts_mw]

Pavl_mw, выкинь эти курсы, они устарели оооочень давно, пятая версия уже года два как не поддерживается, если брать библиотеку, то хотя бы mysqli
при клике ничего не должно срабатывать.
php серверный язык программирования и он не может отслеживать события в браузере. для этого есть javascript, он отслеживает события в браузере, а потом может инициализировать обновления в бд через ajax
или просто при переходе на страницу, тогда да, можно обновить запись.
а твоя проблема в этом условии

if(mysql_num_rows($current_ip) != 1) {

обновление происходит в нем, а следовательно он не попадает в условие и повторное обновление не происходит

а если с одного IP зайдут два пользователя?  ) у тебя немного некорректно составлена логика

вынеси из условия

mysql_query("UPDATE img SET hosts = hosts + 1 WHERE id = '".$_GET["id"]."'");	

Сообщение отредактировал MattCutts - 3.5.2020, 16:41

[MattCutts_mw]

и тебе нужно не $row = mysql_fetch_assoc($res); а mysql_num_rows($res)
открой документацию или на курсе не учат ее читать?
mysql_num_rows — Возвращает количество рядов результата запроса
mysql_fetch_assoc — Возвращает ряд результата запроса в качестве ассоциативного массива

разницу видишь?

https://www.php.net/manual/ru/ref.mysql.php


Сообщение отредактировал MattCutts - 3.5.2020, 16:51

[Pavl_mw Topic Starter]

MattCutts_mw, 
Если изменить код на:

if(mysql_num_rows($current_ip) === 1) {

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

mysql_query("UPDATE img SET hosts = hosts + 1 WHERE id = '".$_GET["id"]."'");     

тоже - одни нули в таблицах.

То, что Вы пишите об устаревших курсах - это спасибо, я учту.

а если с одного IP зайдут два пользователя?  )

 
А как с одного ip могут зайти два разных пользователя?  :huh:  С двух разных ip  - один пользователь может зайти, а двое с одного как?
Это, наверное уже другая тема? (нужно будет куки и сессии подключать, насколько понимаю).

[MattCutts_mw]

А как с одного ip могут зайти два разных пользователя? :huh: С двух разных ip - один пользователь может зайти, а двое с одного как?

наверное они находятся в одной подсети, например у предприятия может быть один статичный IP адрес и 100 сотрудников выходя в сеть будут светить им, для тебя это будет один пользователь, а сессия ненадежна, закрыл браузер и все. нужна более уникальная связка данных, поищи в интернете.

сейчас найду пятый пых )


Сообщение отредактировал MattCutts - 3.5.2020, 17:01

[Pavl_mw Topic Starter]

наверное они находятся в одной подсети, например у предприятия может быть один статичный IP адрес и 100 сотрудников выходя в сеть будут светить им.  

 
Вообще-то, да. Например, в какой-либо организации есть wi-fi, который идёт от одного компьютера (с одного ip-адреса) на разные компьютеры. Но это, как раз следующая тема. Куки, сессии я дальше буду изучать.

[MattCutts_mw]

бросай ты этот курс лучше сразу, учись правильным вещам изначально, уже 20-й год на  дворе