|
|
|
| Написал я здесь не давно скрипт статистики (по своему:)), зацените
подробности в Readme. | |
|
|
|
|
|
|
|
для: Lenux
(15.10.2005 в 13:11)
| | Ну если интересует конструктивная критика, то держите:)
1. файлы admin и count перепутаны местами:)
2. в файле count у вас аж 5 SQL запросов. из них 4 ничего полезного, в общем-то, не делают. Раз уж вы информацию все равно складываете в одну таблицу, то и считали бы хиты в ней.
3. БД вы скармливаете данные безо всякой проверки. То есть чтобы ее взломать, достаточно просто в URL написать sql запрос.
4. Таблица page у вас довольно быстро разрастется до фантастических размеров, так что анализаровать и хранить ее будет довольно неудобно. Лучше бы под повторяющиеся данные отдельные таблицы.
5. крутить счетчик лучше средствами MYSQL:
UPDATE pages SET f_count=f_count+1 WHERE p_file='$script_name'
|
один запрос к базе я вам сэкономил:)
В общем, это после просмотра файла счетчика по диагонали. Ставить пока не буду, так что админку не заценить.
Удачи! | |
|
|
|
|
 5.6 Кб |
|
|
для: Loki
(15.10.2005 в 23:41)
| | Переписал и усовершенствовал. | |
|
|
|
|
|
|
|
для: Lenux
(16.10.2005 в 10:10)
| | Глянул... пункт 1 и 3 поправили.
Обратил внимание что отображается время генерации страницы. Но учитывая что счетчик инклудится в основной код, это время неверно. | |
|
|
|