то, что бросилось в глаза...

Misha v.3 18.07.2003 14:47

1. postprocess в приведенном виде - ЗЛО. посмотрите сколько вы сэкономите байтиков (парсер автоматически делает оптимизацию).

2. совершенно не ясно, зачем описывать конфигурационный auto.p и зачем в него класть КОД. это - конфигурационный файл, в нем рекомендуется определять кодировки, строки подключения, методы по фатальной обработки ошибок, но не код...

3. термины... "Класс @main[] - это основной класс". @main[] - это МЕТОД, или в вашей терминологии - процедура, но никак не класс (хотя учитывая тот факт, что в документации парсера это называется "метод", наверное стоит придерживаться такой-же терминологии). из-за подобных вещей читать обяснение невозможно. читать приходится код. также мне кажется, что комментарии не должны нести смысл документации и ликбеза: в них стоит писать то, что относится к коду (что делает этот код, а не разжевывать синтаксис/цитировать ФАК).

4. rusage-у место в постпроцессе...

5. ^if(-f "/html/classes/class_navigat.p"){} - не понятно зачем.
есть переменная CLASS_PATH. расположите все в ней и не работайте с путями в таком виде. к тому-же классы лучше распологать вне веб пространства, в этом случае даже если кто-то забудет/снесет ограничение доступа в данный каталог, ничего страшного не произойдет, с веба никто не придет.

6.

^switch[$navigat_table.align]{
^case[1]{top}
^case[2]{left}
^case[3]{center}
^case[4]{right}
^case[5]{bottom}
}

- попробуйте использовать вместо этого хеши. уверен, вам понравится. как? есть документация. код будет красивее и понятнее...

7. ...UPDATE menu_item SET align = '$form:navigat_align'...
сколько раз об одном и том-же: -> SET align = ^form:navigat_align.int(0) и т.д. это - числа!

8.

---