c#:TcpClient NetworkStreamの実装について

Question

###確認したいこと
以下のコードの書き方は問題ないでしょうか？
直接TcpClientのGetStreamからNetworkStreamを叩いていますが、
クローズ処理をしていなかったりで気持ち悪く感じます。

気になる点として、
・ネットワークストリームはゲットしたものをそのまま使って問題ないかどうか（クローズはいらない？）
・受信バッファのクリアの仕方は問題ないか。
の2点です。

他になにか気になる点が有れば、教えていただけると幸いです。
よろしくおねがいします。

TcpClient tcpClient = 初期化、コネクト済み;
byte[] byteBlank = 送信するデータ;

if (tcpClient.Connected == true)
{
    //受信バッファクリア
    while (tcpClient.GetStream().ReadByte() != -1) ;
    try
    {
        tcpClient.GetStream().Write(byteBlank, 0, byteBlank.Length);
    }
    catch
    {
        //ここにエラー処理
    }
}

Answer 1

TCPClientのGetStreamの実装は

public NetworkStream GetStream() {
    if(Logging.On)Logging.Enter(Logging.Sockets, this, "GetStream", "");
    if (m_CleanedUp){
        throw new ObjectDisposedException(this.GetType().FullName);
    }
    if (!Client.Connected) {
        throw new InvalidOperationException(SR.GetString(SR.net_notconnected));
    }
    if (m_DataStream==null) {
        m_DataStream = new NetworkStream(Client, true);
    }
    if(Logging.On)Logging.Exit(Logging.Sockets, this, "GetStream", m_DataStream);
    return m_DataStream;
}

このようになっています。
最初にGetStreamを呼んだときはm_DataStreamがnullのため、新たにNetworkStreamが生成されますが、それ以降はm_DataStreamがそのまま返されます。
ですので、Closeを行うとそれ以降はCloseされたStreamが返されることになります。
Closeを行う際は、TCPClientをCloseする時に行う形になるかと思います。

Comments

[kikukiku]

GetStreamは何回呼んでも、同じNetworkStreamのインスタンスを返すんですね。調べて頂いてありがとうございます。勉強になります。

[saki_ABC]

実装まで確認していただきありがとうございます。

同じStreamが帰ってくるとのことなので、切断時にclose処理でやってみます！

Answer 2

・ネットワークストリームはゲットしたものをそのまま使って問題ないかどうか（クローズはいらない？）

　取得したNetworkStreamは、利用が終わったらCloseする必要があります。
ちなみに、TcpClientも利用が終わったらCloseする必要があります。
おそらく、GetStreamを何回呼び出しても同じNetworkStreamのインスタンスを返すと
思いますが、そんな使い方をしたことがないため、確証はありません（知識なし）。
なので、GetStreamで取得したNetworkStreamのインスタンスを
変数に入れて利用し、使い終わったら、Closeする流れがよいと思います。

・受信バッファのクリアの仕方は問題ないか。

　受信バッファのクリアをしたいのに、なぜかwriteしています。
writrは送信バッファに書き込む命令で、
やりたいことと、コードが不一致しています。
受信バッファをクリアしたいのであるならば、
すべて読み取って、何も利用せずに破棄すれば良いと思います。

===追記===========================
コードをちゃんとみていませんでした。
ReadByteにて受信バッファから読み取っているので問題ないと思います。