Klammern

[Anonymo]

Hallihallo liebe mseu User.
Da ich mich in letzter Zeit ans Plugins programmieren Rantaste, jedoch ein Probleme mit Eclipse habe wollte ich euch mal fragen ob ihr euch das ansehen könnt:

package me.enderelf.joinmessage;

import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.GameMode;
import org.bukkit.Location;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.event.player.PlayerJoinEvent;
import org.bukkit.event.player.PlayerQuitEvent;
import org.bukkit.plugin.java.JavaPlugin;

public class main extends JavaPlugin implements Listener{
   
   @Override
   public void onEnable() {
     this.getServer().getPluginManager().registerEvents(this,  this);
     System.out.println("[SkyPvP]Plugin erfolgreich aktiviert!");
     
   }

   @Override
   public void onDisable() {
     System.out.println("[SkyPvP] Plugin erfolgreich deaktiviert!");
     
   }
   
   public boolean onCommand(CommandSender sender, Command cmd, String cmdlabel, String[] args) {
   
    Player p = (Player) sender;
     
    if(cmd.getName().equalsIgnoreCase("regeln"))
    {
    if(p.hasPermission("SkyPvP.regeln"))
    {
    p.sendMessage("§8[SkyPvP] §6Hier stehen bald die Regeln!");
    return true;
      }else{
         p.sendMessage("§8[SkyPvP] §6Das darfst du nicht!");
    return true;
    }
    }
    if(cmd.getName().equalsIgnoreCase("SkyPvP"))
    {
    if(p.hasPermission("SkyPvP.SkyPvP"))
    {
    p.sendMessage("§8[SkyPvP] §6Das Plugin wurde von EnderElf geschrieben!");
    return true;
      }else{
         p.sendMessage("§8[SkyPvP] §6Das darfst du nicht!");
    return true;
    }
    }
    if(cmd.getName().equalsIgnoreCase("event19"))
    {
    if(p.hasPermission("SkyPvP.event19"))
    {
    Bukkit.broadcastMessage("§4Sky §8> §9Event um 19 Uhr!");
    return true;
      }else{
         p.sendMessage("§8[SkyPvP] §6Das darfst du nicht!");
    return true;
    }
    }
    if(cmd.getName().equalsIgnoreCase("event20"))
    {
    if(p.hasPermission("SkyPvP.event20"))
    {
    Bukkit.broadcastMessage("§4Sky §8> §9Event um 20 Uhr!");
    return true;
      }else{
         p.sendMessage("§8[SkyPvP] §6Das darfst du nicht!");
    return true;
    }
    }
    if(cmd.getName().equalsIgnoreCase("infos"))
    {
    if(p.hasPermission("SkyPvP.infos"))
    {
    p.sendMessage(ChatColor.DARK_GREEN + "Infos");
    p.sendMessage(ChatColor.AQUA + "Test");
    return true;
      }else{
         p.sendMessage("§8[SkyPvP] §6Das darfst du nicht!");
    return true;
    }
    }
    if(cmd.getName().equalsIgnoreCase("eventj"))
    {
    if(p.hasPermission("SkyPvP.eventjetzt"))
    {
    Bukkit.broadcastMessage("§4Sky §8> §9Event jetzt!");
    return true;
      }else{
         p.sendMessage("§8[SkyPvP] §6Das darfst du nicht!");
    return true;
    }
    }
    if(cmd.getName().equalsIgnoreCase("skype"))
    {
    if(p.hasPermission("SkyPvP.skype"))
    {
    p.sendMessage("§4Skype: §5QuantumSupp");
    return true;
    }else{
         p.sendMessage("§8[SkyPvP] §6Das darfst du nicht!");
    return true;
    }
    }
    if(cmd.getName().equalsIgnoreCase("bewerben"))
    {
    if(p.hasPermission("SkyPvP.bewerben"))
    {
    p.sendMessage("§2Bitte schreibe eine Bewerbung mit wenigstens"
         + " 150 Zeichen an unseren Skype account: QuantumSupp"
         + " in der Bewerbung sollte dein Ingame Name stehen."
         + " Um dich zu Bewerben musst du mindestens 13 Jahre alt sein.");
    return true;
    }else{
         p.sendMessage("§8[SkyPvP] §6Das darfst du nicht!");
    return true;
    }
    }
    if(cmd.getName().equalsIgnoreCase("cc"))
    {
    if(p.hasPermission("SkyPvP.cc"))
    {
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("");
    Bukkit.broadcastMessage("§8[SkyPvP] §6Der Chat wurde von " + p.getName() + " gesäubert!");
    return true;
    }else{
         p.sendMessage("§8[SkyPvP] §6Das darfst du nicht!");
     
    return true;
    }
    }
    if(cmd.getName().equalsIgnoreCase("ccp"))
    {
    if(p.hasPermission("SkyPvP.ccp"))
    {
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("");
       p.sendMessage("§8[SkyPvP] §6Du hast deinen Chat gelöscht! ");
       return true;
    }else{
         p.sendMessage("§8[SkyPvP] §6Das darfst du nicht!");
    return true;
    }
    }
     
    return false;
   }
     @EventHandler
     public void onJoin(PlayerJoinEvent e) {
       Player p = e.getPlayer();
       {
         this.getServer().getScheduler().scheduleSyncDelayedTask(this, new Runnable() {

           public void run() {
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage("");
             p.sendMessage(ChatColor.GOLD + "[Sky-PvP]");
             p.sendMessage(ChatColor.DARK_GRAY + "_______________________");
             p.sendMessage("");
             p.sendMessage(ChatColor.BLUE + "Pro Kill bekommst du" + ChatColor.RED + " 5 Level");
             p.sendMessage(ChatColor.DARK_GRAY + "_______________________");
             e.setJoinMessage("");
       }
     @EventHandler
     public void onquit(PlayerQuitEvent e) {
       e.setQuitMessage("");
       }
     @EventHandler
       public void ondeath(PlayerDeathEvent e) {
         e.setDeathMessage("");
     }
         }

mein Problem:
In Zeile 541 und 370 werden die } klammern rot Makiert, jedoch sollte es eigentlich richtig sein.
Danke für jede nette antwort ;D

Lg Lukas

 

[Victini151]

Huhu, Einrückung würde deinem Code sehr gut tun - es würde auch unglaublich bei der Fehlersuche helfen.

 

[FelixKlauke]

Huhu, Schleifen und Java Name Conventions würden deinem Code sehr gut tun - Es würde auch unglaublich bei der Analyse und Skalierbarkeit helfen.

 

[Verschraubt]

Habe ein paar Fehler gefunden. Der Code hier müsste passen. PS: Es gibt da so ne schöne Tastenkombination([Strg] + [Shift] + F)

Link für pastebin: http://pastebin.com/hv0n7X8N

//Bearbeitet

 

[Victini151]

int sched;
    int i = 1;

    @EventHandler
    public void onJoin1(final PlayerJoinEvent e) {
        final Player p = e.getPlayer();
    
    
    
        sched = Bukkit.getScheduler().scheduleAsyncRepeatingTask(this, new Runnable() {
        
            @Override
            public void run() {
                p.sendMessage("");
                i++;
                if (i >= 150){
                    Bukkit.getScheduler().cancelTask(sched);
                    p.sendMessage(ChatColor.GOLD + "[Sky-PvP]");
                    p.sendMessage(ChatColor.DARK_GRAY + "_______________________");
                    p.sendMessage("");
                    p.sendMessage(ChatColor.BLUE + "Pro Kill bekommst du" + ChatColor.RED + " 5 Level");
                    p.sendMessage(ChatColor.DARK_GRAY + "_______________________");
                    e.setJoinMessage("");
                }
            }
        }, 1, 1);
    }

Ich muss ein Zitat aus diesem Forum verwenden:
"Dieser Code ist auf so viele Arten so falsch"

A) Eine Async-Task ist hier absolut nicht das was genutzt werden soll. Stichwort Threadsicherheit. Und das mit der Joinmessage dürfte sogar ne Race-Condition sein.

B) Eine Schleife tuts auch. Dafür braucht man den Scheduler nicht bemühen.

C) Nach dem ersten Spieler gehts zu 100% nimmer, da der Zähler hängt.

 

[Verschraubt]

Ja, mag sein, dass der Code nicht gerade ideal ist (Habe ihn kurzfristig hier geschrieben), aber er soll als Inspiration für einfachere Möglichkeiten dienen.