Eduardo Lucio Eduardo Lucio - 1 month ago 18
C++ Question

c++ - Save the contents of a "std::vector<unsigned char>" to a file

I use the function below "writeFileBytes" to write the contents of a

std::vector<unsigned char>
to a file. I want to use the type "unsigned char" to save the file (note that is made a cast for "char"). The reason for that question is because "unsigned char" has compatibility with any byte ("00000000" bits, for example). When we use "char" we have some problems with the manipulation of certain "invalid" chars.

See this topic What is the most suitable type of vector to keep the bytes of a file? for more information about the issues with "00000000" bits (1 byte).

void writeFileBytes(const char* filename, std::vector<unsigned char>& fileBytes){
std::ofstream file(filename, std::ios::out|std::ios::binary);
file.write(fileBytes.size() ? (char*)&fileBytes[0] : 0,
std::streamsize(fileBytes.size()));
}

writeFileBytes("xz.bin", fileBytesOutput);


Is there a way to use the type "unsigned char" natively to write to a file?

This concern really make sense?




UPDATE I:

Is there a way to use the type "unsigned char" natively to write to a file? -> YES!

Following the krzaq's guidelines!

void writeFileBytes(const char* filename, std::vector<unsigned char>& fileBytes){
std::ofstream file(filename, std::ios::out|std::ios::binary);
std::copy(fileBytes.cbegin(), fileBytes.cend(),
std::ostream_iterator<unsigned char>(file));
}





UPDATE II:

This concern really make sense? -> In some ways, YES!

""

Answer

It doesn't matter, char* can be used to access any kind of data and it'll work correctly. But if you don't want to use the explicit cast, maybe use std::copy and std::ostreambuf_iterator:

copy(fileBytes.cbegin(), fileBytes.cend(),
     ostreambuf_iterator<char>(file));

alternatively, you can call

copy(fileBytes.cbegin(), fileBytes.cend(),
     ostream_iterator<char>(file));

But it will do the same thing, only possibly slower.

Btw: you can't pass a null pointer to write, that's UB.