Форум: Форум PHPФорум ApacheФорум Регулярные ВыраженияФорум MySQLHTML+CSS+JavaScriptФорум FlashРазное
Новые темы: 0000000
PHP на примерах (2 издание). Авторы: Кузнецов М.В., Симдянов И.В. Объектно-ориентированное программирование на PHP. Авторы: Кузнецов М.В., Симдянов И.В. Программирование. Ступени успешной карьеры. Авторы: Кузнецов М.В., Симдянов И.В. PHP Puzzles. Авторы: Кузнецов М.В., Симдянов И.В. PHP 5. На примерах. Авторы: Кузнецов М.В., Симдянов И.В., Голышев С.В.
ВСЕ НАШИ КНИГИ
Консультационный центр SoftTime

Форум PHP

Выбрать другой форум

 

Здравствуйте, Посетитель!

вид форума:
Линейный форум Структурный форум

тема: Система статистики Sin count
 
 автор: Lenux   (15.10.2005 в 13:11)   письмо автору
 
 

Написал я здесь не давно скрипт статистики (по своему:)), зацените
подробности в Readme.

   
 
 автор: Loki   (15.10.2005 в 23:41)   письмо автору
 
   для: 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'

один запрос к базе я вам сэкономил:)

В общем, это после просмотра файла счетчика по диагонали. Ставить пока не буду, так что админку не заценить.
Удачи!

   
 
 автор: Lenux   (16.10.2005 в 10:10)   письмо автору
5.6 Кб
 
   для: Loki   (15.10.2005 в 23:41)
 

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

   
 
 автор: Loki   (16.10.2005 в 15:13)   письмо автору
 
   для: Lenux   (16.10.2005 в 10:10)
 

Глянул... пункт 1 и 3 поправили.
Обратил внимание что отображается время генерации страницы. Но учитывая что счетчик инклудится в основной код, это время неверно.

   
Rambler's Top100
вверх

Rambler's Top100 Яндекс.Метрика Яндекс цитирования