The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



"Релиз открытой биллинговой системы  ABillS 0.5"
Версия для распечатки Пред. тема | След. тема
Форум Разговоры, обсуждение новостей
Исходное сообщение [ Отслеживать ]
Подсказка: Доступны два режима работы форума: "Раскрыть нити" и "Свернуть нити".
. "Релиз открытой биллинговой системы  ABillS 0.5" +/
Сообщение от AsmodeuS (?), 25-Дек-09, 13:18 
>>По подробней
>
>1) почему у вас в инсталяции по умолчанию файлы графиков (xml) пишутся
>в директорию /cgi-bin/admin

чтобы не лазили кому не лень в этот каталог

>2) нет единой системы логирования, поэтому процедура log_print у вас определена раз
>5
>
>./Abills/HTML.pm:sub log_print {
>./Abills/PDF.pm:sub log_print {
>./Abills/nas.pl:sub log_print2 {
>./Abills/XML.pm:sub log_print {
>./libexec/config.pl:sub log_print  {

вывод лога в разных форматах, некоторые лишние

>
>3) Модуль - это огромный файл webinterface c мешаниной из perl кода,
>sql комманд, и html тегов.

webinterface нет не одного SQL   запроса, все они в лаере взаимодействия с базой в каталоге AbillS/mysql/. Иногда встречаются HTML  теги но стараюсь их вынести в библиотеки визуализации HTML, XML, CONSOLE,PDF


>4) В коде куча строк вида,
>   #  print $prepaids{0}."\n";
>   #  print "LIMIT: $RESULT->{TRAFFIC_LIMIT} USED: $used_traffic->{TRAFFIC_SUM}";

да много встречается постараюсь выбросить поскорее

>5) интенсивно используются конструкции вида
>$list = get_smth()
>if ($list->[0]->[3]) { bla bla }
>те жесткая привязка на индексы, если что-то менять, то во первых сложно
>найти где это используется ( удобнее пользоваться именнованными хешами, грепом удобно
>искать где это используется), а во вторых стоит чуть изменить порядок
>следования элементов то все летих нахер.

Пока иследую новые способы работы с этим

>6) странная структура каталогов.
>несколько странно обнаружить модуль Auth.pm в директории, Abills/mysql/Auth.pm
>А если вы будете добавлять например PG, вы создадите Abills/pg/Auth.pm ?

Модуль взаимодействия с базой для авторизатора

>PS: Только не надо воспринимать меня как человка который пытается опустить ваш
>продукт, он работает, и многих устраивает ( в том числе и
>меня ), но качество кода там низкое. Ради интереса посмотрите на
>код постгреса. Проекту 4 года ( или больше), ваш уровень как
>разработчика несомненно вырос, поэтому и надо рефакторить код, чтобы не тащить
>ошибки юнности в светлое будующее :)

Большое спасибо мы учтём в будущем все Ваши комментарии, мнение со стороны очень ценно для нас.

Ответить | Правка | Наверх | Cообщить модератору

Оглавление
Релиз открытой биллинговой системы  ABillS 0.5, opennews, 23-Дек-09, 14:28  [смотреть все]
Форумы | Темы | Пред. тема | След. тема



Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру