Switch from characters to numerical tile IDs

2 files changed, 20 insertions, 29 deletions

diff --git a/src/view/map.ts b/src/view/map.ts
index d75ea98..d7c9ce9 100644
--- a/src/view/map.ts
+++ b/src/view/map.ts
@@ -1,38 +1,38 @@
 import MapData from '../model/data/MapData';
-import { mapValues, nextPowerOf2 } from '../util';
+import { nextPowerOf2 } from '../util';
 
 import Renderer from './renderer/Renderer';
 import { SpriteCoords, SpriteView, SpriteViewBuilder } from './sprite';
-import { loadImages, mkTexture } from './util/image';
+import { loadImage, mkTexture } from './util/image';
 
-interface TileMap {
+interface Tileset {
 	texture: WebGLTexture;
-	tiles: Map<string, SpriteCoords>;
+	tiles: SpriteCoords[];
 }
 
-function loadTiles(tiles: Map<string, string>): Promise<Map<string, HTMLImageElement>> {
-	return loadImages(mapValues((t) => `resources/sprite/tile/${t}.png`, tiles));
+function loadTiles(tiles: string[]): Promise<HTMLImageElement[]> {
+	return Promise.all(tiles.map((t) => loadImage(`resources/sprite/tile/${t}.png`)));
 }
 
-function mkTileMap(
+function mkTileset(
 	gl: WebGLRenderingContext,
-	tiles: Map<string, HTMLImageElement>,
-): TileMap {
+	tiles: HTMLImageElement[],
+): Tileset {
 	const tileSize = 32;
 
-	const canvasDim = nextPowerOf2(Math.sqrt(tiles.size));
+	const canvasDim = nextPowerOf2(Math.sqrt(tiles.length));
 	const canvasSize = canvasDim * tileSize;
 
 	const canvas = document.createElement('canvas');
 	canvas.width = canvas.height = canvasSize;
 
 	let x = 0, y = 0;
-	const map: Map<string, SpriteCoords> = new Map();
+	const tileCoords: SpriteCoords[] = [];
 	const ctx = canvas.getContext('2d') as CanvasRenderingContext2D;
 
-	for (const [k, tile] of tiles) {
+	for (const tile of tiles) {
 		ctx.drawImage(tile, x * tileSize, y * tileSize);
-		map.set(k, [x / canvasDim, y / canvasDim, (x + 1) / canvasDim, (y + 1) / canvasDim]);
+		tileCoords.push([x / canvasDim, y / canvasDim, (x + 1) / canvasDim, (y + 1) / canvasDim]);
 
 		x++;
 		if (x === canvasDim) {
@@ -43,31 +43,28 @@ function mkTileMap(
 
 	return {
 		texture: mkTexture(gl, canvas),
-		tiles: map,
+		tiles: tileCoords,
 	};
 }
 
-function addSprite(builder: SpriteViewBuilder, tileMap: TileMap, x: number, y: number, tile: string) {
-	if (tile === ' ')
+function addSprite(builder: SpriteViewBuilder, tileset: Tileset, x: number, y: number, tile: number) {
+	if (tile === 0)
 		return;
 
-	const tilePos = tileMap.tiles.get(tile);
-	if (!tilePos)
-		throw new Error('invalid tile specifier in map data');
-
+	const tilePos = tileset.tiles[tile - 1];
 	builder.addSprite([x, y, x + 1, y + 1], tilePos);
 }
 
 export async function loadMap(r: Renderer, map: MapData): Promise<SpriteView> {
 	const tiles = await loadTiles(map.tiles);
-	const tileMap = mkTileMap(r.getContext(), tiles);
+	const tileset = mkTileset(r.getContext(), tiles);
 
-	const builder = new SpriteViewBuilder(r, tileMap.texture);
+	const builder = new SpriteViewBuilder(r, tileset.texture);
 
 	for (const layer of map.layers)
 		for (let x = 0; x < map.width; x++)
 			for (let y = 0; y < map.height; y++)
-				addSprite(builder, tileMap, x, y, layer[y][x]);
+				addSprite(builder, tileset, x, y, layer[y][x]);
 
 	return builder.build();
 }
diff --git a/src/view/util/image.ts b/src/view/util/image.ts
index 50debde..73edbba 100644
--- a/src/view/util/image.ts
+++ b/src/view/util/image.ts
@@ -1,5 +1,3 @@
-import { mapValuesAsync } from '../../util';
-
 export function loadImage(url: string): Promise<HTMLImageElement> {
 	return new Promise((resolve, reject) => {
 		const img = new Image();
@@ -9,10 +7,6 @@ export function loadImage(url: string): Promise<HTMLImageElement> {
 	});
 }
 
-export function loadImages(urls: Map<string, string>): Promise<Map<string, HTMLImageElement>> {
-	return mapValuesAsync(loadImage, urls);
-}
-
 export function mkTexture(gl: WebGLRenderingContext, src: HTMLCanvasElement|HTMLImageElement): WebGLTexture {
 	const texture = gl.createTexture();
 	if (!texture)