не похоже это на класс :)
Misha v.3 14.05.2009 15:08
это набор методов, который повязан на переменных из MAIN.
из того, что не нравится лично мне:
- ссылки в MAIN
- описание локальных переменных отсутствует как класс
- неадекватное название переменных (например первое упоминание $id в @userId[] -- это совсем не $id)
- использование try только для того, чтобы заткнуть ругань.
- достаточно большое количество повторов кода. например в том-же @userId[] можно один раз в самом начале написать $result(0).
- "Вы успешно вышли из системы! Но возникла проблема с БД!" -- ахренительно.
- '^if($form:password eq $form:password2){' в registration лучше вынести до проверки наличия пользователя (- 1 sql запрос в этом случае)
- зачем вам ^now.sql-string[] при добавлении пользователя в БД? чем не угодил NOW()?
- $email[$form:email]^email.trim[] => ^form:email.trim[] и т.д.
- html (мы уже давно выдаём xml)
