Текстовые файлы на С

[Raptor]

Написал две функции. Первая записывает в файл, вторая считывает и выводит на экран.

Но почемуто когда открываешь файл, независимо от того - в ручную или через прогу - вместо информациия всякаячушь типа
"  "@јT &@ "
или 0;

Мож кто-нибудь знает в чем дело.
Заранее спасибо!

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

struct rec
{
  char name;
  int tries;
double ti;
  int punkti;
	 };
void zapis_v_fail(char Name, int tries, double ti, int punkti)
{
struct rec Info;
FILE *f;
f=fopen("rez.dat","a");
Info.name=Name;
Info.tries=tries;
Info.punkti=punkti;
Info.ti=ti;
fwrite(&Info,sizeof(struct rec),1,f);
fclose(f);
}

fileopen()
{
int izmers,i;
char neim;
FILE *f;
	 struct rec Info;
 f=fopen("rez.dat","r");
	 if (!f)
{
	gotoxy(20,17);
	textcolor(RED);
	cprintf("FAILU NEVAR NOLASIT");
	textcolor(WHITE);
	getch();
	goto END;
	}
izmers=sizeof(struct rec);
fseek(f,0,SEEK_SET);
i=0;
gotoxy(5,10);  // KUR VAJAG IZVADIT TABULAS VIRSRAXTUS
printf("Vaards        Meg skaits       Laiks      Punkti");
while ( fread(&Info,izmers,1,f) == 1 ) {
gotoxy(5,11+i);
printf("%s    %d     %d    %i", Info.name, Info.tries, Info.ti, Info.punkti);
i++;
}
fclose(f);
getch();
END:

}

Здесь double ti - переменная несущая значение difftime(second,first);

[B_N]

Потому, что размер поля "name" в структуре - 1 байт (и в определении zapis_v_fail - тоже). Насколько я понимаю, там должна быть строка(судя по "printf("%s.......", тогда нужно хотя бы объявлять массив.

[puporev]

Чтобы файл можно было читать "глазами", он должен быть текстовым, т. е. с расширением *txt. Все прочие файлы читаются только программой.

[Stilet]

он должен быть текстовым, т. е. с расширением *txt.

Я наверное немного поправлю: не просто с расширением, а еще скорее всего числовые переменные записываются как набор байт т.е. не особо читаемый формат.

[Raptor]

Спасибо всем за помощь, попробую исправить.

[Raptor]

С первой разобрался.
Вот,если кого интересует, исправленный вар. 1 функции.

#include<stdio.h>
#include<conio.h>
#include<stdlib.h>
struct rec
{
  char name[32];
  int tries;
	double ti;
  int punkti;
	 }Info;


void zapis_v_fail(char Name[32], int *tries, double ti, int punkti)
{getch();
FILE *f;
f=fopen("2.txt","a");
//Name="Padla";
Info.tries=*tries;
Info.punkti=punkti;
Info.ti=ti;
printf("\n");
gotoxy(5,10);

fprintf(f, "%10s      %5i      %5.0f      %5d\n", Name, Info.tries, Info.ti, Info.punkti);
fclose(f);
}

[B_N]

Raptor, учтите только, что так делать - очень плохой тон и потенциальная опасность стековых эксплойтов. Лучше всё-таки через указатели делать, через старый добрый LPSTR (ну или PCHAR, unsigned char*, как больше нравится)

[Raptor]

А можно чуть-чуть подробнне? Спасибо.
Окончательный вариант:

#include<stdio.h>
#include<conio.h>
#include<stdlib.h>
struct rec
{
  char name[32];
  int tries;
	double ti;
  int punkti;
	 }Info;

void zapis_v_fail(int *tries, double ti, int punkti)
{gotoxy(25,21);
printf("Vase imja: ");
gotoxy(39,21);
scanf("%s", Info.name);
FILE *f;
f=fopen("2.txt","a");
Info.tries=*tries;
Info.punkti=punkti;
Info.ti=ti;
printf("\n");
gotoxy(5,10);

fprintf(f, "%10s      %5i      %5.0f      %5d\n", Info.name, Info.tries, Info.ti, Info.punkti);
gotoxy(1,25);
textbackground(GREEN);
cprintf("     Information saved     ");
fclose(f);
}

[B_N]

Постараюсь поподробнее. Дело в том, что когда Вы определяете вектора фиксированной длины прямо в стуктуре, есть опасность забыть 1) и поместить такую структуру в стеке в качестве локальной переменной, 2) забыть перед копированием в такую "строку" сравнить размеры источника и приемника. Отсюда вытекает опасность, что Вам в функцию передадут строку длиннее ожидаемой и смогут "перебить" адрес возврата из функции. Это и есть классический стековый эксплойт - страшно опасная вещь, если не принимать меры по предотвращению. И самые эффективные меры, это прежде всего проверять размер параметра. В крайнем варианте я не видел у Вас кода, заполняющего структуру снружи функции, но конструкция "scanf("%s", Info.name);" выгядит очень своеобразно , уж извиняйте, все таки быстродействие тоже немаловажный фактор, и кроме того это опасно: scanf не знает размера буфера и будет забивать его пока \0 не попадется.

[Raptor]

Тоесть, Вы советуете использовать здесь вместо scanf другую функцию типа gets?