[C] existiert die Zieldatei?

[myhonor]

Für alle, die das Programm nachcompilieren lassen wollen: Der Fehler ist nach der Eingabe von y/n. Die Datei wird zwar erstellt, es findet aber ein Memory-Leak statt, was nicht sein darf.

#include <stdio.h>
#include <stdlib.h>

int main(int argc, char **argv)
{
	FILE *filename, *oldfile;
	char *name = NULL;
	char answer;

	name = malloc(30);
	printf("file? ");
	scanf("%s", name);

	if( (oldfile = fopen(name, "r")) != NULL)
	{
		printf("file %s exists and will be replaced. "
			"Do you wand to proceed? (y/n): ", name);
		scanf("%s", &answer);
	}

	if( (answer == 'y') || (oldfile == NULL))
	{
		filename = fopen(name, "w+");
		fprintf(filename, "Ich fange mit einem Header an!\nCool");
		fclose(filename);
	}

	free(name);
	return 0;
}

 

[Kamikazecarsten]

In Zeile 14

if( (oldfile = fopen(name, "r")) != NULL)

öffnest du die Datei mit fopen.
Ist die Datei existent und du bestätigst mit "y" das Überschreiben öffnest du die Datei in Zeile 23

filename = fopen(name, "w+");

ein weiteres mal ohne vorher den Filehandle geschlossen zu haben.

 

[vfl_freak]

Das "fclose" für "oldfile" fehlt übrigens, wie mir gerade auffällt .....

#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char **argv)
{
    FILE *filename, *oldfile;
    char *name = NULL;
    char answer;
 
    name = malloc(30);
    printf("file? ");
    scanf("%s", name);
 
    if( (oldfile = fopen(name, "r")) != NULL)
    {
        printf("file %s exists and will be replaced. "
            "Do you wand to proceed? (y/n): ", name);
        scanf("%s", &answer);
// !
    fclose(filename);
// !
    }
 
    if( (answer == 'y') || (oldfile == NULL))
    {
        filename = fopen(name, "w+");
        fprintf(filename, "Ich fange mit einem Header an!\nCool");
        fclose(filename);
    }
 
    free(name);
    return 0;
}

Gruß
Klaus

 

[myhonor]

ihr meint, weil oldfile kein fclose hat, wird der memory leak ausgelöst. Danke

 

[Kamikazecarsten]

Genau, so wie es Klaus in seinem Beispiel getan hat.

 

[myhonor]

OK, Problem gelöst, wenn die Datei existiert. Das nächste Problem ist, wenn die Datei noch nicht existiert (sie muss neu erstellt werden). Laut Valgrind ist der Fehler in der 2. If-Abfrage (answer == 'y') || (oldfile == NULL)

Ich hab jetzt die Bedingungen vertauscht (zuerst oldfile, dann answer). In beiden Fällen (existiert/existiert nicht) funktioniert es ohne Memory leak und ohne Errors (obwohl die Bedingung nicht erfüllt ist, wenn es keine Datei gibt, und trotzdem kein ML).

Danke an alle beteiligten. Ein großteil der Übung ist geschaft.

 

[vfl_freak]

Laut Valgrind ist der Fehler in der 2. If-Abfrage (answer == 'y') || (oldfile == NULL)
Ich hab jetzt die Bedingungen vertauscht (zuerst oldfile, dann answer).
In beiden Fällen (existiert/existiert nicht) funktioniert es ohne Memory leak und ohne Errors (obwohl die Bedingung nicht erfüllt ist, wenn es keine Datei gibt, und trotzdem kein ML).

Danke an alle beteiligten. Ein großteil der Übung ist geschaft

Äääh - nur mal so zur Info:
Deine Bedingung ist eine ODER-Verknüfung
D. h. die Bedingung ist wahr, wenn eine von beiden Teilbedingungen wahr
"A oder B" ist genau das Gleiche wie "B oder A" .......
Es ist nur die Frage, wie es der Compiler abhandelt!

Im Regelfall wird zuerst A geprüft und wenn A TRUE ist, dann B nicht mehr ! ! ! !
Lediglich wenn A FALSE ergibt, wird B geprüft ! ! !

Ergo: prüft Du erst, ob "oldfile == NULL" ist und es IST NULL, dann ist es völlig egal ob du ein 'y' oder sonstwas eingegeben hast ! ! ! ! !
Das solltest Du so leicht gemischten Bedingungen immer dabei beachten !

Gruß
Klaus