atmega übersprint Anweisung an bestimmter Stelle

[Chris T. (chris0086)]

Moin Community,
ich habe ein Problem und erkenne den Fehler nicht.
Das Problem ist das nach dem Ausführen einer Funktion der anschließnde 
Teil nicht richtig abgearbeitet wird. Hier erst mal der Code:

while(1)
{

  if(wert_anfordern == 1)
  {

    RX_start = 0;
    //USART_Transmit('|');
    wert_anfordern = 0;
    //Force_enable();
    SF_length = Build_String(count,requested_value,frame_send,werte);
    frame_send[SF_length]= CRC_Get(frame_send,SF_length);

    Frame_Change(SF_length,frame_send); //<--wahrscheinlich auslösende Funktion 
    //USART_SendS(frame_send,SF_length);
    //USART_Transmit('B'); //O
    receive_complete = 1;
  }
  if(receive_complete == 1) // Es wurden Daten empfangen
  {
    receive_complete = 0;
    USART_Transmit('|'); //O
    

  }
}

Inhalt der Funktion Frame_Change

void Frame_Change(uint8_t length,uint8_t *frame_send)
{
  uint8_t i,change,j;
  uint8_t frame_copy[16];
  j= 5;
  change = 0;
  LED_PORT ^= (1<<LED2);
  for(i=0;i<5;i++)
  {
    frame_copy[i] = frame_send[i];
  }
  for(i=5; i<=length;i++)
  {
    if((frame_send[i] == 0x02) || (frame_send[i] == 0x2B) || (frame_send[i] == 0xFE))
    {
      frame_copy[j] = frame_send[i];
      frame_copy[j+1] = 0x00;
      j+=1;
      change = 1;
    }
    else
    {
      if((frame_send[i] == 0x11))
      {
        frame_copy[j] = 0xFE;
        frame_copy[j+1] = 0x12;
        j+=1;
        change = 1;
      }
      else
      {
        if((frame_send[i] == 0x13))
        {
          frame_copy[j] = 0xFE;
          frame_copy[j+1] = 0x14;
          j+=1;
          change = 1;
        }
        else
        {
          frame_copy[j] = frame_send[i];
        }
      }
    }
    j+=1;
  }
  for(i=0;i<j;i++)
  {
    USART_Transmit(frame_copy[i]);
  }
  //return j;
}

Der Funktionsinhalt wird soweit richtig durchlaufen d.h. es wird am ende 
alles auf richtig auf die serielle Schnittstelle gesendet.

Danach müsste die Variable receive_complete auf 1 gesetzt werden und die 
If Bedingung durchlaufen werden und ich müsste ein '|' Zeichen an der 
seriellen Schnittstelle empfangen.
Tu ich aber nicht.
Kommentiere ich den Funktionsaufruf aus so wird die If Anweisung 
durchlaufen und ich erhalte mein '|' Zeichen an der seriellen.

Wo kann denn da der Fehler liegen?

Grüße Christian

[Max D. (max_d)]

Ist reveive_complete als volatile deklariert?

[Chris T. (chris0086)]

Max D. schrieb:
> Ist reveive_complete als volatile deklariert?

Ja ist es.

[Karl H. (kbuchegg)]

Wie gross ist frame_send definiert?
Welcher Wert ergibt sich hier

    SF_length = Build_String(count,requested_value,frame_send,werte);

für SF_length?

Wahrscheinlich hast du ein Array überlaufen. Es könnte entweder 
SF_length oder frame_copy in der zweiten Funktion sein.

frame_copy braucht in Wirklichkeit auch keiner. Anstatt die zeichen erst 
mal mühsam in einem Array zu sammeln um sie dann doch wieder nur einzeln 
auszugeben, könntest du auch gleich die Zeichen ausgeben, so wie du auf 
sie stösst. Hätte den angenehmen Nebeneffekt, dass du kein Array 
brauchst, das du somit auch nicht überlaufen kannst.

[Marc V. (Firma: Vescomp) (logarithmus)]

Chris tian schrieb:
> Wo kann denn da der Fehler liegen?

 Dass du nicht den Teil mit Variablen und Deklarationen mitgesendet
 hast. Wahrscheinlich hast du die 'receive_complete ' totgerechnet.

 Zum 1000-sten Mal:
 Immer das ganze Programm mit Deklarationen, header etc. mitschicken.
 Wenn es zu lang ist, als besondere Datei uploaden.

[Thomas H. (Firma: CIA) (apostel13)]

Es gibt do so was wie einen Debugger...

[Karl H. (kbuchegg)]

Das hier

  for(i=5; i<=length;i++)

ist auch dubios.

In einem Vektor der Länge 5, ist der größte sinnvolle Index 4.

  0, 1, 2, 3, 4

zähl nach: sind genau 5 Stück.

Wenn du daher einen String mit der Länge length hast (vorausgesetzt die 
Längenangabe ist korrekt), dann ist der größte Index length-1, oder eben

  for( i = 5; i < length; i++ )

so wie das eben bei Arrays deren Länge man kennt der Normalfall in einer 
for-Schleife ist: 'kleiner' und nicht 'kleiner gleich'.


Ausgeglichen wird der Fehler dadurch, dass du in der aufrufenden 
Funktion die Länge um 1 zu niedrig zählst. Denn hier

    frame_send[SF_length]= CRC_Get(frame_send,SF_length);

wird der Frame um 1 länger, was du allerdings nicht in SF_length 
berücksichtigst.

Jetzt könnte man natürlich sagen: was solls. Beim Aufrufer ist die Länge 
um 1 zu gering, in der Funktion gehts dafür um 1 zu weit - kommt doch am 
Ende wieder alles richtig raus.

Ist trotzdem keine gute Idee. Denn irgendwann rufst du die Funktion von 
einer anderen Stelle aus auf, und dann übergibst du die tatsächlich 
richtige Länge und dann baut die Funktion grossartigen Mist.
Mach die Dinge gleich richtig, so dass jede Funktion genau das tut was 
sie zu tun hat und keine Querverbindungen dergestalt bestehen, dass die 
eine Funktion fehlerhaft ist und quasi als Ausgleich eine andere 
Funktion den gegenteiligen Fehler macht, so dass sich die Fehler 
gegenseitig aufheben.