rustrogueliketutorial icon indicating copy to clipboard operation
rustrogueliketutorial copied to clipboard

1.9 suggestion - mention draw_map before ranged_target

Open Raymond26 opened this issue 5 years ago • 0 comments

Before 1.9, the last mention of draw_map() and the code below occurs in 1.6 where it is below the delete_the_dead() call.

damage_system::delete_the_dead(&mut self.ecs);

draw_map(&self.ecs, ctx);

let positions = self.ecs.read_storage::<Position>();
let renderables = self.ecs.read_storage::<Renderable>();
let map = self.ecs.fetch::<Map>();

for (pos, render) in (&positions, &renderables).join() {
    let idx = map.xy_idx(pos.x, pos.y);
    if map.visible_tiles[idx] { ctx.set(pos.x, pos.y, render.fg, render.bg, render.glyph) }
}

The setting of backgrounds in ranged_target() gets overwritten by the map drawing/rendering code, and it's not explicitly mentioned in the tutorial that it needs to be moved up to the beginning of tick().. might be helpful add that.

Raymond26 avatar Apr 13 '20 05:04 Raymond26