[C#] Variable wird nicht übernommen

[Draco (Gast)]

Ich habe hier ein Problem, mit welchem ich gerade nicht weiter komme:


Ich habe ein Programm welches mir Daten an einen µC sendet und wieder 
empfängt. Das funktioniert eigentlich auch ganz gut - bis auf den 
Umstand das eine Variable aus dem Receive Handler nicht übernommen wird:

//..
volatile string varHardwareVersion;                    //Volatile
//..

private void butInfoClick(object sender, EventArgs e)  //Button Event Handler
{
  FormControllerInfo f = new FormControllerInfo();     //New Form from FormControllerInfo

  f.mComPort = ComPort.PortName;                       //Variable set 

  Gauge.mGetHardwareVersion();                         //Set Command to µC
  ComSend(Gauge.mByteToSend);                          //Send Command to µC
  Gauge.SetupGauge(0x02);                              //Reset µC to default   

  f.mVersionString = varHardwareVersion;              //!!! Hier Fehler !!!!
  this.Text = varHardwareVersion;                     //!!! Hier Fehler !!!! 

  f.Show();                                            //Show Form
}


//... Der Com Port Receive Handler....

public void DataReceivedHandler(object sender, SerialDataReceivedEventArgs e)
{
   SerialPort sp = (SerialPort)sender;          // ComPort from object sender

   string indata = sp.ReadLine();               // Get uart data
   char[] inchar = indata.ToArray();            // convert to byte array

   //vol_AppHead(indata);                       // Debug: Work (volatile)
            
   varHardwareVersion = inchar[0].ToString();  // First byte to string: work
   vol_AppHead(varHardwareVersion);             // show value: Work!!

   switch (inchar[0])
   {
       case 'x':
           break;
       case 'v':
           break;
       case 'V':
           //varHardwareVersion = indata;
       break;
    }
}

Soo... also der String wird empfangen und der erste Byte in die Variable 
varHardwareVersion geschrieben (spielt auch keine Rolle ob der indata 
String geschrieben wird, kommt aufs gleiche raus), dann wird sie mir 
über eine geschützte Funktion (vol_AppHead) ausgegeben. Das klappt 
wunderbar.

Nun aber ist die Variable wieder zurück in butInfoClick wieder leer. Da 
wo in den Kommentaren !!!HIER FEHLER!!! steht. Jedoch ist sie doch aber 
volatile geschützt, wieso ist sie denn dann leer wenn sie doch aber 
beschrieben wird und auch angezeigt wird?!

Ich stehe aufm Schlauch :D

[Peter II (Gast)]

volatile ist doch kein Schutz!

Eventuell kommt ja schon die nächste Nachricht an und setzt die Variable 
auf einen neue Wert. DataReceivedHandler ist ja Asynchron.

Außerdem ist sp.ReadLine(); im DataReceivedHandler keine Gute Idee, dort 
sollte man nur das lesen was auch da ist und nicht warten.

[Draco (Gast)]

Na in dem Sinne geschützt das Threadübergreifend darauf zugegriffen 
werden kann. Aber die Variable wird ausschlaußlich an diesen beiden 
Stellen genutzt.

Hmmm... du meinst quasi das der Programmcode weiter ausgeführt wird in 
butInfoClick, die Variable da noch garnicht gesetzt ist wenn sie 
übergeben wird. Und der ReceiveHandler erst später "zuschlägt"?!

Da muss ich mir was einfallen lassen das butInfoClick erst weitermacht 
wenn varHardwareVersion gesetzt ist.

[Peter II (Gast)]

Draco schrieb:
> Hmmm... du meinst quasi das der Programmcode weiter ausgeführt wird in
> butInfoClick, die Variable da noch garnicht gesetzt ist wenn sie
> übergeben wird. Und der ReceiveHandler erst später "zuschlägt"?!

ja, du wartest doch nirgends.


Du könntest das ganze ohne Event machen, einfach in butInfoClick den 
comport mit readline lesen. Dann blockiert aber dein Programm, wenn 
niemand antwortet.

[Draco (Gast)]

Sooo....


ich habe mal, auf deinen Ratschlag hin, das sp.ReadLine() zu 
sp.ReadExisting() geändert.

Die butInfoClick hab ich nun um eine While Schleife ergänzt die solange 
läuft wie varHardwareVersion == "" ist. Das funktioniert nun. Ich muss 
nun nur noch ne Abbruchbedingung aus der Schleife machen, sollte nach 
einer gewissen Zeit keine Daten ankommen.

Interessant wäre auch, die ComPort direkt nach dem Senden auszulesen. 
Aber da kommt mir ja der ReceiveHandler dazwischen bzw kollidieren sie 
sich ja. Kann ich während des Programmablaufes den Eventhandler 
"wegnehmen"?

ComPort.DataReceived += new SerialDataReceivedEventHandler(DataReceivedHandler);

[Draco (Gast)]

Peter II schrieb:
> Du könntest das ganze ohne Event machen, einfach in butInfoClick den
> comport mit readline lesen. Dann blockiert aber dein Programm, wenn
> niemand antwortet.

Naja zumindest nur solange wie Timeout in ComPort gesetzt ist, solange 
darf das warten und das Programm stillstehen :D

[Peter II (Gast)]

Draco schrieb:
> Interessant wäre auch, die ComPort direkt nach dem Senden auszulesen.
> Aber da kommt mir ja der ReceiveHandler dazwischen bzw kollidieren sie
> sich ja. Kann ich während des Programmablaufes den Eventhandler
> "wegnehmen"?

ja sollte gehen, aber warum setzt du ihn überhaupt?

[Draco (Gast)]

Peter II schrieb:
> ja sollte gehen, aber warum setzt du ihn überhaupt?

Weil ich von dem µC immer "Pings" bekomme (Case x im Eventhandler). Der 
Code ist aber rausgeflogen für hier.

Ich habe nun die butInfoClick angepasst um die Daten dann direkt zu 
lesen. Klappt nun wunderbar. Ich danke dir ganz doll!

[c]

private void butInfoClick(object sender, EventArgs e)
{
   ComPort.DataReceived -= DataReceivedHandler;      //Remove 
EventHandler

   FormControllerInfo f = new FormControllerInfo();
   f.mComPort = ComPort.PortName;

   Gauge.mGetHardwareVersion();
   ComSend(Gauge.mByteToSend);
   Gauge.SetupGauge(0x02);

   try                                                 //Catch Timout
   {
     varHardwareVersion = ComPort.ReadLine();          //ReadLine
   }
   catch
   {
     varHardwareVersion = "Error reading Port";
   }

   ComPort.DataReceived += new 
SerialDataReceivedEventHandler(DataReceivedHandler);  //Set EventHandler

   f.mVersionString = varHardwareVersion;
   f.Show();
}

[Draco (Gast)]

Peter II schrieb:
> ja sollte gehen, aber warum setzt du ihn überhaupt?

Weil ich von dem µC immer "Pings" bekomme (Case x im Eventhandler). Der 
Code ist aber rausgeflogen für hier.

Ich habe nun die butInfoClick angepasst um die Daten dann direkt zu 
lesen. Klappt nun wunderbar. Ich danke dir ganz doll!

private void butInfoClick(object sender, EventArgs e)
{
   ComPort.DataReceived -= DataReceivedHandler;      //Remove EventHandler
            
   FormControllerInfo f = new FormControllerInfo();
   f.mComPort = ComPort.PortName;

   Gauge.mGetHardwareVersion();
   ComSend(Gauge.mByteToSend);
   Gauge.SetupGauge(0x02);

   try                                                 //Catch Timout
   {
     varHardwareVersion = ComPort.ReadLine();          //ReadLine
   }
   catch
   {
     varHardwareVersion = "Error reading Port";
   }

   ComPort.DataReceived += new SerialDataReceivedEventHandler(DataReceivedHandler);  //Set EventHandler

   f.mVersionString = varHardwareVersion;
   f.Show();
}

EDIT.... Waaahhh falscher Knopf! :D