1 files changed, 31 insertions, 20 deletions

diff --git a/src/attack.cpp b/src/attack.cpp
index 550b849..e91b4cd 100644
--- a/src/attack.cpp
+++ b/src/attack.cpp
@@ -1,10 +1,13 @@
 #include <attack.hpp>
+
 #include <components.hpp>
 #include <engine.hpp>
 #include <particle.hpp>
+#include <player.hpp>
+#include <world.hpp>
 
-constexpr int shortSlashLength = 100;
-constexpr int longSlashLength = 200;
+constexpr int shortSlashLength = 20;
+constexpr int longSlashLength = 40;
 
 // math helpers because we don't trust stdlib
 template<typename T>
@@ -20,6 +23,11 @@ bool inrange(float point, float left, float right, float range)
 		(point > left && point < right);
 }
 
+bool inrange(float point, float left, float right)
+{
+	return point > left && point < right;
+}
+
 void AttackSystem::receive(const AttackEvent& ae)
 {
 	attacks.emplace_front(ae);
@@ -31,17 +39,35 @@ void AttackSystem::update(entityx::EntityManager& en, entityx::EventManager& ev,
 	(void)ev;
 	(void)dt;
 
+	// handle attacking entities
+	en.each<Hit, Position>([&](entityx::Entity p, Hit& hit, Position& ppos) {
+		bool die = false;
+		en.each<Health, Position, Solid>([&](entityx::Entity e, Health& health, Position& pos, Solid& dim) {
+			if (!e.has_component<Player>() && inrange(ppos.x, pos.x, pos.x + dim.width) && inrange(ppos.y, pos.y - 2, pos.y + dim.height)) {
+				health.health -= hit.damage;
+				game::engine.getSystem<ParticleSystem>()->addMultiple(15, ParticleType::SmallBlast,
+					[&](){ return vec2(pos.x + dim.width / 2, pos.y + dim.height / 2); }, 300, 7);
+				die = !hit.pierce;
+			} else if (WorldSystem::isAboveGround(vec2(ppos.x, ppos.y - 5)))
+				die = true;
+		});
+
+		if (die)
+			p.destroy();
+	}); 
+
+	// handle emitted attacks
 	for (const auto& a : attacks) {
 		switch (a.type) {
 		case AttackType::ShortSlash:
 		case AttackType::LongSlash:
 			en.each<Position, Solid, Health>(
 				[&a](entityx::Entity e, Position& pos, Solid& dim, Health& h) {
-					(void)e;
-					if (e.has_component<Player>())
+					if (!(e.has_component<Player>() ^ a.fromplayer))
 						return;
 
-					if (inrange(a.pos.x, pos.x, pos.x + dim.width, shortSlashLength)) {
+					if (inrange(a.pos.x, pos.x, pos.x + dim.width, HLINES(shortSlashLength)) &&
+						inrange(a.pos.y, pos.y, pos.y + dim.height)) {
 						h.health -= a.power;
 						game::engine.getSystem<ParticleSystem>()->addMultiple(15, ParticleType::DownSlash,
 							[&](){ return vec2(pos.x + dim.width / 2, pos.y + dim.height / 2); }, 300, 7);
@@ -49,21 +75,6 @@ void AttackSystem::update(entityx::EntityManager& en, entityx::EventManager& ev,
 				}
 			);
 			break;
-		case AttackType::SmallBoom:
-			en.each<Position, Solid, Health>(
-				[&a](entityx::Entity e, Position& pos, Solid& dim, Health& h) {
-					(void)e;
-					if (e.has_component<Player>())
-						return;
-
-					if (inrange(a.pos.x, pos.x, pos.x + dim.width, shortSlashLength)) {
-						h.health -= a.power;
-						game::engine.getSystem<ParticleSystem>()->addMultiple(15, ParticleType::SmallBlast,
-							[&](){ return vec2(pos.x + dim.width / 2, pos.y + dim.height / 2); }, 300, 7);
-					}
-				}
-			);
-			break;
 		default:
 			break;
 		}