C#フォームアプリケーションでのソケット通信プログラムについて

Question

前提・実現したいこと

C# フォームアプリケーションでソケット通信を行うプログラムを作成しています。

button1を押すと接続先とソケット通信を行い、
button2を押すとソケット通信を利用したコマンドのやり取りを行うようになっています。

プログラム(フォーム)を閉じるときはソケット通信をcloseするように組んでいるのですが
button1、button2、FormClosing、それぞれで最初に宣言した TcpClient client を
呼び出したいので、public partial class Form1 : Form　のところで宣言してあります。

button1を押して接続がうまくできなかった場合、環境を整えてから
再度button1を押して接続トライできるようにプログラムを構成したいのですが、
下記ソースだとbutton1を押して接続がうまくできなかった時点でTcpClient clientが
消えてしまい、再度button1を押してもclientを再び宣言しなくてはいけない状態になります。

button1の中で TcpClient client を宣言してしまうと、button2、FormClosingのところで
clientの呼び出しができないため、どのようにプログラムを組めば良いか悩んでいます。

初歩的な内容になりますが、どなたかご教授頂けないでしょうか？

該当のソースコード

using System;
using System.Collections.Generic;
using System.Drawing;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.Net.Sockets;
using System.Net;
using System.Threading;

namespace Connect
{
    public partial class Form1 : Form
    {
        IPAddress ipAddress = IPAddress.Parse("192.168.3.39");　//IPアドレス入力
        int port = 2999;　//接続ポート

        //ソケット通信
        public TcpClient client = new TcpClient();


        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            //ソケット通信開始
            try
            {
                int timeout = 500;　//接続タイムアウト時間 (msec)

                Task con_task = client.ConnectAsync(ipAddress, port);　//接続先から応答がないとフリーズしてしまうのでTask処理
                if (!con_task.Wait(timeout))　// 接続拒否などは AggregateException
                {
                    throw new SocketException(10060);　// 10060 -> タイムアウト
                }
                else
                {
                    MessageBox.Show("Socket通信 接続成功");　//接続成功
                }
            }
            catch (SocketException)　//接続エラー
            {
                client.Close();
                MessageBox.Show("Socket通信 応答無し");
            }
            catch (AggregateException)　//接続拒否
            {
                client.Close();
                MessageBox.Show("Socket通信 接続拒否");
            }
        }

        private void Form1_FormClosing(object sender, FormClosingEventArgs e)
        {
            client.Close();
            MessageBox.Show("Socket closed");
        }

        private void button2_Click(object sender, EventArgs e)
        {
            if (client.Connected)
            {
                try
                {
                    NetworkStream stream = client.GetStream();
                    //データ通信の構文↓（省略）
                    //
                    //
                }

                catch (Exception)
                {
                    MessageBox.Show("応答無し");
                }
            }
            else
            {
                MessageBox.Show("Socket通信  接続できていません");
            }

        }
    }
}

Comments

[gentaro]

> 下記ソースだとbutton1を押して接続がうまくできなかった時点でTcpClient clientが
消えてしまい

あいまいな表現はやめましょう。
消えるというのはnullになるという意味ですか？
それとも何らかのエラーが発生して利用が不可能になる、という意味ですか？
いずれにせよ、エラーは発生していると思われるため、そのエラーの内容を質問文に書きましょう。

[tayamay]

あいまいな表現、申し訳御座いません。
nullになっていました。エラー内容は下記のとおりです。
Task con_task = client.ConnectAsync(ipAddress, port)　← System.NullReferenceException:
オブジェクト参照がオブジェクト インスタンスに設定されていません。

接続ができなかったときにclient.Close()していますが、これをとってしまうと
再度button1を押して再接続させようとするとソケットの重複が起きてしまいます。
エラー文は下記の通りです。↓
System.InvalidOperationException:
'同じソケット上で別の非同期操作の実行中に BeginConnect を呼び出すことはできません。'

Answer 1

button1_Clickでnullチェックを行い、nullの場合にインスタンスを生成すればご所望の動作ができるとおもいます。

        //ソケット通信
        public TcpClient client;

        private void button1_Click(object sender, EventArgs e)
        {
            if (client == null)
                client = new TcpClient();
            //ソケット通信開始
            try
            {