可以动态生成这样的代码吗?或者这是一个主要的代码味道?如何做得更好?
我是网络新手,但我一直都在磕磕绊绊,我不明白为什么。
// Create a js function that applies foo to each group of controls
foreach (KeyValuePair<string, Dictionary<Control, string>> pair in maps)
{
js.Append(pair.Key);
js.Append("=function(grid){Prm.remove_endRequest(");
js.Append(pair.Key);
js.Append(");if(grid && grid._element)"); // ... blah blah blah
}
page.ClientScript.RegisterClientScriptBlock(page.GetType(), key + "Ajax",
js.ToString(), true);
答案 0 :(得分:4)
在你开始在整个地方做这件事之前,我不认为它是一种气味。
但是,请考虑这些将在未来有所帮助的变化:
编写数据驱动的JS函数,只动态生成所需的数据。这样,您的所有JS都可以隐藏在快速静态文件中,而您的服务器只会发送数据。 这是一个比(2)和(3)更好的设计变更 - 它确实不是那么难。只需考虑当前代码生成器所需的数据,提供该数据而不是JS代码,然后将JS代码包装在接受该数据作为输入的“工厂函数”中。
使用JS代码的模板就像使用HTML模板一样。这样,当你真的只想更改一些变量名时,你不必在C#流控制/数据控制代码中徘徊。我建议使用它协助的视图名称命名模板文件。如果您有 Home.aspx
,那么您可能会使用JS代码模板 Home_DoCrazyGridThing.js
, Home_DoOtherCrazyThing.js
。您可以编写一个简单的模板引擎,也可以使用许多现有的模板引擎。
在生成代码时创建一个薄层,这样就可以明显地对维护者做了什么。也就是说,拥有一个具有不同智能水平的JSCodeGenerator
类(了解语言或只允许您在其中转储字符串或与(2)中的emplate引擎接口)。