DelegateだとかInvokeだとかそんなんしたくない・・・

Question

C#でシリアル通信アプリをつくっており、受信した直後や、タイマー割り込みの
中でＧＵＩを操作しているのですが、ＧＵＩにアクセスするたびにDelegateだとか
Invokeだとか書いて、メソッドを作らないとダメで非常に面倒で読みにくいです。

GUIを操作するメソッド一つにして、その中で全てのGUIを操作するようにして
引数で制御するようにしてみましたが、それを呼び出すたびに操作しないGUIに
対しても引数を全部設定しなければならず手間で見にくいです。

これはどうにもならないものなのでしょうか？

ソースコードは金澤ソフト設計さんのを流用しているので勝手に張るのもよくない
のでそちらを見てください。

Comments

[y_waiwai]

どこを見ればいいんでしょうか

Answer 1

Control.CheckForIllegalCrossThreadCalls プロパティを False にすると、呼び出しスレッドのチェックを行わなくなります。

「Control.CheckForIllegalCrossThreadCalls プロパティ」
https://learn.microsoft.com/ja-jp/dotnet/api/system.windows.forms.control.checkforillegalcrossthreadcalls?view=netframework-4.8

大抵の操作は、ウインドウズメッセージを介して行われるため、これで充分かと思います。

.NET Framework の場合

透過プロキシを使うことによって、public メソッドや public プロパティ等、全ての操作を Invoke を介して行うことが出来ます。

using System;
using System.Reflection;
using System.Runtime.Remoting.Messaging;
using System.Runtime.Remoting.Proxies;
using System.Windows.Forms;

public static class ControlUtil
{
    public static T CreateProxy<T>(T target) where T : Control {
        var proxy = new ControlProxy(target);
        return (T)proxy.GetTransparentProxy();
    }

    private class ControlProxy: RealProxy
    {
        Control _Instance;

        public ControlProxy(Control target)
            : base(target.GetType()) {
            _Instance = target;
        }

        public sealed override IMessage Invoke(IMessage msg) {
            try {
                IMethodMessage mm = msg as IMethodMessage;
                object[] args = mm.Args;
                MethodInfo method = (MethodInfo)mm.MethodBase;
                object ret = Invoke(method, args);
                return new ReturnMessage(
                    ret, args, args.Length, mm.LogicalCallContext, (IMethodCallMessage)msg);

            } catch (Exception ex) {
                if (ex.InnerException != null)
                    return new ReturnMessage(ex.InnerException, (IMethodCallMessage)msg);
                return new ReturnMessage(ex, (IMethodCallMessage)msg);
            }
        }

        public object Invoke(MethodInfo mi, object[] args) {
            if (_Instance.InvokeRequired) {
                Func<MethodInfo, object[], object> d = Invoke;
                return _Instance.Invoke(d, mi, args);
            } else {
                var result = mi.Invoke(_Instance, args);
                var parameters = mi.GetParameters();
                for (int i = 0; i < parameters.Length; i++) {
                    var p = parameters[i];
                    if (p.ParameterType.IsByRef && args[i] is Control control) {
                        var proxy = new ControlProxy(control);
                        args[i] = proxy.GetTransparentProxy();
                    }
                }
                return result;
            }
        }
    }
}

使い方

using System;
using System.Threading;
using System.Threading.Tasks;
using System.Windows.Forms;

public partial class Form1 : Form
{
    public Form1() {
        InitializeComponent();
    }

    private void button1_Click(object sender, EventArgs e) {
        Task.Run(OtherTask);
    }

    private void OtherTask() {
        Form1 form1 = ControlUtil.CreateProxy(this);
        for (int i = 0; i <= 10; i++) {
            Thread.Sleep(200);
            form1.label1.Text = i.ToString(); // label1 は public でないとダメ
        }
    }
}

.NET Core 系の場合

RealProxy は使えなくなりました。
代わりに DispatchProxy クラスが提供されていますが、残念ながらインターフェイスを介してしか使えないので RealProxy ほど使い勝手は良くないようです。
https://learn.microsoft.com/ja-jp/dotnet/api/system.reflection.dispatchproxy?view=net-7.0

Comments

[milkypai]

非常に使い勝手がよくなりました。
こんな方法があるなんて、驚きました。
本当にありがとうございました。

[KOZ6.0]

すみません、透過プロキシ作成のコーディングミスってました。
メソッド実行時、ByRef でコントロールを 返す場合は、透過プロキシを返すようにしなければいけません。
これにより、CheckForIllegalCrossThreadCalls = false にする必要はなくなります。

Answer 2

GUIアプリでシリアル通信を実用的に実装しようと思ったら、そのDelegateだとかInvokeだとかってのは必須となります。
食わず嫌いせずに、そこらへんがんばって学習しましょう

Comments

[milkypai]

学習とかじゃなく、質問に書いてある通り書く工数を減らしたい、読みやすくしたいんですね

[y_waiwai]

そのキーワードを使いたくないってだけなら、ラムダ式なんかを使えば、ある程度それは実現できます
まあそれで工数が減るとか読みやすくなるかってのはちと疑問なところではありますが。