Serial port latch Application C#

Question

I'm making an application that is reading the serial port and updating data in a WindowsForms application screen.

Sometimes when you try to close the program is locked. I'm using delegate.

What should I be doing wrong?

void sp1_LineReceived(object sender, LineReceivedEventArgs Args)
{
  Invoke(new Action(() =>
  {
    // execute code
  }));
}

Init

public FormPrincipal()
{
  InitializeComponent();
  spSistema.LineReceived += new LineReceivedEventHandler(sp1_LineReceived);
  // next codes
}

Outher codes

public partial class FormPrincipal : Form
{       
  SerialPort spSimulador = new SerialPort();        
  public static PortaSerial spSistema = new PortaSerial();

It is very likely to cause deadlock when you use the Close() method while the device is sending. Use BeginInvoke() or remove Close() or run Close on a worker thread. — Hans Passant, Jan 28 '14 at 18:28

score 2 · Accepted Answer · edited May 23 '17 at 12:03

2

In general, BeginInvoke is preferable to Invoke, as it won't block. See here. You may well have a deadlock here.

void sp1_LineReceived(object sender, LineReceivedEventArgs Args)
{
  BeginInvoke(new Action(() =>
  {
    // execute code
  }));
}

edited May 23 '17 at 12:03

Community

1
1

answered Jan 28 '14 at 18:22

Jon B

51,025
31
133
161

Serial port latch Application C#

1 Answers1