The OpenNET Project / Index page

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



Индекс форумов
Составление сообщения

Исходное сообщение
"Релиз открытой биллинговой системы  ABillS 0.5"
Отправлено vgray, 25-Дек-09 12:10 
>По подробней

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 тегов.

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 года ( или больше), ваш уровень как разработчика несомненно вырос, поэтому и надо рефакторить код, чтобы не тащить ошибки юнности в светлое будующее :)


 

Ваше сообщение
Имя*:
EMail:
Для отправки ответов на email укажите знак ! перед адресом, например, !user@host.ru (!! - не показывать email).
Более тонкая настройка отправки ответов производится в профиле зарегистрированного участника форума.
Заголовок*:
Сообщение*:
  Введите код, изображенный на картинке: КОД
 
При общении не допускается: неуважительное отношение к собеседнику, хамство, унизительное обращение, ненормативная лексика, переход на личности, агрессивное поведение, обесценивание собеседника, провоцирование флейма голословными и заведомо ложными заявлениями. Не отвечайте на сообщения, явно нарушающие правила - удаляются не только сами нарушения, но и все ответы на них. Лог модерирования.



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

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