error: class, interface, or enum expected

[toni_latenz]

Hallo zusammen.

Gerade versuche ich Java Programmierungsgrundlagen zu lernen. Dazu benutze ich das Buch "Programmieren lernen mit Minecraft Plugins".

Bei einer der Übungen gibt mir die Shell (Ubuntu 14.04 LTS) den folgenden Error für den nachfolgenden Code aus:

java:47: error: class, interface, or enum expected
}
^
1 error

der Code lautet:

package simple;
import net.canarymod.plugin.Plugin;
import net.canarymod.logger.Logman;
import net.canarymod.Canary;
import net.canarymod.commandsys.*;
import net.canarymod.chat.MessageReceiver;
import net.canarymod.api.entity.living.humanoid.Player;
import com.pragprog.ahmine.ez.EZPlugin;
import net.canarymod.api.world.effects.SoundEffect;
import net.canarymod.api.world.position.Location;

public class Simple extends EZPlugin
{ public static long howlong(int years)
  { return years * 365 * 24 * 60 * 60; }

  @Command(aliases = { "simple" },
  description = "simple plugin",
  permissions = { "*" },
  toolTip = "/simple")
  public void simpleCommand(MessageReceiver caller, String[] parameters)
  { if (caller instanceof Player)
  { Player me = (Player)caller;
  // Put your code after this line:

  int myage = 35;
  int twicemyage = myage * 2;
  float volume = 0.5f;
  float pitch = 1.0f;
  double dayOnIo = 152853.504;
  String myname = "Toni Latenz";
  long secondsOld = howlong(35); }

  {
  me.chat("My age is" + myage);
  me.chat("My name is" + myname);
  me.chat("My age doubled is:" + twicemyage);
  me.chat("A day on Io in seconds is:" + dayOnIo);
  me.chat("My lifetime in seconds is:" + secondsOld); }

  { Location loc = me.getLocation();
  playSound(loc, SoundEffect.Type.GHAST_SCREAM, volume, pitch);
  }

  // ...and finish your code before this line.
  }
}
}

Zeile 47, in welcher der Error scheinbar entsteht ist die letzte geschweifte Klammer....
Ich habe bereits versucht die letzte Klammer wegzulassen oder noch eine hinzuzufügen. (Man übersieht ja mal schnell was..)

Hat jemand eine Idee wo mein Fehler genau liegt?
Im Voraus besten Dank!

 

[JStein52]

Formatiere deinen Code doch mal gescheit mit Einrückungen und so. Dann siehst du den Fehler wahrscheinlich auf den ersten Blick. Und lass mal diese ganzen überflüssigen Klammern weg. Die machen einen ja meschugge.

 

[kneitzel]

Also eine Klammer ist am Ende auf jeden Fall zuviel. Die Anzahl der öffnenden und schließenden Klammern muss gleich sein.

Und dann hast Du da ein paar Blöcke, die unnötig sind. Es macht keinen Sinn, Befehle einfach so in { } zu setzen.

Das Problem ist aber nicht ganz am Ende sondern der Block vom if ist zu klein. Wenn Du innerhalb von { } eine Variable definierst, dann ist diese Variable nur innerhalb dieses Blocks bekannt. Derzeit schließt Du den Block vom if nur um dann einen neuen Block zu erstellen, in dem Du die lokale Variable me erneut verwenden willst. Das geht natürlich nicht.

Also entferne die beiden "} {" Kombinationen.

Dann noch die letzte } entfernen und dann könnte es evtl. gehen mit dem compilieren.

Konrad

 

[toni_latenz]

Danke JStein52 und Konrad.

Ich hab den Code nochmal formatiert und denke das er so recht gut lesbar ist. Trotzdem verlangt der javac bzw build.sh beim Kompilieren einen Semikolon an einer Stelle, an der er für mich keinen Sinn macht. Selbst wenn ich dort einen platziere, ändert das die Fehlermeldung nicht.

Hier nochmal der neu formatierte Code:

package simple;
import net.canarymod.plugin.Plugin;
import net.canarymod.logger.Logman;
import net.canarymod.Canary;
import net.canarymod.commandsys.*;
import net.canarymod.chat.MessageReceiver;
import net.canarymod.api.entity.living.humanoid.Player;
import com.pragprog.ahmine.ez.EZPlugin;
import net.canarymod.api.world.effects.SoundEffect;
import net.canarymod.api.world.position.Location;

  public class Simple extends EZPlugin {

    public static long howlong(int years) {
      return years * 365 * 24 * 60 * 60;
    }

      @Command(aliases = { "simple" },
            description = "simple plugin",
            permissions = { "*" },
            toolTip = "/simple")

            public void simpleCommand(MessageReceiver caller, String[] parameters) {
              if (caller instanceof Player) {
                Player me = (Player)caller;
      // Put your code after this line:

                int myage = 35;
                int twicemyage = myage * 2;
                float volume = 0.5f;
                float pitch = 1.0f;
                double dayOnIo = 152853.504;
                String myname = "Toni Latenz";
                long secondsOld = howlong(35);
              }

                  {
                  me.chat("My age is" + myage);
                  me.chat("My name is" + myname);
                  me.chat("My age doubled is:" + twicemyage);
                  me.chat("A day on Io in seconds is:" + dayOnIo);
                  me.chat("My lifetime in seconds is:" + secondsOld);
                  }

                    {
                    Location loc = me.getLocation();
                    playSound(loc, SoundEffect.Type.GHAST_SCREAM, volume, pitch);
                    }

      // ...and finish your code before this line.
  }
}

Die Fehlermeldung(en) der Shell beim Versuch zu Kompilieren sind folgende:

Compiling with javac...
src/simple/Simple2.java:13: error: ';' expected
{public static long howlong(int years)
                                      ^
src/simple/Simple2.java:20: error: class, interface, or enum expected
  public void simpleCommand(MessageReceiver caller, String[] parameters)
         ^
src/simple/Simple2.java:25: error: class, interface, or enum expected
      int myage = 35;
      ^
src/simple/Simple2.java:26: error: class, interface, or enum expected
      int twicemyage = myage * 2;
      ^
src/simple/Simple2.java:27: error: class, interface, or enum expected
      float volume = 0.5f;
      ^
src/simple/Simple2.java:28: error: class, interface, or enum expected
      float pitch = 1.0f;
      ^
src/simple/Simple2.java:29: error: class, interface, or enum expected
      double dayOnIo = 152853.504;
      ^
src/simple/Simple2.java:30: error: class, interface, or enum expected
      String myname = "Toni Latenz";
      ^
src/simple/Simple2.java:31: error: class, interface, or enum expected
      long secondsOld = howlong(35);
      ^
src/simple/Simple2.java:34: error: class, interface, or enum expected
      me.chat("My age is" + myage);
      ^
src/simple/Simple2.java:35: error: class, interface, or enum expected
      me.chat("My name is" + myname);
      ^
src/simple/Simple2.java:36: error: class, interface, or enum expected
      me.chat("My age doubled is:" + twicemyage);
      ^
src/simple/Simple2.java:37: error: class, interface, or enum expected
      me.chat("A day on Io in seconds is:" + dayOnIo);
      ^
src/simple/Simple2.java:38: error: class, interface, or enum expected
      me.chat("My lifetime in seconds is:" + secondsOld);
      ^
src/simple/Simple2.java:40: error: class, interface, or enum expected
      Location loc = me.getLocation();
      ^
src/simple/Simple2.java:41: error: class, interface, or enum expected
      playSound(loc, SoundEffect.Type.GHAST_SCREAM, volume, pitch);
      ^
16 errors

Was mach ich nur falsch?

 

[JStein52]

Was ist das da :

      @Command(aliases = { "simple" },
            description = "simple plugin",
            permissions = { "*" },
            toolTip = "/simple")

gehört da vielleicht noch ein Semikolon dahinter ??

Und dann sind da immer noch einige }{ zuviel. Dein if-Block hört doch viel weiter unten auf. Mach doch mal diese überflüssigen {} weg !! Siehe Kommentar von Konrad oben zu der Variablen me !!

 

[JStein52]

import net.canarymod.plugin.Plugin;
import net.canarymod.logger.Logman;
import net.canarymod.Canary;
import net.canarymod.commandsys.*;
import net.canarymod.chat.MessageReceiver;
import net.canarymod.api.entity.living.humanoid.Player;
import com.pragprog.ahmine.ez.EZPlugin;
import net.canarymod.api.world.effects.SoundEffect;
import net.canarymod.api.world.position.Location;

public class Simple extends EZPlugin {

    public static long howlong(int years) {
        return years * 365 * 24 * 60 * 60;
    }

    @Command(aliases = {"simple"},
            description = "simple plugin",
            permissions = {"*"},
            toolTip = "/simple")

    public void simpleCommand(MessageReceiver caller, String[] parameters) {
        if (caller instanceof Player) {
            Player me = (Player) caller;
            // Put your code after this line:

            int myage = 35;
            int twicemyage = myage * 2;
            float volume = 0.5f;
            float pitch = 1.0f;
            double dayOnIo = 152853.504;
            String myname = "Toni Latenz";
            long secondsOld = howlong(35);

            me.chat("My age is" + myage);
            me.chat("My name is" + myname);
            me.chat("My age doubled is:" + twicemyage);
            me.chat("A day on Io in seconds is:" + dayOnIo);
            me.chat("My lifetime in seconds is:" + secondsOld);

            Location loc = me.getLocation();
            playSound(loc, SoundEffect.Type.GHAST_SCREAM, volume, pitch);
        }
        // ...and finish your code before this line.
    }
}

wenn ich das bei mir so mache meckert er zwar die fehlenden packages an die du importierst. Der Rest scheint ok zu sein.