try fix memory error

This commit is contained in:
Simon From Jakobsen 2025-10-29 13:07:33 +01:00
parent 7584609f33
commit 012d021a08
2 changed files with 69 additions and 65 deletions

View File

@ -1,3 +1,4 @@
#include <mutex>
#include <thread> #include <thread>
#include <functional> #include <functional>
#include "GameRenderer.hpp" #include "GameRenderer.hpp"
@ -17,37 +18,37 @@ Game::~Game()
void Game::update(std::stop_token stop_token) void Game::update(std::stop_token stop_token)
{ {
while (!stop_token.stop_requested()) { while (!stop_token.stop_requested()) {
game_mutex.lock(); {
auto lock = std::lock_guard(game_mutex);
ticks++; ticks++;
player.update(); player.update();
for (Arrow *arrow : arrows) { for (Arrow *arrow : arrows) {
arrow->update(); arrow->update();
}
for (Zombo *zombo : zombos) {
zombo->update(player.x, player.y);
}
if (ticks % 1000 == 0) {
zombos.push_back(new Zombo(&renderer, 20.0, 20.0)); // doesn't work :((((
}
renderer.redraw();
map.check_bounds(player.x, player.y);
} }
for (Zombo *zombo : zombos) {
zombo->update(player.x, player.y);
}
if (ticks % 1000 == 0) {
zombos.push_back(new Zombo(&renderer, 20.0, 20.0)); // doesn't work :((((
}
renderer.redraw();
map.check_bounds(player.x, player.y);
game_mutex.unlock();
std::this_thread::sleep_for(16666us); std::this_thread::sleep_for(16666us);
} }
} }
void Game::draw() void Game::draw()
{ {
const std::lock_guard lock(game_mutex); std::lock_guard lock(game_mutex);
renderer.clear_screen(0x80, 0x40, 0xFF, 0xFF); renderer.clear_screen(0x80, 0x40, 0xFF, 0xFF);
@ -64,61 +65,65 @@ void Game::draw()
void Game::run() void Game::run()
{ {
update_thread = std::jthread(std::bind_front(&Game::update, this)); auto update_thread = std::jthread(std::bind_front(&Game::update, this));
while (true) { while (true) {
SDL_Event e; SDL_Event e;
SDL_WaitEvent(&e); SDL_WaitEvent(&e);
if (e.type == SDL_QUIT) { {
break; auto lock = std::lock_guard(game_mutex);
}
if (e.type == SDL_KEYDOWN) { if (e.type == SDL_QUIT) {
switch (e.key.keysym.sym) { break;
case SDLK_RIGHT:
case SDLK_d:
player.x_vel = player.speed;
break;
case SDLK_LEFT:
case SDLK_a:
player.x_vel = -player.speed;
break;
case SDLK_DOWN:
case SDLK_s:
player.y_vel = player.speed;
break;
case SDLK_UP:
case SDLK_w:
player.y_vel = -player.speed;
break;
} }
}
if (e.type == SDL_KEYUP) { if (e.type == SDL_KEYDOWN) {
switch (e.key.keysym.sym) { switch (e.key.keysym.sym) {
case SDLK_RIGHT: case SDLK_RIGHT:
case SDLK_d: case SDLK_d:
case SDLK_LEFT: player.x_vel = player.speed;
case SDLK_a: break;
player.x_vel = 0; case SDLK_LEFT:
break; case SDLK_a:
case SDLK_DOWN: player.x_vel = -player.speed;
case SDLK_s: break;
case SDLK_UP: case SDLK_DOWN:
case SDLK_w: case SDLK_s:
player.y_vel = 0; player.y_vel = player.speed;
break; break;
case SDLK_UP:
case SDLK_w:
player.y_vel = -player.speed;
break;
}
} }
}
if (e.type == SDL_MOUSEBUTTONDOWN && e.button.button == 1) { if (e.type == SDL_KEYUP) {
arrows.push_back(new Arrow(&renderer, player.x, player.y, player.angle)); switch (e.key.keysym.sym) {
} case SDLK_RIGHT:
case SDLK_d:
case SDLK_LEFT:
case SDLK_a:
player.x_vel = 0;
break;
case SDLK_DOWN:
case SDLK_s:
case SDLK_UP:
case SDLK_w:
player.y_vel = 0;
break;
}
}
int mouse_x, mouse_y; if (e.type == SDL_MOUSEBUTTONDOWN && e.button.button == 1) {
SDL_GetMouseState(&mouse_x, &mouse_y); arrows.push_back(new Arrow(&renderer, player.x, player.y, player.angle));
player.angle = std::atan2(mouse_y - renderer.screen_height / 2, mouse_x - renderer.screen_width / 2); }
int mouse_x, mouse_y;
SDL_GetMouseState(&mouse_x, &mouse_y);
player.angle = std::atan2(mouse_y - renderer.screen_height / 2, mouse_x - renderer.screen_width / 2);
}
draw(); draw();
} }

View File

@ -19,7 +19,6 @@ private:
std::vector<Arrow*> arrows; std::vector<Arrow*> arrows;
std::vector<Zombo*> zombos; std::vector<Zombo*> zombos;
std::jthread update_thread;
std::mutex game_mutex; std::mutex game_mutex;
unsigned int ticks = 0; unsigned int ticks = 0;