Ошибка при записи в бинарный файл

@MayaNash · Регистрация: 24.08.2011

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

Задание - записать в бинарный файл последовательность из N байт вида "00001111 11110000 00001111 ...".
Почему-то 11110000 записывается неправильно. Это 240 в десятичной и F0 в шестнадцатеричной системе, но при считывании получается FFFFFFF0. И дублируется последний байт. Почему?

C++

#include <iostream>
#include <fstream>
using std::ifstream;
using std::ofstream;
using std::cin;
using std::cout;
using std::endl;
using std::ios;
 
void main()
{
    int n;
    setlocale(LC_ALL,"rus");
    cout << "Введите N: ";
    cin >> n; // вводим количество байтов
 
    char byte[2] = {15,240};
    ofstream fout("binfile",ios::binary);
    for (int i = 0; i < n; i++)
        fout.write(&byte[i%2], 1);
    fout.close();
 
    ifstream fin("binfile",ios::binary);
    char ch;
    while (!fin.eof())
    {
        fin.read(&ch,1);
        cout << std::hex << (int)ch << " ";
    }
    cout << endl;
    fin.close();
    system("pause");
}

@nmcf · 09.05.2014, 11:14

У тебя массив byte фиксированной длины. Для чего вводить n? А если введут 10? Будет чтение за границами массива. И почему byte[i%2], а не byte[i]?

@MayaNash · 09.05.2014, 11:31 **[ТС]**

Обрати внимание, что если введут 10, то все будет нормально. i%2 - это остаток от деления на 2, то есть или 0 или 1. Так что выхода за границы по сути быть не может. Вопрос вообще не в этом.

Добавлено через 2 минуты
Задание конечно глупое, но как ни крути, в файле должны чередоваться 2 байта - 15 и 240. И всего должно быть N байтов.

@0x10 · 09.05.2014, 11:39

Керра, пишется и читается все верно. Второй байт - действительно 0xF0. Но вспомним, что char - знаковый. Т.е. это значение интерпретируется как -16. Приводим к int - получаем то же значение -16, но уже в ячейчке размером 4 байта. Что в дополнительном коде как раз 0xfffffff0.

@MayaNash · 09.05.2014, 11:43 **[ТС]**

Ясно, и что делать? abs не помогает

@nmcf · 09.05.2014, 11:44

Лишний байт у тебя выходит из-за того, что флаг конца файла устанавливается только после неудачного чтения, поэтому одна лишняя итерация выполняется после чтения последнего символа. Тебе нужно после read(), например peek() вызвать, чтобы флаг был установлен. Или ещё что-то придумать, может читать исходя из размера файла.

@0x10 · 09.05.2014, 11:48

Керра, в выводе можно вот такую загогулину воткнуть.

C++

1	static_cast<int>(static_cast<unsigned char>(ch))

@MayaNash · 09.05.2014, 11:52 **[ТС]**

nmcf, 0x10, спасибо, помогло

@nmcf · 09.05.2014, 11:53

У меня вот так работает:

C++

1	(int)(unsigned char)ch

Ошибка при записи в бинарный файл

Решение

Решение

@nmcf 7789 / 6556 / 2984 Регистрация: 14.04.2014 Сообщений: 28,661
	09.05.2014, 11:14	2
	У тебя массив byte фиксированной длины. Для чего вводить n? А если введут 10? Будет чтение за границами массива. И почему byte[i%2], а не byte[i]? 0

@MayaNash 1296 / 469 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	09.05.2014, 11:31 [ТС]	3
	Обрати внимание, что если введут 10, то все будет нормально. i%2 - это остаток от деления на 2, то есть или 0 или 1. Так что выхода за границы по сути быть не может. Вопрос вообще не в этом. Добавлено через 2 минуты Задание конечно глупое, но как ни крути, в файле должны чередоваться 2 байта - 15 и 240. И всего должно быть N байтов. 0

@0x10 3257 / 2059 / 351 Регистрация: 24.11.2012 Сообщений: 4,909
	09.05.2014, 11:39	4
	Керра, пишется и читается все верно. Второй байт - действительно 0xF0. Но вспомним, что char - знаковый. Т.е. это значение интерпретируется как -16. Приводим к int - получаем то же значение -16, но уже в ячейчке размером 4 байта. Что в дополнительном коде как раз 0xfffffff0. 0

@MayaNash 1296 / 469 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	09.05.2014, 11:43 [ТС]	5
	Ясно, и что делать? abs не помогает 0

@nmcf 7789 / 6556 / 2984 Регистрация: 14.04.2014 Сообщений: 28,661
	09.05.2014, 11:44	6
	Сообщение было отмечено MayaNash как решение Решение Лишний байт у тебя выходит из-за того, что флаг конца файла устанавливается только после неудачного чтения, поэтому одна лишняя итерация выполняется после чтения последнего символа. Тебе нужно после read(), например peek() вызвать, чтобы флаг был установлен. Или ещё что-то придумать, может читать исходя из размера файла. 1

@MayaNash 1296 / 469 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	09.05.2014, 11:52 [ТС]	8
	nmcf, 0x10, спасибо, помогло 0

	09.05.2014, 11:53