[ABD] GeoMod

Coagulant · Сообщение **Coagulant** » 21.02.2005 21:24

Описание мода: Показывает флаг страны пользователя во viewtopic.php, основываясь на его ip. Страна определяется по бд ip2country, поставляемой вместе с модом.
Версия мода: 0.2.1
Cложность установки: Низкая
Время, необходимое для установки: ~ 7 минут
Демо: Посмотреть как это выглядит можно здесь
Скриншот: Различные флаги

Скачать: geomod_021.zip (555 кб) (мод+база ip+флаги)

Устарело:

Уважаемым гуру, просьба подсказать, как прооптимизировать запрос и базу, ибо работает медленно. Любые комментарии/пожелания приветствуются

Структура базы:

Код: Выделить всё

CREATE TABLE `phpbb_ip2country` (
  `ip_from` varchar(8) NOT NULL default '',
  `ip_to` varchar(8) NOT NULL default '',
  `country_short` char(2) NOT NULL default '',
  `country` varchar(50) NOT NULL default ''
) TYPE=MyISAM;

INSERT INTO `phpbb_ip2country` VALUES ('0206be38', '0206be3f', 'GB', 'UNITED KINGDOM');

Запрос (модифицирован стандартный из viewonline.php):

Код: Выделить всё

$sql = "SELECT u.user_id, u.username, u.user_allow_viewonline, u.user_level, s.session_logged_in, s.session_time, s.session_page, s.session_ip, i.country
	FROM ".USERS_TABLE." u, ".SESSIONS_TABLE." s LEFT JOIN ".IP2COUNTRY_TABLE." i
	ON (s.session_ip>=i.ip_from AND s.session_ip<=i.ip_to)
	WHERE u.user_id = s.session_user_id
		AND s.session_time >= ".( time() - 300 ) . "
	ORDER BY u.username ASC, s.session_ip ASC";

Vladson · Сообщение **Vladson** » 21.02.2005 21:42

Оригинал

Код: Выделить всё

$sql = "SELECT u.user_id, u.username, u.user_allow_viewonline, u.user_level, s.session_logged_in, s.session_time, s.session_page, s.session_ip
	FROM ".USERS_TABLE." u, ".SESSIONS_TABLE." s
	WHERE u.user_id = s.session_user_id
		AND s.session_time >= ".( time() - 300 ) . "
	ORDER BY u.username ASC, s.session_ip ASC";

модифицированый

Код: Выделить всё

$sql = "SELECT u.user_id, u.username, u.user_allow_viewonline, u.user_level, s.session_logged_in, s.session_time, s.session_page, s.session_ip, i.country
	FROM ".USERS_TABLE." u, ".SESSIONS_TABLE." s, ".IP2COUNTRY_TABLE." i
	WHERE u.user_id = s.session_user_id
		AND s.session_time >= ".( time() - 300 ) . "
		AND s.session_ip >= i.ip_from
		AND s.session_ip <= i.ip_to
	ORDER BY u.username ASC, s.session_ip ASC";

Не проверял, но должно быть так

Coagulant · Сообщение **Coagulant** » 21.02.2005 21:51

Vladson
А как быть, если ip не определится по базе? С таким запросом пользователь просто пропадет из списка. Хотя такой случай (не определится страна по базе) я думаю, маловероятен. Однако, опять же, для локальных форумов не будет работать.

Егор Наклоняев · 21.02.2005 21:57

Гм. А почему не использовать бинарную версию файла, которая свободна распространяется, плюс имеются все библиотеки?

Добавлено спустя 4 минуты 35 секунд:

Кстати, примерно месяца три назад я обсуждал с Xpert стоит ли делать этот MOD или нет и решил его не делать.

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

Coagulant · Сообщение **Coagulant** » 21.02.2005 22:11

Егор Наклоняев писал(а):Гм. А почему не использовать бинарную версию файла, которая свободна распространяется, плюс имеются все библиотеки?

А смысл? Разве будет быстрее? (простите моё невежество)

Vladson · Сообщение **Vladson** » 21.02.2005 22:42

А как быть, если ip не определится по базе? С таким запросом пользователь просто пропадет из списка.

Естественно, но это самый простой способ повысить быстродействие (правда будет настолько малый прирост что я вообще сомневаюсь будет ли он вовсе)

Coagulant · Сообщение **Coagulant** » 21.02.2005 23:24

Vladson
Для почти пустой таблицы посетителей ваш вариант у меня дает выйгрыш в 1.1 секунды (оно и понятно, LEFT JOIN с выражением отбора - не лучший вариант).Выходит 1.2 и 0.09 соответственно.

Пробовал индексировать таблицу phpbb_ip2country различными способами, но эффекта не дает, видимо надо править запрос как-то.

Coagulant · Сообщение **Coagulant** » 23.02.2005 0:49

Предложенный вариант запроса без LEFT JOIN для 100 пользователей онлайн :wink: дает 1.9599 сек. Оригинальный запрос - 0.0564 сек. Надо бы и его прооптимизировать как-то...

Проблему мода для локалки можно решить добавлением соответствующих диапазонов в таблицу. Получаем страну LAN

Код: Выделить всё

INSERT INTO `phpbb_ip2country` VALUES ('0a000000', '0affffff', 'LN', 'LAN');
#10.0.0.0 - 10.255.255.255
INSERT INTO `phpbb_ip2country` VALUES ('7f000000', '7fffffff', 'LN', 'LAN');
#127.0.0.0 - 127.255.255.255
INSERT INTO `phpbb_ip2country` VALUES ('ac100000', 'ac1fffff', 'LN', 'LAN');
#172.16.0.0 - 172.31.255.255
INSERT INTO `phpbb_ip2country` VALUES ('c0a80000', 'c0a8ffff', 'LN', 'LAN');
#192.168.0.0 - 192.168.255.25

Coagulant · Сообщение **Coagulant** » 09.03.2005 19:52

Был найден более быстрый запрос (несколько десятых секунды выйгрыша):

Код: Выделить всё

$sql = "SELECT u.user_id, u.username, u.user_allow_viewonline, u.user_level, s.session_logged_in, s.session_time, s.session_page, s.session_ip, i.country 
   FROM ".SESSIONS_TABLE." s STRAIGHT_JOIN ".IP2COUNTRY_TABLE." i STRAIGHT_JOIN ".USERS_TABLE." u
   WHERE u.user_id = s.session_user_id 
      AND s.session_time >= ".( time() - 300 ) . " 
      AND s.session_ip
      BETWEEN i.ip_from
      AND i.ip_to
   ORDER BY u.username ASC, s.session_ip ASC";

Ясно, что подобные конструкции в принципе не смогут обладать должным быстродействием (сотые доли секунды).

Как вариант, можно определять страну для каждого пользователя при создании сессии и хранить её в соответствующей таблице, просто добавив еще одно поле. Что думаете по этому поводу? Не будет ли еще медленнее в итоге?

Предполагается добавить приблизительно вот такой sql запрос при создании сессии:

Код: Выделить всё

SELECT country
FROM phpbb_ip2country
WHERE ip_from <= 'dee10000'
ORDER BY ip_from
LIMIT 1

Coagulant · Сообщение **Coagulant** » 11.03.2005 17:25

Вышла версия 0.2.0

Ссылка в первом сообщении темы.

Код: Выделить всё

##   2005-03-12 - Version 0.2.0
##              - IP-to-Country Database updated (01.03.2005)
##              - Russian country names
##              - Country flags in viewonline.php
##              - SQL-query improved

Carbofos · Сообщение **Carbofos** » 17.03.2005 17:44

Посмотрал Гео - понравилось, классная идея

Посмотрел базу данных - понравилось меньше. Там все страны прямо текстом выписаны. Может, лучше заменить их на коды? Я так думаю, одного байта на все страны хватить должно

Тогда во-первых размер уменьшится, во-вторых, может и еще быстрее будет

Coagulant · Сообщение **Coagulant** » 17.03.2005 18:29

Посмотрел базу данных - понравилось меньше. Там все страны прямо текстом выписаны. Может, лучше заменить их на коды? Я так думаю, одного байта на все страны хватить должно

Вы не правы. Придется еще одну таблицу соответствий id страны и названия делать, а это лишний join. Место сэкономим, а в скорости потеряем.

Carbofos · Сообщение **Carbofos** » 17.03.2005 18:35

Ну, я не настаиваю, я не спец

Кстати, где можно почитать о быстродействии различных команд в SQL?

Или (совсем улет) - закинуть страны в lang-файлы.

Тогда вообще не важно будет, на каком форуме (имею ввиду язык) МОД ставить

В смысле, тут еще надо подумать, где скорости больше потеряется, в join-е или в Апаче при загрузке и парсинге отдельного файла (все, уже вижу

)

Coagulant · Сообщение **Coagulant** » 17.03.2005 18:44

Идея с lang-файлами мне нравится, я думаю, это оптимальный вариант.
Попробую реализовать, думаю не будет сильно медленнее.

Coagulant · Сообщение **Coagulant** » 20.03.2005 16:25

Код: Выделить всё

##   2005-03-19 - Version 0.2.1
##              - geomod_db_inst.php included for easy database install (mysql only)
##              - Country names moved to lang_main.php to manage mod translations
##              - Russian translation included

В следующей версии планируется добавить страну и флаг в профиль пользователя (автодетект с возможностью вручную поменять).