Поглядел я немного на пхп-исходники, образовались замечания, может они и нн важны автору, просто оставлю их тут:
- 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, потому на объективность не претендую.
основная идея - так как система управляет домом, безопасностью в доме, на нее рассчитывают люди, то должен быть порог отказоустойчивости. и это критично.
