taliban 8 ноя 2008 в 16:01

темплейты в JavaScript

3 мин

1.8K

Чулан

Комментарии 32

linki 8 ноя 2008 в 16:42

интересно, а если будет много шаблонов, какова будет скорость выполнения?

taliban 8 ноя 2008 в 17:04

если честно, скорость я не проверял, но этот метод не претендует бить скоростные характеристики, он просто освобождает от кода который Вы писали бы для этих «много шаблонов» в реализации DOM, потому как если делать все это через DOM, то размер кода был бы гораздо больше

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

alexshelkov 8 ноя 2008 в 18:27

Пару замечаний, если позволите:
1. Подобная функциональность элементарно реализуется без всяких классов, т.е чтобы класс имел право на существование нужно расширить функциональность.
2. Для каждого нового темплейта создаться куча лишних функций(точнее 1 функция evaluate), надо переписать класс так, чтобы использовать прототип, да и вообще функция evaluate(и название лучше заменить на eval =) ) сделана не очень хорошо. Или вообще убрать класс и сделать функцию =)
3(знаю соврал). Ну что за скобки "#{", "}" почему они разные? Неудобно ведь =)
4. В framework'ах есть подобная функциональность.

Я не придираюсь, просто решил оставить мнение, насколько оно обосновано судить вам =)

taliban 8 ноя 2008 в 19:26

1 — это реализовано в классе а не в функции итд для того чтобы один парсер соответствовал одному темплейту, расширить функциональность ничего не мешает никому, берете и расширяете :)
2 — для нового темплейта создается не одна функция а один класс, класс можно дополнить методами, название функции «evaluate» оставил как была изначально (я выковырял этот парсер из фреймворка)
3 — скобки оставил как были изначально (меняется регулярное выражение, меняются скобки)
4 — подобная функциональность была только в одном фреймворке, именно из него я и вытянул эту штуку, потому как им не пользуюсь а фича понадобилась

а чем функция evaluate сделана не очень хорошо? помоему преллестно, свои задачи выполняет замечательно, проста

alexshelkov 8 ноя 2008 в 19:43

1. Ну дык раз уж создавать, а тем более потом выкладывать в общее пользование класс, то стоит сделать его более функциональным. Я рассматриваю класс в том виде, в котором он выложен вами, тут нужно сильно много расширять =).
3. Да, признаю, это легко поменять.
4. Ясно.

Функция evaluate сделана плохо тем, что ее можно написать гораздо удобней для чтения. Помещать «смысл» функции целиком в return, через метод встроенного класс, где используется еще 1 анонимная функция… ну не знаю, мне это кажется несколько странным.

AndreyTS 9 ноя 2008 в 11:59

>… Помещать «смысл» функции целиком в return… кажется несколько странным.

ну это очень индивидуально) раньше я бы с вами согласился, а теперь, после более тесного знакомства с функциональными языками (к каковым в значительной степени относится JavaScript), я бы, напротив, даже переписал бы её в ещё более функциональном стиле. Так и яснее, и короче, на мой взгляд:

this.evaluate = function(object)
{
return _template.replace(/(#\{.*?\})/g, function(match)
{
var key = match.substring(2, match.length-1);
return (undefined !== object[key] )? object[key]: "";
})
}