Функция чтения из файла на WinAPI

@Gelo123321 · Регистрация: 05.01.2012

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

Пытаюсь создать функцию на WinAPI для чтения данных с файла. Ниже приведен мой пример, но он не работает. И к тому же у моей функции есть один серьезный недостаток: как я понимаю будет возвращаться не вся информация, а только 1 строка, остальное будет теряться.

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
string readFromFile(string nameFile)
{
    HANDLE hFile;
    string buffer = " ";
    string pathFile = ".\\" + nameFile + ".txt";
    hFile = CreateFile(pathFile.c_str(),
                       GENERIC_READ,
                       0,
                       0,
                       OPEN_EXISTING,
                       0,
                       0);
    if (!hFile) {
        CloseHandle(hFile);
        return "-1";
        }
     else {
        for (;;) {
            DWORD dwBytesRead;
            if (!ReadFile(hFile,
                          &buffer,
                          sizeof(buffer),
                          &dwBytesRead,
                          0)) {
                CloseHandle(hFile);
                return "0";
            }
         if (dwBytesRead == 0) break;
        }
    }
    CloseHandle(hFile);
    return buffer;
}

Как переделать функцию так, что бы она возвращала всю информацию с файла, в каком направлении копать?

Убежденный · 15.11.2014, 14:17

C++
1
string pathFile = ".\\" + nameFile + ".txt";

Ненадежно. Лучше писать полный путь к файлу.

C++
1
2
3
4
if (!hFile) {
        CloseHandle(hFile);
        return "-1";
        }

Во-первых, CreateFile в случае ошибки возвращает INVALID_HANDLE_VALUE.
Расширенную информацию в этом случае можно извлечь с помощью GetLastError.
Во-вторых, возврат "-1" - это никуда не годится. Что, если в файле записано "-1" ?
Как клиент поймет, ошибка это или нет ?

C++
1
2
3
4
5
6
7
8
9
10
for (;;) {
            DWORD dwBytesRead;
            if (!ReadFile(hFile,
                          &buffer,
                          sizeof(buffer),
                          &dwBytesRead,
                          0)) {
                CloseHandle(hFile);
                return "0";
            }

"sizeof (buffer)" в данном случае эквивалентно "sizeof (string)".
А "&buffer" - адрес памяти, где хранится объект string, не строка.

Так что неудивительно, что этот код не работает.

@Gelo123321 · 15.11.2014, 14:40 **[ТС]**

Сообщение от Убежденный

C++
1
string pathFile = ".\\" + nameFile + ".txt";

Ненадежно. Лучше писать полный путь к файлу.

Хорошо, переделал!

Сообщение от Убежденный

C++
1
2
3
4
if (!hFile) {
 CloseHandle(hFile);
 return "-1";
 }

Во-первых, CreateFile в случае ошибки возвращает INVALID_HANDLE_VALUE.
Расширенную информацию в этом случае можно извлечь с помощью GetLastError.
Во-вторых, возврат "-1" - это никуда не годится. Что, если в файле записано "-1" ?
Как клиент поймет, ошибка это или нет ?

Спасибо, переделал (правда выдает 6Yh2<Yh2. Кажется, здесь что-то не так.):

C++
1
2
3
4
5
6
7
if (!hFile) {
        CloseHandle(hFile);
        strstream stream;
        stream << GetLastError();
        string error(stream.str());
        return error;
        }

Сообщение от Убежденный

"sizeof (buffer)" в данном случае эквивалентно "sizeof (string)".
А "&buffer" - адрес памяти, где хранится объект string, не строка.

Не понимаю, как переделать(

Убежденный · 15.11.2014, 14:45

Я обычно делаю как-то так:

C++
1
2
3
4
5
6
7
8
9
10
DWORD const BuffSize = ...; // Размер буфера.
vector<char> Buffer(BuffSize);
 
if (FALSE == ReadFile(hFile, &Buffer[0], BuffSize, &nBytes, NULL))
{
    THROW_EXC("ReadFile failed with last error 0x%.8lx.", GetLastError());
}
 
string Text;
Text.append(&Buffer[0], nBytes);

@Gelo123321 · 15.11.2014, 15:16 **[ТС]**

Убежденный, спасибо, переделал по вашему примеру, теперь ошибки нет, но все же выводится не совсем то, что надо. В файле записано '1111', а мне выводит 'ц'.

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
string readFromFile(string nameFile)
{
    HANDLE hFile;
    string error, text;
    DWORD const bufferSize = 1024;
    DWORD dwBytesRead;
    vector<char> buffer(bufferSize);
    hFile = CreateFile(nameFile.c_str(),
                       GENERIC_READ,
                       0,
                       0,
                       OPEN_EXISTING,
                       0,
                       0);
    if (!hFile) {
        CloseHandle(hFile);
        error = GetLastError();
        return error;
        }
     else {
        for (;;) {
            if (!ReadFile(hFile,
                          &buffer,
                          bufferSize,
                          &dwBytesRead,
                          0)) {
                CloseHandle(hFile);
                error = GetLastError();
                return error;
            }
         if (dwBytesRead == 0) break;
        }
    }
    text.append(&buffer[0], dwBytesRead);
    CloseHandle(hFile);
    return text;
}

Убежденный · 15.11.2014, 16:47

И снова:

C++
1
2
3
4
5
if (!hFile) {
        CloseHandle(hFile);
        error = GetLastError();
        return error;
        }

Такая обработка ошибок недопустима.
"if (!hFile)" срабатывает, если "hFile == NULL".
Но CreateFile в случае ошибки возвращает INVALID_HANDLE_VALUE, а
это значение не равно NULL !

Далее, GetLastError нужно звать до CloseHandle, иначе его значение
может потеряться.

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

В файле записано '1111', а мне выводит 'ц'.

Покажите, как выводите.

@Gelo123321 · 15.11.2014, 17:32 **[ТС]**

Сообщение от Убежденный

Покажите, как выводите.

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
string readFromFile(string nameFile)
{
    HANDLE hFile;
    string error, text;
    DWORD const bufferSize = 1024;
    DWORD dwBytesRead;
    vector<char> buffer(bufferSize);
    hFile = CreateFile(nameFile.c_str(),
                       GENERIC_READ,
                       0,
                       0,
                       OPEN_EXISTING,
                       0,
                       0);
    if (hFile == INVALID_HANDLE_VALUE) {
        CloseHandle(hFile);
        error = GetLastError();
        return error;
        }
     else {
        for (;;) {
            if (!ReadFile(hFile,
                          &buffer,
                          bufferSize,
                          &dwBytesRead,
                          0)) {
                CloseHandle(hFile);
                error = GetLastError();
                return error;
            }
         if (dwBytesRead == 0) break;
        }
    }
    text.append(&buffer[0], dwBytesRead);
    CloseHandle(hFile);
    return text;
}

Вывод:

C++
1
2
3
4
5
6
7
8
9
case 'r': {
                string nameFile;
                cout << "Enter name of file from you want to read the data: ";
                cin  >> nameFile;
                cout << "File >> "
                     << readFromFile(nameFile)
                     << '\n';
                break;
            }

Убежденный · 15.11.2014, 17:37

C++
1
2
3
4
5
6
7
8
9
10
for (;;) {
            if (!ReadFile(hFile,
                          &buffer,
                          bufferSize,
                          &dwBytesRead,
                          0)) {
                CloseHandle(hFile);
                error = GetLastError();
                return error;
            }

Не "&buffer", а "&buffer[0]". Это же разные вещи.

@Gelo123321 · 15.11.2014, 18:15 **[ТС]**

Убежденный, поставил "&buffer[0]". Теперь выводит пустую строку.

Убежденный · 15.11.2014, 18:28

Ну не знаю, проверяйте что возвращает CreateFile, возникают ли
ошибки на ReadFile, сколько байт прочитано и т.п.
В качественно написанных программах все это проверяется на
каждом шаге, и если есть хотя бы одно отступление от "плана",
функция возвращает ошибку или исключение. А в этом коде
наоборот: ошибки проглатываются, в итоге ничего не понятно.

Вызвал readFromFile, получил пустую строку, и теперь гадай:
то ли файл открыть не удалось, то ли прочитать, то ли файл
нулевой длины, то ли в нем пустая строка...

Добавлено через 9 минут
Нужно сделать что-то типа такого:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
string readFromFile(string const & Filename)
{
    file File = file::open(Filename);
 
    if (!File)
    {
        THROW_EXCEPTION("file::open failed (last error = 0x%.8lx).", GetLastError());
    }
 
    size_t const Size = File.get_size();
 
    if ( (0 == Size) || (Size > FILE_TOO_BIG) )
    {
        THROW_EXCEPTION("Invalid file size (%Iu).", Size);
    }
 
    vector<char> Buffer(Size);
 
    if (false == File.read(&Buffer[0], Size))
    {
        THROW_EXCEPTION("file::read failed (last error = 0x%.8lx).", GetLastError());
    }
 
    string RetValue;
    RetValue.append(&Buffer[0], Size);
    return RetValue;
}

Здесь любые "неожиданности" под контролем: нулевой (или слишком большой)
размер файла, ошибка открытия файла, ошибка чтения... Случается неприятность -
сразу выбрасывается исключение с текстом и кодом GetLastError.
Ресурсы контролируются обертками file, vector и string.

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@Gelo123321 22 / 22 / 8 Регистрация: 05.01.2012 Сообщений: 282
	15.11.2014, 18:15 [ТС]
	Убежденный, поставил "&buffer[0]". Теперь выводит пустую строку. 0