Array aus SD Karte laden

[Georg (Gast)]

Hallo,

folgendes Problem:

Ich würde gerne ein Kennfeld aus einer Text-Datei in ein Array laden.

Ich verwende einen ATmega32 und programmiere mit AVR Studio/GGC

Das Kennfeld ist in der Textdatei besteht aus einer variablen Anzahl aus 
Stützstellen, z.B.:

(0;300) (500;200) (1000;300) ...

Das erste Element ist einen Drehzahl, das Zweite das dazugehörige 
Moment.

Das einlesen der Werte ist soweit kein Problem. Ich weiß nur nicht wie 
ich sie in ein Array ablegen kann, da die Anzahl der Wertepaar variabel 
ist.

Vielleicht kann mir da jemand weiterhelfen.


Gruß
Georg

[Karl H. (kbuchegg)]

Georg schrieb:

> Das einlesen der Werte ist soweit kein Problem. Ich weiß nur nicht wie
> ich sie in ein Array ablegen kann, da die Anzahl der Wertepaar variabel
> ist.

Daran solltest du arbeiten:
Die darf zwar im File variabel sein, im Mega32 willst du aber ein fix 
dimensioniertes Feld haben. Du musst daher das Feld mit einer Größe 
anlegen, so dass du auf jeden Fall durchkommen wirst.

Natürlich gibt es auch Möglichkeiten, wie man sich Arrays dynamisch 
wachsen lassen kann. Auf einem Mega32 ist das allerdings nicht sinnvoll. 
Die Verwaltung des ganzen kostet dir mehr, als wie wenn du einfach den 
freien Speicher komplett in das Array steckst. Und wenn der freie 
Speicher aus ist, dann ist er aus. Da geht dann nichts mehr. Dann schon 
lieber soviel wie möglich in das Array investieren, als 10% vom speicher 
für die Verwaltung von dynamisch allokiertem Speicher zu reservieren.

[Robert L. (lrlr)]

generell:

kommt auf die Auswertung an, wenn du mehr werte einlesen musst, als du 
speicher hast, z.b. den Mittelwert über 50 werte (vor und nach dem 
aktuell) rechnen willst, mach zwei buffer mit je 100 werten..


wenns nur um ein paar wenige werte geht, den maximalwert reservieren 
(wie schon vorgeschlagen)

oder in deinem fall z.b. Fix 101 werte.. (0%-100%) und jeden der 100ert 
werte durch z.b interpolieren ermitteln..) dann wäre nachherige zugriffe 
einfacher (einheitlicher)

[Georg (Gast)]

Alles klar, benutze jetzt eine feste Anzahl an Werten.

Hab gerade noch ein anderes Problem und zwar werden Zeiche wie '(' oder 
'[' nicht erkannt.

z.B.

if(strcmp(buffer,'['){
 ...
}

Hier wird nicht in die if-Struktur gesprungen, auch wenn in der 
Textdatei das entsprechende Zeichen vorhanden ist. In buffer steht 
jeweils ein eingelesenes Zeichen aus der Text-Datei.

Ander Zeichen werden gefunden

z.B.

if(strcmp(buffer,'a'){
...
}

... funktioniert.


Jemand eine Idee woran das liegen könnte?

[Karl H. (kbuchegg)]

Georg schrieb:
> Alles klar, benutze jetzt eine feste Anzahl an Werten.
>
> Hab gerade noch ein anderes Problem und zwar werden Zeiche wie '(' oder
> '[' nicht erkannt.
>
> z.B.
>

> 
> if(strcmp(buffer,'['){
>  ...
> }
> 
>

Schmeiss dein C-Buch weg, und kauf dir ein neues. Deines taugt nichts, 
wenn es dir den Unterschied zwischen '[' und "[" und damit den 
Unterschied zwischen einem einzelnem Zeichen und einem String nicht klar 
macht.

> Jemand eine Idee woran das liegen könnte?

Könnte es sein, dass du gar kein C Buch hast?
Betsell dir am besten heute noch eines bei Amazon (oder einem sonstigen 
Buchhandel deiner Wahl). In der Zwischenzeit kannst du mit der 
Minimal-Kurzfassung aus der FAQ zum Thema "Strings" vorlieb nehmen.
http://www.mikrocontroller.net/articles/FAQ#Wie_funktioniert_String-Verarbeitung_in_C.3F
(Da gibt es einen Absatz der rot hervorgehoben ist. Wie gemünzt für 
deine Situation)

Und setz den Warning-Level deines Compilers wieder auf ein vernünftiges 
Mass hoch. Deine Version hätte nie ungestraft durch den Compiler gehen 
dürfen.

> if(strcmp(buffer,'a'){
> ...
> }
>
> ... funktioniert.

Zitat: Das glaub ich nicht, Tim.

[Georg (Gast)]

Jaahh ... ich habe natürlich "[" geschrieben ... sorry, vertippt.

Das Problem bleibt aber das gleich. Sonst noch jemand einen Tipp.


Gruß

[Karl H. (kbuchegg)]

Georg schrieb:
> Sonst noch jemand einen Tipp.

Aber natürlich:
Poste kompletten Code. Auf keinen Fall aber Codeschnipsel die du 
abtippst.

Dann kann man sich auch auf die Suche nach den Fehlern machen, die du
a) wirklich begangen hast
b) selber nicht siehst

Aber ich tippe mal ins Blaue:
Deine Stringterminiierung stimmt nicht.

[Georg (Gast)]

Ja, ich denke die Stringterminierung war's.

Ich habe den buffer so deklariert

buffer[1]

da ich ja immer ein Zeichen einlese.

Da ich ja nun die FAQs gelesen habe,
;-)

 weiß ich, dass ein String ja immer auch aus einer 'Endmarkierung' 
besteht.

Nun habe ich den buffer so deklariert:

buffer[2]

Nun funktioniert es. Danke für den Hinweis!

Gruß
Georg

PS: Ich hoff ich hab das auch richtig erkannt und keinen totalen Mist 
erzählt. ;-)

[Karl H. (kbuchegg)]

Georg schrieb:
> Ja, ich denke die Stringterminierung war's.
>
> Ich habe den buffer so deklariert
>
>

buffer[1]

>
> da ich ja immer ein Zeichen einlese.

... stellt sich die Frage, warum du dann überhaupt auf Strings setzt?
Einzelne Character zu bearbeiten geht einfacher und schneller

Aber ich seh schon: Du willst deinen Code nicht herzeigen, auf das man 
dir ein paar Techniken und Verfahren zeigen könnte, die dir das Leben 
einfacher machen könnten.

[Georg (Gast)]

Hier der Code:

#define TORQUE 0
#define REV 1

char ReadBuffer[2];
char mapBuffer[6];
uint16_t map[50][2];
uint8_t map_index;
uint8_t byte_count=0;



void get_map(void){
  do{
    byte_count=Fread(ReadBuffer,1);
      if(byte_count!=1)break;
                        if(strcmp(ReadBuffer,"[")==0){
        mapBuffer[0] = '\0';
        do{
          byte_count=Fread(ReadBuffer,1);
          if(byte_count!=1)break;
          if(strcmp(ReadBuffer,";")==0)break;
          strcat(mapBuffer,ReadBuffer);
        }while(1);
        map[map_index][TORQUE]=atoi(mapBuffer);
        mapBuffer[0] = '\0';
        do{
          byte_count=Fread(ReadBuffer,1);
          if(byte_count!=1)break;
          if(strcmp(ReadBuffer,"]")==0)break;
          strcat(mapBuffer,ReadBuffer);
        }while(1);
        map[map_index][REV]=atoi(mapBuffer);
        }
        map_index++;
  }while(1);

}

[Georg (Gast)]

Die Struktur der Daten in der Textdatei:

[0;300][500;2300][1000;4300][1500;2300] ...

[Georg (Gast)]

Würde natürlich sehr gerne lernen, wie man sowas besser macht.


Gruß
Georg

[Karl H. (kbuchegg)]

Du arbeitest gefährlich! Die strcat an mapBuffer sind alle nicht 
abgesichert. Mit einer Falscheingabe kannst du dein Programm ganz leicht 
-> BUMM!

Definiere dir char Arrays mit denen du auf Strings gehst nicht so auf 
knirsch! Lass dir ein wenig Platz um zumindest kleinere Fehler auffangen 
zu können.

Arbeite mit sizeof um Arraygrenzen zu prüfen, das mindeste sind aber ein 
paar #define um Arraygrößen festzulegen und dann mit diesen #define die 
Bereichsgrenzen im Programm zu prüfen.

Fread erwartet von dir lediglich die Adresse eines Speichers, an dem es 
die von dir vorgegebene Anzahl an Zeichen ablegen kann. Da du sowieso 
nur 1 Zeichen haben willst, kannst du ihm auch die Adresse eines char 
übergeben, in welches Fread das Zeichen reingibt. Dann brauchst du nicht 
die teuren Funktionen strcat, strcmp zur Weiterverarbeitung benutzen.

Wenns leicht geht, rechne auch immer damit, dass die Eingaben Fehler 
enthalten. Ein Leerzeichen da ein Tabulator dort. Schreib den Code nach 
Möglichkeit so, dass dir die nicht in die Quere kommen.

#define TORQUE 0
#define REV 1

#define MAP_SIZE 50

char ReadBuffer
char mapBuffer[6];
uint16_t map[MAP_SIZE][2];
uint8_t map_index;
uint8_t byte_count = 0;
uint8_t nextMap = 0;

void get_map(void)
{
  while( Fread( &ReadBuffer, 1 ) != -1 ) {

    if( ReadBuffer == '[' ) {

      nextMap = 0;
      while( Fread( &ReadBuffer, 1 ) != -1 ) {

        if( ReadBuffer == ';' )
          break;

        if( nextMap < sizeof( mapBuffer ) - 1 )
          mapBuffer[ nextMap++ ] = ReadBuffer;
      }
      mapBuffer[ nextMap ] = '\0';
      if( map_index < MAP_SIZE )
        map[map_index][TORQUE] = atoi( mapBuffer );

      nextMap = 0;
      while( Fread( &ReadBuffer, 1 ) != -1 ) {

        if( ReadBuffer == ']' )
          break;

        if( nextMap < sizeof( mapBuffer ) - 1 )
          mapBuffer[ nextMap++ ] = ReadBuffer;
      }
      mapBuffer[ nextMap ] = '\0';
      if( map_index < MAP_SIZE )
        map[map_index][REV] = atoi( mapBuffer );
    }
    map_index++;
  }
}

In dieser Funktion kommt der Code um eine Zahl einzulesen 2 mal vor. 
Wäre ein Kandidat um in eine Funktion ausgelagert zu werden. Der 
Codegröße und der Übersicht von get_map würde das zu Gute kommen. Oder 
hast du bemerkt, dass auch jedes Leerezeichen zwischen den Klammern 
ebenfalls den map_index um 1 erhöht. Ich habs auch nicht gesehen, ehe 
ich refakturiert habe

uint16_t get_number( char Terminator )
{
  uint8_t i = 0;

  while( Fread( &ReadBuffer, 1 ) != -1 ) {

    if( ReadBuffer == Terminator )
       break;

    if( i < sizeof( mapBuffer ) - 1 )
      mapBuffer[ i++ ] = ReadBuffer;
  }
  mapBuffer[ i ] = '\0';

  return atoi( mapBuffer );
}

void get_map(void)
{
  uint16_t torque;
  uint16_t rev;

  while( Fread( &ReadBuffer, 1 ) != -1 ) {

    if( ReadBuffer == '[' ) {
     
      torque = get_number( ';' );
      rev    = get_number( ']' );

      if( map_index < MAP_SIZE ) {
        map[map_index][TORQUE] = torque;
        map[map_index][REV]    = rev;

        map_index++;
      }
    }
  }
}

[sebastians (Gast)]

Du könntest auch einen ganz anderen Ansatz nehmen: Gib dem 
Microcontroller eine Binärdatei. Mach die Umwandlung von Text- in 
Binärdatei offline. Dann kannst du eine Skriptsprache deiner Wahl 
nehmen, mit der sich einfach Texte parsen lassen. Syntaxfehler erkennst 
du schon beim konvertieren, brauchst dich im Microcontroller nicht mehr 
drum kümmern. Und der Code im Microcontroller wird kleiner+schneller.

[Georg (Gast)]

@Karl heinz Buchegger
Danke für die vielen Tipps, hab es so umgesetzt. Werde noch versuchen 
möglichst viele Fehleingaben abzufangen.

@sebastians
Danke für den Vorschlag. Finde es aber recht gut, einfach in eine 
Textdatei schreiben zu können.


Gruß
Georg

PS. Wie kommt es eigentlich zu der Kompiler-Warnung:
warning: pointer targets in passing argument 1 of 'Fread' differ in 
signedness

[Karl H. (kbuchegg)]

Georg schrieb:
> PS. Wie kommt es eigentlich zu der Kompiler-Warnung:
> warning: pointer targets in passing argument 1 of 'Fread' differ in
> signedness

Schau dir den Protoypen von Fread an.
Höchst wahrscheinlich will Fread einen Pointer auf einen unsigned char 
habe, du übergibst ihm aber die Adresse eines char.

[Georg (Gast)]

Ja, natürlich, so ist es.

Warum eigentlich

  while( Fread( &ReadBuffer, 1 ) != -1 ) {

sollte es nicht

  while( Fread( &ReadBuffer, 1 ) != 0 ) {

heißen?

Zumindest bleibt das Prog. in der Funktion get_map hängen.

[Karl H. (kbuchegg)]

Mein Fehler
Durch deine dichte Schreibweise

    byte_count=Fread(ReadBuffer,1);
      if(byte_count!=1)break;

hab ich ein Minus gesehen, wo keines ist

Bau dir doch Leerzeichen ein. Das kostet dir nichts! Seit du 6 Jahre alt 
bist, hast du dein Gehirm darauf trainiert, dass beim Lesen dort wo ein 
Leerraum ist, ein neues Wort anfängt. Benutze dieses Training! Dann kann 
man auch Programmtext 'im ganzen Satz' lesen ohne erst mit den Augen vor 
und rück durch den Buchstabensalat zu müssen um rauszufinden wo etwas 
anfängt und wo es wieder aufhört.

    byte_count = Fread( ReadBuffer, 1 );
      if( byte_count != 1 )
        break;

Die Absicht ist daher:
Aus der Schleife raus, wenn Fread nicht 1 Zeichen lesen konnte. Die 
Schleife soll also laufen, solange Fread 1 Zeichen lesen kann

    while( Fread( &ReadBuffer, 1 ) == 1 ) {

(kommt an anderen Stellen auch noch vor)

[Georg (Gast)]

Alles klar. Werde in Zukunft versuchen, übersichtlicher zu 
programmieren.

Noch mal zu der signed-warnung.

Ich benutze die Funktionen atoi() und Fread() ja jeweils mit der 
Variablen ReadBuffer. Fread() möchte einen unsigned char-Pointer und 
atoi einen char-Pointer.

Ich bekommen also bei einem der Funktionsaufrufe immer die 
signed-Warning.

Wie bekomme ich die nun weg?

[Karl H. (kbuchegg)]

Georg schrieb:

> Wie bekomme ich die nun weg?

Die kannst du an dieser Stelle getrost wegcasten. Nach dem Motto: Wir 
wissen das das passt, also geben wir der Funktion was sie haben will.

 while( Fread( (unsigned char*)&ReadBuffer, 1 ) == 1 ) {
   ...