Fehler Meldung "else without if"

pumpgun99

Neues Mitglied
ich komm nicht voran es zeigt immer diese Fehler Meldung "else without if"
Bin für jede hilfe dankbar:

package de.pumpgun99.plugin1.commands;

import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;

public class FlyCommand implements CommandExecutor {

@Override
public boolean onCommand(CommandSender s, Command cmd, String label, String[] args){
if(s.hasPermission("fly.own")){
if(s instanceof Player){
Player p = (Player)s;
if(!p.getAllowFlight()){
p.setAllowFlight(true);
p.setFlying(true);
p.sendMessage("Lets fly!");
}else{
p.setAllowFlight(false);
p.setFlying(false);
p.sendMessage("No fly");
}


} else {
s.sendMessage("§c Du bist ein kein Spieler bitte Logge dich ein ");

}else {
s.sendMessage("§c Du hast keine Rechte ");

return false;
}

}
}
}
 

LimDul

Top Contributor
Code bitte in Code-Tags, dann bleiben die Einrückungen erhalten:
Code:
package de.pumpgun99.plugin1.commands;

import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;

public class FlyCommand implements CommandExecutor {

    @Override
    public boolean onCommand(CommandSender s, Command cmd, String label, String[] args){
        if(s.hasPermission("fly.own")){
            if(s instanceof Player){
                Player p = (Player)s;
                if(!p.getAllowFlight()){
                    p.setAllowFlight(true);
                    p.setFlying(true);
                    p.sendMessage("Lets fly!");
                }else{
                    p.setAllowFlight(false);
                    p.setFlying(false);
                    p.sendMessage("No fly");
                }


        } else {
                s.sendMessage("§c Du bist ein kein Spieler bitte Logge dich ein  ");

            }else {
            s.sendMessage("§c Du hast keine Rechte  ");

        return false;
    }

    }
    }
}

Das erste was auffällt - die Einrückungen sind willkürlich. Daher ist es gerade extream schwer zu erkennen ,welches else zu welchem if gehört (Die Verschachtelungstiefe ist auf jeden Fall auch zu hoch, der Code ist kaum lesbar). Aber korrigieren wir mal die Einrückungen (Die erste und letzte öffenende Klammer von der Klasse lass ich mal weg)

Java:
public boolean onCommand(CommandSender s, Command cmd, String label, String[] args){
    if(s.hasPermission("fly.own")){
        if(s instanceof Player){
            Player p = (Player)s;
            if(!p.getAllowFlight()){
                p.setAllowFlight(true);
                p.setFlying(true);
                p.sendMessage("Lets fly!");
            }else{
                p.setAllowFlight(false);
                p.setFlying(false);
                p.sendMessage("No fly");
            }
        } else {
            s.sendMessage("§c Du bist ein kein Spieler bitte Logge dich ein  ");
    }else {
        s.sendMessage("§c Du hast keine Rechte  ");

       return false;
    }
}
}
Was fällt auf? Da ist eine schließende Klammer zu viel. Vermutlich muss die schließende Klammer hinter der "No Fly" Ausgabe weg, und dann sollte alles passen.

Bei sowas generell die Formatierung der IDE nutzen, dass kann jede gute IDE - dann merkt man schnell, das mit den Klammern was im Argen liegt.
 
K

kneitzel

Gast
Was auf jeden fall auch nicht gehen kann (und wohl den Fehler vervorruft:
Java:
} else {
    s.sendMessage("§c Du bist ein kein Spieler bitte Logge dich ein  ");
} else

Zwei else hintereinander geht natürlich nicht. ==> Die }, die am Ende zu viel ist, fehlt da wohl.

Hier ist meine generelle Forderung:
Teile und Herrsche!

Also mach mehr Methoden! Gerade als Anfänger sollte man Methoden kurz und prägnant halten mit guten Namen. Dann hat man auch automatisch eine Chance, seinen eigenen Code nachzuvollziehen.

Und sowas kann man schnell kürzen / aufteilen:

Ursprünglicher Code - mit verbesserter } und das return false ans Ende gestellt, denn alle Durchgänge müssen einen Wert zurück geben!:
Java:
public boolean onCommand(CommandSender s, Command cmd, String label, String[] args) {
    if(s.hasPermission("fly.own")){
        if(s instanceof Player){
            Player p = (Player)s;
            if(!p.getAllowFlight()){
                p.setAllowFlight(true);
                p.setFlying(true);
                p.sendMessage("Lets fly!");
            }else{
                p.setAllowFlight(false);
                p.setFlying(false);
                p.sendMessage("No fly");
            }
        } else {
            s.sendMessage("§c Du bist ein kein Spieler bitte Logge dich ein  ");
        }
    } else {
        s.sendMessage("§c Du hast keine Rechte  ");
    }
    return false;
}

So man nur Commands erlaubt, wenn die Rechte stimmen und ein Spieler vorliegt, dann wäre eine Vereinfachung, dies an den Anfang zu ziehen:
Java:
public boolean onCommand(CommandSender s, Command cmd, String label, String[] args) {
    if(!s.hasPermission("fly.own")) {
        s.sendMessage("§c Du hast keine Rechte  ");
        return false;
    }
    
    if(!(s instanceof Player) {
        s.sendMessage("§c Du bist ein kein Spieler bitte Logge dich ein  ");
        return false;
    }
    
    // Restlicher code ...
}

Das ist dann sozusagen eine Validierung am Anfang der Methode ...

Aber ansonsten werden Funktionalitäten raus gezogen, also z.B. mit einer Methode flyPlayer oder so:
Java:
public boolean onCommand(CommandSender s, Command cmd, String label, String[] args) {
    if(!s.hasPermission("fly.own")) {
        s.sendMessage("§c Du hast keine Rechte  ");
        return false;
    }

    if(!(s instanceof Player) {
        s.sendMessage("§c Du bist ein kein Spieler bitte Logge dich ein  ");
        return false;
    }

    return flyPlayer((Player) s);
}

public boolean flyPlayer(Player p) {
    if(!p.getAllowFlight()){
        p.setAllowFlight(true);
        p.setFlying(true);
        p.sendMessage("Lets fly!");
    }else{
        p.setAllowFlight(false);
        p.setFlying(false);
        p.sendMessage("No fly");
    }
    return true;
}

So in der Art. Auch die Methode könnte man noch unterteilen oder umstrukturieren, aber ich wollte er nur einmal vom Prinzip her zeigen....
 

temi

Top Contributor
So in der Art. Auch die Methode könnte man noch unterteilen oder umstrukturieren, aber ich wollte er nur einmal vom Prinzip her zeigen....
Ich würde die Hilfsmethoden allerdings "private" machen. Sie werden ja i. d. R. nicht von außerhalb der Klasse aufgerufen, bzw. sollen das auch nicht werden können: private boolean flyPlayer(Player p)
 
Ähnliche Java Themen
  Titel Forum Antworten Datum
M Compiler-Fehler Fehler Meldung java.lang.NumberFormatException: empty String Java Basics - Anfänger-Themen 2
K Fehler Meldung java.lang.NumberFormatException: empty String Java Basics - Anfänger-Themen 1
S Fehler Meldung und Runntimeexaption Java Basics - Anfänger-Themen 15
F Kann mir jemand bei dem Fehler helfen? Java Basics - Anfänger-Themen 6
Juelin jedit Fehler beim starten Java Basics - Anfänger-Themen 2
I Frage Thymeleaf -> Fehler ignorieren und mit "" ersetzen? Java Basics - Anfänger-Themen 15
E Matrizenmultiplikation Fehler Java Basics - Anfänger-Themen 0
Z Fehler Zeiterfassungsprogramm Anzeige Java Basics - Anfänger-Themen 3
C SwingWorker mit Fehler Java Basics - Anfänger-Themen 2
monsterherz Fehler Semikolon fehlt - ich weiss aber nicht wo da noch eines hin sollte... Java Basics - Anfänger-Themen 21
G Compiler-Fehler Fehler in Zeile 1 beheben, wie? Java Basics - Anfänger-Themen 9
W Fehler in der Datei pom.xml Java Basics - Anfänger-Themen 19
marcelnedza Finde meinen Fehler in einer Methode nicht, Java Karol Java Basics - Anfänger-Themen 15
monsterherz einfache Methode mit Fehler den ich nicht finde Java Basics - Anfänger-Themen 21
monsterherz if / else if mit Fehler den ich leider nicht finde Java Basics - Anfänger-Themen 11
N Interpreter-Fehler Compiler zeigt keine Fehler an, aber das Programm läuft nicht (BlueJ) Java Basics - Anfänger-Themen 2
ohneInformatik; Dynamische Zinsen. Wo liegt der Fehler? Java Basics - Anfänger-Themen 4
Fiedelbambu deriveFont Fehler wer kann Helfen? Java Basics - Anfänger-Themen 4
I Ical4j / Kalender einlesen von URL - Fehler: "Unparseable date" Java Basics - Anfänger-Themen 16
Lion.King Fehler in double und int Java Basics - Anfänger-Themen 7
H BlueJ: was genau ist hier der Fehler? Java Basics - Anfänger-Themen 14
berserkerdq2 Habe ein Spiel entwickelt, dass immer in der 4 Runde einen cast-Fehler erhält Java Basics - Anfänger-Themen 3
D Datentypen LocalDate.parse() ergibt Fehler Java Basics - Anfänger-Themen 5
stormyark Fehler beim überschreiben einer Variable Java Basics - Anfänger-Themen 1
T String Array Fehler beim Index Java Basics - Anfänger-Themen 3
N Fehler "Cannot instantiate the type" Java Basics - Anfänger-Themen 3
L Ich weis nicht was der Fehler ist! Java Basics - Anfänger-Themen 14
L30nS JNI Fehler, JRE und JDK Java Basics - Anfänger-Themen 8
E Executable jar file fehler Java Basics - Anfänger-Themen 9
S Fehler beim Programm Java Basics - Anfänger-Themen 2
U Warum kriege ich hier eine nullpointer exception, sehe den Fehler nicht (swing) Java Basics - Anfänger-Themen 1
J Syntax-Fehler? Java Basics - Anfänger-Themen 2
Jose05 Fehler im Programm feststellen Java Basics - Anfänger-Themen 2
S Methoden 2 non-static Methoden, trotzdem Fehler "non static method can not be referenced from a static context" Java Basics - Anfänger-Themen 9
G Taschenrechner ergibt Fehler in if-Abfrage Java Basics - Anfänger-Themen 6
I Fehler bei for-Schleife Java Basics - Anfänger-Themen 6
lol5443 Tic Tac Toe Fehler Java Basics - Anfänger-Themen 5
K Fehler bei der Implementierung Java Basics - Anfänger-Themen 6
N Fehler im Code (Aufgabe für Anfänger) Java Basics - Anfänger-Themen 11
W Verschachtelte If-else --> finde meinen Fehler nicht Java Basics - Anfänger-Themen 30
J Fehler bei array aus anderer Klasse Java Basics - Anfänger-Themen 3
H Fehler bei integer Division Java Basics - Anfänger-Themen 28
C Fehler beim erstellen eines Objektes Java Basics - Anfänger-Themen 3
N Was bedeutet dieser Fehler Java Basics - Anfänger-Themen 2
fuerteb Compiler-Fehler Methode wird nicht bzw. als Fehler erkannt Java Basics - Anfänger-Themen 4
Lion.King Fehler Java Basics - Anfänger-Themen 5
AlexG. Nullpointer exeption Fehler Java Basics - Anfänger-Themen 0
C Fehler im Code Java Basics - Anfänger-Themen 10
J Anfänger, Fehler; "Der Hund liegt begraben" Java Basics - Anfänger-Themen 3
Aqtox Hallo ich muss für die Schule ein Wuerfell Duell erstellen jedoch habe ich ein fehler Java Basics - Anfänger-Themen 4
V Wer findet den Fehler :) Java Basics - Anfänger-Themen 12
B ArrayIndexOutOfBoundsException, ich finde den Fehler nicht? Java Basics - Anfänger-Themen 10
A Compiler Fehler - not a statement Java Basics - Anfänger-Themen 2
Arita welche Fehler gibt es noch? wie kann ich es noch vervollständigen Java Basics - Anfänger-Themen 15
S Fehler bei Code mit SubStrings für mich nicht auffindbar. Java Basics - Anfänger-Themen 4
S Kriege Fehler "Exception in thread" beim Benutzen von SubStrings. Java Basics - Anfänger-Themen 2
H Logik Fehler erkennen Java Basics - Anfänger-Themen 21
T Fehler in Caesar-Chiffre Java Basics - Anfänger-Themen 7
R Fehlermeldung aber WO liegt der Fehler? Java Basics - Anfänger-Themen 7
B Nicht reproduzierbarer Fehler bei Kompilierung - Shortcut "Bereinigung" Compiler ? Java Basics - Anfänger-Themen 4
Nerdinfekt BMI Rechner, fehler beim Zurückgeben des Strings? Java Basics - Anfänger-Themen 2
P Was bedeutet dieser Fehler? Java Basics - Anfänger-Themen 31
KogoroMori21 Java Datum Differenz (kleiner Fehler) Java Basics - Anfänger-Themen 10
N java.util.InputMismatchException Fehler Java Scanner Java Basics - Anfänger-Themen 5
H Fehler: NullPointerException und ich weiß net warum Java Basics - Anfänger-Themen 4
R Ich sehe meinen fehler nicht Java Basics - Anfänger-Themen 8
Johannes_ece Fehler: Application Terminated (TypeError): var$0.$moveToolTo is not a function Java Basics - Anfänger-Themen 4
GermanPie Fehler in Putty (kein Hauptmanifestattribut, in jar) Java Basics - Anfänger-Themen 4
M Scannen von *.txt - Dateien; wo sind der oder die Fehler? Java Basics - Anfänger-Themen 4
P Methoden aufrufen - Fehler Java Basics - Anfänger-Themen 20
JavaClap "Bruchrechner" liefert Fehler/keine Ausgabe bei Addition und Subtraktion Java Basics - Anfänger-Themen 0
B if Clause Fehler Java Basics - Anfänger-Themen 2
G Fibonacci Zahlenreihe Fehler Java Basics - Anfänger-Themen 4
A Fehler beim Ausführen einer class Datei Java Basics - Anfänger-Themen 6
B Fehler, aber ich weiß nicht warum Java Basics - Anfänger-Themen 3
C system cannot be resolved Fehler in Eclipse Java Basics - Anfänger-Themen 18
J Fehler im Code, aber ich weiß nicht wieso! Java Basics - Anfänger-Themen 6
M Compiler-Fehler Fehler beim Ausführen des Codes Java Basics - Anfänger-Themen 25
M While-Schleifen-Fehler Java Basics - Anfänger-Themen 4
N Fehler bei JUnit Test Java Basics - Anfänger-Themen 5
C Projekte in 2 versch. Arbeitsbereichen: auf ein Projekt verweisen (ohne Fehler zu bekommen) Java Basics - Anfänger-Themen 8
R Java SQL Fehler! Java Basics - Anfänger-Themen 4
L non-static Fehler Java Basics - Anfänger-Themen 16
C Fehler beim Speichern (Build projekt) Java Basics - Anfänger-Themen 42
L Methoden Wie Löse ich ext Methoden Aufruf Fehler? Java Basics - Anfänger-Themen 3
F Methoden Bitte Helft mir meinen Fehler zu finden. Möchte in diesem Bankenprogramm durch die Konsoleneingabe auswählen welches Konto reduziert und welches erhö Java Basics - Anfänger-Themen 17
C Fehler bei der Compilierung Java Basics - Anfänger-Themen 1
T Mein Programm hat Fehler Java Basics - Anfänger-Themen 4
S Warum dieser Fehler? Java Basics - Anfänger-Themen 1
B Fehler bei Ausführung Java Basics - Anfänger-Themen 5
Kirby.exe Fehler beim Ausgeben Java Basics - Anfänger-Themen 2
X java.lang.NullPointerException fehler ? Java Basics - Anfänger-Themen 1
L Wo ist der Fehler? Java Basics - Anfänger-Themen 87
J Fehler in Programm: Index -1 out of bounds for length 0 Java Basics - Anfänger-Themen 5
M JOptionPane Fehler bei "Abbrechen" des Fensters Java Basics - Anfänger-Themen 10
N Fehler bei string Attribut! Java Basics - Anfänger-Themen 18
W Wo liegt der Fehler? Java Basics - Anfänger-Themen 8
G Palindromtest mit Angabe WO der Fehler ist Java Basics - Anfänger-Themen 2
J Wo ist der Fehler im Programmcode? Java Basics - Anfänger-Themen 7
J Fehler den ich nicht kapiere Java Basics - Anfänger-Themen 9

Ähnliche Java Themen

Neue Themen


Oben