PC-PLC接続 DMの表示の仕方

Question

現在パソコンとシーケンサー(PLC:KEYENCE KV-8000)をイーサーネットで繋げています

C#でプログラムは書いています
MR(補助接点)の読書きはできまして
DM(データメモリ)の書込みはできました

DMの読込をした後にtextbox1に表示しようとしても
うまく表示しないので教えて頂けないでしょうか
宜しくお願い致します

using System;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.Net.Sockets;

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

    private readonly TcpClient tcpClient = new TcpClient();
    private NetworkStream netstream;

    private void Form1_Load(object sender, EventArgs e)
    {
        tcpClient.Connect("192.168.3.1", 8501);
        netstream = tcpClient.GetStream();

        timer1.Enabled = true;
        timer1.Interval = 1000;
    }

    private void timer1_Tick(object sender, EventArgs e)
    {
        string sendMsg = "RD MR011" + '\r';
        byte[] byteRecieveMessage = SendAndRecieve(sendMsg);
        string read = Encoding.ASCII.GetString(byteRecieveMessage, 0, 1);

        if (read == "1")
        {
            sendMsg = "WR DM003 20" + '\r';
            byteRecieveMessage = SendAndRecieve(sendMsg);
            

            sendMsg = "RD DM100" + '\r';
            byteRecieveMessage = SendAndRecieve(sendMsg);
            string b = Encoding.ASCII.GetString(byteRecieveMessage, 0, 1);

            textBox4.Text = b;
        }
        else
        {
        }

    }

    private byte[] SendAndRecieve(string sendMsg)
    {
        netstream = tcpClient.GetStream();
        netstream.ReadTimeout = 10000;
        netstream.WriteTimeout = 10000;

        Encoding enc = Encoding.ASCII;
        byte[] sendBytes = enc.GetBytes(sendMsg);

        netstream.Write(sendBytes, 0, sendBytes.Length);
        System.IO.MemoryStream ms = new System.IO.MemoryStream();

        byte[] resBytes = new byte[50];
        int resSize = 0;

        do
        {
            resSize = netstream.Read(resBytes, 0, resBytes.Length);
            ms.Write(resBytes, 0, resSize);

        } while (netstream.DataAvailable || resBytes[resSize - 1] != '\n');

        return resBytes;
    }

    private void Form1_FormClosed(object sender, FormClosedEventArgs e)
    {
        netstream.Close();
        tcpClient.Close();
    }
}

}

Comments

[y_waiwai]

このままではコードが読みづらいので、質問を編集し、</>(コードの挿入)ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

[SurferOnWww]

質問する際は何を何で作っているかを質問の一行目に書きましょう。（例: Windows Forms アプリを Visual Studio 2022 でフレームワークを .NET Framework 4.8 として作っています・・・とか）

コードの表示が崩れています。質問をアップしたらきちんと書けているかどうか見直すということはしないのですか？

[y_waiwai]

で、うまく表示しないとはどういうふうになるんでしょうか

[SurferOnWww]

質問者さん、無言ですが、回答がついているのでそれらに対するフィードバックを返してください。役に立った/立たなかったぐらいはすぐに返せるのでは？ 役に立たなかったならどこがダメかを書くとより期待に近い回答が出てくるかも。とにかく無言は NG です。

Answer 1

SendAndRecieve が返す値が間違ってそうです。
せっかく受信した内容を MemoryStream に書き込んでいるのに、return resBytes と、最後に受信したバッファの内容を返しています。
ここは、return ms.ToArray(); が正しいのでは？

string b = Encoding.ASCII.GetString(byteRecieveMessage, 0, 1);
もまずいですね。先頭１バイトしか変換していない。

いっそのこと SendAndRecieve の戻り値は string にして、return enc.GetStriing(ms.ToAray(), 0, ms.Length - 1); としたほうがよさそう。

while (netstream.DataAvailable || resBytes[resSize - 1] != '\n');
も、まずいです。
メッセージが長くなり、複数のパケットに分割されると、すべてのデータを受信する前に DataAvailable プロパティが False になり、ループを抜けてしまいます。
while (resSize > 0 && resBytes[resSize - 1] != '\n');
として、デリミタを受信するまでループしてください。(通信仕様がわからないのでデリミタが '\n' でいいかは不明)

Comments

[KOZ6.0]

受信メッセージが 50 バイト以内なら動いちゃうんでしょうね

[MASA]

ありがとうございます
やってみます

[KOZ6.0]

回答に追記しました

Answer 2

すみません
textbox4.text = d;
dの中にデータが入って来ていないのでコードの書き方がおかしいのでしょうか

Comments

[SurferOnWww]

質問者は回答欄に書かないでください。訳が分からなくなりますので。回答へのレスは回答欄の下のコメント欄に書いてください。この回答は削除してください（削除できなければ、内容を削除して「コメント欄に移動」とでもかいてください）。

Answer 3

DMの読込をした後にtextbox1に表示しようとしてもうまく表示しない

timer1_Tick イベントハンドラの、

string b = Encoding.ASCII.GetString(byteRecieveMessage, 0, 1);

で変数 b にはデータは取れているが、次の行の、

textBox4.Text = b;

で textBox4 に表示されないということですか？

であれば、多分 timer1_Tick イベントハンドラが UI スレッドとは別のスレッドで動いているからだと思います。Windows Forms アプリのようですが、であれば Invoke メソッド を使ってみてください。以下のような感じ。

this.Invoke((Action)(() => this.textBox4.Text = b));