SteamWar/FightSystem
SteamWar/FightSystem
Archiviert
13
1
Fork 0
Code Issues 6 Pull-Requests Releases Wiki Aktivität

Arrows Stopping in Techhider Blocks #208

Manuell gemergt
Lixfel hat 17 Commits von arrow-in-techhider nach master 2021-01-02 09:20:42 +01:00 zusammengeführt
Diskussion 3 Commits 17 Geänderte Dateien 2 +13 -21
1 geänderte Dateien mit 13 neuen und 21 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit 7b055a37be werden angezeigt - Alle Commits anzeigen

30
FightSystem_Main/src/de/steamwar/fightsystem/listener/ArrowStopper.java
Datei anzeigen

@ -37,7 +37,7 @@ public class ArrowStopper extends BasicListener {
private BukkitTask task; private BukkitTask task;
private static final Vector NULL_VECTOR = new Vector(0, 0, 0); private static final Vector NULL_VECTOR = new Vector(0, 0, 0);
private static final BlockFace[] BLOCK_FACES = {BlockFace.UP, BlockFace.DOWN, BlockFace.EAST, BlockFace.WEST, BlockFace.NORTH, BlockFace.SOUTH};
Lixfel hat 2020-12-30 07:31:12 +01:00 kommentiert
Veraltet
Review
Link kopieren

Statt der (aufwändigen) Verwaltung von LAST_LOCATIONS wäre es wsl. eleganter, mit entity.getVelocity() / 20 zu arbeiten. Das dürfte einiges an Verwaltungsaufwand sparen. Man könnte dann wsl. in run auch mit world.getEntitiesWithClass() arbeiten.

Statt der (aufwändigen) Verwaltung von LAST_LOCATIONS wäre es wsl. eleganter, mit entity.getVelocity() / 20 zu arbeiten. Das dürfte einiges an Verwaltungsaufwand sparen. Man könnte dann wsl. in run auch mit world.getEntitiesWithClass() arbeiten.
Chaoscaot hat 2020-12-30 13:38:59 +01:00 kommentiert
Veraltet
Review
Link kopieren

?, entity.getVelocity() gibt einen Vektor zurück, welchen man nicht einfach durch 20 teilen kann

?, entity.getVelocity() gibt einen Vektor zurück, welchen man nicht einfach durch 20 teilen kann
Lixfel hat 2020-12-30 13:45:18 +01:00 kommentiert
Veraltet
Review
Link kopieren

.divide?

.divide?
Chaoscaot hat 2020-12-30 14:10:49 +01:00 kommentiert
Veraltet
Review
Link kopieren

braucht einen Vektor

braucht einen Vektor
Lixfel hat 2020-12-30 14:14:27 +01:00 kommentiert
Veraltet
Review
Link kopieren

Du brauchst ja sowieso nur die einzelnen Komponenten, also teile doch einfach die durch 20....

Du brauchst ja sowieso nur die einzelnen Komponenten, also teile doch einfach die durch 20....
public ArrowStopper() { public ArrowStopper() {
super(Config.TechhiderActive ? EnumSet.of(FightState.RUNNING) : EnumSet.noneOf(FightState.class)); super(Config.TechhiderActive ? EnumSet.of(FightState.RUNNING) : EnumSet.noneOf(FightState.class));
} }
@ -74,28 +74,20 @@ public class ArrowStopper extends BasicListener {
private boolean checkBlocks(Block start, Block end) { private boolean checkBlocks(Block start, Block end) {
Block cursor = start; Block cursor = start;
while (cursor.getY() != end.getY()) { while (!cursor.getLocation().equals(end.getLocation())) {
boolean negativ = cursor.getY() - end.getY() < 0; BlockFace nearest = BlockFace.SELF;
BlockFace face = negativ?BlockFace.UP:BlockFace.DOWN; double nearestDistance = cursor.getLocation().distance(end.getLocation());
if(checkBlock(cursor)) for (BlockFace face : BLOCK_FACES) {
Lixfel hat 2020-12-21 07:56:04 +01:00 kommentiert
Veraltet
Review
Link kopieren

Das kannst du nicht Asynchron machen (es dauert 50ms+, bis du einen Chunk für Blockabfragen bekommst), das produziert hier nur Threads.

Das kannst du nicht Asynchron machen (es dauert 50ms+, bis du einen Chunk für Blockabfragen bekommst), das produziert hier nur Threads.
return true; Block relative = cursor.getRelative(face);
cursor = cursor.getRelative(face); double distance = relative.getLocation().distance(end.getLocation());
if(distance < nearestDistance) {
Lixfel hat 2021-01-02 08:30:14 +01:00 kommentiert
Review
Link kopieren

Ok, wenn ich jetzt richtig verstanden habe, was diese Funktion macht, dann tut sie linear zwischen den Blöcken interpolieren? Ich meine, das ist zwar nicht die korrekte Minecraft-Berechnungsreihenfolge, soll mir aber auch recht sein.

Ok, wenn ich jetzt richtig verstanden habe, was diese Funktion macht, dann tut sie linear zwischen den Blöcken interpolieren? Ich meine, das ist zwar nicht die korrekte Minecraft-Berechnungsreihenfolge, soll mir aber auch recht sein.
nearestDistance = distance;
nearest = face;
} }
while (cursor.getX() != end.getX()) {
boolean negativ = cursor.getX() - end.getX() < 0;
BlockFace face = negativ?BlockFace.EAST:BlockFace.WEST;
if(checkBlock(cursor))
return true;
cursor = cursor.getRelative(face);
} }
cursor = cursor.getRelative(nearest);
while (cursor.getZ() != end.getZ()) {
boolean negativ = cursor.getZ() - end.getZ() < 0;
BlockFace face = negativ?BlockFace.SOUTH:BlockFace.NORTH;
if(checkBlock(cursor)) if(checkBlock(cursor))
return true; return true;
cursor = cursor.getRelative(face);
} }
return false; return false;