Почему не работает код с++?

Question

[FarmaCeduS]

Всем привет. Ребят, подскажите пожалуйста почему не работает код. Задача сделать программу которая будет инвертировать текст, т.е. загружать исходный с 1 файла и сохранять инвертированный в файл 2. Я написал вот такой код, но он не работает.

#include <iostream>
#include <fstream>
using namespace std;

int main ()
{
	int temp;
	setlocale (LC_ALL, "Russian");
	char buf1;
	ifstream in ("1.txt", ios::in|ios::binary);
	ofstream out ("2.txt", ios::out|ios::binary);
	
	if (!in)
	{
		cout << "Fail open 1 fail";
		abort ();
	}
	if (!out)
	{
		cout << "Fail open 2 fail";
		abort ();
	}
	in.seekg (0,ios::end); //получаем размер входящего файла для создания цикла
	temp = in.tellg ();
	in.seekg (ios::end); //устанавливаем указатель для считывания входящего файла с конца
	out.seekp (ios::beg); // устанавливаем указатель для записи считанного байта в файл №2
	for (int i=0; i < temp; i++) //запускаем цикл считывания и записи.
	{
		in.get (buf1);
		out.put (buf1);
		in.seekg (-1, ios::cur); //устанавливаем указатель входящего файла на -1 байт от текущего положения (текущее должно быть конец входящего файла)
		
	}

return 0;
}

Answer 1

[Александр Ручкин]

in.seekg (ios::end);
Чтобы прочесть последний символ, стоять надо не в конце, а на один символ раньше. В вашем случае читать уже нечего.

in.seekg (-1, ios::cur);
Этим вы только отшагнёте обратно на один символ, который только что прочли.

Если файлы невелики, проще будет прочесть его весь прочесть в std::vector, а потом просто скопировать в выходной файл:

std::vector<char> cts;
in.seekg(0, in.end);
cts.resize(in.tellg());
in.seekg(0, in.beg);

in.read(&ctr[0], ctr.size());

std::ostream_iterator<char> out_it(out);
std::copy(cts.rbegin(), cts.rend(), out);

Comments

[FarmaCeduS]

Благодарю за ответ но не могли бы вы прокоментировать свой код 1,3 и две останих строчки, что они делают.

[FarmaCeduS]

Почему в моём коде в цикле не происходит изменения когда я меняю курсор на -1 относительно текущего положения. т.е. выводит одно значения?

[Александр Ручкин]

FarmaCeduS: потому что текущее положение смещается на 1 при чтении, и вы его смещаете на 1 назад, т.е. туда, где оно было до чтения.

[Александр Ручкин]

FarmaCeduS: а что именно непонятно? Сначала я просто узнаю размер файла, затем resize'ом выделяю достаточную память, затем читаю весь файл.
out_it - это специальный итератор, который пишет в выходной поток, в нашем случае - в файл out.
Ну и по этому итератору я копирую все данные из массива. Но чтобы копировать в обратном порядке, я использую не пару итераторов begin/end, а пару rbegin/rend, которая пробегает контейнер в обратном направлении.

[FarmaCeduS]

Александр Ручкин: Я просто только учусь) STL шаблоны еще не учил.