辅助函数 - 希望 DRYness

Question

我喜欢使用辅助函数的想法，但我在设计方面遇到了一些困难。

我有一个 Javascript 函数，可以将实体添加到列表框 - 看起来或多或少像这样：

function MyEntityAdd(o) {
    var li = $('<li />').append(o.Product.Name);
    $('#SomeListbox').append(li);
}

...以两种方式调用：1）用于在页面加载时显示项目：

$(function() {
    @foreach (var item in Model) {
        @: MyEntityAdd({ Product: { Name: '@item.Product.Name' }});
    }
}

以及，2）用于添加项目当用户（通过 ajax 调用）创建一个新实体时到现有页面：

$.ajax({ type: 'post',
    url: '/MyEntity/Create',
    data: { ... },
    success: function(o) {
        MyEntityAdd(o)
    }
});

现在我的问题：我需要格式化产品，并通过辅助函数完成它：

@helper FormatProduct(MyEntity e)
{
    <strong>@e.Product.Name</strong> @e.Version
}

所以现在我可以像这样重新定义我的 Js（参数现在是平坦的，不是嵌入的物体对象）：

function MyEntityAdd(o) {
    var li = $('<li />').append(o.Product);
    $('#SomeListbox').append(li);
}

像这样调用：

$(function() {
    @foreach (var item in Model) {
        @: MyEntityAdd({ Product: '@Html.FormatProduct(item.Product)' });
    }
}

......一切都很好。除了现在 success ajax 调用不起作用，因为 Create 操作返回一个 JSON 对象...所以我必须在控制器 ie 内格式化产品返回：

[HttpPost]
public ActionResult Create(string Name)
{
    MyEntity e = new MyEntity(Name);
    db.MyEntities.Add(e);
    db.SaveChanges();

    return Json(e);
}

我还必须在这一端复制格式化功能：

[HttpPost]
public ActionResult Create(string Name)
{
    MyEntity e = new MyEntity(Name);
    db.MyEntities.Add(e);
    db.SaveChanges();

    return Json(new { Product = MyServerSideFormattingFunction(e) });
}

这很奇怪（不干燥）。给任何人带来灵感吗？

Answer 1

你的质疑是对的。就我个人而言，我喜欢 DRY，但我看到的关于您完全不喜欢的最终想法的最糟糕的事情是您将在控制器中添加 HTML，这从根本上来说是不好的做法，因为这是您的视图的工作。

我认为辅助函数很棒，但虽然它们只是服务器端，但它们并不是客户端使用的正确工具。

因此，我会将您的 MyEntityAdd 函数重写为：

function MyEntityAdd(o) {
    var productDisplay = '<strong>' + o.Product.Name + '</strong> ' + o.Version;
    var li = $('<li />').append(productDisplay);
    $('#SomeListbox').append(li);
}

一切都应该像使用辅助方法之前一样就位，当然它是 DRY 的。有时最简单的方法是最好的:)