[ANSI C] Obsługa plików

[szasza 0]

Hi,

 

napisałem program w ANSI C, który ma za zadanie utworzyć plik, zapisać do niego dane z klawiatury oraz odczytać jego zawartość.

 

#include <stdio.h>
#include <stdlib.h>
void tworz(FILE* f)
{
   char nazwa[256];
   printf("Podaj nazwe pliku do utworzenia: >\n");
   scanf("%s", nazwa);
   f = fopen(nazwa, "w");
   fclose(f);
}

int zapis(FILE* f)
{
   char tekst[1024];
   printf("Podaj tekst do zapisu:> \n");
   scanf("%s", tekst);
  if ((f=fopen(tekst, "w"))==NULL)
  {
       printf ("Nie mogę otworzyć pliku  do zapisu!\n");
       return 0;
   }
   fprintf (f, "%s", tekst);
   fclose (f);
}
void odczyt(FILE* f)
{
   char nazwa[256];
   char ch;
   printf("Podaj nazwe pliku: >\n");
   scanf("%s", nazwa);
   f = fopen(nazwa, "r");

   ch = getc(f);
   while(ch != EOF)
   {
       putchar(ch);
       ch = getc(f);
   }
}
main()
{
   FILE* plik;

   tworz(plik);
   zapis(plik);
   odczyt(plik);

   getchar();
   getchar();
}

Po wywołani funkcji odczyt otrzymuję błąd o naruszeni pamięci.

Proszę o pomoc w naprawie tego kodu, dziękuję i pozdrawiam!

[BlackBishop 1]

int zapis(FILE* f)
{
   char tekst[1024];
   printf("Podaj tekst do zapisu:> \n");
   scanf("%s", tekst);
  if ((f=fopen(tekst, "w"))==NULL)   <-------------- otwierasz do zapisu plik o nazwie którą zapisujesz jako tekst a nie plik z 'f' 
                                                     ani też z podanj nazwy
  {
       printf ("Nie mogę otworzyć pliku  do zapisu!\n");
       return 0;
   }
   fprintf (f, "%s", tekst);
   fclose (f);
}
}

 

dwa to bez sensu podajesz FILE jako param jak go nie wykorzystujesz

[szasza 0]

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

char* tworz(FILE* f)
{
   char nazwa[256];
   printf("Podaj nazwe pliku do utworzenia: >\n");
   scanf("%s", nazwa);
   f = fopen(nazwa, "w");
   fclose(f);
   return nazwa;
}

int zapis(FILE* f)
{
   char tekst[9600];
   char nazwa[256];
   printf("Podaj nazwe pliku: >\n");
   scanf("%s", &nazwa);
   printf("Podaj tekst do zapisu:> \n");
   scanf("%s", tekst);
   f = fopen(nazwa, "w");
   fprintf (f, "%s", tekst);
}
void odczyt( char* s)
{
   FILE* p;
   char buf[128];

   p = fopen(s, "r");
   while(fgets(buf, 50, p))
   {
       printf("%s", buf);
   }
   fclose(p);
}

main()
{
   FILE* plik;

   char* nazwa = tworz(plik);
   zapis(plik);
   odczyt(nazwa);

   getchar();
   getchar();
}

Nieco zmodyfikowałem ten kod. Jednak nadal po zapisie otrzymuję błąd o naruszeniu pamięci. Masz może jakiś pomysł gdzie znajduje się błąd?

 

Pozdrawiam

[BlackBishop 1]

bo nie czytasz warringów...

 

c.cpp: In function ‘char* tworz(FILE*)’:
c.cpp:6: warning: address of local variable ‘nazwa’ returned

 

zmień to na

char* tworz(FILE* f)
{
   char *nazwa = new char[256];
   printf("Podaj nazwe pliku do utworzenia: >\n");
   scanf("%s", nazwa);
   f = fopen(nazwa, "w");
   fclose(f);
   return nazwa;
}

 

i dodatkowo nie sprawdzasz błędów otwarcia pliku

[cube0 0]

   char *nazwa = new char[256];

 

 

W C nie ma operatora new tylko funkcje malloc() i calloc()

 

   char *nazwa = (char *)malloc(256);