Aktuelles Datum in Datei speichern.

radiac

Mittlerweile habe ich herausgefunden, dass es nicht an fclose liegt. Das Programm stürzt beim Ende der function ab. Also wenn er wieder zurück in die Main soll. Dort kommt er jedoch nie an, zumindest werde keine Haltepunkte die direkt nach dem Funtkionsaufruf gesetzt sind erreicht.

Hat jemand eine Idee? Muss mann Strukturen usw. am Funktionsende auf NULL setzen?!

SeppJ

Erkär doch mal, wie fscanf funktioniert.

radiac

Das kann ich jetzt nicht ganz nachvollziehen

Mein fscanf schreibt wie gesagt in die mitgegebene Datei einen String mit Formatspezifikationen. Die Anzahl und Typen der Spezifikationen stimmen mit den mitgegebenen Parametern ein. Der Zeiger auf die Datei passt auch und in der Datei wird alles richtig geschrieben. Beim fclose kommt es ja auch nicht zu fehlern, wie ich mittlerweile festgestellt habe.

Oder übersehe ich da was? Bin wie gesagt ein absoluter Neuling in C.

radiac schrieb:

Mein fscanf schreibt wie gesagt in die mitgegebene Datei einen String mit Formatspezifikationen. Die Anzahl und Typen der Spezifikationen stimmen mit den mitgegebenen Parametern ein. Der Zeiger auf die Datei passt auch und in der Datei wird alles richtig geschrieben. Beim fclose kommt es ja auch nicht zu fehlern, wie ich mittlerweile festgestellt habe.

Oder übersehe ich da was? Bin wie gesagt ein absoluter Neuling in C.

Der Prototyp von fscanf sieht so aus: int fscanf ( FILE * stream, const char * format, ... );

Dabei stehen die ... für eine variable Anzahl von Parametern.

Den Typ und die Anzahl der Paramter ermittelt fscanf aus den Formatspecifieren in dem Formatstring. Für jeden Formatspecifier muss also auch Parameter angegeben sein.

Wieviel Formatspecifier hast du?

Vielleicht verstehst du dann auch die Warnung vom Compiler.
Bzw. schaltest diese ein und beachtest sie auch.

radiac

Soweit ich es überblicken kann stimmen Format und Anzahl der Parameter überein. Entdeckt du da bei mir einen Fehler?

Müsste dann nicht auch der Fehler bei der fprintf-Anweisung auftauchen? Oder hab ich hier einen Denkfehler?

€: Ich hab es übrigens schon mit einer einfach Anweisung fprintf("TEST"); probiert, da erhalte ich denselben Fehler. Ich vermute also, dass es nicht an dieser Anweisung liegt.

Wie gesagt, auch das flcose ist nicht der Fehler, wie zuerst von mir vermutet. Erst bei Ende der Funktion tritt der Fehler auf, unabhängig davon, welche Anweisungen ich nach fclose noch hinzufüge.

radiac schrieb:

while(fscanf(dat,"%s")!=EOF)

radiac schrieb:

Soweit ich es überblicken kann stimmen Format und Anzahl der Parameter überein.

Wo werden die Daten für das %s abgespeichert?

SeppJ

radiac schrieb:

Mein fscanf schreibt wie gesagt in die mitgegebene Datei einen String mit Formatspezifikationen.

Sicher?

Oder übersehe ich da was? Bin wie gesagt ein absoluter Neuling in C.

Deswegen gebe ich dir ja auch einen Hinweis auf den Fehler. Vielleicht solltest du weniger wild herum experimentieren und stattdessen mal den Hinweisen folgen, die dir gegeben werden? Experimentieren ist ja schön und gut, aber ich zitiere mal:

c.rackwitz schrieb:

Wenn du selber Code schreibst, musst du ihn auch verstehen. Code ist kein Haufen von wahllos zusammengeschmissenen Buchstaben und Zeichen, Code ist Logik pur. Du musst genau wissen, warum du wo und welches Zeichen setzt.

Wenn du bei fscanf nicht genau weißt, wie es funktioniert (oder dir jemand sagt, dass du es nicht weißt, obwohl du selber das anders siehst), dann wäre es vielleicht Zeit für ein bisschen RTFM. Hier findest du Referenzen zu C-Funktionen:
http://www.cplusplus.com/reference/clibrary/
http://en.cppreference.com/w/c

radiac

Ach, jetzt ich erstmal das ich hier ständig fscanf statt fprintf schreibe :-\ Sorry dafür.

Also das Problem tritt ja nicht bei der Funktion mit fscanf auf. Mir ist klar, das ich mit

fopen_s(&dat, datei,"r");

	while(fscanf(dat,"%s")!=EOF)
				i++;

	fclose(dat);

den ausgelesenen String nirgends speichere. Ich verwende diese Funktion nur dafür, um zu ermitteln wieviele Datensätze in der Datei gespeichert sind. Deswegen soll er einfach alle Zeilen durchgehen und eben i hochzählen. Das funktoniert auch problemlos, er beendet diese Hilfsfunktion ohne abzustürzen usw.

Mein Problem liegt wie gesagt hier:

[code="c"]void kunde_erfassen()
{
FILE *datei;
KUNDE neu_kunde;
struct tm datum;
time_t zeit;
int variante_datum;

//aktuelle Kundennummer holen
neu_kunde.k_id=akt_id(k_dat); ---> Das ist die obige Unterfunktion

printf("\nBitte Daten zum Kunden Nummer %d eingeben!", neu_kunde.k_id);

//Datum auswählen lassen
printf("\nAktuelles Datum fuer Vertragsbeginn? (1=ja, sonst manuelle Eingabe) ");scanf_s("%d", &variante_datum);

//aktuelles Datum als Vertragsdatum
if(variante_datum==1)
{
time(&zeit); //aktuelle Zeit
localtime_s(&datum, &zeit);
neu_kunde.vertrag_beginn.tm_year=datum.tm_year;
neu_kunde.vertrag_beginn.tm_mon=datum.tm_mon;
neu_kunde.vertrag_beginn.tm_mday=datum.tm_mday;
}
else //Datum manuell einlesen
{
fflush(stdin);
printf("\nBitte Tag eingeben (dd):");scanf_s("%d",&neu_kunde.vertrag_beginn.tm_mday);
printf("Bitte Monat eingeben (mm):");scanf_s("%d",&neu_kunde.vertrag_beginn.tm_mon);
printf("Bitte Jahr eingeben (yyyy):");scanf_s("%d",&neu_kunde.vertrag_beginn.tm_year);
}

//restliche Daten erfassen
printf("\nWeitere Kundendaten");
fflush(stdin);
printf("\nName:");gets_s(neu_kunde.name,sizeof(neu_kunde.name));
printf("Vorname:");gets_s(neu_kunde.vorname,sizeof(neu_kunde.vorname));
printf("Strasse:");gets_s(neu_kunde.anschrift.strasse,sizeof(neu_kunde.anschrift.strasse));
printf("Hausnummer:");gets_s(neu_kunde.anschrift.hausnummer,sizeof(neu_kunde.anschrift.hausnummer));
printf("PLZ:");gets_s(neu_kunde.anschrift.plz,sizeof(neu_kunde.anschrift.plz));
printf("Ort:");gets_s(neu_kunde.anschrift.ort,sizeof(neu_kunde.anschrift.ort));

//Datei öffnen, Kunden einschreiben, Datei schließen
fopen_s(&datei, k_dat,"a+");

fprintf(datei,"%d;%s;%s;%s;%s;%s;%s;%d.%d.%d;\n", neu_kunde.k_id, neu_kunde.name, neu_kunde.vorname, neu_kunde.anschrift.strasse, neu_kunde.anschrift.hausnummer, neu_kunde.anschrift.plz, neu_kunde.anschrift.ort, neu_kunde.vertrag_beginn.tm_mday, neu_kunde.vertrag_beginn.tm_mon+1, neu_kunde.vertrag_beginn.tm_year+1900);

fclose(datei);
printf("Test");
}[/code]

Die beiden letzten Anweisungen (fett) werden ausgeführt, auch das "Test" erscheint noch auf der Console. Aber danach bricht das Programm ab. Er kommt auch nicht bis zurück zur Main. Sollte das wirklich an der Unterfunktion oben liegen, in der ich ohne Parameter mitzugeben einfach mittels fscanf die Zeilen durchlaufe? Ich hab das mal mit nem char[90] Parameter probiert --> selber fehler.

radiac schrieb:

Mir ist klar, das ich mit ... (fscanf(dat,"%s") ...
den ausgelesenen String nirgends speichere.

Falsch. Der String wird irgendwo gespeichert. fscanf kann nur anhand der Formatspecifier wissen, wieviel Paramter da noch kommen. Und dann müssen die auch da stehen.
fscanf holst sich die Adresse vom Stack, auch wenn die nicht vom Funktionsaufruf sind. Dieses irgendwo kann bei deinen anderen Daten oder im Programmcode liegen.

Da ich nur [ code][ /code] sehe, bei zitieren aber der Code auftaucht, hier nochmal. Scheint ein Problem mit den [ b][ /b] Tags zu sein

radiac schrieb:

void kunde_erfassen()
{
	FILE *datei;
	KUNDE neu_kunde;
	struct tm datum;
	time_t zeit;
	int variante_datum;

	//aktuelle Kundennummer holen
	neu_kunde.k_id=akt_id(k_dat); // FETT---> Das ist die obige Unterfunktion
	
	printf("\nBitte Daten zum Kunden Nummer %d eingeben!", neu_kunde.k_id);

	//Datum auswählen lassen
	printf("\nAktuelles Datum fuer Vertragsbeginn? (1=ja, sonst manuelle Eingabe) ");scanf_s("%d", &variante_datum);

	//aktuelles Datum als Vertragsdatum
	if(variante_datum==1)
	{
		time(&zeit);    //aktuelle Zeit
		localtime_s(&datum, &zeit);
		neu_kunde.vertrag_beginn.tm_year=datum.tm_year;
		neu_kunde.vertrag_beginn.tm_mon=datum.tm_mon;
		neu_kunde.vertrag_beginn.tm_mday=datum.tm_mday;
	}
	else //Datum manuell einlesen
	{
		fflush(stdin);
		printf("\nBitte Tag eingeben (dd):");scanf_s("%d",&neu_kunde.vertrag_beginn.tm_mday);
		printf("Bitte Monat eingeben (mm):");scanf_s("%d",&neu_kunde.vertrag_beginn.tm_mon);
		printf("Bitte Jahr eingeben (yyyy):");scanf_s("%d",&neu_kunde.vertrag_beginn.tm_year);
	}

	//restliche Daten erfassen
	printf("\nWeitere Kundendaten");
	fflush(stdin);
	printf("\nName:");gets_s(neu_kunde.name,sizeof(neu_kunde.name));
	printf("Vorname:");gets_s(neu_kunde.vorname,sizeof(neu_kunde.vorname));
	printf("Strasse:");gets_s(neu_kunde.anschrift.strasse,sizeof(neu_kunde.anschrift.strasse));
	printf("Hausnummer:");gets_s(neu_kunde.anschrift.hausnummer,sizeof(neu_kunde.anschrift.hausnummer));
	printf("PLZ:");gets_s(neu_kunde.anschrift.plz,sizeof(neu_kunde.anschrift.plz));
	printf("Ort:");gets_s(neu_kunde.anschrift.ort,sizeof(neu_kunde.anschrift.ort));
	
	//Datei öffnen, Kunden einschreiben, Datei schließen
	fopen_s(&datei, k_dat,"a+");

	fprintf(datei,"%d;%s;%s;%s;%s;%s;%s;%d.%d.%d;\n", neu_kunde.k_id, neu_kunde.name, neu_kunde.vorname, neu_kunde.anschrift.strasse, neu_kunde.anschrift.hausnummer, neu_kunde.anschrift.plz, neu_kunde.anschrift.ort, neu_kunde.vertrag_beginn.tm_mday, neu_kunde.vertrag_beginn.tm_mon+1, neu_kunde.vertrag_beginn.tm_year+1900);

	fclose(datei);  //FETT
printf("Test");         //FETT
}

Entweder du machst mehr Debugausgaben mit aktuellen Werten und Zeigern oder du probierst mal den Debugger aus.

Grenze das Problem ein.
Weise neu_kunde.k_id einen festen Wert zu und verzichte auf akt_id(k_dat);
Kommentiere einzelne Zeilen oder Blöcke aus. Minimier den Code auf das nötigste, so dass das Problem gerade noch auftritt.

radiac

Hallo Dirk,

vielen Dank für deine ständige Hilfe. Ich hab schon versucht das Problem einzugrenzen, hab mir Zeiger usw. ausgeben lassen und die Struktur beim debuggen geprüft, sprich, ob alle Daten da ordentlich drin stehen. Ich komme einfach nicht auf die Ursache. Auch das fest zuweisen der k_id hat noch nix gebracht.

Da ich die Aufgabe bis zum 5.12 abgeben muss, werd ich den Fehler erstmal hinten anstellen. Ich verliere zuviel Zeit damit den zu beheben und am Ende verliere ich dadurch vielleicht nur einen Punkt. Ist vielleicht der falsche Ehrgeiz, aber ich muss noch 2 komplette Teilaufgaben implementieren, bei denen ich auch noch nicht so recht weiß wie ich das anstellen soll

Sollte ich die Ursache gefunden haben, gebe ich euch aber Bescheid!