Change region reader to a visitor pattern

Not keeping the whole region in memory reduces the needed space
to less than 5MB (from about 140).
This commit is contained in:
Matthias Schiffer 2015-02-02 01:41:17 +01:00
parent 1e5e315816
commit 8c1629af42
3 changed files with 30 additions and 32 deletions

View file

@ -54,7 +54,7 @@ Region::ChunkMap Region::processHeader(const uint8_t header[4096]) {
return map;
}
Region::Region(const char *filename) {
void Region::visitChunks(const char *filename, ChunkVisitor visitor) {
std::ifstream file;
file.exceptions(std::ios::failbit | std::ios::badbit);
file.open(filename, std::ios::in | std::ios::binary);
@ -68,6 +68,8 @@ Region::Region(const char *filename) {
chunkMap = processHeader(header);
}
bool seen[SIZE][SIZE] = {};
size_t i = 1, c = 0;
while (!file.eof()) {
auto it = chunkMap.find(i);
@ -80,16 +82,19 @@ Region::Region(const char *filename) {
size_t x, z, len;
std::tie(x, z, len) = it->second;
if (chunks[x][z])
if (seen[x][z])
throw std::invalid_argument("duplicate chunk");
seen[x][z] = true;
c++;
uint8_t buffer[len * 4096];
file.read((char *)buffer, len * 4096);
chunks[x][z].reset(new Chunk(Buffer(buffer, len * 4096)));
Chunk chunk(Buffer(buffer, len * 4096));
visitor(x, z, &chunk);
i += len;
c++;
}
if (c != chunkMap.size())

View file

@ -28,6 +28,7 @@
#include "Chunk.hpp"
#include <functional>
#include <memory>
#include <stdexcept>
#include <tuple>
@ -41,23 +42,18 @@ class Region {
public:
static const size_t SIZE = 32;
typedef std::function<void (size_t, size_t, const Chunk *)> ChunkVisitor;
Region() = delete;
private:
typedef std::tuple<size_t, size_t, size_t> ChunkDesc;
typedef std::unordered_map<size_t, ChunkDesc> ChunkMap;
std::unique_ptr<Chunk> chunks[SIZE][SIZE];
static ChunkMap processHeader(const uint8_t header[4096]);
public:
Region(const char *filename);
const Chunk * operator()(size_t x, size_t z) {
if (x >= SIZE || z >= SIZE)
throw std::range_error("Region(): bad coordinates");
return chunks[x][z].get();
}
static void visitChunks(const char *filename, ChunkVisitor visitor);
};
}