2

MVP パターンを使用してアプリケーションを開発しようとしています。

問題は、すべてのコードを手動で配線することです。必要なコードを減らしたいと思っていました。別のソリューションをコピーしようとしましたが、作業できませんでした。私はWinformsを使用しており、ソースとして使用していたソリューションはWPFを使用しています。

それはいくつかの規則で物事を配線します：

ビュー イベントは名前で結び付けられます。例: ビューの「Loaded」イベントは、プレゼンターの「OnLoaded()」メソッドに接続されます。ボタン コマンドは名前で接続されます。例: MoveNext" ボタンは "OnMoveNext()" メソッドに接続されます。グリッドのダブルクリックは名前で接続されます。例: "Actions" のダブルクリックは "OnActionsChoosen(ToDoAction)" に接続されます。

WPF の作業コードは次のとおりです。

    private static void WireListBoxesDoubleClick(IPresenter presenter)
    {
        var presenterType = presenter.GetType();
        var methodsAndListBoxes = from method in GetActionMethods(presenterType)
                                  where method.Name.EndsWith("Choosen")
                                  where method.GetParameters().Length == 1
                                  let elementName = method.Name.Substring(2, method.Name.Length - 2 /*On*/- 7 /*Choosen*/)
                                  let matchingListBox = LogicalTreeHelper.FindLogicalNode(presenter.View, elementName) as ListBox
                                  where matchingListBox != null
                                  select new {method, matchingListBox};

        foreach (var methodAndEvent in methodsAndListBoxes)
        {
            var parameterType = methodAndEvent.method.GetParameters()[0].ParameterType;
            var action = Delegate.CreateDelegate(typeof(Action<>).MakeGenericType(parameterType),
                                                 presenter, methodAndEvent.method);

            methodAndEvent.matchingListBox.MouseDoubleClick += (sender, args) =>
            {
                var item1 = ((ListBox)sender).SelectedItem;
                if(item1 == null)
                    return;
                action.DynamicInvoke(item1);
            };
        }   
    }

    private static void WireEvents(IPresenter presenter)
    {
        var viewType = presenter.View.GetType();
        var presenterType = presenter.GetType();
        var methodsAndEvents =
                from method in GetParameterlessActionMethods(presenterType)
                let matchingEvent = viewType.GetEvent(method.Name.Substring(2))
                where matchingEvent != null
                where matchingEvent.EventHandlerType == typeof(RoutedEventHandler)
                select new { method, matchingEvent };

        foreach (var methodAndEvent in methodsAndEvents)
        {
            var action = (Action)Delegate.CreateDelegate(typeof(Action),
                                                          presenter, methodAndEvent.method);

            var handler = (RoutedEventHandler)((sender, args) => action());
            methodAndEvent.matchingEvent.AddEventHandler(presenter.View, handler);
        }
    }

    private static IEnumerable<MethodInfo> GetActionMethods(Type type)
    {
        return from method in type.GetMethods(BindingFlags.DeclaredOnly | BindingFlags.Public | BindingFlags.Instance)
               where method.Name.StartsWith("On")
               select method;
    }

    private static IEnumerable<MethodInfo> GetParameterlessActionMethods(Type type)
    {
        return from method in GetActionMethods(type)
               where method.GetParameters().Length == 0
               select method;
    }

とにかく、それを WinForm アプリに移植しようとしましたが、成功しませんでした。を に変更しましたRoutedEventHandlersがEventHandlers、 をどうすればよいかわかりませんでしたLogicalTreeHelper。

私はこれにちょっと立ち往生しています。手動で行うこともできましたが、このミニフレームワークは非常に巧妙で、ほとんどクレイジーであることがわかりました。

PS: ソースはhttp://msdn.microsoft.com/en-us/magazine/ee819139.aspxです

編集

私は何かに気づきました。私は頭がおかしいわけではありません。上記のコードはあまりテストに適していませんね。

4

1 に答える 1

1

Ok。私はそれを自分で動かしました。少なくとも他の1人が面白いと思ったので、私は答えを投稿しています。

まず、ビュー

public interface IBaseView
{
    void Show();
    C Get<C>(string controlName) where C : Control; //Needed to later wire the events
}

public interface IView : IBaseView
{
    TextBox ClientId { get; set; } //Need to expose this
    Button SaveClient { get; set; }
    ListBox MyLittleList { get; set; }
}

public partial class View : Form, IView
{
    public TextBox ClientId //since I'm exposing it, my "concrete view" the controls are camelCased
    {
        get { return this.clientId; }
        set { this.clientId = value; }
    }

    public Button SaveClient
    {
        get { return this.saveClient; }
        set { this.saveClient = value; }
    }

    public ListBox MyLittleList
    {
        get { return this.myLittleList; }
        set { this.myLittleList = value; }
    }

    //The view must also return the control to be wired.
    public C Get<C>(string ControlName) where C : Control
    {
        var controlName = ControlName.ToLower();
        var underlyingControlName = controlName[0] + ControlName.Substring(1);
        var underlyingControl = this.Controls.Find(underlyingControlName, true).FirstOrDefault();
        //It is strange because is turning PascalCase to camelCase. Could've used _Control for the controls on the concrete view instead
        return underlyingControl as C;
    }

今プレゼンター：

public class Presenter : BasePresenter <ViewModel, View>
{
    Client client;
    IView view;
    ViewModel viewModel;

    public Presenter(int clientId, IView viewParam, ViewModel viewModelParam)
    {
        this.view = viewParam;
        this.viewModel = viewModelParam;

        client = viewModel.FindById(clientId);
        BindData(client);
        wireEventsTo(view); //Implement on the base class
    }

    public void OnSaveClient(object sender, EventArgs e)
    {
        viewModel.Save(client);
    }

    public void OnEnter(object sender, EventArgs e)
    {
        MessageBox.Show("It works!");
    }

    public void OnMyLittleListChanged(object sender, EventArgs e)
    {
        MessageBox.Show("Test");
    }
}

「魔法」は基本クラスで起こります。wireEventsTo（IBaseViewビュー）内

public abstract class BasePresenter
    <VM, V>
    where VM : BaseViewModel
    where V : IBaseView, new()
{

    protected void wireEventsTo(IBaseView view)
    {
        Type presenterType = this.GetType();
        Type viewType = view.GetType();

        foreach (var method in presenterType.GetMethods())
        {
            var methodName = method.Name;

            if (methodName.StartsWith("On"))
            {
                try
                {
                    var presenterMethodName = methodName.Substring(2);
                    var nameOfMemberToMatch = presenterMethodName.Replace("Changed", ""); //ListBoxes wiring

                    var matchingMember = viewType.GetMember(nameOfMemberToMatch).FirstOrDefault();

                    if (matchingMember == null)
                    {
                        return;
                    }

                    if (matchingMember.MemberType == MemberTypes.Event)
                    {
                        wireMethod(view, matchingMember, method);    
                    }

                    if (matchingMember.MemberType == MemberTypes.Property)
                    {
                        wireMember(view, matchingMember, method);    
                    }

                }
                catch (Exception ex)
                {
                    continue;
                }
            }
        }
    }

    private void wireMember(IBaseView view, MemberInfo match, MethodInfo method)
    {
        var matchingMemberType = ((PropertyInfo)match).PropertyType;

        if (matchingMemberType == typeof(Button))
        {
            var matchingButton = view.Get<Button>(match.Name);

            var eventHandler = (EventHandler)EventHandler.CreateDelegate(typeof(EventHandler), this, method);

            matchingButton.Click += eventHandler;
        }

        if (matchingMemberType == typeof(ListBox))
        {
            var matchinListBox = view.Get<ListBox>(match.Name);

            var eventHandler = (EventHandler)EventHandler.CreateDelegate(typeof(EventHandler), this, method);

            matchinListBox.SelectedIndexChanged += eventHandler;
        }
    }

    private void wireMethod(IBaseView view, MemberInfo match, MethodInfo method)
    {
        var viewType = view.GetType();

        var matchingEvent = viewType.GetEvent(match.Name);

        if (matchingEvent != null)
        {
            if (matchingEvent.EventHandlerType == typeof(EventHandler))
            {
               var eventHandler = EventHandler.CreateDelegate(typeof(EventHandler), this, method);
               matchingEvent.AddEventHandler(view, eventHandler);
            }

            if (matchingEvent.EventHandlerType == typeof(FormClosedEventHandler))
            {
                var eventHandler = FormClosedEventHandler.CreateDelegate(typeof(FormClosedEventHandler), this, method);
                matchingEvent.AddEventHandler(view, eventHandler);
            }
        }
    }
}

私はこれをそのままここで機能させています。プレゼンターのEventHandlerをIViewにあるコントロールのデフォルトイベントに自動配線します。

また、補足として、BindDataメソッドを共有したいと思います。

    protected void BindData(Client client)
    {
        string nameOfPropertyBeingReferenced; 

        nameOfPropertyBeingReferenced = MVP.Controller.GetPropertyName(() => client.Id);
        view.ClientId.BindTo(client, nameOfPropertyBeingReferenced);

        nameOfPropertyBeingReferenced = MVP.Controller.GetPropertyName(() => client.FullName);
        view.ClientName.BindTo(client, nameOfPropertyBeingReferenced);
    }

    public static void BindTo(this TextBox thisTextBox, object viewModelObject, string nameOfPropertyBeingReferenced)
    {
        Bind(viewModelObject, thisTextBox, nameOfPropertyBeingReferenced, "Text");
    }

    private static void Bind(object sourceObject, Control destinationControl, string sourceObjectMember, string destinationControlMember)
    {
        Binding binding = new Binding(destinationControlMember, sourceObject, sourceObjectMember, true, DataSourceUpdateMode.OnPropertyChanged);
        //Binding binding = new Binding(sourceObjectMember, sourceObject, destinationControlMember);
        destinationControl.DataBindings.Clear();
        destinationControl.DataBindings.Add(binding);
    }

    public static string GetPropertyName<T>(Expression<Func<T>> exp)
    {
        return (((MemberExpression)(exp.Body)).Member).Name;
    }

これにより、バインディングから「マジックストリング」が排除されます。INotificationPropertyChangedでも使用できると思います。

とにかく、誰かがそれが役に立つと思うことを願っています。そして、コードの臭いを指摘したいのであれば、私は完全に大丈夫です。

于 2012-11-30T18:07:26.983 に答える