Array mit strncmp durchsuchen

[Christoph H. (christoph_b)]

Hallo

Ich scheitere nun an folgender einfachen Aufgabe ;-)
Hier erstmal der Code:

      char output_array[100][40];
      check=UB_Uart_ReceiveString(COM3,buf);//buf =empfangsbuffer
      if(check==RX_READY) {
        //i++;
        if(strlen(buf)>40)
        {
          i++;
          UB_Led_Toggle(LED_RED);
          ptr=strtok(buf,delimiter);

           for(array_search=0;array_search<100;array_search++)
           {
             if (strncmp (output_array[array_search],ptr,16) == 0)
             {
               UB_Led_Toggle(LED_ORANGE);
               break;
             }
           }

            //sprintf (output_array[i],"%s",ptr);
          while (ptr != NULL)
            {

             sprintf (output_array[f],"%s",ptr);
             ptr = strtok(NULL, delimiter);
             if(f<i)
             {
               f=i;
               e=f-1;
             }
            }

Ich teile nun meine Zeichenfolge und speicher sie in dem 
Mehrdimensionales Array "char output_array[100][40];"

Die Zeichenfolge die im Array gespeichert wird schaut z.B. so aus.
10:68:3f:24:6e:e5 5a020c "" -54
08:d4:2b:1e:40:22 1a010c "" -66
ac:7b:a2:a3:92:5d 7e010c "" -43

Wenn die ersten 16 Zeichen der Zeichenfolge schon im Array vorhanden 
ist, soll er nicht wieder eingetragen werden.

Leider klappt das nicht.

Wenn ich die Zeichenfolge in einer anderen Reihenfolge empfange 
fünktioniert die Überprüfung nicht. Wieso weis ich nicht.

[Stefan F. (Gast)]

Der Quelltext ist unvollständig und ich verstehe deine 
Problembeschreibung nicht. Fang nochmal von vorne an und erkläre in Ruhe 
und ausführlich, was das Programm tun soll und was es stattdessen tut.

>10:68:3f:24:6e:e5 5a020c "" -54
>08:d4:2b:1e:40:22 1a010c "" -66
>ac:7b:a2:a3:92:5d 7e010c "" -43

Was soll das darstellen?

[MaWin (Gast)]

Christoph B. schrieb:
> Leider klappt das nicht.

Vereinfacht gesagt: Du kannst einfach nicht programmieren.

Dein Code tut etwas anderes als du beschreibst.

Wenn die ersten 16 Zeichen im empfangenen Strin in einem der 100 
Einträge vorhanden sind, blinkt die LED.

Der String wird an einem obskuren Delimiter (vielleicht Semikolon) 
zerlegt, an irgendeiner Stelle im Arry abgelegt und dann werden 
irgendwelche Variablen beeinflusst, womit man wohl erreichen will daß 
der nächste Teil des Sttings woanders hin kommt.

Nein, da ist so viel dermassen durcheinander und erkennbar ohne PLan 
zusammenkopiert, man kann dir nicht die gesamten fehlenden 
Programmierkenntnisse beibringen. Frag einen, der das kann.

Man wird sicher nicht einen unendlich langen Strong einlesen.
Man wird sicher nicht in einem unsortierten Array nach einer 
Übereinstimmung des Anfangs suchen.
Man wird sicher nicht per strtok in undefiniert lange Abschnitte 
zerlegte Strings in möglicherweise zu kurze Array schreiben.
Man tut sich auch keinen Gefallen, ein Format zu erwarten aber nicht mal 
ansatzweise zu prüfen, ob die Daten das Format haben. Und wenn man es 
analysiert hat, braucht man keine Strings mehr zu vergleichen, sondern 
z.B. 6 Bytes.
Und ein sprintf zu verwenden, um einen String zu kopieren, ist ein 
deutliches Zeichen mangelnder Kompetenz.

> UB_Uart_ReceiveString(

Hmm, wenn das die Funktion ist
http://mikrocontroller.bplaced.net/wordpress/?page_id=2838
ist die auch saumässig schlecht definiert.

Kommt 50 Zeichen lang kein CarriageReturn wird einfach der Speicher 
überschrieben.

Vergiss die vorgefertige Library. Die ist so schlecht wie dein Code. 
Frag also auch nicht deren Ersteller ob er dir hilft.