Добавление структуры в файл

@Asdf · Регистрация: 18.03.2012

Author24 — интернет-сервис помощи студентам

Вечер добрый! В задании сказано, реализовать просмотров содержания файла(т.е. вывод всех структур), также структуры, удаление и подсчёт общего количества книг. При написании кода, столкнулся с проблемой добавлении структуры в файл, при компиляции позволяет ввести автора, и название книги, но в сам файл этого не добавляет. Подскажите, как мне решить проблему?

C++

#include <iostream>
#include <windows.h>
#include <conio.h>
 
struct BOOK
{
    char Author[50];
    char Name[50];
    int Quantity;
};
 
//struct BOOK edit[3];
//int N = 3;        Возможна проблема здесь, не уверен.
//HANDLE *hFile;
 
 
 
 
void add();
void del();
void equal();
void prosmotr();
 
using namespace std;
 
void enter()
{
 
for(int i = 0; i < N; i++)
{
  printf("Vvedite avtora knigi:\n");
  scanf("%s", &edit[i].Author);
}
 
for(int k = 0; k < N; k++)
{
  printf("Vvedite nazvanie knigi:\n");
  scanf("%s", &edit[k].Name);
}
WriteFile(hFile, &edit, sizeof(edit), NULL, NULL);
}
 
void main()
{
    HANDLE hFile;
    hFile = CreateFile("book.dat",GENERIC_WRITE,0,NULL,OPEN_ALWAYS,FILE_ATTRIBUTE_NORMAL,NULL);
    if (hFile!=INVALID_HANDLE_VALUE)
    {
        cout<<"book.dat "<<GetLastError()<<endl;
 
        FILETIME CreationTime, LastAccessTime, LastWriteTime;
        GetFileTime(hFile, &CreationTime, &LastAccessTime, &LastWriteTime);
        cout<<CreationTime.dwHighDateTime<<endl;
 
        SYSTEMTIME SystemTime;
        FileTimeToSystemTime(&CreationTime, &SystemTime);
        cout<<SystemTime.wDay<<'.'<<SystemTime.wMonth<<'.'<<SystemTime.wYear<<endl;
        cout<<SystemTime.wHour<<'.'<<SystemTime.wMinute<<'.'<<SystemTime.wSecond<<endl;
 
        BY_HANDLE_FILE_INFORMATION FileInformation;
        GetFileInformationByHandle(hFile, &FileInformation ); 
        FileTimeToSystemTime( &FileInformation.ftLastAccessTime ,&SystemTime  );
        cout <<SystemTime.wDayOfWeek <<endl;
    }
    if (hFile = CreateFile("book.dat",GENERIC_WRITE,0,NULL,OPEN_EXISTING,FILE_ATTRIBUTE_NORMAL,NULL)) 
    {
        enter();
    }
    else
    {
        ReadFile(hFile, &edit, sizeof(edit), NULL, NULL);
    }
 
    CloseHandle(hFile);
    cout<<"Press Any Key To Exit..."<<endl;
    getch();
}

@castaway · 25.11.2012, 01:19

Что-то ты начудил. Где у тебя определена переменная hFile, к которой в 41-й строке обращается функция WriteFile ?

@Asdf · 25.11.2012, 03:03 **[ТС]**

В 15 строке, просто скрыл коментами, чтобы указать на то место, где предпологаемо ошибка прячется

@castaway · 25.11.2012, 10:23

Сообщение от Asdf

В 15 строке

Ну тогда не понятно почему ты определил его как указатель.
А в 46-й строке у тебя что?
И почему ты два раза открываешь файл (47-я и 66-я) ?

@Asdf · 25.11.2012, 16:02 **[ТС]**

Забыл убрать, т.к. он там и не нужен, просто пробовал. В 46 тож можно убрать, а в 47 и 66, мне надо, чтобы сначала файл создался, а если он есть, то выдал его имя и временные атрибуты, а там где if, я только его открываю, чтобы записать в файл, OpenFile читал, что уже не используется. Или уже через 47 файл у меня уже открытый получается?

@castaway · 25.11.2012, 16:05

Я ничего не понял. Покажи исправленный код.

@Asdf · 25.11.2012, 16:08 **[ТС]**

Вот.

C++

#include <iostream>
#include <windows.h>
#include <conio.h>
 
struct BOOK
{
    char Author[50];
    char Name[50];
    int Quantity;
};
 
struct BOOK air[3];
int N = 3;
HANDLE hFile;
 
 
 
void add();
void del();
void equal();
void prosmotr();
 
using namespace std;
 
void enter()
{
 
for(int i = 0; i < N; i++)
{
  printf("Vvedite avtora knigi:\n");
  scanf("%s", &air[i].Author);
}
 
for(int k = 0; k < N; k++)
{
  printf("Vvedite nazvanie knigi:\n");
  scanf("%s", &air[k].Name);
}
WriteFile(hFile, &air, sizeof(air), NULL, NULL);
}
 
void main()
{
    //HANDLE hFile;
    hFile = CreateFile("book.dat",GENERIC_WRITE,0,NULL,OPEN_ALWAYS,FILE_ATTRIBUTE_NORMAL,NULL);
    if (hFile!=INVALID_HANDLE_VALUE)
    {
        cout<<"book.dat "<<GetLastError()<<endl;
        FILETIME CreationTime, LastAccessTime, LastWriteTime;
        GetFileTime(hFile, &CreationTime, &LastAccessTime, &LastWriteTime);
        cout<<CreationTime.dwHighDateTime<<endl;
        SYSTEMTIME SystemTime;
        FileTimeToSystemTime(&CreationTime, &SystemTime);
        cout<<SystemTime.wDay<<'.'<<SystemTime.wMonth<<'.'<<SystemTime.wYear<<endl;
        cout<<SystemTime.wHour<<'.'<<SystemTime.wMinute<<'.'<<SystemTime.wSecond<<endl;
        BY_HANDLE_FILE_INFORMATION FileInformation;
        GetFileInformationByHandle(hFile, &FileInformation ); 
        FileTimeToSystemTime( &FileInformation.ftLastAccessTime ,&SystemTime  );
        cout <<SystemTime.wDayOfWeek <<endl;
    }
    if (hFile = CreateFile("book.dat",GENERIC_WRITE,0,NULL,OPEN_EXISTING,FILE_ATTRIBUTE_NORMAL,NULL)) 
    {
        enter();
    }
    else
    {
        ReadFile(hFile, &air, sizeof(air), NULL, NULL);
    }
 
    CloseHandle(hFile);
    cout<<"Press Any Key To Exit..."<<endl;
    getch();
}

@castaway · 25.11.2012, 16:16

Сообщение от Asdf

Или уже через 47 файл у меня уже открытый получается?

Конечно.
Закрой его между 60-й и 61-й строкой (последний код).

@Asdf · 25.11.2012, 16:23 **[ТС]**

Закрыл. Получается, когда в if, я открываю файл, то можно так и оставить в CreateFile(OPEN_EXISTING) или, что то другое ставить?

@castaway · 25.11.2012, 16:36

Можно так и оставить. Первый CreateFile гарантирует то что файл существует.

@Asdf · 25.11.2012, 16:38 **[ТС]**

Но всёравно ничего не записывается в файл. ( Где я накололся?

@castaway · 25.11.2012, 16:47

WriteFile предпоследний параметр не может быть равен NULL, если последний NULL.
Сделай так:

C

1
2
3

DWORD dwWritten;
...
WriteFile( hFile, &air, sizeof( air ), &dwWritten, NULL );

@Asdf · 25.11.2012, 16:54 **[ТС]**

dwWritten это то, сколько байтов всего записано?

@castaway · 25.11.2012, 16:55

В 32-й и 38-й строках знак амперсанда не нужен.

Добавлено через 35 секунд

Сообщение от Asdf

dwWritten это то, сколько байтов всего записано?

Да, сколько записалось. Её можно не использовать, но в функцию передать надо.

@Asdf · 25.11.2012, 17:02 **[ТС]**

Т.е. ты имел ввиду 33, 39?

Добавлено через 57 секунд
Понятно.

Добавлено через 4 минуты
Получается с ReadFile в 68 строке тоже нельзя оставить в предпоследнем параметре NULL?

@castaway · 25.11.2012, 17:06

Сообщение от Asdf

Т.е. ты имел ввиду 33, 39?

Нет, 32 и 38 из 7-го поста.

Сообщение от Asdf

Получается с ReadFile в 68 строке тоже нельзя оставить в предпоследнем параметре NULL?

Да, там такая же ситуация.

@Asdf · 25.11.2012, 17:09 **[ТС]**

А dwWritten можно оставить не давая ему никаких значений? Просто DWORD dwWritten?

@castaway · 25.11.2012, 17:12

Да, можно.

@Asdf · 25.11.2012, 17:37 **[ТС]**

Так, пока что всёравно файл пустой

Добавлено через 16 минут
То, что сейчас есть, но не записывается ничего в файл.

C++

#include <iostream>
#include <windows.h>
#include <conio.h>
 
struct BOOK
{
    char Author[50];
    char Name[50];
    int Quantity;
};
 
struct BOOK air[3];
int N = 3;
HANDLE hFile;
 
 
 
void add();
void del();
void equal();
void prosmotr();
 
using namespace std;
 
void add()
{
DWORD dwWritten;
 
for(int i = 0; i < N; i++)
{
  printf("Vvedite avtora knigi:\n");
  scanf("%s", air[i].Author);
}
 
for(int k = 0; k < N; k++)
{
  printf("Vvedite nazvanie knigi:\n");
  scanf("%s", air[k].Name);
}
WriteFile(hFile, &air, sizeof(air), &dwWritten, NULL);
}
 
void main()
{
    //HANDLE hFile;
    hFile = CreateFile("book.dat",GENERIC_WRITE,0,NULL,OPEN_ALWAYS,FILE_ATTRIBUTE_NORMAL,NULL);
    if (hFile!=INVALID_HANDLE_VALUE)
    {
        cout<<"book.dat "<<GetLastError()<<endl;
        FILETIME CreationTime, LastAccessTime, LastWriteTime;
        GetFileTime(hFile, &CreationTime, &LastAccessTime, &LastWriteTime);
        cout<<CreationTime.dwHighDateTime<<endl;
        SYSTEMTIME SystemTime;
        FileTimeToSystemTime(&CreationTime, &SystemTime);
        cout<<SystemTime.wDay<<'.'<<SystemTime.wMonth<<'.'<<SystemTime.wYear<<endl;
        cout<<SystemTime.wHour<<'.'<<SystemTime.wMinute<<'.'<<SystemTime.wSecond<<endl;
        BY_HANDLE_FILE_INFORMATION FileInformation;
        GetFileInformationByHandle(hFile, &FileInformation ); 
        FileTimeToSystemTime( &FileInformation.ftLastAccessTime ,&SystemTime  );
        cout <<SystemTime.wDayOfWeek <<endl;
        CloseHandle(hFile);
    }
    if (hFile = CreateFile("book.dat",GENERIC_WRITE,0,NULL,OPEN_EXISTING,FILE_ATTRIBUTE_NORMAL,NULL)) 
    {
        add();
    }
    else
    {
        DWORD dwReading;
        ReadFile(hFile, &air, sizeof(air), &dwReading, NULL);
    }
 
    CloseHandle(hFile);
    cout<<"Press Any Key To Exit..."<<endl;
    getch();
}

@castaway · 25.11.2012, 17:50

Откомпилировал ничего не изменяя. Все записывается.

@Asdf 1 / 1 / 0 Регистрация: 18.03.2012 Сообщений: 110
	25.11.2012, 16:54 [ТС]	13
	dwWritten это то, сколько байтов всего записано? 0

@castaway 4985 / 3092 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	25.11.2012, 16:55	14
	В 32-й и 38-й строках знак амперсанда не нужен. Добавлено через 35 секунд Сообщение от Asdf dwWritten это то, сколько байтов всего записано? Да, сколько записалось. Её можно не использовать, но в функцию передать надо. 0

@castaway 4985 / 3092 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	25.11.2012, 01:19	2
	Что-то ты начудил. Где у тебя определена переменная hFile, к которой в 41-й строке обращается функция WriteFile ? 0

@Asdf 1 / 1 / 0 Регистрация: 18.03.2012 Сообщений: 110
	25.11.2012, 03:03 [ТС]	3
	В 15 строке, просто скрыл коментами, чтобы указать на то место, где предпологаемо ошибка прячется 0

@castaway 4985 / 3092 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	25.11.2012, 10:23	4
	Сообщение от Asdf В 15 строке Ну тогда не понятно почему ты определил его как указатель. А в 46-й строке у тебя что? И почему ты два раза открываешь файл (47-я и 66-я) ? 0

@Asdf 1 / 1 / 0 Регистрация: 18.03.2012 Сообщений: 110
	25.11.2012, 16:02 [ТС]	5
	Забыл убрать, т.к. он там и не нужен, просто пробовал. В 46 тож можно убрать, а в 47 и 66, мне надо, чтобы сначала файл создался, а если он есть, то выдал его имя и временные атрибуты, а там где if, я только его открываю, чтобы записать в файл, OpenFile читал, что уже не используется. Или уже через 47 файл у меня уже открытый получается? 0

@castaway 4985 / 3092 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	25.11.2012, 16:05	6
	Я ничего не понял. Покажи исправленный код. 0

@castaway 4985 / 3092 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	25.11.2012, 16:16	8
	Сообщение от Asdf Или уже через 47 файл у меня уже открытый получается? Конечно. Закрой его между 60-й и 61-й строкой (последний код). 0

@Asdf 1 / 1 / 0 Регистрация: 18.03.2012 Сообщений: 110
	25.11.2012, 16:23 [ТС]	9
	Закрыл. Получается, когда в if, я открываю файл, то можно так и оставить в CreateFile(OPEN_EXISTING) или, что то другое ставить? 0

@castaway 4985 / 3092 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	25.11.2012, 16:36	10
	Можно так и оставить. Первый CreateFile гарантирует то что файл существует. 0

@Asdf 1 / 1 / 0 Регистрация: 18.03.2012 Сообщений: 110
	25.11.2012, 16:38 [ТС]	11
	Но всёравно ничего не записывается в файл. ( Где я накололся? 0

@Asdf 1 / 1 / 0 Регистрация: 18.03.2012 Сообщений: 110
	25.11.2012, 17:02 [ТС]	15
	Т.е. ты имел ввиду 33, 39? Добавлено через 57 секунд Понятно. Добавлено через 4 минуты Получается с ReadFile в 68 строке тоже нельзя оставить в предпоследнем параметре NULL? 0

@castaway 4985 / 3092 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	25.11.2012, 17:06	16
	Сообщение от Asdf Т.е. ты имел ввиду 33, 39? Нет, 32 и 38 из 7-го поста. Сообщение от Asdf Получается с ReadFile в 68 строке тоже нельзя оставить в предпоследнем параметре NULL? Да, там такая же ситуация. 0

@Asdf 1 / 1 / 0 Регистрация: 18.03.2012 Сообщений: 110
	25.11.2012, 17:09 [ТС]	17
	А dwWritten можно оставить не давая ему никаких значений? Просто DWORD dwWritten? 0

@castaway 4985 / 3092 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	25.11.2012, 17:12	18
	Да, можно. 0

@castaway 4985 / 3092 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	25.11.2012, 17:50	20
	Откомпилировал ничего не изменяя. Все записывается. 0