do While schleife

[andy (Gast)]

Hallo,
Mit der do While schleife lese ich eine Datei von der SD-Card in einen 
puffer(site_puffer)ein.
Dieser ist aber nur 1500 byte gross, meine Datei aber grösser.

Meine überlegung ist erstmal count_a bis 1499 in den puffer zuschreiben 
und dann zusenden, dann count_a wieder auf null setzen und weiter in den 
puffer schreiben bis mein Ende zeichen kommt.

aber irgendwie haut das mit meiner if else abfrage nicht hin.

könnte mir da einer weiterhelfen

mfg

static char site_puffer[1500];

            do
             {
              res = f_read(&fsrc, &by,  1, &br);       // Daten lesen
         //site_puffer[count_a]=by;
              count_a++;
       
       //ist count_a = 1499 dann erstmal die daten senden
       if ((count_a == 1499))
        {
               PSOCK_SEND_STR(&s->sout,site_puffer);
         
         sprintf( read_write_buffer, "count_a:%d", count_a );
               DrawString(0,0,read_write_buffer,&Arial_16x25,WHITE,RED);            
        
         count_a = 0;
        }
         
       else
        {
          site_puffer[count_a]=by;
        }         
         
                
      }
              while ((by != '%'));

[Andreas S. (Firma: Schweigstill IT) (schweigstill)]

Und Du meinst, dass irgendjemand Lust hat, sich Deinen hingerotzten 
Quellcode näher anzuschauen, wenn nicht einmal die Einrückungen stimmen?

[andy (Gast)]

Andreas Schweigstill schrieb:
> Und Du meinst, dass irgendjemand Lust hat, sich Deinen
> hingerotzten
> Quellcode näher anzuschauen, wenn nicht einmal die Einrückungen stimmen?


ich habe Höflichst gefragt.
Und solch eine Aussage ist ja wohl unterstes Niveau
junge junge

[Andreas S. (Firma: Schweigstill IT) (schweigstill)]

andy schrieb:
> ich habe Höflichst gefragt.

In der deutschen Sprache wird eine Frage mit einem Fragezeichen 
abgeschlossen. Das hast Du nicht getan.

Und "aber irgendwie haut das mit meiner if else abfrage nicht hin." ist 
keine angemessene Fehlerbeschreibung.

[Daniel A. (daniel-a)]

Dieser code veruhrsacht alptraume!

Ungetestet:

static char site_puffer[1500];
count_a=0;
bool run=1;
while(run) {
  res = f_read(&fsrc, &by,  1, &br);
  if(by=='%'||res<=0)
    run = 0;
  else
    site_puffer[count_a++] = by;
  if(count_a >= sizeof(site_puffer)-1 || !run){
    site_puffer[count_a] = 0;
    PSOCK_SEND_STR(&s->sout,site_puffer);
    sprintf( read_write_buffer, "count_a:%d", count_a );
    DrawString(0,0,read_write_buffer,&Arial_16x25,WHITE,RED);            
    count_a = 0;
  }
}

PS: Hier kann es noch zu einem Bufferoverflow bei read_write_buffer 
kommen...

[Dirk B. (dirkb2)]

site_puffer[0] wird nie beschrieben
und wenn der Puffer voll ist, wird das vorher empfangene Zeichen auch 
nicht abgespeichert.

[Jim (Gast)]

hast Du mal in einem C Forum gefragt?
Da gibt es meist brauchbare Antworten oder im Roboternetz.de
Wenn Du glück hasst, weist Dich jemand auf Tippfehler hin :-) und auf 
unkorrektes Deutsch lol

[spontan (Gast)]

Hallo Andy, was verstehst du an dem copy&paste-code nicht?

Von dir ist er ja wohl nicht.

Also: Problem beschreiben, Fragen stellen -> Antworten erhalten.

[Klaus W. (mfgkw)]

andy schrieb:
> ich habe Höflichst gefragt.
> Und solch eine Aussage ist ja wohl unterstes Niveau
> junge junge

Er hat aber Recht.
Vielleicht gefällt dir der Ton nicht, aber inhaltlich gibt es nichts zu 
meckern - wenn dir jemand helfen soll, kann man wenigstens lesbaren 
Quelltext und eine ordentliche Beschreibung verlangen.

[foobar (Gast)]

Ich hab ja den Verdacht, dass count_a nen Byte ist ...

[Joe F. (easylife)]

a) sicher stellen, dass count_a mit 0 initialisiert ist
b) die von dir auskommentierte Stelle für "site_puffer[count_a] = by;" 
war schon richtig. Du willst ja den Buffer ab Index 0 füllen.
c) ich nehme stark an, du willst auch den Rest der Datei, der nach einem 
Vielfachen von 1499 bytes gelesen wurde abschicken -> "if ((count_a == 
1499) || (by == '%'))"
d) Ein String wird in C üblicherweise mit einem Nullbyte terminiert. 
Dies fehlt bei dir.

also so sollte es gehen:

static char site_puffer[1500];

count_a = 0;

do
{
  res = f_read(&fsrc, &by, 1, &br);       // Daten lesen
  site_puffer[count_a] = by;
  count_a++;

  // ist count_a = 1499 oder das Ende der Datei erreicht, Daten senden
  if ((count_a == 1499) || (by == '%'))
  {
    site_puffer[count_a] = 0; // terminate string
    
    PSOCK_SEND_STR(&s->sout, site_puffer);
    
    sprintf(read_write_buffer, "count_a:%d", count_a);
    DrawString(0, 0, read_write_buffer, &Arial_16x25, WHITE,RED);            
    
    count_a = 0;
  }
} while (by != '%');

[Joe F. (easylife)]

Und so läuft es vermutlich schneller:

static char site_buffer[1500];
char * delimiter;

do
{
  res = f_read(&fsrc, site_buffer, 1499, &br);       // Daten lesen

  site_buffer[br] = 0;  // kann sein, dass das überflüssig ist
  
  delimiter = strchr(site_buffer, '%');
  if (delimiter)
  {
    *delimiter = 0;      // truncate at delimiter position
    br = delimiter - site_buffer;  // calculate length of remaining bytes in buffer
  }
  PSOCK_SEND_STR(&s->sout, site_buffer);
  
  sprintf(read_write_buffer, "bytes read: %d\n", br);
  DrawString(0, 0, read_write_buffer, &Arial_16x25, WHITE,RED);            
} while ((br == 1499) && (delimiter == NULL));

[Joe F. (easylife)]

Steht das '%' eigentlich wirklich mitten im File, oder ist das das 
letzte Zeichen im File und du nimmst das als Hilfskonstruktion, weil du 
nicht wusstest, wie man das Ende des Files erkennt?
Wenn du das gesamte File übertragen willst, wirds natürlich einfacher. 
Dann kannst du dir die Suche nach dem '%' komplett sparen.