Unreachable Code

So i began to learn bukkit programming with a help of a tutorial. After some learning of the programming i started to mess around with commands and created my first command. That worked perfect but then i created my second command and that then both commands didn't work and at the if line of the second command it says "unreachable Code" but in my opinion it should be reachable. Can you please help me fixing the problem?
My Code:

Code:java

package de.nielster.test;
 
import java.util.List;
 
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin {
 
    //Laden des Plugins
    public void onLoad() {
        super.onLoad();
        System.out.println("Plugin wurde erfolgreich geladen");
    }
 
    //aktivieren des Plugins
    public void onEnable() {
        super.onEnable();
        System.out.println("Pluigin wurde aktviert");
    }
 
    //deaktivieren des Plugins
    public void onDisable() {
        super.onDisable();
        System.out.println("Plugin wurde deaktiviert");
    }
 
    //Commands
    public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
        Player p = null;
        If (sender instanceof Player); p = (Player) sender;
 
        If (command.getName().equals("Test")); {
        p.sendMessage("Test Nachricht");
 
        return true;
        }
 
 
        If (command.getName().equals("info")); {
        String name = this.getDescription().getName();
        String version = this.getDescription().getVersion();
        List<String> authors = this.getDescription().getAuthors();
        String author = "";
        for (int i = 0; i < authors.size(); i++) author += authors.get(i) + " ";
        p.sendMessage("Name des Plugis: " + name + " " + version + "\n" + "Von:" + author);
 
        return true;
        }
 
    }
 
    private void If(boolean b) {
        // TODO Auto-generated method stub
 
    }
 
}
 

And my plugin.yml:

Code:

name: Test
version: 1.0
author: nielster
main: de.nielster.test.Main
 
commands:
  Test:
      usage: /<command>
      description: nur zum Testen
  info:
      usage: /<command>
      description: informationen
      

And if needed i could comment the code on english.

 

Why are you calling the super methods in your onLoad, onEnable, and onDisable methods?? You should be overriding the super methods with your new ones.

Second, you shouldn't be using .equals on commands, instead equalsIgnoreCase, as not all people that use your plugin are guaranteed to write the commands in lower case.

Third, on line 35 you write "; {" this does nothing. Literally nothing. Remove the semicolon.

Fourth, write all java keywords in lowercase, capitalizing any character in one will remove its functionality.

Fifth, don't write semicolons after an if statement.

Sixth, <removed> learn java and don't waste people's time on the forums to answer questions <removed>.

[edit by JaguarJo: removed insulting language. Please do not take out your frustrations on other forum members. Let's keep this discussion civil please.]

 

While everything you said is correct, give people a break. Jeez. Just because EVERYTHING isn't correct in his code doesn't mean he is a "Lazy ass." The whole point of the forums is to ask for help. Take a freaking chill pill dude.

[edit by JaguarJo: fixed the quote box to reflect mod edits made in earlier post]

 

_Filip I agree. People should go elsewhere to learn Java in my opinion.
es359 I agree. That certainly wasn't the best way to tell them that this is not the place to get help with basic Java.

 

Speed_crafter007 Sorry, but this is not the place for you to request help with Java, this is for people who already have a good understanding and wish to apply that to learning the CraftBukkit API. Until you are good enough at Java to do so, I don't recommend attempting to code for Bukkit at all. Java is an entire language, not just a way of writing things. It's very difficult to learn it on the fly. One thing at a time.

You're right, he obviously does not grasp the basics of Java. But that was pretty rude and disrespectful, him being bad at Java or otherwise. He really isn't worth it.