C Probleme beim öffnen einer *.txt Datei.

[Marcel S. (marcel_s)]

Hallo

ich hoffe die C-Speziallisten können mir bei folgendem
Probleme helfen.

Ich möchte double Zahlen aus der Textdatei m.txt auslesen. Die Datei
möchte ich mit der Funktion MessdatenLesen(datei, messwA); öffnen.


Wenn ich das versuche kommt diese Fehlermeldung:

Unbehandelte Ausnahme bei 0x6ca060a6 in messdaten.exe: 0xC0000005:
Zugriffsverletzung beim Schreiben an Position 0x00000000.

Ich vermute einen Zeigerfehler hab aber schon in einigen Büchern die
Kapitel über Zeiger und Arrays gelesen und nichts gefunden was mir
hilft.

Dateien sind hochgeladen



Vielen Dank



Marcel

[Peter II (Gast)]

while(fscanf_s(mDatei,"%f",mW)!=EOF)

mW muss als Adresse übergeben werden  -> @mW

[Peter II (Gast)]

so weiter:

du speicherst deine Daten überhaupt nicht im Array.

> if((messwA[ret]<=OBEN) && (messwA[ret]>=UNTEN))
ret ist hier die falsche variabel, müsste i sein.

 for(i=0; i<ret; ++i)
  {
    if((messwA[ret]<=OBEN) && (messwA[ret]>=UNTEN))
    {
      ++n;
    }
  }
  retA=i/(ret/100);
  return retA;

wozu die schleife wenn du n überhaupt nicht verwendest?

[Um Gottes Willen (Gast)]

Heiliger Strohsack....

[Marcel S. (marcel_s)]

So soll die Schleife aussehen.

double Auswertung(double messwA[], int ret)
{
  double retA;

  int i, n=0;
  for(i=0; i<ret; ++i)
  {
    if((messwA[ret]<=OBEN) && (messwA[ret]>=UNTEN))
    {
      ++n;
    }
  }
  retA=(n/ret)*100;
  return retA;
}

danke für den Hinweis. Leider sind meine Programmierfähigkeiten noch 
sehr schlecht ich setze alles dran dazu zulernen.

[Joachim D. (Firma: JDCC) (scheppertreiber)]

Um Gottes Willen schrieb:
> Heiliger Strohsack....

Ja.

[Peter II (Gast)]

Marcel S. schrieb:
> So soll die Schleife aussehen.

dann ist sie aber immer noch falsch, lies mir mein Hinweis noch mal 
durch.

[Marcel S. (marcel_s)]

Hallo Peter,

vielen Dank für die hilfreichen Tipps.

Das Programm funktioniert jetzt. Ich würde mich sehr freuen wenn ihr mir 
sagt was ich noch verbessern kann. Ich möchte durch eure Kritik 
dazulernen.

Gruß

Marcel

[Uhu U. (uhu)]

Ich würde eine Kommandozeilen-Applikation draus machen, die den 
Dateinamen über die Kommandozeile erhält.

Wozu ist as system("PAUSE") am Ende gut?

[Marcel S. (marcel_s)]

Uhu Uhuhu schrieb:

> Wozu ist as system("PAUSE") am Ende gut?

Ohne wird das DOS Fenster in Windows immer geschlossen, bevor ich die 
Ausgabe erkennen kann. Gibt es da eine elegantere Lösung?

[Uhu U. (uhu)]

Marcel S. schrieb:
> Gibt es da eine elegantere Lösung?

Wenn du das Kommandozeilenfenster zuerst öffnest und dort den Aufruf 
reinschreibst, bleibt es offen.

Dein Programm, so wie es jetzt ist, ist ein GUI-Programm ohne GUI.

[Karl H. (kbuchegg)]

Marcel S. schrieb:
> Hallo Peter,
>
> vielen Dank für die hilfreichen Tipps.
>
> Das Programm funktioniert jetzt. Ich würde mich sehr freuen wenn ihr mir
> sagt was ich noch verbessern kann. Ich möchte durch eure Kritik
> dazulernen.

ABgesehen von der GUI Diskussion:

gets ist eine "Funktion non grata". Also eine Funktion die du NIEMALS 
benutzen solltest. Immer fgets benutzen!

Du hast keine Kontrolle darüber, was dein Benutzer an der Tastatur alles 
aufführt. Der kann dir da eine Eingabe von

kkkkkkkkkkkkkkkkkkaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaallllllllllllllllllllllddddddddddddddddddddd

reinpfriemeln, welche gets versucht in dein Array zu stopfen. Was 
natürlich schief geht, weil dein Array nur 31 Elemente groß ist. D.h. 
gets, bzw. dein Benutzer schaffen es mit links, dass sich dein Programm 
selber die Variablen zerschiesst. gets kann dich davor nicht schützen. 
fgets kann es.

gets ist eine Funktion, die es so wie sie ist, nie in den C-Standard 
hätte schaffen sollen. Deine beste Option ist es, völlig zu vergessen 
bzw. zu verdrängen, dass diese Funktion überhaupt existiert. Streiche 
sie aus deinem 'Wortschatz'.

    fgets( datei, sizeof(datei), stdin );

[Peter II (Gast)]

Marcel S. schrieb:
> Ich möchte durch eure Kritik
> dazulernen.

> return (n*100)/anzahl;
ist noch fehlerhaft. Diese Rechnung wird mit int gemacht, damit werden 
hier niemals kommastellen rauskommen. Damit man der Return double keinen 
sinn.