C/Zeichen aus Array vergleichen?

[Stefan O. (Gast)]

Hallo zusammen,

seit ein paar Tagen hat mich auch der µC Virus befallen, bei der 
Gelegenheit unternehme ich auch meine ersten Gehversuche mit "C". Und 
schon habe ich mein erstes Problem und Fragen:

Meine selbstgestellte Übungsaufgabe: eine Art "Ablaufsteuerung" in einem 
Array (code) abzulegen. Kommandos sind dabei in Form von einzelne 
Buchstaben hinterlegt.Was ich jetzt hin bekommen habe ist es das Array 
zu splitten und die einzelnen Buchstaben wieder separat auszugeben. 
Woran ich aber schier verzweifle ist Vergleichsoperationen 
durchzuführen.
Wenn ich mit "if(strcmp(element, "A") == 0)" vergleiche funktioniert 
dies nicht richtig.

Leider bringt mich der Compiler Hinweis "passing argument 1 of ‘strcmp’ 
makes pointer from integer without a cast" nur bedingt weiter. Wenn ich 
das richtig verstehe muss ich eine Typenumwandlung durchführen weil 
sonst mit dem Pointer auf die Variable verglichen werden würde. Nur wie? 
Kann mir jemand auf die Sprünge helfen?

Nach einiger Frickelei habe ich jetzt Version2 gebastelt und die Strings 
für den Vergleich "kompatibel" gemacht, die Version funktioniert 
zumindest. Kann ich das auch intelligenter lösen?

[[http://www.mikrocontroller.net/articles/FAQ#wie_schreibt_man_eine_Funktion.2C_die_einen_String_liefert.3F]] 
habe ich gefunden, leider habe ich da aber noch Transferschwierigkeiten.

Schönes Wochenende und besten Dank im vorraus,

Stefan

Code Sniplet Version1:

#include <stdio.h>
#include <string.h>


int main()
{
  char code []="AABCAEE";
  char element;
  int i=0;
  while (i<sizeof(code)-1)
  {
    element = code[i];
    printf("Element %d: %c\n",i,element);
    
    if(strcmp(element, "A") == 0)

    {
      printf("Ausgang A Schalten\n");
    }
    if(strcmp(element, "B") == 0)

        {
          printf("Ausgang B Schalten\n");
        }
  i++;
  }

  return 0;
}

Code Sniplet Version2:

#include <stdio.h>
#include <string.h>


int main()
{
  char code []="AABCAEE";
  char element[2];
  int i=0;
  while (i<sizeof(code)-1)
  {
    element[0] = code[i];

    i++;
    element[1] = '\0';

    if(strcmp(element, "A") == 0)

    {
      printf("Ausgang A Schalten\n");
    }
    if(strcmp(element, "B") == 0)

        {
          printf("Ausgang B Schalten\n");
        }
  }

  return 0;
}

[Stefan E. (sternst)]

Wenn du einzelne Buchstaben vergleichen willst, dann vergleiche auch nur 
Buchstaben und keine Strings:

if (element == 'A')

[Stefan O. (Gast)]

Hi Stefan,

danke für die prompte Antwort. Leider komme ich so nicht zum gewünschten 
Ergebnis (keine Verzweigung in die Bedingung bei "A") und den Hinweis 
"comparison between pointer and integer".

Werde in der Zwischenzeit mal versuchen Variante 2 auf einen µC zu 
portieren.

LG Stefan

[Rufus Τ. F. (rufus)]

Stefan Ott schrieb:
> Leider komme ich so nicht zum gewünschten
> Ergebnis (keine Verzweigung in die Bedingung bei "A") und den Hinweis
> "comparison between pointer and integer".

Dann sieh mal ganz genau hin und vergleiche die von Dir verwendeten 
Zeichen unmittelbar vor und hinter dem A mit dem, was Stefan Ernst 
verwendet hat.

Da ist ein Unterschied.

Tipp:
Stringkonstanten werden in Anführungszeichen geschrieben: "bla",
Zeichenkonstanten aber werden in Hochkommata geschrieben: 'a'.

[Stefan O. (Gast)]

Na klar, so funktioniert es. Ich danke Euch beiden. Da habe ich mir ein 
""" für ein "'" vormachen lassen ;-)

In der Zwischenzeit habe ich das ganze mal mit Euren Korrekturen für den 
ATTINY13 mit 3 LED aufgebaut. Funktioniert einwandfrei und kann schnell 
und flexibel erweitert werden.

Ich hänge das ganze mal im kommentierten Quelltext an.Vielleicht kann 
jemand so etwas noch gebrauchen.

LG Stefan

[nocheinGast (Gast)]

Ich weiß nicht, ob du schon so weit bist, aber für solche 
Mehrfachverzweigungen bietet sich auch "switch" an. Das wird m. E. 
leichter lesbar und ist vor allem nicht so viel zu tippen:

switch (do_action) {
  case 'A':
    led_A_on;
    break;

  case 'a':
    led_A_off;
    break;

  case 'B':
    led_B_on;
    break;
    ...
  default:
    // unbekanntes Zeichen
    break;
}

[Stefan O. (Gast)]

Danke, die Idee mit dem Case Switch schau ich mir mal an. Ich befasse 
mich erst seit 2 Tagen mit C, ich habe vorher überwiegend mit Java 
programmiert, dort gibt es den auch.

LG Stefan

[Stefan O. (Gast)]

Stefan Ott schrieb:
> Danke, die Idee mit dem Case Switch schau ich mir mal an. Ich befasse
> mich erst seit 2 Tagen mit C, ich habe vorher überwiegend mit Java
> programmiert, dort gibt es den auch.
>
> LG Stefan

So sieht es gut aus...

Stefan

[nocheinGast (Gast)]

Du hast da noch ein paar Sachen drin, mit denen du dich gewaltig in die 
Nesseln setzen kannst...

#define led_A_on  PORTB |=_BV(PB0)
#define led_A_off  PORTB &= ~_BV(PB0)
#define led_B_on  PORTB |=_BV(PB1)
#define led_B_off  PORTB &= ~_BV(PB1)
#define led_C_on  PORTB |=_BV(PB2)
#define led_C_off  PORTB &= ~_BV(PB2)

Hast du schon mal probiert, was passiert, wenn in einem dieser Makros 
ein Fehler drin ist? Dann wird der Compiler bei der Fehlermeldung nicht 
die Zeile angeben, in der das Makro steht, sondern die Zeile, in der das 
Makro verwendet wird. Da kann das Fehlersuchen ekelhaft werden. Du 
kannst stattdessen inline-Funktionen erstellen, also welche, die der 
Compiler selber anstelle des Funktionsaufrufs einfügt. Das macht im 
Gegensatz zu den Makros nicht der Präprozessor, du kriegst also eine 
exakte Fehlermeldung:

inline void led_A_on() {
  PORTB |= (1 << PB0);
}

inline void led_A_off() {
  PORTB &= ~(1 << PB0);
}

inline void led_B_on() {
  PORTB |= (1 << PB1);
}

inline void led_B_off() {
  PORTB &= ~(1 << PB1);
}

inline void led_C_on() {
  PORTB |= (1 << PB2);
}

inline void led_C_off() {
  PORTB &= ~(1 << PB2);
}

Dieses _BV-Makro halte ich auch für überflüssig, mehr zu schreiben ist 
es so auch nicht.

DDRB =7;    //Ausgänge definieren (LED PB0-PB2)

Das ist sehr schwer zu lesen. Jemand, der kein Datenblatt hat, ist da 
schnell verloren ("Wo kommt denn die 7 her?"). Besser ist es so:

DDRB = (1 << PB0) | (1 << PB1) | (1 << PB2);

Diese Ausdrücke berechnet der Compiler beim Kompilieren, das Programm 
wird also nicht größer.

while (i<sizeof(program)-1)

"sizeof" ist in C mit Vorsicht zu genießen. Hier funktioniert es, aber 
wie wäre es so:

for (uint8_t i = 0; program[i] != '\0'; i++) {

Schließlich wird der String, den du oben festlegst, von einem '\0' 
abgeschlossen.

Ich hab deinen Code mal etwas umgeschrieben und angehängt.

[Stefan O. (Gast)]

Hallo nocheinGast,

viele Dank für Deine Rückmeldungen. Sind allesamt plausibel und 
nachvollziehbar. Ich werde diese bei meinen weiteren Gehversuchen mit C 
auf jeden Fall berücksichtigen.

LG Stefan