C# winforms, сохраняющее значение из numericupdown

Question

У меня есть данные о serialportRecived, в которых есть 2 метода. LogFile регистрирует данные, а drawSetpoint рисует график.

public void serialPort1_DataRecived (object sender, SerialDataReceivedEventArgs e) 
     { 

      DateTime time = DateTime.Now; 
      string Rdata = serialPort1.ReadLine(); 
      LogFile(Rdata, (int)numericSetpoint.Value); 
      drawSetpoint(time,numericSetpoint.Value.ToString()); 


     } 

Оба метода принимает второй аргумент от управления NumericUpDown, который выглядит следующим образом

public void numericSetpoint_KeyPress(object sender, KeyPressEventArgs e) 
     { 
      if (e.KeyChar == 13) 
      { 

       if (serialPort1.IsOpen) 
       { 
        int setpoint = (int)numericSetpoint.Value; 
        //send to serial port .....      
       } 

Проблема заключается в том, оба метода в моих данных RECIVED событий чтения цифр, как я печатаю их. Например, если я напечатаю 150, LogFile покажет 1,15,150, а функция draw начнет рисовать 1,15,150. Я хотел бы, чтобы обе функции принимали значение из элемента управления numericSetpoint после нажатия клавиши ввода, чтобы получить все значение. Как я мог это сделать?

Answer 1

Вы используете событие KeyPress. Вместо этого подумайте о том, чтобы использовать событие ValueChanged, которое срабатывает только при нажатии кнопки или пользователь покидает управление. https://msdn.microsoft.com/en-us/library/system.windows.forms.numericupdown.valuechanged%28v=vs.110%29.aspx

Answer 2

Похоже, что ваше событие rx данных стреляет во время ввода текста. Я бы попробовал заполнить буфер в вашем rx-событии данных и удержаться на любом протоколировании или графике, пока вы не отправите свои данные. Существует множество «безопасных» способов сделать это, но основная логика такова:

byte[] buffer = new byte[MAX_BUFFER]; 
private volatile bool _ready = false; 
private Object _lock = new Object(); 

public void serialPort1_DataRecived (object sender, SerialDataReceivedEventArgs e) 
{ 

    DateTime time = DateTime.Now; 

    // Either read as bytes or convert string to bytes, add to your buffer 
    // string Rdata = serialPort1.ReadLine(); 

    lock(_lock) 
    { 
     if(_ready) 
     { 
      _ready = false; 
      var myData = your buffer as string 
      // clear buffer 
      LogFile(myData, (int)numericSetpoint.Value); 
      drawSetpoint(time,numericSetpoint.Value.ToString()); 
     } 
    } 

} 

... 

public void numericSetpoint_KeyPress(object sender, KeyPressEventArgs e) 
{ 
    if (e.KeyChar == 13) 
    { 

     if (serialPort1.IsOpen) 
     { 
      int setpoint = (int)numericSetpoint.Value; 
      //send to serial port .....  
      lock(_lock) { _ready = true; }    
     } 
    } 
}