C#のシリアル通信での文字送信

Question

C#での文字のシリアル通信

今バブルメッセンジャーという名前で、入力した文字を一文字ずつ水槽内に泡として出力するものをVB.NETで書かれたプログラムをC#に翻訳して作り直しています。
プログラムはシリアル通信を通してリレー回路が組まれた基盤に送り、それでバルブの開け閉めで文字を作るようにしています。

発生している問題

文字の入力後、送信をする際に基盤の回路が動かずそのままになってしまいます

該当のソースコード

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace bubble_2019
{
    public partial class Form1 : Form
    {
        private byte[] outdata = new byte[10];

        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            serialPort1.PortName = "COM0";
            serialPort1.BaudRate = 9600;
            serialPort1.Parity = System.IO.Ports.Parity.None;
            serialPort1.DataBits = 8;
            serialPort1.Handshake = System.IO.Ports.Handshake.None;
            serialPort1.RtsEnable = false;
            cmdSEND.Enabled = false;
            sendroop.Enabled = false;
            textBox1.Text = "文字を入れてください";
            timer1.Enabled = false;
        }

        private void cmdOPEN_Click(object sender, EventArgs e)
        {
            label2.Text = "エラーはありません";
            if (serialPort1.IsOpen == true)
            {
                serialPort1.Close();
                cmdOPEN.Text = "通信開始";
                comboBox1.Enabled = true;
                cmdSEND.Enabled = false;
                sendroop.Enabled = false;
            }
            else
            {
                try 
                {
                    serialPort1.PortName = comboBox1.SelectedItem.ToString();
                    serialPort1.Open();
                }
                catch
                {
                    label2.Text = "COMポートが間違っています";
                    return;
                }
                cmdOPEN.Text = "通信停止";
                comboBox1.Enabled = false;
                cmdSEND.Enabled = true;
                sendroop.Enabled = true;
            }
        }
        private void SENDdeta()
        {
            string moji;
            int a, b, c, n, mojisu, mojisu2;
            int data1, data2, data3, data4, data5, data6;
            long color;
            Bitmap bitm = new Bitmap(pictureBox1.Image);
            Color co;
            mojisu = textBox1.Text.Length;
            mojisu2 = mojisu;
            serialPort1.DiscardInBuffer();
            moji = textBox1.Text;
            mojisu = moji.Length;

            for (n = 1; n <= mojisu; n++) //バッファ？
            {
                outdata[0] = 0x53;
                outdata[1] = 0x30;
                outdata[2] = 0x30;
                outdata[3] = 0x30;
                outdata[4] = 0x30;
                outdata[5] = 0x30;
                outdata[6] = 0x30;
                outdata[7] = 0x30;
                outdata[8] = 0x30;
                outdata[9] = 0xD;
                
                do
                {
                    if (serialPort1.IsOpen == true)
                    {
                        serialPort1.Write(outdata, 0, outdata.Length);
                        string readData = serialPort1.ReadExisting();
                    }
                }
                while (serialPort1.IsOpen == true);
                for (var Y = 0; Y <= 23; Y++)
                {
                    a = 0;
                    for (var X = 0; X <= 7; X++)
                    {
                        co = bitm.GetPixel((n - 1) * 24 + X, Y);
                        color = co.A;
                        a = a * 2;
                        if (color != 0)
                            a = a + 1;
                    }
                    b = 0;
                    for (var X = 8; X <= 15; X++)
                    {
                        co = bitm.GetPixel((n - 1) * 24 + X, Y);
                        color = co.A;
                        b = b * 2;
                        if (color != 0)
                            b = b + 1;
                    }
                    c = 0;
                    for (var X = 16; X <= 23; X++)
                    {
                        co = bitm.GetPixel((n - 1) * 24 + X, Y);
                        color = co.A;
                        c = c * 2;
                        if (color != 0)
                            c = c + 1;
                    }

                    data1 = (a / 16) % 16;
                    data2 = a % 16 + 0x30;
                    data3 = (b / 16) % 16 + 0x30;
                    data4 = b % 16 + 0x30;
                    data5 = (c / 16) % 16 + 0x30;
                    data6 = c % 16 + 0x30;

                    outdata[0] = 0x53;
                    outdata[1] = (byte)data5;
                    outdata[2] = (byte)data6;
                    outdata[3] = (byte)data5;
                    outdata[4] = (byte)data6;
                    outdata[5] = (byte)data3;
                    outdata[6] = (byte)data4;
                    outdata[7] = (byte)data1;
                    outdata[8] = (byte)data2;
                    outdata[9] = 0xD;

                    do
                    {
                        if (serialPort1.IsOpen == true)
                        {
                            serialPort1.Write(outdata, 0, outdata.Length);
                            string readData = serialPort1.ReadExisting();
                        }
                    }
                    while (serialPort1.IsOpen == true);

                    System.Threading.Thread.Sleep(50);
                }
                outdata[0] = 0x53;
                outdata[1] = 0x30;
                outdata[2] = 0x30;
                outdata[3] = 0x30;
                outdata[4] = 0x30;
                outdata[5] = 0x30;
                outdata[6] = 0x30;
                outdata[7] = 0x30;
                outdata[8] = 0x30;
                outdata[9] = 0xD;

                do
                {
                    if (serialPort1.IsOpen == true)
                    {
                        serialPort1.Write(outdata, 0, outdata.Length);
                        string readData = serialPort1.ReadExisting();
                    }
                }
                while (serialPort1.IsOpen );

                System.Threading.Thread.Sleep(500);
            }
        }

        private void textBox1_TextChanged(object sender, EventArgs e)
        {
            Bitmap mojisend = new Bitmap(pictureBox1.Width, pictureBox1.Height);
            int mojisu, n, X, Y;
            long color;
            Color co;

            pictureBox1.Image = null;
            mojisu = textBox1.Text.Length;

            for (n = 1; n <= mojisu; n++)
            {
                Bitmap mojikioku = new Bitmap(pictureBox2.Width, pictureBox2.Height);
                Graphics g = Graphics.FromImage(mojikioku);
                StringFormat format = new StringFormat();
                Font fnt = new Font("MS 明朝", 18, FontStyle.Regular, GraphicsUnit.Point);
                format.Alignment = StringAlignment.Center; 
                format.LineAlignment = StringAlignment.Near; 
                g.Clear(Color.White); 
                g.DrawString(textBox1.Text. Substring(n-1,1), fnt, Brushes.Black, 13, 0,format); 
                fnt.Dispose();
                g.Dispose();
                pictureBox2.Image = mojikioku;
                Bitmap bitm = new Bitmap(pictureBox2.Image);

                for (Y = 0; Y <= 23; Y++)
                {
                    for (X = 0; X <= 23; X++)
                    {
                        Graphics tenset = Graphics.FromImage(mojisend);
                        co = bitm.GetPixel(X, Y); 
                        color = co.B;
                        if (color == 0)
                        {
                            tenset.DrawEllipse(Pens.Black, (n-1) * 24 + X, Y, 1, 1); 
                            tenset.Dispose();
                            pictureBox1.Image = mojisend; 
                        }
                    }
                }
                pictureBox2.Image.Dispose();
                pictureBox2.Image = null;
            }
        }

        private void Button4_Click(object sender, EventArgs e)
        {
            if (serialPort1.IsOpen == true)

                serialPort1.Close();
            Environment.Exit(0);
        }

        private void cmdSEND_Click(object sender, EventArgs e)
        {
            SENDdeta();
        }

        private void timer1_Tick(object sender, EventArgs e)
        {
            timer1.Enabled = false;
            SENDdeta();
            timer1.Enabled = true;
        }

        private void sendroop_Click(object sender, EventArgs e)
        {
            if (sendroop.Text == "繰り返し送信")
            {
                sendroop.Text = "繰り返し停止";
                timer1.Enabled = true;
            }
            else
            {
                sendroop.Text = "繰り返し送信";
                timer1.Enabled = false;
            }
        }

        private void TextBox1_GotFocus() 
        {
            textBox1.SelectionStart = 0;
            textBox1.SelectionLength = textBox1.Text.Length;
        }

        private void Form1_FormClosing(object sender, FormClosingEventArgs e)
        {
            if (serialPort1.IsOpen == true)
                serialPort1.Close();
            Environment.Exit(0);
        }
    }
}

試したこと

様々なサイトを回り、handshakeやあることを確認したりしました

補足情報

VB.NETのプログラムは6年前に卒業した先輩の残した物なので作成者に聞くことができません

Comments

[brue__akio]

COMポートは接続出来ています、文字の送信の際にリレー回路が正常に動作しなく、基盤はVBだと正常に動きます

[ozwk]

シリアルモニタでもcom0comとTeratermでもとにかく何でもいいのでVB版ソフトと移植したソフトで同じことをしようとしたときに同じデータが送られているかを確認してください。

[brue__akio]

C#のソフトがデータを送れないでいます

[ozwk]

1バイトも？

[brue__akio]

おそらく、もう一度詳しく調べてみます

[ozwk]

SENDdeta()呼ばれてます？

[brue__akio]

遅れてすみません、自分もそこが出来てないんじゃないかと思っていて…

[ozwk]

思うんだったらConsole.WriteLineでもブレークポイント置くでもいいんで確認しましょう

Answer 1

VB.NET で作成した exe は動作するんですよね？

「Telerik JustDecompile」
https://www.telerik.com/products/decompiler.aspx

これを使うと、exe から C# のソースを生成することができるので、生成したソースとコンバージョンしたソースを比較してみてはいかがでしょうか？

Sharp Develop 4 だとソースを直接コンバートできたのですが、サイトが一新されて旧バージョンのダウンロード先がわからなくなってしまいました。(最新バージョンの 5 ではコンバートできません。)

Answer 2

コードは読んでませんが、気のついたところを。

serialPort1.PortName = "COM0";

COM0 というデバイス名はありません。
確認してみては。

Comments

[brue__akio]

記載漏れですみませんが、COMポートはcomboboxで変えていてCOMポートの接続は出来ています

[y_waiwai]

接続ができている、とはどうやってそれを確認していますか？

他のPCにシリアル・ポートをつけて、きちんと送信できているか、をモニタしてみてはどうでしょうか。

[brue__akio]

ポートは接続の際にエラーが出ているかいないかを判別するプログラムで、エラーの出ないCOMポートがあるので接続は出来ていると思います。
その上で入力した内容を送ると回路が動かずフリーズしたままになってしまいます

[y_waiwai]

思いますレベルで言われましても。。

繰り返しますが、ポート番号が間違ってます。
デバイスマネージャで、そのシリアル・ポートのCOM番号確認してみましょうよ

[brue__akio]

デバイスマネージャーで確認した所、COM5でした。そして、COMポートはcomboboxで14まで用意してあるので接続は出来ていました

[y_waiwai]

あなたのいう「接続が出来ている」という言葉の意味がわかりませんが、
とにかく別のPCもってきて、シリアル・ポートつないで通信のモニタしてみては？
Teratermなどのターミナルソフトでシリアルの通信確認できますよ。

[brue__akio]

色々確認してポートは大丈夫なのですが、データの送信が出来ませんでした

[y_waiwai]

送信ができないのはポートが大丈夫とは言いませんｗ
ポート番号間違ってるんじゃないの？
いろいろポート番号変えて、送信できる番号無いか探してみれ

[Mithra]

プログラム－COMポートーシリアルケーブルー基板という構造になっていると思いますが、
この中でデータはどこまで届いているか確認しなさいと助言を貰っているわけですから、
まずはやってみてはいかがですか？
もしくは、試したならどのような方法で確認したかを報告する。

ただ、「出来てます」では回答側も困るでしょう。
なぜなら、問題を解決する能力の無い(だから質問している)人の「思い込み」や
「間違った方法論」での確認したは当てにならないからです。

問題の切り分けという基本レベルの確認から始めてください。