c# - 我最终得到了什么样的设计模式？

Question

我们有一个 Silverlight 应用程序。它有几个页面位于我们 UI 的选项卡内。过去，我们称它们为 SavePage 和 PanelPage。保存页面仅具有编辑记录详细信息、创建新记录和删除屏幕上现有记录的基本功能。 PanelPage 继承自 SavePage。 PanelPage 稍微复杂一些，因为面板会根据您在屏幕上所做的选择变得可见/不可见。

Silverlight 应用程序中的代码非常困惑。但是，最近，我采取了将此代码移植到 Xamarin Forms 中的步骤。我为此做了两次失败的尝试，在第三次尝试时，我得到了适用于所有目标平台的代码:Silverlight、iOS、Android 和 Windows UWP。我对现在的类(class)设计相当满意。它可能会更简单，但这会在一段时间内起到作用。

此设计的要点在于，UI 逻辑是从物理 UI 控件本身中抽象出来的。我从跨两个平台(SavePage 和 PanelPage)的共享代码中删除了 System.Windows 使用。这些页面更像是 MVC 或 MVVC 模式中的“ Controller ”。但是，我不认为我创建的正是这两种模式中的任何一种。但是，我不得不将这些类分成两部分:一部分用于抽象 UI 调用，如 SaveAsync()，另一部分用于平台特定的 UI 调用，如 ReportError。我很难正确命名类，因为我什至不知道我使用的是什么设计模式。

这是一个类图: 以下是一些相关接口(interface)的代码:

public interface IPage : IRecordSelector
{
    /// <summary>
    /// This event should be raised when the busy state of a tab changes
    /// </summary>
    event EventHandler<BusyStateChangedEventArgs> BusyStateChanged;
    object PageElement { get; }
}

public interface IButtonDrivenPage : IPage
{
    Task SaveClickAsync();
    Task DuplicateClickAsync();
    Task DeleteClickAsync();
    Task NewClickAsync();
    event EventHandler<ButtonVisibilityChangedEventArgs> ButtonVisibilityChanged;
    IRecord GetRecord();
}

public interface ISavePage : IButtonDrivenPage, IRequestClose
{
    string DataContextXmlSnapshot { get; }
    bool PromptForChangeCancel { get; }
    IRecord SelectedItem { get; }
    Task SetSelectedItemAsync(IRecord selectedItem);
    event EventHandler SelectedItemChanged;
    void Close();
    void SetAutomationObject(object automationObject);
    ISavePageUIController SavePageUIController { get; }
}

public interface ISavePageUIController: IDisposable
{
    /// <summary>
    /// The UI controller is notifying the page that the UI content has been loaded
    /// </summary>
    event EventHandler ContentLoaded;

    /// <summary>
    /// Prompt the user for a yet or a no
    /// </summary>
    Task<bool> GetYesNoFromPrompt(string message, string title);

    /// <summary>
    /// Report an error to the user
    /// </summary>
    void ReportError(string title, string message, Exception exception);

    /// <summary>
    /// Notifies the UI that the DataContext/Binding context has changed
    /// </summary>
    void SetSelectedItem(IRecord selectedItem);

    /// <summary>
    /// The actual UI object that is displayed on screen as the content of the page
    /// </summary>
    object PageElement { get;  }

    /// <summary>
    /// Clears residual errors from the screen if they exist
    /// </summary>
    void ClearErrors();

    /// <summary>
    /// The record was saved. The selectedItem parameter will be the saved record from the server.
    /// </summary>
    void CurrentRecordSaved(IRecord selectedItem);

    /// <summary>
    /// This event occurs when the UI wants to notify the controller that a Save button has been clicked in the UI somewhere
    /// </summary>
    event EventHandler SaveClicked;
}


public interface IPanelUIController : ISavePageUIController
{
    void CreateAndAddPanelFromContent(PagePanel pagePanel, double? panelHeight);
    IEnumerable<IPagePanel> GetIPagePanelControls();
    void SetHeader(IPanelHeader pageHeader);
    void SetVisiblePanels(IList<bool> visiblePanels);
    void HideAllPanels();
    event EventHandler<RecordsSelectedRoutedEventArgs> PanelPageRecordsSelected;
}

这些接口(interface)已在 Silverlight 和 Xamarin Forms 中成功实现。那么，这是否类似于另一种 UI 设计模式？谁能推荐改进？或者，告诉我需要做什么才能将其转换为更标准的 UI 设计模式？取名怎么样？我应该在这里命名我的类和接口(interface)？

Answer 1

老实说，我不会太执着于成为其中之一(MVC、MVVM 或 MVP)，它们几乎相同，关键是要保持“大 split ”。也就是说，现在，恕我直言，您似乎最接近 MVP(模型 View 展示器)

问题是你有很多混杂的逻辑:IPage 实际上应该只是一个 View 但你有它做的事情 Controller 通常会这样做。与它的 child 一样:ISavePage 有一个名为 SetAutomation 对象的方法，我通常希望在 Controller 中看到它(至少如果我是正确猜测它的功能)。

Adam Freeman 在谈论如何在 ASP.NET MVC 5 中分解这些东西方面做得很好:http://enos.itcollege.ee/~ijogi/Nooks/Pro%20ASP.NET%20MVC%205/Pro%20ASP.NET%20MVC%205.9781430265290.pdf查看第 51 页，他在其中分解了每个项目在概念上应该是什么，这可能有帮助？

我会尝试这样做，因为我们已经确定您的 IPage 确实是一个 Controller 。

public class PageElement
{
    IPageController _controller;

    // these are your models - your controller will simply allow them to be shown with your other methods
    private PageData _data; 
    private PageData _otherData;

    public PageElement(IPageController ctrl)
    {
         _controller = ctrl;
    }
}

public class PageController : IPageController
{
     IPageService _service;

     public PageController(IPageService service)
     {
          _service = service;
     }

     // this is what your button calls when clicked
     public void SaveAsync(object sender, SomeEventArgs args)
     {
           // the service does the actual work
          _service.SaveAsync()
     }
}

public class PageService : IPageService
{
     public void SaveAsync(){ // what it does}

}