Frage zu Java

[Manfred L. (Gast)]

Hallo zusammen,

ich habe gestern ein Programm geschrieben und bin dabei auf folgendes 
gestoßen, was mir unerklärlich erscheint:

Folgender Code:

//Version: 2.0, 10.10.2013
//Author: Samuel Ioannidis
//Code function: Simple program for calculation of average speed
//               based on km/h 
//Copyright: Samuel Ioannidis, 10.10.2013

import java.awt.*;
import java.awt.event.*;

import javax.swing.*;

public class averagespeed extends JFrame
{  
  
    int status=2;
    
    averagespeed()
    {  
      ...
      
      
      JRadioButton hrs = new JRadioButton("hrs");
      hrs.setMnemonic(KeyEvent.VK_H);
        hrs.setActionCommand("hours");
        hrs.addActionListener(new SetTimeAction());
      
      ...
      
    }
    

    
    class SetTimeAction  implements ActionListener
    {
      status = 8;    // warum kommt hier ein Fehler?
      public void actionPerformed(ActionEvent e)
      {
        
        if(e.getActionCommand()=="hours")
        {
          // mache etwas....

        }  
      }    
    }

    
    public static void main(String[] args) 
  {
    averagespeed averagenew = new averagespeed();
    averagenew.pack();
    averagenew.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    averagenew.setSize(600, 200);
  }
}

Warum ist die Variable "status" in der "SetTimeAction"-Klasse nicht 
sichtbar?


Danke und Gruß
Manfred

[Stefan R. (srand)]

Manfred L. schrieb:
> Warum ist die Variable "status" in der "SetTimeAction"-Klasse nicht
> sichtbar?

Ist sie doch. Nur wieso glaubst du, du könntest Code außerhalb von 
Methoden haben?

Falls du das als Initialisierung meinst, das geht das wohl mit passend 
plazierten geschweiften Klammern. Aber sicher bin ich mir da nicht, Java 
ist schon zu lange her.

[Stephan (Gast)]

Manfred L. schrieb:
> class SetTimeAction  implements ActionListener
>     {
>       status = 8;    // warum kommt hier ein Fehler?
>       public void actionPerformed(ActionEvent e)
>       {

Hier wird der Variablen status ein Wert zugewiesen. Das geht nur 
innerhalb einer Methode. Oder es fehlt hier ein Typ-Bezeichner, dann 
wäre es eine Variablendeklaration (neue Variable status, nicht mehr die 
der umgebenden Klasse) mit Initialisierung.
Gruß
S.

[Roland P. (pram)]

Hab auch noch was:

if(e.getActionCommand()=="hours")

mach Stringvergleiche besser ein .equals

Gruß
Roland

[c. m. (Gast)]

Manfred L. schrieb:

 public class averagespeed extends JFrame
 {
     private int status=2;
     public void setStatus(int _value) {
        status=_value;
     }
 ...
 hrs.addActionListener(new SetTimeAction(this));
 ...
     }
     class SetTimeAction(averagespeed _avs)  implements ActionListener
     {
       avs.setStatus(8);
       //status = 8;    // warum kommt hier ein Fehler?
       public void actionPerformed(ActionEvent e)
       {
         if(e.getActionCommand()=="hours")
         {
           // mache etwas....
         }
       }
     }

so ungefähr.

[Tobias K. (t_k)]

@Manfred: Tipp: Access-Modifier besser explizit benennen, das hilft beim 
Verständnis bzgl. Sichtbarkeit. Und beim Suchen im Netz.
@C.m.: Von inner Classes aus ohne Setter Schreiben finde ich jetzt OK. 
Ansonsten gibt's ja keinen echten Sinn für "inner"?

[Manfred L. (Gast)]

Hallo nochmal. Erstmal Danke!

Aber: wenn ich von der Klasse averagespeed erbe, klappt das auch nicht?!
Warum funktioniert das überhaupt, innerhalb der Hauptklasse eine Klasse 
zu definieren?

Wenn ich die Variable status in der Hauptklasse "public" mache sehe ich 
sie in der unteren ebenfalls nicht....

Koenntetihr mir helfen?

[Karl H. (kbuchegg)]

Hier

    class SetTimeAction  implements ActionListener
    {
      status = 8;    // warum kommt hier ein Fehler?
      public void actionPerformed(ActionEvent e)
      {
        
        if(e.getActionCommand()=="hours")
        {
          // mache etwas....

        }  
      }    
    }

fängt doch erst mal eine neue Klasse an.
Daher ja auch das Schlüsselwort 'class'.
Dann definierst du wie die Klasse heisst und von wem sie erbt.

Und dann?
Dann müsstest du erst mal die Methoden der Klasse auflisten. Hast du ja 
hier auch gemacht

public class averagespeed extends JFrame
{  
  
    averagespeed()
    {  
       ...
    }
    
    
    public static void main(String[] args) 
  {
    ....
  }
}

eine ausführbare Anweisung kann nur innerhalb einer Methode einer Klasse 
stehen!
Nur bitte, innerhalb welcher Methode steht dann das

        status = 8;

Da ist keine Methode definiert.


Das hier

public class averagespeed extends JFrame
{  
  
    int status=2;

ist was anderes! Das ist eine Initialisierung! Erkennbar daran, dass die 
Wert-'Zuweisung' bei der Variablendefinition steht. Da wird in erster 
Linie eine Member-Variable definiert

    int status;

und damit man da nicht lang fackeln muss, wird sie auch gleich noch 
intialisiert.

    int status=2;

Aber den Fall hast du ja hier nicht

    class SetTimeAction  implements ActionListener
    {
      status = 8;    // warum kommt hier ein Fehler?

das ist keine Initialisierung. Das ist eine hundsordinäre Zuweisung! Und 
die kann als solches nur in einer Methode stehen. Das kann jetzt im 
Konstruktor sein

    class SetTimeAction  implements ActionListener
    {

      SetTimeAction()
      {
         status = 8;
      }

oder in irgendeiner anderen Member-Funktion

    class SetTimeAction  implements ActionListener
    {

      public void foo()
      {
         status = 8;
      }

aber egal wie, es muss auf jeden Fall innerhalb einer Methode sein!