Arrows Stopping in Techhider Blocks #208
@ -26,7 +26,6 @@ import de.steamwar.fightsystem.states.FightState;
|
||||
import de.steamwar.fightsystem.states.StateDependent;
|
||||
import org.bukkit.Bukkit;
|
||||
import org.bukkit.Location;
|
||||
import org.bukkit.Material;
|
||||
import org.bukkit.block.Block;
|
||||
import org.bukkit.block.BlockFace;
|
||||
import org.bukkit.entity.AbstractArrow;
|
||||
@ -52,18 +51,14 @@ public class ArrowStopper implements StateDependent {
|
||||
if(arrows.isEmpty())
|
||||
return;
|
||||
for (Entity entity : arrows) {
|
||||
if(entity.getTicksLived() > Config.ArrowTechhiderCollision){
|
||||
LAST_LOCATION.remove(entity);
|
||||
continue;
|
||||
}
|
||||
if(((AbstractArrow) entity).isInBlock()) {
|
||||
if(entity.getTicksLived() > Config.ArrowTechhiderCollision ||
|
||||
((AbstractArrow) entity).isInBlock()){
|
||||
|
|
||||
LAST_LOCATION.remove(entity);
|
||||
Lixfel
hat
Auch wenn die arrows Empty sind, sollten die Pfeile aus der LAST_LOCATION entfernt werden (also mehr oder weniger der normale Programmfluss genommen werden), oder? Auch wenn die arrows Empty sind, sollten die Pfeile aus der LAST_LOCATION entfernt werden (also mehr oder weniger der normale Programmfluss genommen werden), oder?
|
||||
continue;
|
||||
}
|
||||
|
Lixfel
hat
Hätte es gerne, dass du zur kognitiven Komplexitätsreduktion einfach am Ende prüfst (beim derzeitigen else), wie lange die Entity schon gelebt hat, und sie dann ggf. einfach nicht mehr einfügst. Hätte es gerne, dass du zur kognitiven Komplexitätsreduktion einfach am Ende prüfst (beim derzeitigen else), wie lange die Entity schon gelebt hat, und sie dann ggf. einfach nicht mehr einfügst.
Lixfel
hat
Bitte auch noch "tote" Arrows/ stillstehende Arrows rauswerfen. Bitte auch noch "tote" Arrows/ stillstehende Arrows rauswerfen.
|
||||
if(!LAST_LOCATION.containsKey(entity))
|
||||
|
Lixfel
hat
Wenn man so häufig e.getKey() verwendet (weiter unten ja auch noch), ist es einfach wesentlich besser lesbar, wenn man das dann einfach mal als temporäre extravariable deklariert. Wenn man so häufig e.getKey() verwendet (weiter unten ja auch noch), ist es einfach wesentlich besser lesbar, wenn man das dann einfach mal als temporäre extravariable deklariert.
|
||||
LAST_LOCATION.put(entity, ((Player) ((AbstractArrow) entity).getShooter()).getEyeLocation());
|
||||
|
Lixfel
hat
Das dürfte glaube ich nicht den Wert aus dem Set entfernen, sondern nur die Entity aus der Welt. LAST_LOCATION wird damit immer weiter zugemüllt. Das dürfte glaube ich nicht den Wert aus dem Set entfernen, sondern nur die Entity aus der Welt. LAST_LOCATION wird damit immer weiter zugemüllt.
|
||||
Location last = LAST_LOCATION.get(entity);
|
||||
LAST_LOCATION.remove(entity);
|
||||
Location last = LAST_LOCATION.remove(entity);
|
||||
|
Lixfel
hat
Bitte reduziere mal deine Map-Operationen. Optimierungsidee: Du machst LAST_LOCATION.remove(), und wenn du null bekommst, nimmst du deine EyeLocation, 1,5 Mapoperationen weniger. Bitte reduziere mal deine Map-Operationen. Optimierungsidee: Du machst LAST_LOCATION.remove(), und wenn du null bekommst, nimmst du deine EyeLocation, 1,5 Mapoperationen weniger.
|
||||
if(checkBlocks(entity.getLocation().getBlock(), last.getBlock()))
|
||||
entity.remove();
|
||||
else
|
||||
|
Lixfel
hat
Ich glaube, replace() wäre hier die angebrachtere Operation Ich glaube, replace() wäre hier die angebrachtere Operation
|
||||
@ -78,7 +73,7 @@ public class ArrowStopper implements StateDependent {
|
||||
|
||||
@Override
|
||||
|
Lixfel
hat
Eine solche Funktionalität sollte bereits mit dem Techhider bzw. RecordSystem enthalten sein, wenn würde ich diese generalisieren. Eine solche Funktionalität sollte bereits mit dem Techhider bzw. RecordSystem enthalten sein, wenn würde ich diese generalisieren.
|
||||
public void enable() {
|
||||
task = Bukkit.getScheduler().runTaskTimerAsynchronously(FightSystem.getPlugin(), this::run, 1, 0);
|
||||
task = Bukkit.getScheduler().runTaskTimerAsynchronously(FightSystem.getPlugin(), this::run, 1, 1);
|
||||
}
|
||||
|
||||
|
Lixfel
hat
Spieler heben keine Pfeile auf (ist unterbunden) Spieler heben keine Pfeile auf (ist unterbunden)
|
||||
@Override
|
||||
|
||||
Landen arrows wirklich in Blöcken?