Array Probleme

Was du da schreibst ist reichlich verworren.

Aber schau dir mal zum Thema 'Wir funktionieren Strings in C'
mal folgendes an:

http://www.mikrocontroller.net//articles/FAQ#Wie_funktioniert_String-Verarbeitung_in_C.3F

Das sollte einiges klären.

so jetzt bin ich registriert^^

Also ich empfange auch ASCII Zeichen, und das ist mein größtes Problem.

Ich werde mir jetzt mal den Link angucken und schreiben was bei 
rumgekommen ist.

Jonas G. wrote:
> Ich werde mir jetzt mal den Link angucken und schreiben was bei
> rumgekommen ist.

Das Hauptproblem in deiner Fragestellung ist, dass nicht klar
ist worauf deine Frage abzielt. Es gibt da mehrere Interpretations-
möglichkeiten. Insbesondere ist nicht ganz klar, was dein
das Ergebnis sein soll.
Falls du noch Fragen hast, denke also daran die Fragestellung
zu präzisieren. Insbesondere hilft es meist, wenn du angibst
welche Datentypen involviert sind. Daraus kann man dann meist
dein eigentliches Problem erraten.

Okay,

ich habe jetzt mal gelesen und ausprobiert. Leider ohne wirklichen 
erfolg.
Ich möchte wissen, wie ich die Zeichen, die ich per UART bekomme und in 
"unsigned char c" speichere in den array "char temp_str[40]" speichern, 
sodass im arry später alle empangenen zeichen (Zahlen und Buchstaben) 
enthalten sind.

Leider gibt es Probleme, wenn ich c also UDR in den Array specihern 
möchte.
Meldung:
code.c:227: warning: passing argument 2 of 'strcat' makes pointer from 
integer without a cast
Das ist mein Code:

1

SIGNAL(SIG_UART_RECV)

2

{

3

  char temp_str[40];

4

        unsigned char c;

5

  c = UDR;

6

  strcat(temp_str,c);

7

8

  for(int i=0; i<40; i++){      //40 nur zum Test um den gesammten array auszulesen.

9

  uart_send_c(temp_str[i]);

10

  }

11

12

}

Wenn ich statt "c" einen String wie "TEST" einfüge, bekomme ich zwar 
keinen Fehler aber falsche Zeichen im Hyperterminal.

Jonas

Das was du schriebts stimmt zwar und so hat es mit der ausgabe auch 
funktioniert. Doch ich brauche unbedingt alle empangenen Zeichen in 
einer Variable oder muss die zusammen an eine Funktion übergben.

Hier die Funktion:

1

void stepper(int drive, int steps, int dir, int mode){..}

So möchte ich dann übergeben:

1

stepper(1, [Alle Zeichen bzw. kompl. Array], 1, 0);

Hoffe ihr verliehrt nicht die gedult mit mir^^

Jonas

Jonas G. wrote:
> Okay,
>
> ich habe jetzt mal gelesen und ausprobiert. Leider ohne wirklichen
> erfolg.
> Ich möchte wissen, wie ich die Zeichen, die ich per UART bekomme und in
> "unsigned char c" speichere in den array "char temp_str[40]" speichern,
> sodass im arry später alle empangenen zeichen (Zahlen und Buchstaben)
> enthalten sind.

Ganz einfach: Indem du sie ins Array zuweist:

  temp_str[naechstes_Zeichen] = UDR;

und natürlich danach naechstes_Zeichen um 1 erhöhen, sodass
das nächste Zeichen dann im Array an die nächste Position
kommt.

  naechstes_Zeichen++;

Also:

1

char temp_str[40];

2

int  nachstes_Zeichen;

3

4

SIGNAL(SIG_UART_RECV)

5

{

6

  temp_str[naechsts_Zeichen] = UDR;

7

  naechstes_Zeichen++;

8

}

Ein String wird in C in einem Array gespeichert. Die einzelnen
Elemente des Arrays sind die Zeichen die den String aufbauen.
Du kannst daher auf jedes einzelne Zeichen mit ganz normaler
Array Sytax zugreifen.
Erst dann wenn du das Array als Ganzes als String auffassen
möchtest, kommen die str... Funktionen ins Spiel.

johnny.m wrote:
> BTW: Du solltest möglichst anstelle des veralteten SIGNAL-Makros das
> aktuelle ISR verwenden.

Werde ich spätestens ändern, wenn die übergabe an die stepper funktion 
läuft. Eigendlcih mache ich das auch mit ISR. Keine Ahnung, warum ich es 
diesmal nciht gemacht haben xD

Jonas G. wrote:
> Das was du schriebts stimmt zwar und so hat es mit der ausgabe auch
> funktioniert. Doch ich brauche unbedingt alle empangenen Zeichen in
> einer Variable

Nochmal. Das geht nicht.
Alle empfangenen Zeichen zusammengenommen bilden einen String.
Es gibt aber keinen Datentyp für Strings! Es gibt nur ein Array

> Hier die Funktion:
>

1

> void stepper(int drive, int steps, int dir, int mode){..}

2

>

Das ist aber was ganz anderes. Diese Funktion möchte
Zahlenwerte und keine Strings.

Du musst dich also damit beschäftigen, wie du aus einem
String "123" den Zahlenwert 123 bekommst.

>

1

> stepper(1, [Alle Zeichen bzw. kompl. Array], 1, 0);

2

>

Hier:
             ********************************

hier musst du aus dem String "123" den Zahlenwert 123 machen.

Wenn der String korrekt aufgebaut ist (das abschliessende '\0'
nicht vergessen!) dann geht das so

  int Zahl = atoi( temp_str );

atoi macht aus einem entsprechend geformten String den
entsprechenden Zahlenwert. Da steckt nichts
geheimnusvolles dahinter. Angenommen du mueestes das
machen und ich geb die die Zeichen einzeln. Fangen wir
an.
Zunächst mal ist deine Zahl eine 0
Dann geb ich dir eine '5' (Also den ASCII Code für 5)
Du ziehst davon den Code für '0' ab, nimmst die bisherige
Zahl mal 10 und addierst die 5

   Zahl = 10 * Zahl + ( '5' - '0' )

Da Zahl vorher 0 war, kriegt Zahl als neuen Wert 5

Jetzt geb ich dir '8'. Davon die '0' abgezogen gibt 8, also
den Zahlenwert, der dem Zeichen '8' entspricht.
Und wieder nimmst du die bisherige Zahl mal 10
(also 5 * 10 -> 50 ) und addierst diese 8. Macht 58

Das nächste Zeichen das ich dir gebe ist eine '3'

   '3' - '0'     ->  3
   58 * 10   -> 580  und dazu noch die 3 addiert -> 583

Dann geb ich dir '7'

   '7' - '0'     -> 7
   583 * 10 -> 5830 + 7 -> 5837

Jetzt solltest du das Prinzip schon gesehen haben. In C Code
gegossen ist das dann:

1

  int ToNumber( char* String )

2

  {

3

    int i = 0;

4

    int Zahl = 0;

5

6

    while( String[i] != '\0' ) {

7

      Zahl = 10 * Zahl + ( String[i] - '0' );

8

      i++;

9

    }

10

11

    return Zahl;

12

  }

Zusammengefasst: Von der UART kriegst du einen String.
In diesem String steckt die Zeichenmässige Repräsentierung
einer Zahl. Die kann man wieder als Zahl zurückgewinnen und
dann natürlich in die stepper-Funktion weitergeben

So dann starte ich mal:
Ich möchte eine Reihe von Zahlen und Buchstaben an den µC schicken und 
verarbeiten. Später soll mit diesem Daten die Funktion für die 
Schrittmoren "gefüttert" werden.
Dazu muss immer die ISR immer die empfangnen Zeichen speichern, was jezt 
auch funktioniert (hatte vorher irgendwie ein Brtee vorm Kopf). Code 
s.u.
Ich übergbe dann nach jedem Befehl ein Zeichen, was das Ende angibt. 
Dann muss der µC die Empfangenen Zeichen an die Funktion übergben hier 
ein Bsp.:
Gesendete Zeichen PC -> µC:
A;1;300;1;0;
Spätere Verarbeitung:
A=Stepper-Funktion
1=Motor 1
300 = 300 Schritte
1 = Linkslauf
0 = Vollschritt

So sieht mein Ziel aus. Doch das Eigendliche und oben besprochene 
Problem ist das, dass ich nicht weiß, wie ich die Zeichenkette aus 
mehreren Array-Einträgen übergbe. z.B.: die 300 wären ja 3 Einträge 
3-0-0 und wie übergbe ich die zusammen an die Funktion.

Hoffe es verwirrt nicht nochmehr...

Jonas

P.S.: Wie ich die Daten "auseinanderreiße" weiß ich und die 
Stepperfunktion ist auch fertig.

Hier die Codes:

1

//////////////////////UART-ISR zum Empfangen

2

SIGNAL(SIG_UART_RECV)

3

{

4

  temp_str[NextChar] = UDR;

5

  NextChar++;

6

7

  for(int i=0; i<40; i++){

8

  uart_send_c(temp_str[i]);

9

  }

10

11

}

12

//////////////////////Deklaration der Stepperfunktion

13

void stepper(int drive, int steps, int dir, int mode)...

@   Karl heinz Buchegger:
Ich glaube in deinem Post ist die Antwort für mein Problem.
Ich teste jezt mal und dann melde ich meinen erfolg.

Jonas

> A;1;300;1;0;
> Spätere Verarbeitung:
> A=Stepper-Funktion
> 1=Motor 1
> 300 = 300 Schritte
> 1 = Linkslauf
> 0 = Vollschritt
>
> So sieht mein Ziel aus. Doch das Eigendliche und oben besprochene
> Problem ist das, dass ich nicht weiß, wie ich die Zeichenkette aus
> mehreren Array-Einträgen übergbe. z.B.: die 300 wären ja 3 Einträge
> 3-0-0 und wie übergbe ich die zusammen an die Funktion.
>
> Hoffe es verwirrt nicht nochmehr...

Ganz im Gegenteil: Jetzt ist alles klar.
Du entdeckst gerade das ein String etwas anderes als eine Zahl ist,
selbst wenn in dem String "123" drinnensteht.
Lösung: schon gepostet: Du musst die Zahl aus dem String wieder
zurückgewinnen.

ERFOLG:

1

SIGNAL(SIG_UART_RECV)

2

{

3

  temp_str[NextChar] = UDR;

4

5

  for(int i=0; i<40; i++){

6

  uart_send_c(temp_str[i]);

7

  }

8

  if (temp_str[NextChar] == 0x0D) {

9

  int zahl = atoi(temp_str);

10

  stepper(1,zahl,1,0);}

11

  NextChar++;

12

}

Wenn ich zB 400 sende und dann Enter drücke dreht sich der Motor 1 um 
genau 400 Schritte. (ist nen Motor mit 400 Stepps daher leicht zu 
erkennen)

Danke an alle und Sorry wenn ich euch verwirtt habe^^

Jonas

NextChar wieder auf 0 zurücksetzen, sonst überlaufst du
das Array irgendwann.

Falk wrote:
> @Jonas G.
>
> Ich glaube du hasst da noch ein Verständnissproblem was Interrupts
> angeht. Im Interrupt wird beim UART immer nur ein Zeichen gelesen und
> gespeichert, die Verarbeitung erfolgt zu 99% ausserhalb des Interrupts.

Ne, den Code den ich da gepostet habe, der war nur zum Testen. Ich werde 
dass dans im Main-PRG machen.

Mir ging es dort nur um den Erfolg und den Test^^

Jonas

naja unbauchbar ist falsch, da es ja funktonier hat.
Jetzt wird rx_flag immer überprüft...
So muss ich natürlich jede Eingabe mit [Enter] abschließen.

1

SIGNAL(SIG_UART_RECV)

2

{

3

  temp_str[NextChar] = UDR;

4

  if (temp_str[NextChar] == 0x0D) {

5

  rx_flag=1;}

6

  else {

7

  NextChar++;

8

             }

9

}

stimmt, die kann ich auch ins ISR schreiben.
hatte sie im Main PRG