Pull to refresh

Comments 6

> Код прозрачен так, что пояснений особо думаю не потребуется.

Вы даете длинный код без хайлайтинга на очень красивом, но очень трудно читаемом языке и говорите, что он прозрачен %)
Согласен. Но безуспешно пободавшись с парсером тэгов оставил так. :) Бум исправлять.
Делал что-то подобное, только у меня были такие отличия от вашего способа:

1) URL не жестко зашит в код js, а соответствует принципам REST. Соответственно, js-движок умеет понимать, какого типа запрос (и в случае, если запрос идет через xhr, отсылает и через PUT, и DELETE (если нужно, опять же)).

2) Я так понимаю, вы обрабатываете каждый url отдельно, потому что хотите по разному отображать результаты работы формы — у меня же такой задачи не стояло, у меня было по одной форме на страницу и достаточно было иметь один метод callback для каждой формы. Кстати, есть мнение, что в 99% задач дела обстоят точно также.

3) Ну, еще было бы неплохо (по крайней мере, я так сделал), прямо на этапе сбора данных с формы делать их валидацию, но у вас это записано в todo.

4) Ну и совершенно непонятно, что делает в форме inline-javascript — нет никаких причин его там держать. Но раз уж код вы писали за 15 минут, то простительно.

P.S. для того, чтобы не париться с подствекой кода, подготовьте пост в хабраредакторе.
Автору на заметку:
parent::Init();

Вызывать не нужно т.к. конструктор Zend_Form это делает сам.
Sign up to leave a comment.

Articles

Change theme settings