MajorDoMo

Привет! Вначале скажу что проект - супер, автору - респект!

Поглядел я немного на пхп-исходники, образовались замечания, может они и нн важны автору, просто оставлю их тут:

• 1. код и шаблоны перемешаны, организация данных(файлов) хромает
  
  2. мультиязычность намного лучше делать путем использования get_text а не изобретать пхп-аналоги. таким образом намного проще будет и переводчикам и разработчикам
  
  3. нет автолоадера а "либы" грузятся скриптом от проекта 2004 года...
  
  4. нет "код-стайла". в разных файлах вижу разное форматирование, отступы - 1 пробел, переводы строк - где win где linux, кодировка то утф8 то цп1251... конечно можно и так но если над проектом будет работать > 1 программиста - то становится весьма неприятным упущением
  
  5. php течет. конечно, в последних версиях (5.3/5.4) появился есть более-менее нормальный сборщик мусора, однако php все равно течет. основной цикл на пхп - ок, но что будет если (когда) он упадет? предусмотрен ли этот момент?
  
  6. обработка ошибок - отдельный разговор. возьмем БД. в текущем контексте предполагается что коннект всегда один, ок. если происходит ошибка - она пишется в лог, ок. вот только возьмем пример из cycle.php:

  Код: Выделить всё

  $objects=SQLSelect("SELECT ID, TITLE FROM objects WHERE $o_qry");
  $total=count($objects);

  и что что ошибка? код то выполняется дальше не проверяя, есть ли ошибка или нет...
  Сюда же - почему закомментировано set_error_handler - не отлавливаются ошибки там где это руками не реализовано и почему не используются эксепшены - с ними было бы намного проще...?
  
  n. некоторые решения в коде меня удивляют - к примеру, зачем для проверки синтаксиса кода загонять этот код в фаил, почему было бы его не загнать в STDIN при вызове php -l?

PS замечу, что смотрел я код минут 15, потому на объективность не претендую.
основная идея - так как система управляет домом, безопасностью в доме, на нее рассчитывают люди, то должен быть порог отказоустойчивости. и это критично.

Спасибо!
Код на самом неде "не на выставку"
Но тем и хорош open source, что проблемы на виду и со временем исправятся...