Уведомления

Группа в Telegram: @pythonsu

#1 Янв. 24, 2007 10:01:44

slivlen
От:
Зарегистрирован: 2006-07-06
Сообщения: 764
Репутация: +  0  -
Профиль   Отправить e-mail  

Покритикуйте прогу

asv13
прочел, все из-за несовместимости настроек редакторов.
Молодец. 5. Давай зачетку. :)



Офлайн

#2 Янв. 24, 2007 21:27:40

dvs
От:
Зарегистрирован: 2006-05-22
Сообщения: 176
Репутация: +  3  -
Профиль   Отправить e-mail  

Покритикуйте прогу

asv13,
Мелкие замечания:

вместо
“SELECT id, crashed, min_end, sec_end, pos FROM games WHERE id IN (SELECT max(id) FROM games)”
надо бы
“SELECT id, crashed, min_end, sec_end, pos FROM games WHERE id=(SELECT max(id) FROM games)”

вместо
'UPDATE games SET crashed = ? WHERE id=(?)',(True, self.num)
можно
'''UPDATE games SET crashed = ‘True’ WHERE id=(?)''', self.num)

вместо bla_bla.has_key(the_key)
сейчас модно the_key in bla_bla
(и вроде побыстрей должно работать, лень проверять сейчас)

PS. У меня твой скрипт отрабатывает за 20-21 сек, по-моему и так неплохо
PPS. Win32, _namemapper.pyd привинтил, c psyco работать отказался :(



Отредактировано (Янв. 24, 2007 21:29:14)

Офлайн

#3 Янв. 24, 2007 21:55:32

bialix
От:
Зарегистрирован: 2006-07-13
Сообщения: 774
Репутация: +  1  -
Профиль   Отправить e-mail  

Покритикуйте прогу

slav0nic
asv13
вам же сказали, читайте РЕР8 )
не все юзают скайте, в других может съехать код, да и “стандарт” в питоне таб==4 пробела
Как однажды написал Гвидо в своем блоге: “если в коде стоит отступ в 4 пробела – этот код я писал для своих проектов, если в 2 пробела – то для Google. Если в коде стоит Tab – этот код писал не я”



Офлайн

#4 Янв. 24, 2007 22:02:02

dvs
От:
Зарегистрирован: 2006-05-22
Сообщения: 176
Репутация: +  3  -
Профиль   Отправить e-mail  

Покритикуйте прогу

И ещё мне стало интересно, зачем в этой проге используется SQLite (точнее зачем что-то сохранять на диск между запусками, на кэширование не похоже :| )



Офлайн

#5 Янв. 25, 2007 12:49:34

asv13
От:
Зарегистрирован: 2007-01-22
Сообщения: 130
Репутация: +  0  -
Профиль   Отправить e-mail  

Покритикуйте прогу

dvs
Спасибо за SQL-ликбез, поправлю поделюсь результатами. Скрипт за 20 сек считает только малеьнкий лог 400 Кб в примере - а логи весят и 50 и более Мб, мой тестовый 22 Мб обрабатывается 5 минут (Centrino 1.6 Ghz), а некотрые тестирующие оставляли считать на всю ночь )).

dvs
И ещё мне стало интересно, зачем в этой проге используется SQLite (точнее зачем что-то сохранять на диск между запусками, на кэширование не похоже
Лог парсится в базу, куда заносятся только новые игры, все результаты через SQL формируются - через конфиг можно добавлять новые таблицы. Предыдущая версия проги обходилась без SQL. Всё это для тех кто не хочет или не может ставить себе apache (играет по локалке на работе например) но нуждается в статистике. Конечно страшное дело сколько файлов на диске сохраняется.
В любом случае если даже буду переводить на apache и тп, время в 1 секунду на выполнение запроса - многовато.
bialix
Как однажды написал Гвидо ..
Я бы сильно озадачился с двумя пробелами. Такой минимализм - как будто дисковое пространство у Google в дефиците :) а табы при этом вне закона.



Офлайн

#6 Янв. 26, 2007 14:36:35

niro
От:
Зарегистрирован: 2006-10-04
Сообщения: 178
Репутация: +  0  -
Профиль   Отправить e-mail  

Покритикуйте прогу

Я бы сильно озадачился с двумя пробелами. Такой минимализм - как будто дисковое пространство у Google в дефиците smile а табы при этом вне закона.
Давайте пофлудим на это тему ?!



Офлайн

#7 Янв. 28, 2007 20:43:19

OlDer
От:
Зарегистрирован: 2006-07-19
Сообщения: 133
Репутация: +  0  -
Профиль   Отправить e-mail  

Покритикуйте прогу

asv13

if not tmpl.has_key(tmpl_name):
tmpl = Template(file = tmpl_name, filter = EncodeUnicode)
Так, к слову. Тут говорят что .has_key() is deprecated, use ‘in’



Офлайн

Board footer

Модераторировать

Powered by DjangoBB

Lo-Fi Version