Это просто ужасно ...
Зачем нам твои комментарии? Зачем в данном случае HTML-код? Зачем использовать строковые включения, напряжные для любого фреймворка, когда есть возможность корректно и понятно включить требуемый код? Куда делись твои закрывающие знаки semicolon?
Плюс-минус твой код можно трансформировать в подобный:
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
$(document).ready(function(){
$.getJSON("weekendtask.json", function(data){
$.each(data.Buttons, function(_, button){
$.each(button.Conditions, function(_, condition){
var but = $("<input/>").attr({type:"button", value:condition.ButtonText}).css({backgroundColor:condition.Colors.BGColor, color:condition.Colors.FontColor});
but.on("click", {action:condition.JavascriptAction}, function(event){eval(event.data.action);});
$("#ulObj").append($("<li/>").append(but)).append("test</br>");//можно все свести в одну строку, но для читаемости кода разбил на несколько
});
});
});
});
А в общем - это жуть ... Как раз тот случай, когда программист не полностью понимает язык и механизмы в нем, но уже вовсю использует фреймворк и лепит откровенный говнокод.
Главная проблема - строка, в которой находится скрипт, который должен выполниться при клике, так не будет работать, как ты написал. Выполняется код, а не строковая константа. Да и по большому счету, передача скрипта таким образом - это несерьезное решение, так обычно не делают.
Еще пару замечаний:
- проследи за строкой, которую ты "лепишь" - там неправильное соответствие кавычек;
- каждый язык имеет определенный необязательный (или негласный) синтаксис; js имеет ряд правил, которых придерживаются, в том числе написание функций и переменных с маленькой буквы, в том числе внутри json.
А в остальном - настолько данный код далек от приятного, понятноинтуитивного и качественного кода, что даже не находятся слова назвать что еще не так. Кстати, это очень присуще двум группам программеров: на бейсике и на ASP (.NET). Что-то там такое дают, что клиентская часть у них получается сделанной через одно место. Тут, конечно, мои извинения - мнение сугубо личное и субъективное, прошу не принимать близко и не обижаться.
