Countdown Fehler

[MrPyro13]

Hallo,

kannst mir jemand sagen, was an diesem Code falsch ist?

package de.mrpyro13.ImpComs;
import org.bukkit.Bukkit;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.plugin.Plugin;
public class iCountdown_Command implements CommandExecutor {
@Override
public boolean onCommand(final CommandSender sender, Command cmd, String label, String[] args) {
//SCRIPT ANFANG
int countdown;
sender.sendMessage("§aDer Countdown wurde gestartet");
countdown = Bukkit.getScheduler().scheduleSyncRepeatingTask((Plugin) this,new Runnable() {
int x;
int high = 11;
@Override
public void run() {
if(high != 0) {
high--;
sender.sendMessage("§6Countdown läuft noch §b " + high + " §5Sekunden");
} else {
sender.sendMessage("§aCountdown vorbei");
Bukkit.getScheduler().cancelTask(countdown);
}
}
}, 0, 20);
//SCRIPT ENDE
return true;
}
}

Bukkit.getScheduler().cancelTask(countdown);

Hier wird das countdown angestrichen, man soll die Variable oben (int countdown) final(en)

Final ich es wird in diesem Abschnitt das countdown angestrichen:

countdown = Bukkit.getScheduler().scheduleSyncRepeatingTask((Plugin) this,new Runnable() {

 

[games6471]

Ein kleiner Hinweis. Du kannst nicht einfach ein CommandExecutor zu Plugin casten.

Bukkit.getScheduler().scheduleSyncRepeatingTask((Plugin) this,new Runnable()

 

[MrPyro13]

Schreibe ich aber nur das this, dann wird der scheduleSyncRepeatingTask unterstrichen :/

 

[FelixKlauke]

Hallo,

kannst mir jemand sagen, was an diesem Code falsch ist?

package de.mrpyro13.ImpComs;
import org.bukkit.Bukkit;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.plugin.Plugin;
public class iCountdown_Command implements CommandExecutor {
@Override
public boolean onCommand(final CommandSender sender, Command cmd, String label, String[] args) {
//SCRIPT ANFANG
int countdown;
sender.sendMessage("§aDer Countdown wurde gestartet");
countdown = Bukkit.getScheduler().scheduleSyncRepeatingTask((Plugin) this,new Runnable() {
int x;
int high = 11;
@Override
public void run() {
if(high != 0) {
high--;
sender.sendMessage("§6Countdown läuft noch §b " + high + " §5Sekunden");
} else {
sender.sendMessage("§aCountdown vorbei");
Bukkit.getScheduler().cancelTask(countdown);
}
}
}, 0, 20);
//SCRIPT ENDE
return true;
}
}

Hier wird das countdown angestrichen, man soll die Variable oben (int countdown) final(en)

Final ich es wird in diesem Abschnitt das countdown angestrichen:

Ein kleiner Hinweis. Du kannst nicht einfach ein CommandExecutor zu Plugin casten.

Bukkit.getScheduler().scheduleSyncRepeatingTask((Plugin) this,new Runnable()

Du musst daran denken, deine mainclass da zu uebergeben und keine x-beliebige. Erzeuge eine neue Instanz deiner mainclass im Konstruktor. Diese kannst du dann uebergeben.

 

[MrPyro13]

Du musst daran denken, deine mainclass da zu uebergeben und keine x-beliebige. Erzeuge eine neue Instanz deiner mainclass im Konstruktor. Diese kannst du dann uebergeben.

Und das ging nochmal wie?

 

[games6471]

Du kannst die Plugininstanz wie folgt über dem Konstruktor übergeben:

http://wiki.bukkit.org/Plugin_Tutorial#Using_a_separate_CommandExecutor_class

public class MyPluginCommandExecutor implements CommandExecutor {
    private final MyPlugin plugin;

    public MyPluginCommandExecutor(MyPlugin plugin) {
        this.plugin = plugin; // Store the plugin in situations where you need it.
    }

Und bei der Erstellung einer Instanz (new ...) dieser Klasse die Hauptklasse übergeben. Das bedeutet new blabla(instance) Also in der Hauptklasse (JavaPlugin) mit this.

new MyPluginCommandExecutor(this)

 

[FelixKlauke]

In der Klasse mit dem Scheduler:

private <namedermain> inst;

public <namederklasse>(<Namedermain> plugin){
    this.inst = plugin;
}

Und dann dem Scheduler "inst" uebergeben.

In der Mainklasse machste dann:

new <namederschedulerklasse>(this);

Versuch aber erst, den simn zu verstehen und es dann zu kopieren!

 

[MrPyro13]

Das funktioniert jetzt. Jedoch geht das mit dem countdown nicht

package de.mrpyro13.ImpComs;

import org.bukkit.Bukkit;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.plugin.Plugin;

public class iCountdown_Command implements CommandExecutor {

    private Main inst;

    public iCountdown_Command(Main plugin){
        this.inst = plugin;
    }

    @Override
    public boolean onCommand(final CommandSender sender, Command cmd, String label, String[] args) {
        //SCRIPT ANFANG
    
        int countdown;
        sender.sendMessage("§aDer Countdown wurde gestartet");
        countdown = Bukkit.getScheduler().scheduleSyncRepeatingTask(inst, new Runnable() {
            int x;
            int high = 11;
            @Override
            public void run() {
                if(high != 0) {
                    high--;
                    sender.sendMessage("§6Countdown läuft noch §b " + high + " §5Sekunden");
                } else {
                    sender.sendMessage("§aCountdown vorbei");
                    Bukkit.getScheduler().cancelTask(countdown);
                }
            }
        }, 0, 20);
    
        //SCRIPT ENDE
        return true;
    }

}

Das was ich rot markiert habe, buggt: Ich fahre über den Fehler, als Problemlösung kommt das:

final int countdown;

Dann ist es immernoch rot markiert und es kommt als Problemlösung das:

final int countdown = 0;

Jetzt ist das rot markierte richtig, aber das grün markierte wird rot unterstrichen und möchte das final weg haben.
Langsam drehe ich durch

@xXSasukeXx und @games6471
Programmierst ihr auch in extra Klassen oder eher in der Main?

 

[games6471]

Einfache Alternative:

new BukkitRunnable() {
            int x;
            int high = 11;

            @Override
            public void run() {
                if (high != 0) {
                    high--;
                    sender.sendMessage("§6Countdown läuft noch §b " + high + " §5Sekunden");
                } else {
                    sender.sendMessage("§aCountdown vorbei");
                    this.cancel();
                }
            }
        }.runTaskTimer(inst, 0, 20);

Man sollte seinen Programmcode logisch in verschiedene Klassen unterteilen und nicht alles in eine Klasse stopfen. Das verbessert deutlich die Übersichtlichkeit und viele weitere Dinge.

 

[deleted133530]

Wenn du keine weitere Verwendung für sie hast, könntest du die Variablen "countdown" und "x" einfach löschen... Momentan werden sie jedenfalls nicht benutzt.

 

[MiCrJonas]

Programmierst ihr auch in extra Klassen oder eher in der Main?

Du solltest auf jeden Fall extra Klassen benutzen, dafür ist Java da.

 

[MrPyro13]

Ich nutze auch extra Klassen ^^