bartek/base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Split InteractiveEntity to separate listeners and remove position checks

Browse Source
This commit is contained in:
Bartłomiej Przemysław Pluta
2021-04-01 20:12:25 +02:00
parent e1e6d897cd
commit f5f8ec1595
3 changed files with 28 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
api/src/main/java/com/bartlomiejpluta/base/api/entity/EntityStepInListener.java Normal file
View File

@@ -0,0 +1,7 @@
package com.bartlomiejpluta.base.api.entity;
import com.bartlomiejpluta.base.api.move.Movement;
public interface EntityStepInListener extends Entity {
void onEntityStepIn(Movement movement, Entity entity);
}

4
api/src/main/java/com/bartlomiejpluta/base/api/entity/InteractiveEntity.java → api/src/main/java/com/bartlomiejpluta/base/api/entity/EntityStepOutListener.java
View File

@@ -2,8 +2,6 @@ package com.bartlomiejpluta.base.api.entity;
import com.bartlomiejpluta.base.api.move.Movement;
public interface InteractiveEntity extends Entity {
void onEntityStepIn(Movement movement, Entity entity);
public interface EntityStepOutListener extends Entity {
void onEntityStepOut(Movement movement, Entity entity);
}

34
engine/src/main/java/com/bartlomiejpluta/base/engine/world/map/layer/object/DefaultObjectLayer.java
View File

@@ -3,7 +3,8 @@ package com.bartlomiejpluta.base.engine.world.map.layer.object;
import com.bartlomiejpluta.base.api.ai.NPC;
import com.bartlomiejpluta.base.api.camera.Camera;
import com.bartlomiejpluta.base.api.entity.Entity;
import com.bartlomiejpluta.base.api.entity.InteractiveEntity;
import com.bartlomiejpluta.base.api.entity.EntityStepInListener;
import com.bartlomiejpluta.base.api.entity.EntityStepOutListener;
import com.bartlomiejpluta.base.api.map.layer.object.ObjectLayer;
import com.bartlomiejpluta.base.api.map.layer.object.PassageAbility;
import com.bartlomiejpluta.base.api.map.model.GameMap;
@@ -28,7 +29,8 @@ public class DefaultObjectLayer extends BaseLayer implements ObjectLayer {
@Getter
private final ArrayList<Entity> entities = new ArrayList<>();
private final List<InteractiveEntity> interactiveEntities = new ArrayList<>();
private final List<EntityStepInListener> stepInListeners = new ArrayList<>();
private final List<EntityStepOutListener> stepOutListeners = new ArrayList<>();
@Getter
private final PassageAbility[][] passageMap;
@@ -53,8 +55,12 @@ public class DefaultObjectLayer extends BaseLayer implements ObjectLayer {
layer.entities.remove(entity);
}
if (entity instanceof InteractiveEntity) {
interactiveEntities.add((InteractiveEntity) entity);
if (entity instanceof EntityStepInListener) {
stepInListeners.add((EntityStepInListener) entity);
}
if (entity instanceof EntityStepOutListener) {
stepOutListeners.add((EntityStepOutListener) entity);
}
entity.setStepSize(stepSize.x(), stepSize.y());
@@ -67,8 +73,12 @@ public class DefaultObjectLayer extends BaseLayer implements ObjectLayer {
public void removeEntity(Entity entity) {
entities.remove(entity);
if (entity instanceof InteractiveEntity) {
interactiveEntities.remove(entity);
if (entity instanceof EntityStepInListener) {
stepInListeners.remove(entity);
}
if (entity instanceof EntityStepOutListener) {
stepOutListeners.remove(entity);
}
entity.onRemove(this);
@@ -168,23 +178,15 @@ public class DefaultObjectLayer extends BaseLayer implements ObjectLayer {
@Override
public void notifyEntityStepIn(Movement movement, Entity entity) {
var target = movement.getTo();
for (var listener : interactiveEntities) {
if (listener.getCoordinates().equals(target)) {
for (var listener : stepInListeners) {
listener.onEntityStepIn(movement, entity);
}
}
}
@Override
public void notifyEntityStepOut(Movement movement, Entity entity) {
var source = movement.getFrom();
for (var listener : interactiveEntities) {
if (listener.getCoordinates().equals(source)) {
for (var listener : stepOutListeners) {
listener.onEntityStepOut(movement, entity);
}
}
}
}