【发布时间】:2018-12-25 21:44:22
【问题描述】:
我正在使用一个可以从内存中加载 BMP 图像的库。
我有一个代表 BMP 的类。
要从内存中加载,我必须提供一个指向内存中一些 BMP 格式数据的指针和一个用于该数据大小的变量。 (void* data, size_t length)
我想将我的数据存储在std::vector 中。 (避免手动内存管理)
我试图编写一个函数来返回一个std::vector<unsigned char>,但我认为我得到的不是很好。
std::vector<unsigned char> BMP::BITMAP::SaveMem() const
{
// memory storage
std::vector<unsigned char> memory;
BITMAPFILEHEADER f_head;
f_head.bfType = ushort_rev(((WORD)'B' << 0x08) | ((WORD)'M' << 0x00));
f_head.bfSize = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER) + m_width_memory * m_height;
f_head.bfReserved1 = 0;
f_head.bfReserved2 = 0;
f_head.bfOffBits = sizeof(BITMAPFILEHEADER) + sizeof(BITMAPINFOHEADER);
// build standard bitmap file header
BITMAPINFOHEADER i_head;
i_head.biSize = sizeof(BITMAPINFOHEADER);
i_head.biWidth = m_width;
i_head.biHeight = m_height;
i_head.biPlanes = 1;
i_head.biBitCount = m_bit_count;
i_head.biCompression = 0;
i_head.biSizeImage = m_width_memory * m_height;
i_head.biXPelsPerMeter = 0;
i_head.biYPelsPerMeter = 0;
i_head.biClrUsed = 0;
i_head.biClrImportant = 0;
// alloc
memory.resize(f_head.bfSize);
std::copy(&f_head, &f_head + sizeof(f_head), memory.at(0));
std::copy(&i_head, &i_head + sizeof(i_head), memory.at(0) + sizeof(f_head));
// write data
for(unsigned int y = 0; y < m_height; ++ y)
{
std::copy(&m_data[y * m_width_memory], m_data[y * m_width_memory + 3 * m_size_x], memory.at(0) + sizeof(f_head) + sizeof(i_head));
}
}
显然这不能编译。我想不出std::copy 的任何替代方案。这真的是适合这项工作的工具吗?
为了让它编译,我认为我应该将memory.at(x) 更改为memory.data() + x...这样做我将使用原始指针——这就是为什么我认为std::copy 不会比memcpy 更好.
我能对此提出一些建议吗?这有点不合逻辑,如果我早先知道这个要求,我会将我的像素数据存储在unsigned char 中,位图文件标题在数据之前。可惜现在改设计需要做很多工作,所以我宁愿不去碰它。
【问题讨论】:
-
改变设计有那么可怕吗?由于这种错误的设计,修复错误需要多长时间?
-
@JVApen 这并不理想,可能会引入更多错误,因为有大量代码依赖于在二进制级别操作 BMP 数据
标签: c++ vector binary-data stdcopy