Fehler an Plugin

[herberti]

Ich finde den Fehler bei deisem Code nicht:

http://pastebin.com/0Hr6jJp1

Ich bekomme pausenlos Test ausgegeben obwohl es nur alle 60 Sekunden ausgegeben werden sollte.
Meine vermutung ist, dass etwas an diesem Teil nicht stimmt.

Calendar aktuellezeit = Calendar.getInstance(Locale.GERMANY);
    long difference = aktuellezeit.getTimeInMillis() - hmap.get(event.getPlayer()).getTimeInMillis();
    int seconds = (int)(difference / 1000 % 60);
    if(seconds <= 60)

 

[ChunkNorris]

lade das mal bei pastebin oder so hoch, das kann ja kein schwein lesen.

 

[Calenria]

http://pastebin.com/5DaD0pdh

du gibst auch test aus wenn der Player nicht in der HashMap ist und übrigens sollte man keine Player Objekte in ner HashMap speichern.

Edit: Warum speicherst du das Calendar Objekt auch in der HashMap du brauchst doch eh nur den Timestamp da reicht ein System.currentTimeMillis().

Schau mal ob das besser funktioniert, nicht getestet.

public class Reward extends JavaPlugin implements Listener {    
    public HashMap<String, Long> hasw = new HashMap<String, Long>();


    @Override
    public void onEnable() {
        this.getServer().getPluginManager().registerEvents(this, this);
    }


    @EventHandler
    public void onPlayerMove(PlayerMoveEvent event) {
        Player player = event.getPlayer();
        Location loc = player.getLocation().add(0.0D, -1.0D, 0.0D);
        if (hasw.containsKey(player.getName())) {
            long difference = System.currentTimeMillis() - hasw.get(player.getName());
            int seconds = (int) (difference / 1000 % 60);
            if (seconds <= 60) {
                if (loc.getBlock().getTypeId() == 9) {
                    player.sendMessage("Test 1");
                }
            }
        } else {
            hasw.put(player.getName(), System.currentTimeMillis());
            if (loc.getBlock().getTypeId() == 9) {
                player.sendMessage("Test 2");
            }
        }
    }
}

 

[herberti]

Jetzt wird garnichts mehr ausgegeben und der Spieler kommt auch nicht in die Hashmap.
Test soll auch ausgegeben werden wenn der Spieler noch nicht in der Hashmap ist, danach ist er es ja.

 

[Calenria]

Kommt davon wen man es nicht Testet, hab es geändert

 

[herberti]

Jetzt wird allerdings bei jeder bewegung Test 1 ausgegeben.

 

[Calenria]

Jetzt wird allerdings bei jeder bewegung Test 1 ausgegeben.

klar weil die Sekunden ja kleiner als 60 sind. Was soll das den überhaupt werden?

 

[herberti]

Wenn man Wasser betritt soll alle 60 Sekunden was ausgegeben werden.
Aber nur wenn man auf Wasser steht und nur alle 60 Sekunden.

 

[Calenria]

public class Reward extends JavaPlugin implements Listener {
    public HashMap<String, Long> hasw = new HashMap<String, Long>();


    @Override
    public void onEnable() {
        this.getServer().getPluginManager().registerEvents(this, this);
    }


    @EventHandler
    public void onPlayerMove(PlayerMoveEvent event) {
        Player player = event.getPlayer();
        Location loc = player.getLocation().add(0.0D, -1.0D, 0.0D);
        if (hasw.containsKey(player.getName())) {
            long difference = System.currentTimeMillis() - hasw.get(player.getName());
            long seconds = (difference / 1000 % 60);
            if (loc.getBlock().getTypeId() != 9) {
                hasw.remove(player.getName());
                player.sendMessage("Spieler hat Wasser verlassen");
                return;
            } else {
                if (seconds == 59) {
                    player.sendMessage("Spieler 60 Sekunden im Wasser");
                    hasw.remove(player.getName());
                }
            }


        } else {
            if (loc.getBlock().getTypeId() == 9) {
                hasw.put(player.getName(), System.currentTimeMillis());
                player.sendMessage("Spieler im Wasser");
            }
        }
    }
}

 

[herberti]

Ich hab mich etwas schlecht ausgedrückt.
Einfach gesagt: Wenn man über z.B. Stone läuft bekommt man ein Item das aber nur alle 60 Sekunden

 

[Calenria]

Also ohne reset wenn er das Wasser verlässt?

 

[herberti]

Also ohne reset wenn er das Wasser verlässt?

Ja allerdings klappt das mit dem 60 Sekunden im Wasser nicht.

 

[Calenria]

Läuft bei mir, ich hab mal die Sekunden ausgeben lassen, klappt soweit alles.

public class Reward extends JavaPlugin implements Listener {
    public HashMap<String, Long> hasw = new HashMap<String, Long>();


    @Override
    public void onEnable() {
        this.getServer().getPluginManager().registerEvents(this, this);
    }


    @EventHandler
    public void onPlayerMove(PlayerMoveEvent event) {
        Player player = event.getPlayer();
        Location loc = player.getLocation().add(0.0D, -1.0D, 0.0D);
        if (hasw.containsKey(player.getName())) {
            long difference = System.currentTimeMillis() - hasw.get(player.getName());
            long seconds = (difference / 1000 % 60);
            System.out.println(seconds);
            if (seconds == 59) {
                if (loc.getBlock().getTypeId() == 9) {
                    player.sendMessage("Spieler 60 Sekunden im Wasser");
                    hasw.remove(player.getName());
                }
            }
        } else {
            if (loc.getBlock().getTypeId() == 9) {
                hasw.put(player.getName(), System.currentTimeMillis());
                player.sendMessage("Spieler im Wasser");
            }
        }
    }
}

 

[herberti]

Ja die Sekunden werden gezählt soweit geht es auch, aber es kommt nach den 59 Sekunden nie Spieler 60 Sekunden im Wasser und wenn man danach das Wasser wieder betritt Spieler im Wasser.

 

[Calenria]

Hab den EventHandler nochmal angepasst:

Grad noch mal ein wenig überlegt, im Grunde müsstest du das ganze andersherum aufziehen und jedesmal die Sekunden hochzählen wenn ein Spieler auf dem gewünschten Block läuft. Also bei jedem PlayerMoveEvent wo der Spieler auf dem Block ist 2 Ticks hochzählen. Soweit ich weiß kommt das Event nur alle Zwei Ticks beim Plugin an.

 

[herberti]

Bei mir fängt es erneut zum zählen an und gibt kein 60 Sek im Wasser aus.

 

[Calenria]

Mal andersherum, hoffe das wars dan nu

public class Reward extends JavaPlugin implements Listener {
    public HashMap<String, Long> hasw   = new HashMap<String, Long>();
    final private static long    MINUTE = 1200L;


    @Override
    public void onEnable() {
        this.getServer().getPluginManager().registerEvents(this, this);
    }


    @EventHandler
    public void onPlayerMove(PlayerMoveEvent event) {
        Player player = event.getPlayer();
        Block block = player.getLocation().getBlock().getRelative(BlockFace.DOWN);
        if (hasw.containsKey(player.getName())) {
            if (block.getTypeId() == 9) {
                long currTicks = hasw.get(player.getName()) + 1L;
                hasw.put(player.getName(), currTicks);
                if (currTicks >= MINUTE) {
                    player.sendMessage("Spieler 60 Sekunden im Wasser");
                    hasw.remove(player.getName());
                }
            }
        } else {
            if (block.getTypeId() == 9) {
                hasw.put(player.getName(), 0L);
            }
        }
    }
}

 

[herberti]

Nein, leider noch immer das selbe wie vorhin.

 

[Calenria]

Bei klappt es nun einwandfrei, es zählt auf nur wenn das Wasser mindestens 2 Blöcke tief ist und man schwimmt.

 

[herberti]

Bei mir nicht
Sobald ich ins Wasser gehe steht da Spieler im Wasser.
Aber es kommt nie 60 Sekunden im Wasser auch wenn es wieder von 0 zu zählen beginnt.