Question

我可以使用一些关于重构的建议。在我的应用程序中，用户可以动态添加新的表单字段;的CustomField。对于每种类型（文本，下拉列表，复选框等），都定义了ViewModel（TextBoxViewModel，DropDownViewModel，CheckboxViewModel等）。

当我发布表单时，会执行相应的编辑操作，并且我会读取每个自定义字段以存储它们的值。

目前实施工作但很难看;我通过所有ViewModel类型切换/ case / if / else，并根据我执行所需逻辑的类型。

这是当前的实施：

private static void MapToModel(Ticket ticket, TicketViewModel model)
    {
        ticket.Id = model.Id;
        ticket.Name = model.Name;

        ticket.Attributes.Clear();
        foreach (var cvm in model.Controls)
        {
            var attribute = new TicketAttribute
            {
                Id = cvm.Id,
                Name = cvm.Name,
            };

            if (cvm is TextBoxViewModel)
            {
                attribute.Value = ((TextBoxViewModel) cvm).Value;
            }else if (cvm is DropDownListViewModel)
            {
                attribute.Value = ((DropDownListViewModel)cvm).Values;
            }
            ticket.Attributes.Add(attribute);
        }
    }

我想将此重构为类似的东西，但不要将所有逻辑都放在ViewModel中。我能想到的最好的是访问者模式，我将向ViewModel类添加一个Accept方法，并使用访问者执行所需的逻辑：

这仍然需要AddAttribute方法中类型的相同切换逻辑：

foreach (var cvm in model.Controls)
    {
        ticket.Attributes.AddAttribute(cvm);
    }

这需要ViewModel类中的逻辑

foreach (var cvm in model.Controls)
    {
        ticket.Attributes.Add(cvm.AddAttribute);
    }

我想重构它以创建更通用的方法，以便将来添加新类型的字段时，我不必使用新结构更新所有代码以检查类型。

[提供帮助后的解决方案]

我不得不施放对象，我不能在IControlViewModel的不同实现中使用不同的返回类型，所以这是我必须解决的一部分，但总的来说这很漂亮。

  ticket.Attributes = model.Controls
    .OfType<IControlViewModel>()
    .Select(cvm => new TicketAttribute {
        Id = cvm.Id,
        Name = cvm.Name,
        Value = (string)cvm.OutputValue
        })
    .ToList();


  public interface IControlViewModel
    {
        string Id { get; }
        string Name { get; }
        object OutputValue { get; }
    }

    public abstract class ControlViewModel : IControlViewModel
    {
        public string Id { get; set; }
        public abstract string Type { get; }
        public string Label { get; set; }
        public string Name { get; set; }
        public bool Visible { get; set; }
        public abstract object OutputValue { get; }
    }

    public class TextBoxViewModel : ControlViewModel
    {
        public override string Type
        {
            get { return "textbox"; }
        }
        public override object OutputValue
        {
            get
            {
                return Value;
            }
        }

        public string Value {set; }
     }

Answer 1

1）创建一个界面，定义您将在每个视图模型上具有输出值属性

public interface IControlViewModel
{
    object OutputValue{get;}
}

2）在每个视图模型中实现接口：

public TextBoxViewModel: IControlViewModel
{
    ...
    public object OutputValue
    {
        get 
        {   
            //return whatever is your expected output value from control
            return Value; 
        }
    }
    ...
}

3）然后，您可以使用此单个LINQ语句获取所有属性：

ticket.Attributes = model.Controls
    .OfType<IControlViewModel>()
    .Select(cvm => new TicketAttribute {
        Id = cvm.Id,
        Name = cvm.Name,
        Value = cvm.OutputValue
        })
    .ToList();

4）即使您创建了新的控件类型，此代码也能正常工作，只需确保在新的视图模型中实现接口。

控制器中的viewmodel上的switch / case，任何重构建议？

1 个答案: