Почему я не могу скопировать файл?

Question

[Ternick]

Доброго времени суток :)
Открываю файл в бинарном режиме для чтения так:

wstringstream wss;
wifstream wif(basePath, ios::binary);
if (wif.is_open()) {
    wss << wif.rdbuf();
    wif.close();
}

Открываю файл в бинарном режиме для записи так:

wofstream wof(tempFileName, ios::binary);
if (wof.is_open()) {
    wof << wss.str().data();
    wof.close();
}

По итогу записывает 3 байта и всё.
И интересует вопрос, во что можно обернуть бинарные данные из файла, чтобы их можно было записать в исполняемый файл. {Пробовал массив wchar_t, в нём содержалось около 30 кб данных и чтобы скомпилировать его приходилось крайне долго ждать.}

Comments

[Евгений Шатунов]

Покажи код процесса копирования данных между файлами.

[Ternick]

Евгений Шатунов, Не бейте тапками, я знаю о существовании WinApi функций для копирования файлов, но суть в том, что мне нужно не столько переместить файл, сколько прочитать его, расшифровать и сохранить в другом месте. Те два куска кода сверху это и есть копирование {Проблема в том,что я не могу даже просто скопировать их, я уже все перетыкал и перепробовал.}.

[Евгений Шатунов]

Ternick, ну ты уже понимаешь что копирования я там и не увидел.
wss.str().data() - это ты взял строковое представление данных и просто записал его в другой файл.
Так дело не пойдет...

А что тебе документация говорит? Какими еще способами ты можешь получить данные из файла?

[Ternick]

Евгений Шатунов, Ну там есть ещё wss.binary, но тоже не шипко меняет ситуацию ... Не понял суть вопроса этого "Какими еще способами ты можешь получить данные из файла?".

[Евгений Шатунов]

Ternick,

Не понял суть вопроса

Этот вопрос подталкивает тебя к изучению документации. По своей сути у тебя такой вопрос, решение которого занимает не более чем одно прочтение документации. Т.е. прочитал и понял как действовать.
А ты потерял уже уйму времени на ожидании ответа.

Отвыкай писать без std, ты себе яму роешь.
std::wstringstream - это не тот тип, что нужен тебе. Это объект абстракции над текстовым файлом с "широкой" кодировкой. Такой стрим оперирует строками с типом std::wstring.
Изучи документацию к std::basic_fstream. Это то, что тебе нужно.

Еще одно. wof << wss.str().data(); - стоит забыть про operator <<.
Я сейчас говорю тезисы без обоснования. Просто обоснование ты действительно можешь и сам найти. Сейчас ты только усложняешь себе жизнь используя подобные практики.

[Ternick]

Евгений Шатунов, Я вам больше скажу, я из-за подобных глупых вопросов уже 2 дня потерял, но ничего с этим сделать не могу. Чтобы гуглить, нужно знать о чём гуглить, а тут проблема.
А в чём собственно проблема std ?

[Евгений Шатунов]

Ternick, про std: 1, 2.

Чтобы гуглить, нужно знать о чём гуглить, а тут проблема.

Документация. Тебе не нужен гугл чтобы ходить по прямым ссылкам или искать во внутреннем поиске документации.

[Ternick]

Евгений Шатунов, Мыслей нет. Этот вариант, тоже не работает.

wifstream wif(tempName, ios::binary);
	if (wif.is_open()) {
		wif.seekg(0, wif.end);
		size_t size = (size_t)wif.tellg();
		wif.seekg(0, wif.beg);
		wchar_t* encryptedData = new wchar_t[size];
		wif.get(encryptedData, size);
		wif.close();
		delete[] encryptedData;
	}

[Ternick]

Евгений Шатунов, ЮХУУУУУ

wfstream wif(L"logs.dat", ios::binary | ios::in);
	if (wif.is_open()) {
		wif.seekg(0, wif.end);
		size_t size = (size_t)wif.tellg();
		wif.seekg(0, wif.beg);
		wchar_t* encryptedData = new wchar_t[size];
		ZeroMemory(encryptedData, size);
		wif.read(encryptedData, size);
		wif.close();
		wofstream wof(L"temp.dat", ios::binary);
		if (wof.is_open()) {
			wof.write(encryptedData, size);
			wof.close();
		}
		delete[] encryptedData;
	}

А вот этот действительно работает :)
Спасибо.
Это же правильный вариант или не совсем лучший вариант решения моего вопроса ?

[Евгений Шатунов]

Ternick, да, в общем случае это верный подход.
Теперь ты полноправно можешь выписать свой код в качестве ответа и пометить его решением своего вопроса. :)

Однако, на следующие моменты можно обратить свое внимание.
wchar_t* encryptedData = new wchar_t[size];
Тут можно упростить себе жизнь с помощью std::vector. Еще тебе совершенно не нужен wchat_t для работы с бинарным файлом. Лучше будет взять std::byte, если тебе доступен C++17, или uint8_t. Т.е. буферы будет лучше из однобайтовых составить.
ZeroMemory(encryptedData, size) занулит только size байт, а не элементов, т.е. он пройдет только половину твоего текущего буфера.
wif.read(encryptedData, size) - тоже воспользуется лишь половиной буфера, т.к. зачитает size байт, а не size двубайтовых.
И с записью тоже так же.

[Ternick]

Евгений Шатунов, Спасибо, крайний вопрос остался. fstream.read принимает только wchar_t*, streamsize*.
А как мне записать данные в массив byte ? Я пробовал перекастовать, но видимо не судьба. Не ужели придётся сначала всё записывать в wchar_t, а потом перебирать массив и записывать в массив байтов ??

[Евгений Шатунов]

Ternick, тут вопросом остается то, почему ты используешь wfstream?
Ты ведь явно со студией работаешь, если тебе приходится пути в wchar_t передавать.
У студии есть свое расширение базового шаблона std::basic_stream со своим конструктором от широкой строки.
Первый параметр шаблона std::basic_stream говорит о том, в каких элементах следует интерпретировать тело файла.