# Re: Обновление ii-db-utils
vit01 (mira, 1) → Andrew Lobanov – 01:40:17 2016-05-18
vit01>> Убрано много дублирований кода
AL> Можешь показать где? А то я тот ещё быдлокодер =)
Было весьма странно наблюдать в скриптах статистики, как ты по 3 раза проверяешь количество аргументов и делаешь 3 разных условия с одним и тем же Usage :)
А ещё код парсинга даты 2 раза один и тот же.
Объединить скрипты решил из-за того, что обнаружил их полную одинаковость (за исключением алгоритма подсчёта).
Кстати, там у тебя баг один есть. Если все заданные эхи пустые (либо в них нет сообщений за данный период времени), то value_of_division обращается в 0, и происходит деление на 0.
Вообще, если интересно, ты сам можешь исходники посмотреть.
vit01>> (например, эхи вводятся из stdin, а не через конфиг)
AL> Вот про конфиг это зря. ИМХО, конечно.
С ним просто кода больше (а заголовок не есть важно). Если делать моим путём, то можно и вот так:
или вот так:
vit01 (mira, 1) → Andrew Lobanov – 01:40:17 2016-05-18
vit01>> Убрано много дублирований кода
AL> Можешь показать где? А то я тот ещё быдлокодер =)
Было весьма странно наблюдать в скриптах статистики, как ты по 3 раза проверяешь количество аргументов и делаешь 3 разных условия с одним и тем же Usage :)
А ещё код парсинга даты 2 раза один и тот же.
Объединить скрипты решил из-за того, что обнаружил их полную одинаковость (за исключением алгоритма подсчёта).
Кстати, там у тебя баг один есть. Если все заданные эхи пустые (либо в них нет сообщений за данный период времени), то value_of_division обращается в 0, и происходит деление на 0.
Вообще, если интересно, ты сам можешь исходники посмотреть.
vit01>> (например, эхи вводятся из stdin, а не через конфиг)
AL> Вот про конфиг это зря. ИМХО, конечно.
С ним просто кода больше (а заголовок не есть важно). Если делать моим путём, то можно и вот так:
echo "pipe.2032 ii.14 develop.16" | ./visual-stats.py -t echoareas -s 2016.04.18 -e 2016.05.18
или вот так:
cat my_config | ./visual-stats.py ........