Anfängerfrage zu strlen

[Bitkipper (Gast)]

Hallo,

ich versuche mit dem folgenden Code die Länge eines Strings zu 
ermitteln.

#include <string.h>
char CMD_String[32];

...


strcpy( CMD_String, "Hallo Welt" );
  
   
    while (1) 
    {
    
    volatile int Laenge = strlen(CMD_String); // <------
    for (len = 0; len < Laenge ; len++);
    {
       bufferAddToEnd  (&UART0_TX_Buffer,CMD_String[len]); 
    }
    UCSR0B |= (1<< UDRIE0); // USART0 Data Register Empty Interrupt Enable

------------------
Der Code liefert als Stringlänge immer eine 0 zurück, wo liegt mein 
Fehler?

Danke

[Rene H. (Gast)]

Hmm, ich sehe da keinen Fehler. Dann liegt es vermutlich an dem Code den 
du uns nicht zeigst.

[Kaj (Gast)]

Funktioniert bei mir einwandfrei...

Bitkipper schrieb:
> Der Code liefert als Stringlänge immer eine 0 zurück
Woher weißt du das? Wie hast du das festgestellt?

[Bitkipper (Gast)]

Hallo,

hier mal das komplette main.c file

#include "global.h"
#include <avr/io.h>
#include <avr/interrupt.h>
#include <util/delay.h>
#include <string.h>
#include "AVRlib/buffer.h"

#define  buffer_size   32
char UART0_TX_Buffer_String[buffer_size];
cBuffer UART0_TX_Buffer;
char CMD_String[32];


ISR( USART0_UDRE_vect ) 
  {
    UDR0 = bufferGetFromFront (&UART0_TX_Buffer);
    if (UART0_TX_Buffer.datalength = 0)
    {
      UCSR0B &= ~(1<< UDRIE0); // USART0 Data Register Empty Interrupt Disable
    }
  }
  
void InitUart0 (void)
{
  // UART0
  UBRR0 = (config_UBRR0 & 0x0FFF);    // Baudrateneinstellung nur die niederwertigsten 12Bit
  UCSR0B |= (1<<TXEN0);          // Enable TX Funktion 
}



int main(void)
{  
  uint8_t len = 0;
  // Debug
   DDRB = 0x00;
   DDRB |= (1<<PB0)| (1<<PB1);
   // Init Sendebuffer für UART0
   bufferInit(&UART0_TX_Buffer, UART0_TX_Buffer_String, buffer_size);
   
  InitUart0();  // UART0
  
  UCSR0B |= (1<< UDRIE0);          // USART0 Data Register Empty Interrupt Enable
  sei();

  strcpy( CMD_String, "Hallo Welt" );
  
   
    while (1) 
    {
    
    
    volatile int Laenge = strlen(CMD_String);
    for (len = 0; len < Laenge ; len++);
    {
       bufferAddToEnd  (&UART0_TX_Buffer,CMD_String[len]); 
    }
    UCSR0B |= (1<< UDRIE0); // USART0 Data Register Empty Interrupt Enable
    
  }
}

Zur Frage von Kaj:
Ich habe auf der for.. Zeile einen Breakpoint gesetzt und über die 
Watchfunktion den Variableninhalt von Laenge kontrolliert.

Umgebung:
- Win 10
- AVR Studio 7
- Controller AT Mega 644PA

Danke

[Walter S. (avatar)]

Bitkipper schrieb:
> cBuffer UART0_TX_Buffer;
was bedeutet diese Zeile?

>     if (UART0_TX_Buffer.datalength = 0)
das ist bestimmt nicht gewünscht,
Warnings einschalten UND beachten

[Dirk B. (dirkb2)]

Bitkipper schrieb:
> Ich habe auf der for.. Zeile einen Breakpoint gesetzt und über die
> Watchfunktion den Variableninhalt von Laenge kontrolliert.

Möglicherweise existiert die "Variable" nur im Register.

Es ist ungünstig, zweimal durch den String zu gehen

    for (len = 0; CMD_String[len] != '\0' ; len++);
    {
       bufferAddToEnd  (&UART0_TX_Buffer,CMD_String[len]); 
    }

macht das in einem Durchgang.

[for (Gast)]

Walter S. hat ja schon fast alles gesagt.

Von mir: Deine For-Schleife wird unendlich oft aufgerufen, der TX-Buffer 
wird überlaufen, noch bevor die Software was senden kann.