Найти - Пользователи
Полная версия: Покритикуйте прогу
Начало » Python для экспертов » Покритикуйте прогу
1 2
slivlen
asv13
прочел, все из-за несовместимости настроек редакторов.
Молодец. 5. Давай зачетку. :)
dvs
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 работать отказался :(
bialix
slav0nic
asv13
вам же сказали, читайте РЕР8 )
не все юзают скайте, в других может съехать код, да и “стандарт” в питоне таб==4 пробела
Как однажды написал Гвидо в своем блоге: “если в коде стоит отступ в 4 пробела – этот код я писал для своих проектов, если в 2 пробела – то для Google. Если в коде стоит Tab – этот код писал не я”
dvs
И ещё мне стало интересно, зачем в этой проге используется SQLite (точнее зачем что-то сохранять на диск между запусками, на кэширование не похоже :| )
asv13
dvs
Спасибо за SQL-ликбез, поправлю поделюсь результатами. Скрипт за 20 сек считает только малеьнкий лог 400 Кб в примере - а логи весят и 50 и более Мб, мой тестовый 22 Мб обрабатывается 5 минут (Centrino 1.6 Ghz), а некотрые тестирующие оставляли считать на всю ночь )).
dvs
И ещё мне стало интересно, зачем в этой проге используется SQLite (точнее зачем что-то сохранять на диск между запусками, на кэширование не похоже
Лог парсится в базу, куда заносятся только новые игры, все результаты через SQL формируются - через конфиг можно добавлять новые таблицы. Предыдущая версия проги обходилась без SQL. Всё это для тех кто не хочет или не может ставить себе apache (играет по локалке на работе например) но нуждается в статистике. Конечно страшное дело сколько файлов на диске сохраняется.
В любом случае если даже буду переводить на apache и тп, время в 1 секунду на выполнение запроса - многовато.
bialix
Как однажды написал Гвидо ..
Я бы сильно озадачился с двумя пробелами. Такой минимализм - как будто дисковое пространство у Google в дефиците :) а табы при этом вне закона.
niro
Я бы сильно озадачился с двумя пробелами. Такой минимализм - как будто дисковое пространство у Google в дефиците smile а табы при этом вне закона.
Давайте пофлудим на это тему ?!
OlDer
asv13

if not tmpl.has_key(tmpl_name):
tmpl = Template(file = tmpl_name, filter = EncodeUnicode)
Так, к слову. Тут говорят что .has_key() is deprecated, use ‘in’
This is a "lo-fi" version of our main content. To view the full version with more information, formatting and images, please click here.
Powered by DjangoBB