C# (Visual Studio) string erstellen.

[Marion (Gast)]

Hallo,

ich möchte mit einer GUI einen String an einen AVR senden.

Der AVR erwartet Werte wie "0,12,124,42\n".

         private void button1_Click(object sender, EventArgs e)
        {
            string str;
            str = "0";
          str += Convert.ToString(trackBar1.Value);
            str += ",";
            str += Convert.ToString(trackBar2.Value);
            str += ",";
            str += Convert.ToString(trackBar2.Value);
            str += "\n";
            serialPort.WriteLine(str);
        }

Leider liefert mir die Funktion das nicht. Was ist da genau falsch?

[Peter II (Gast)]

Marion schrieb:
> Leider liefert mir die Funktion das nicht.

und was liefert sie?

Man kann es auch besser lesbar schreiben.

string str = string.Format("0,{0},{1},{2}\n", trackBar1.Value, trackBar2.Value, trackBar2.Value );
serialPort.WriteLine(str);

[Wolfgang (Gast)]

Marion schrieb:
> Leider liefert mir die Funktion das nicht. Was ist da genau falsch?

Zähl mal deine Kommata im String.

[Marion (Gast)]

serialPort.WriteLine(string.Format(",{0},{1},{2}\n", trackBar1.Value, trackBar2.Value, trackBar2.Value));

Liefert nicht die Anzahl erwünschter ','. Es wird lediglich 1 ',' 
gesendet

[Marion (Gast)]

Sende ich über Putty "1,2,3,4\n" wird es auf dem AVR korrekt 
"entschlüsselt".

[Peter II (Gast)]

Marion schrieb:
> Liefert nicht die Anzahl erwünschter ','. Es wird lediglich 1 ','
> gesendet

glaube ich nicht.

Bei mir geht es. Die Zwischenvariable hilft übrigens beim Debuggen, dann 
sieht man was wer sendet.

[bluppdidupp (Gast)]

Beachten sollte man, das WriteLine() selbst noch ein NewLine-Zeichen 
anhängt.
Beim Senden von Zeichenketten statt Bytes kann es generell nicht 
schaden, SerialPort.Encoding und SerialPort.NewLine besser auf 
gewünschte Werte zu setzen.

[Roland .. (rowland)]

Marion schrieb:
> Leider liefert mir die Funktion das nicht.

Also was sendet der Code denn jetzt, was steht in "str"?

[Marion (Gast)]

      private void button2_Click(object sender, EventArgs e)
        {
            string str = string.Format("1,{0},{1},{2}", trackBar1.Value, trackBar2.Value, trackBar3.Value);
            serialPort.WriteLine(str);
            lbSent.Items.Add(str);
        }

Sendet laut dem Ausgabefenster "1,0,0,255". Die Werte passen soweit, 
jedoch bekomme ich vom AVR die Antwort

"LED   1 (hue,sat,val)  : 0, 0, 0"

    #ifdef DEBUG_UART
    static char uart_puffer[32];
    #endif
    
    #define NUM_FIELDS  5
    static uint8_t fieldIndex = 0;
    static uint16_t values[NUM_FIELDS];
    static uint8_t i = 0;
    
    // RXC0 Flag gesetzt
    if(uart_available())
    {
      // Zeichen aus UDR0 lesen
      char c = uart_getc();

      if(c >= '0' && c <= '9')
      {
        if(fieldIndex < NUM_FIELDS)
        {
          values[fieldIndex] = (values[fieldIndex] * 10) + (c - '0');
        }
      }
      else if(c == ',')
      {
        fieldIndex++;
      }
      else
      {
        // Einzelne Led setzen
      //  if((values[0] >= 0x0f) && (values[0] < (0x10 + NUM_LEDS)))
        if(values[0] < NUM_LEDS)
        {
          #ifdef DEBUG_UART
          sprintf(uart_puffer, "LED %3d (hue,sat,val)\t:%3d,%3d,%3d\r\n", values[0], values[1], values[2], values[3]);
          uart_puts(uart_puffer);
          #endif
          hsv[values[0]].h = values[1];
          hsv[values[0]].s = values[2];
          hsv[values[0]].v = values[3];        
          //hsv[values[0]-0x10].h = values[1];
          //hsv[values[0]-0x10].s = values[2];
          //hsv[values[0]-0x10].v = values[3];
        }
        // Alle Leds setzen
      //  else if(values[0] == (0x10+NUM_LEDS))
        else if(values[0] == NUM_LEDS)
        {
          #ifdef DEBUG_UART
          sprintf(uart_puffer, "LEDS    (hue,sat,val)\t:%3d,%3d,%3d\r\n", values[1], values[2], values[3]);
          uart_puts(uart_puffer);
          #endif
          for(uint16_t led = 0; led < NUM_LEDS; led++)
          {
            hsv[led].h = values[1];
            hsv[led].s = values[2];
            hsv[led].v = values[3];
          }
        }
        else
        {
        uart_puts("ELSE\n");
          //not yet
        }
        // Index zuruecksetzen
        fieldIndex = 0;
        // Array loeschen
        for(uint8_t i = 0; i < NUM_FIELDS; i++) values[i] = 0;
      }
    }

[Peter II (Gast)]

Marion schrieb:
> Sendet laut dem Ausgabefenster "1,0,0,255". Die Werte passen soweit,
> jedoch bekomme ich vom AVR die Antwort

setzte mal noch wie schon von bluppdidupp geschrieben das Encoding auf 
ASCII.

[Marion (Gast)]

Wie mache ich das, habe von C# nicht viel Ahnung und brauche nur ein 
kleines Programm. Hab ansonsten erst einmal nicht weiter vor mit C# zu 
arbeiten.

[Marion (Gast)]

Okay,

das Encoding scheint schon auf ASCII zu stehen.

            serialPort.Encoding = Encoding.GetEncoding(28591);

Beim AVR kommt einfach kein 2. ',' an.

[Roland .. (rowland)]

Probier mal statt der WriteLine-Methode das hier (WriteLine sendet wie 
schon von bluppdidupp angemerkt selbst LF/CR mit):

Byte[] Buffer = Encoding.ASCII.GetBytes(str);
serialPort.Write(Buffer, 0, Buffer.Length);

[Peter II (Gast)]

Marion schrieb:
> Beim AVR kommt einfach kein 2. ',' an.

dann lass doch den AVR einfach mal zurückschicken, was er bekommt. Es 
müsste ja schon das erste Zeichen reichen was er nicht erkannt hat.

sprintf(uart_puffer, "LED %3d (hue,sat,val)\t:%3d,%3d,%3d (char=%x)\r\n", values[0], values[1], values[2], values[3], c);

[Stefan S. (sschultewolter)]

Hmmm,

das Problem hab ich scheinbar an der falschen Stelle gesucht. Wieso es 
aber mit Putty geht, ist mir ein Rätsel.

Ich habe im Programm ein delay_ms(10). Davor wird ein WS2812 Strip 
beschrieben (80Leds). Nehme ich beide Teile raus, funktioniert es auch 
mit dem Visual C#.

Was kann ich da ggf. machen mit der UART Schnittstelle? Interruptbetrieb 
ist nicht möglich, da die Leds das blockieren.

void readUartInput(void)
{
  #define MAX    50
  static uint8_t strComplete = 0;
  static uint8_t strCount = 0;
  static char string[MAX+1] = "";
  
  if(uart_available())
  {
    char c = uart_getc();
    if(strComplete == 0)
    {
      if(c == '\n' || c == '\r' || strCount >= MAX)
      {
        string[strCount] = '\0';
        strCount = 0;
        strComplete = 1;
      }
      else
      {
        string[strCount] = c;
        strCount++;
      }
    }
  }
  if(strComplete == 1)
  {
    uart_puts(string);
    uart_puts("\r\n");
    strComplete = 0;
  }
}

[Peter II (Gast)]

Stefan S. schrieb:
> Was kann ich da ggf. machen mit der UART Schnittstelle? Interruptbetrieb
> ist nicht möglich, da die Leds das blockieren.

dann zeig mal das komplette Programm. Niemand weiß was hinter 
uart_available und uart_getc steckt.

> Wieso es aber mit Putty geht, ist mir ein Rätsel.
vermutlich weil du viel langsamer tippst als C# sendet.

[Peter II (Gast)]

Nachtrag.

die könntest einfach die Baudrate runter drehen, dann hat der µC mehr 
zeit für die Verarbeitung.

[Marion (Gast)]

uint8_t uart_getc(void)
{
  while (!(UCSR0A & (1<<RXC0)))   // warten bis Zeichen verfuegbar
  ;
  return UDR0;                   // Zeichen aus UDR an Aufrufer zurueckgeben
}

uint8_t uart_available(void)
{
  return ((UCSR0A & (1<<RXC0)));
}

Baudrate werde ich mal runterdrehen.

[Peter II (Gast)]

und wo im Programm steht das delay?

[Roland .. (rowland)]

Stefan S. schrieb:
> Interruptbetrieb ist nicht möglich, da die Leds das blockieren.

Verstehe ich nicht.

Den String im C# Code zeichenweise mit Pausen senden (und hoffen, dass 
der UART-Puffer nicht die Zeichen sammelt), dann hat der µC auch genug 
Zeit.