Запись вектора в бинарный файл, циклом — C++?

Question

[kytcenochka]

Есть файл input.txt с таким содержимым:
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

Записала содержимое в вектор и хочу записать в бинарный файл data.dat только поэлементно с индексом строки. Файл должен иметь структуру: индекс (4 байта) + 6 значений из вектора по 2 байта.

01 00 00 00 01 00 02 00 03 00 04 00 05 00 06 00 02 00 00 00 07 00 08 00 и т.д.

Вот мой код, но пишет неправильно

#include "stdafx.h"
#include <iostream>
#include <fstream>
#include <vector>
#include <iterator>
#include <algorithm>

using namespace std;

int main()
{
    std::fstream input("input.txt");
    std::vector<short> vec; // значения из файла
    std::copy(std::istream_iterator<int>(input), std::istream_iterator<int>(), std::back_inserter(vec));
    std::copy(vec.begin(), vec.end(), std::ostream_iterator<int>(std::cout, " "));

    ofstream fout("data.dat", std::ios::out | ios::binary);
    int countstring = 5;
    int index = 0;
    int row = 6;
    for (int i = 0; i <= countstring; i++) {
        index = i + 1;
        fout.write((char *)&index, sizeof index);
        for (size_t j = 0; j < row; ++j)
        {
            fout.write((char*)&vec[i * 6], row * sizeof(short));
        }
    }
    fout.close();
    return 0;

Подскажите, как исправить второй цикл, чтоб записывать поэлементно

Comments

[Adamos]

Второй цикл стоит исправить так, чтобы в нем использовалась переменная цикла.
А в первом стоит задуматься - то ли i = 0, то ли i <= countstring. Вряд ли то и другое одновременно.

Answer 1

[Александр Ананьев]

#include "stdafx.h"
#include <iostream>
#include <fstream>
#include <vector>
#include <iterator>
#include <algorithm>


int main()
{
	std::fstream input("input.txt");
	std::vector<short> vec; // значения из файла
	std::copy(std::istream_iterator<int>(input), std::istream_iterator<int>(), std::back_inserter(vec));
	std::copy(vec.begin(), vec.end(), std::ostream_iterator<int>(std::cout, " "));

	std::ofstream fout("data.dat", std::ios::out | std::ios::binary);		
	const size_t columnCount = 6;
	for (size_t i = 0; i < vec.size(); i+= columnCount)
	{
		int index = i / columnCount + 1;
		fout.write((char *)&index, sizeof(int));
		for (size_t j = 0; j < columnCount; j++)
		{
			fout.write((char*)&vec[i + j], sizeof(short));
		}
	}
	
	fout.close();
	return 0;
}

Comments

[kytcenochka]

Спасибо!
Вопрос еще возник, добавила второй файл из которого добавляю значение в каждую строку
input2.txt:
0 0 0 0 1 1
0 0 0 0 1 1
0 0 0 0 1 1
0 0 0 0 1 1
0 0 0 0 1 1

Почему только в первой строке пишет верное значение 30 00, потом уже не то

int main()
{
	std::fstream input("input.txt");
	std::fstream input2("input2.txt");

	std::vector<short> vec; // значения из файла input.txt
	std::copy(std::istream_iterator<int>(input), std::istream_iterator<int>(), std::back_inserter(vec));
	std::copy(vec.begin(), vec.end(), std::ostream_iterator<int>(std::cout, " "));

	std::vector<int> vec2; // значения из файла input2.txt
	std::copy(std::istream_iterator<int>(input2), std::istream_iterator<int>(), std::back_inserter(vec2));
	std::copy(vec2.begin(), vec2.end(), std::ostream_iterator<int>(std::cout, " "));

	std::ofstream fout("data.dat", std::ios::out | std::ios::binary);
	const size_t columnCount = 6;

	for (size_t i = 0; i < vec.size(); i += columnCount)
	{
		int index = i / columnCount + 1;
		fout.write((char *)&index, sizeof(int));

		for (size_t j = 0; j < columnCount; j++)
		{
			fout.write((char*)&vec[i + j], sizeof(short));
		}

               // Запись значений второго файла
		unsigned short m_bytesCount = 0; // запись 1й строки в 2 байта

		for (size_t k = 0; k < columnCount; k++)
		{

			if (vec2[i+k] == 1)
				m_bytesCount |= 1 << i+k;
		}
		fout.write((char *)&m_bytesCount, sizeof m_bytesCount);
	}
	fout.close();
	return 0;

[Александр Ананьев]

kytcenochka, Не совсем понятно, что вы пытаетесь сделать.

[kytcenochka]

В файле input2.txt хранятся наборы битов, каждый бит может иметь одно из двух значений – 0 или 1.
Значения каждой строки должны хранится в двух байтах.
Строку 0 0 0 0 1 1 можно записать в два байта как 30 00 используя побитовые операторы, а именно присваивание с побитовой операцией ИЛИ |=.

Вот в первой строке записывается правильно, а затем идет сбой

01 00 00 00 01 00 02 00 03 00 04 00 05 00 06 00 30 00 02 00 00 00 07 00 08 00 09 00 0А 00 0В 00 0С 00 00 0С 03 00 00 00 0D 00 0E 00 0F 00 10 00 11 00 12 00 00 00

[Александр Ананьев]

kytcenochka, думаю, как-то так

for (size_t k = 0; k < columnCount; k++)
    {

      if (vec2[i+k] == 1)
        m_bytesCount |= 1 << i+k;
      else
        m_bytesCount = 0;
    }
    fout.write((char *)&m_bytesCount, sizeof m_bytesCount);

[kytcenochka]

Александр Ананьев, тоже самое

Answer 2

[Rsa97]

индекс (4 байта) + 6 значений из вектора по 2 байта

Значит правильно выбирайте размер данных. int, в общем случае, может иметь размер 16, 32 или 64 бита, в зависимости от системы и компилятора, short - от 8 до 64 бит.
Используйте int32_t для 4-байтовых и int16_t для 2-байтовых чисел (__int32 и __int16 в старых версиях Visual C++).

Cтруктура вектора содержит не значения, записанные в вектор, а указатели на начало и конец блока памяти, выделенного для хранения этих значений (https://stackoverflow.com/questions/27466373/memor... Реализация этой структуры зависит от компилятора. Поэтому в строке

fout.write((char*)&vec[i * 6], row * sizeof(short));

вы получаете не данные, а байты из структуры вектора.