c# Serial Communication

Question

1.50/5 (2 votes)

See more:

I have written a code in c# for receiving data through serial communication and display it in a text box. I am able to do that. But I am receiving data in following format, e.g.

1234 2345 3456 5667
or

1234 3245 5678
But I want that every incoming byte should overwrite the previous byte and in the text box we should see just 1 byte and then another byte overwriting the previous one.

Please suggest how to do that?

My code for ser comm is as follows :

C#

private void SerialPort_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e) 
{ 
    RxString = SerialPort.ReadExisting(); 
    this.Invoke(new EventHandler(DisplayText)); }

private void DisplayText(object sender, EventArgs e)
{
    txtIncomingData.AppendText(RxString);
}

Posted 2-Mar-15 17:19pm

Member 11493170

Updated 2-Mar-15 17:25pm

RajeeshMenoth

v2

Add a Solution

2 solutions

Solution 2

The solution below works, but it gives blank bytes in between data.

C#

    SerialPort serialPort;

    private delegate void SetTextDeleg(string text);

    private void Form1_Load(object sender, EventArgs e)
    {
          serialPort= new SerialPort("COM6", 4800, Parity.None, 8, StopBits.One);
          serialPort.Handshake = Handshake.None;
          serialPort.DataReceived += new SerialDataReceivedEventHandler(sp_DataReceived);
          serialPort.ReadTimeout = 500;
          serialPort.WriteTimeout = 500;
          serialPort.Open();
}

private void btnStart_Click(object sender, EventArgs e)
{
      try
      {
           if(!serialPort.IsOpen)
                serialPort.Open();
      }
      catch (Exception ex)
      {
           MessageBox.Show("Could not open Serial Port");
      }
}

void sp_DataReceived(object sender, SerialDataReceivedEventArgs e)
{
     Thread.Sleep(500);
     string data = serialPort.ReadLine();
     this.BeginInvoke(new SetTextDeleg(si_DataReceived), new object[] { data });
}

private void si_DataReceived(string data)
{
      txtIncomingData.Text = data.Trim();
}

Posted 6-Mar-15 17:18pm

Member 11493170

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Sergey Alexandrovich Kryukov · Accepted Answer · 2015-03-02T18:37:00

You misunderstood Control.Invoke. This is not event invocation, so your object sender and EventArgs arguments are superfluous. Having this made-up artificial limitation, you have done bad thing: passing the text through the outer context relative to your method. Instead, you should use the event delegate with any signature you need:

C#

string newData = SerialPort.ReadExisting(); // local variable to be used in next line
this.Invoke(new System.Action<string>((data) => {
   txtIncomingData.AppendText(data);
}), newData);

If you need to override the character, not append, assuming txtIncomingData is you text box, replace

C#

txtIncomingData.AppendText(data);

with

C#

txtIncomingData.Text = data;

—SA