Wo ist der Fehler in Zeile 10?

[StefanK]

Hallochen allerseits, ich versuche mich grad mit Java und bin noch der echte Anfänger - also bitte nicht
lächeln. Der folgende Code bringt beim Kompilieren leider einen Fehler...Ich habe nur keine Ahnung
wie er zu beheben ist.

public class Volumenrechnen {
  // Anfang Variablen
       int hight;
       int wight;
       int depht;
  // Ende Variablen

  // Anfang Ereignisprozeduren
       void volumen (){
            hight * wight * depht;
       }
             
  public static void main(String[] args) {
    Volumenrechnen quader = new Volumenrechnen();
        int Volumen;
        quader.hight = 3;
        quader.wight = 4;
        quader.depht = 5;
        Volumen = Volumenrechnen.volumen();
      System.out.println("Das Volumen des Würfels ist: " + Volumen);
  }
  // Ende Ereignisprozeduren
}

Danke für jeden Lösungsvorschlag.
Stefan

 

[benders]

Hallo Stefan,

Deine Funktion volumen gibt nix zurück.

void int volumen () {
return higth*wigth*depth;
}

ist besser.

Bernd

 

[SlaterB]

> void int volumen ()
ist ja mal ein Vorschlag

int volumen ()
ist noch besser, vielleicht gar
public int volumen ()

-------

width
ist ein gutes englisches Wort, oder wenigstens weight (Gewicht)

edit: oh, da waren ja noch mehr Rechtschreibfehler , siehe weiter unten

 

[Murray]

Wenn du eine Fehlermeldung bekommst, solltest du die normalerweise auch posten - hier ist es aber wohl einfach genug: dort steht keine vollständige Anweisung, sondern quasi nur die rechte Seite einer Zuweisung.

     void volumen (){
            int vol = hight * wight * depht;
     }

Wahrscheinlich soll die Methode aber den Wert nicht nur berechnen, sondern auch etwas damit machen (z.B ihn zurückgeben?):

     int volumen (){
            return hight * wight * depht;
     }

<OT>
Höhe: height
Breite: width
Tiefe: depth
</OT>

//EDIT: Typos

 

[HG]

Hi.

Bernd meinte bestimmt das richtige

int volumen()
{
return height * width * depth
}

also ohne das void, das wuerde ja sonst keinen Sinn machen.

So und da wir grad dabei sind, und anderen Fragen vorbeugen wollen:

Da du volumen() aus statischem Kontext aufrufst, muss die Methode auch statisch sein

static int volumen() {....}

Und das gleiche fuer die 3 Variablen.

Gruesse
HG

//EDIT: man, mal wieder zu spaet...

 

[Murray]

H
Da du volumen() aus statischem Kontext aufrufst, muss die Methode auch statisch sein

static int volumen() {....}

Und das gleiche fuer die 3 Variablen.

Den Compiler mag das zwar zufriedenstellen, aber normalerweise sollte man Eigenschaften nicht ohne triftigen Grund statisch machen. Besser also: die Methode und die Variablen an der Instanz belassen, und die Methode über das Objekt aufrufen,
also statt

int volumen = Volumenrechnen.volumen();

lieber

int volumen = quader.volumen();

 

[HG]

Richtig.
Aber da er ja erstmal generelle Sachen lernen will, kann man das schon mal so machen.

Ich wollte ihm nur helfen sein Bsp zum laufen zu bringen

 

[SlaterB]

um es mal genau zu nehmen:
nene, dass darf man auf keinen Fall so machen, nix falsches angewöhnen

in diesem Fall schon gar nicht, da der gesamte Code eh auf nicht-statisch ausgelegt ist,
nur dieser eine Aufruf nicht