Variable von Objekten in einer Methode überprüfen

[TFRLucky]

Hi, ich würde gerne wissen, wie ich überprüfen wie viele Objekte eingeschaltet sind:

public class Application {
    public static void main(String[]args){
        Radio r1 = new Radio();
        Radio r2 = new Radio();
        IceMachine ice = new IceMachine();
        Ship.devices.add(r1);
        Ship.devices.add(r2);
        Ship.devices.add(ice);
        r1.on();
        r2.on();
        ElectronicDevice.numberOfElectronicDevicesSwitchedOn(r1, r2, ice);
        Ship.countDevicesSwitchedOn();
    }
}

In der Klasse Application werden meiner Objekte der Klasse ElectronicDevice:

public class ElectronicDevice {
    static boolean isOn = false;

    public void on() {
        isOn = true;
        System.out.println("An");
    }

    public void off() {
        isOn = false;
        System.out.println("Aus");
    }


    public static int numberOfElectronicDevicesSwitchedOn(ElectronicDevice... devices) {
        int x = 0;
        if (isOn) {
            x++;
        }


        System.out.println(x);
        return x;
    }
}

erstellt. Wie überprüfe ich jetzt, welche Objekte eingeschaltet sind?

 

[KonradN]

Deine Frage verstehe ich nicht. Du hast doch schon einen gewissen Code geschrieben. Wenn der nicht das macht, was Du möchtest, dann solltest Du den Fehler beschreiben.

Ansonsten solltest Du Dir einmal anschauen, was das Schlüsselwort static macht - speziell bei der Variable isOn.

 

[TFRLucky]

Hi, danke für die schnelle Antwort. Das Problem welches ich habe, ist, dass ich eine nicht statische Variable nun mal nicht in einer statischen Methode abfragen kann. "numberOfElectronicDevicesSwitchedOn" muss aber static sein, damit ich die Methode in der main Methode von Application aufrufen kann. Das Programm gibt mir aus, dass nur 1 Gerät an ist. Wie sorge ich also dafür, dass jedes Objekt einzeln überprüft wird?

 

[KonradN]

Hi, danke für die schnelle Antwort. Das Problem welches ich habe, ist, dass ich eine nicht statische Variable nun mal nicht in einer statischen Methode abfragen kann. "numberOfElectronicDevicesSwitchedOn" muss aber static sein, damit ich die Methode in der main Methode von Application aufrufen kann. Das Programm gibt mir aus, dass nur 1 Gerät an ist. Wie sorge ich also dafür, dass jedes Objekt einzeln überprüft wird?

Du hast in einer statischen Methode nicht die Instanz this. Aber die brauchst du doch auch nicht. Du willst doch die Geräte prüfen, die als Parameter übergeben worden sind. Mit ElectronicDevice... devices hat Du ein ein vararg mit ElectronicDevice Instanzen. Auf diese kannst Du in der Methode zugreifen um deren isOn zu prüfen.

 

[TFRLucky]

Die Methode sieht jetzt so bei mir aus:

public static int numberOfElectronicDevicesSwitchedOn(ElectronicDevice... devices) {
        int x = 0;
        for(int i = 0; i < devices.length; i++) {
            if (ElectronicDevice.isOn) {
                x++;
            }
        }


        System.out.println(x);
        return x;
    }

Mit der for schleife will ich überprüfen wieviele Instanzen übergeben wurden. Dann frage ich mit if ab ob isOn von der Instanz true ist. Aber die Ausgabe gibt immer an, dass alle on sind auch wenn ich nur eine Instanz on ist

 

[KonradN]

Dann kommen wir noch einmal auf den ersten Punkt von mir zurück: Was ändert sich denn durch das static bei isOn?

 

[TFRLucky]

Ja, ich weiß, dass static die Variable für die ganze Klasse speichert. Wie kann ich aber eine nicht statische Variable in einer statischen Methode überprüfen?

 

[KonradN]

Ja, ich weiß, dass static die Variable für die ganze Klasse speichert. Wie kann ich aber eine nicht statische Variable in einer statischen Methode überprüfen?

Du hast das schlicht falsch verstanden. Um eine nicht statische Variable nutzen zu können brauchst Du eine Instanz der Klasse. Das ist die wichtige Aussage.

In einer statischen Methode hast du nicht die Instanz "this". Somit kannst Du nicht auf this zugreifen (und somit auch nicht auf Alles, was verkürzt für this.xxxx steht).

Aber natürlich kannst Du auf die Variable zugreifen, wenn Du eine Instanz hast (und die Zugriffsrechte es erlauben).

Also probiere es doch einfach aus.

 

[TFRLucky]

Wenn ich isOn zu einer non-static Variable mache und ich über ElectronicDevice.isOn auf sie zugreifen kommt die Fehlermeldung: "Non-static field 'isOn' cannot be referenced from a static context", obwohl ich in meiner main Methode 3 Instanzen der Klasse ElectronicDevice erstellt habe

 

[KonradN]

Wenn ich isOn zu einer non-static Variable mache und ich über ElectronicDevice.isOn auf sie zugreifen kommt die Fehlermeldung: "Non-static field 'isOn' cannot be referenced from a static context", obwohl ich in meiner main Methode 3 Instanzen der Klasse ElectronicDevice erstellt habe

Ja, das ist auch richtig. Wenn Du es zu einer Instanzvariable machst, dann kannst Du nicht mehr über die Klasse zugreifen.

Daher ist da also immer noch etwas falsch in der Methode. Wir drehen uns hier massiv im Kreis:

a) Du hast mehrere Gegenstände und die Gegenstände haben einen Status. In Deinem Fall soll das ein Flags ein: isOn.
Das muss dann doch eine Sache sein, die in jedem Gegenstand ist. Jedes einzelne elektrische Gerät kann an oder aus sein. Unabhängig von allen anderen elektrischen Geräten. Eine globale Variable, die genau ein einziges Mal existiert, macht also absolut keinen Sinn, oder?

b) Wenn Du wissen willst, wie voll der Tank von irgendwelchen Autos ist: Wo schaust Du dann nach? Fragst Du beim Hersteller nach damit der dann mal auf dem Bauplan nachschaut? Wo findet sich der Tankinhalt der Autos?

Wenn Du sowas baust, dann bau es immer ein Schritt nach dem anderen:

Als erstes Bildest Du das ab, was abgebildet sein soll. Ein elektrisches Gerät soll eingeschaltet sein können. Dann machst Du dies. Das wäre der Punkt a.

Wenn Du das fertig hast, dann kannst Du es testen. Du kannst also etwas machen wie:

Radio radio1 = new Radio();
Radio radio2 = new Radio();
assert(!radio1.isOn);
assert(!radio2.isOn);
radio1.on();
assert(radio1.isOn);
assert(!radio2.isOn);

Dann kannst Du eine Methode schreiben wie dein statische Methode. Und da kannst Du Dir dann in Ruhe überlegen, worauf Du eigentlich zugreifen willst und so.

 

[Neumi5694]

Wenn ich isOn zu einer non-static Variable mache und ich über ElectronicDevice.isOn auf sie zugreifen kommt die Fehlermeldung: "Non-static field 'isOn' cannot be referenced from a static context", obwohl ich in meiner main Methode 3 Instanzen der Klasse ElectronicDevice erstellt habe

Es geht letztendlich um die Frage, was die Variable beschreiben soll: Soll isOn den Status eines spezfischen ElectronicDevices beschreiben oder den Status ALLER Devices?
Falles um alle geht, dann ist statisch ok, dann können die verschiedenen Devices aber auch nicht verschiedene Werte haben.

Falls du jedes Device einzeln beschreiben willst, dann muss es eine nicht statische Instanzvariable sein.

Dann allerdings musst du auch sagen, welches Device dich nun interessiert, wenn du darauf zugreifst.
Gib einfach an, um welches Device es geht, dann klappt auch der nicht statische Zugriff (so wie es auch gedacht ist).

Statische Variable:
Device.IsOn(); //alles, was ein Device ist, ist on

Nicht statische Instanzvariable:
Device thisDevice = new Device();
thisDevice.isOn(); //dieses eine Device ist on.

 

[TFRLucky]

Also muss ich immer händisch in die Methode rein gehen und immer wenn ich eine Instanz hinzugefügt habe die Instanz in der Methode berücksichtigen?

static ElectronicDevice r1 = new ElectronicDevice();
    static ElectronicDevice r2 = new ElectronicDevice();
    static ElectronicDevice ice = new ElectronicDevice();

    public static int numberOfElectronicDevicesSwitchedOn(ElectronicDevice... devices) {
        int x = 0;
        for(int i = 0; i < devices.length; i++) {
            if (ElectronicDevice.r1.isOn) {
                x++;
            }
            if (ElectronicDevice.r2.isOn) {
                x++;
            }
            if (ElectronicDevice.ice.isOn) {
                x++;
            }

        }

        return x;
    }

 

[KonradN]

Nein! Vergiss endlich das static. Du brauchst absolut nichts mit static! Und vor allem solltest Du aufhören so zu raten.

Überlege doch erst einmal, was die Methode public static int numberOfElectronicDevicesSwitchedOn(ElectronicDevice... devices) machen soll. Kannst Du das in Worten beschreiben?

 

[TFRLucky]

Die Methode soll jede Insatnz auf isOn überfrüfen und zurück geben bei wieviel isOn true ist

 

[KonradN]

Die Methode soll jede Insatnz auf isOn überfrüfen und zurück geben bei wieviel isOn true ist

Das ist eine pauschale Aussage, mit der niemand etwas anfangen kann weil einfach zu ungenau.

Wenn Du so eine Methode beschreiben willst, dann betrachte immer die Eingabe und sag, was mit der Eingabe passieren soll um dann zu einer Ausgabe zu kommen.

Fangen wir bei der Eingabe an: Was für eine Eingabe hat denn die Methode?

 

[TFRLucky]

Der Methode werden beliebig viele Instanzen übergeben. Dann soll die Methode für jede der übergebenen Instanzen überprüfen ob isOn true ist und wiedergeben.

 

[KonradN]

Der Methode werden beliebig viele Instanzen übergeben. Dann soll die Methode für jede der übergebenen Instanzen überprüfen ob isOn true ist und wiedergeben.

Ja, so klingt das schon viel besser.

Dann wäre die nächste Frage: Wie kannst Du denn bei so einem varargs Parameter auf die Instanzen zugreifen?

Das evtl. direkt bei dem Code der Methode aus #5 einbauen

 

[Neumi5694]

(kleiner Tip: Das Geheimnis, wie man auf eine Instanz zugreift, ist: Anstatt des Klassennamens die Variable verwenden, die auf die Instanz verweist).

 

[TFRLucky]

Wenn ich die Methodeso schreibe:

public static int numberOfElectronicDevicesSwitchedOn(ElectronicDevice... devices) {
        int x = 0;
        for(int i = 0; i < devices.length; i++) {
            if (ElectronicDevice.isOn) {
                x++;
            }

        }

        return x;
    }
}

kommt der Fehlercode "Non-static field 'isOn' cannot be referenced from a static context", ich weiß aber leider nicht, wie ich sonst auf alle Instanzen zugreifen soll

 

[Neumi5694]

Wenn ich die Methodeso schreibe:

public static int numberOfElectronicDevicesSwitchedOn(ElectronicDevice... devices) {
        int x = 0;
        for(int i = 0; i < devices.length; i++) {
            if (ElectronicDevice.isOn) {
                x++;
            }

        }

        return x;
    }
}

kommt der Fehlercode "Non-static field 'isOn' cannot be referenced from a static context", ich weiß aber leider nicht, wie ich sonst auf alle Instanzen zugreifen soll

Was lässt dich glauben, dass ElectronicDevice "alle Instanzen" ist? Das ist genau gar keine Instanz, das ist eine Klasse.

Du hast in deinem Quellcode eine Aufzählung aller Instanzen. Finde sie. Dann greife auf die einzelnen Elemente dieser Aufzählung zu.


MyClass myClassInstance1 = new MyClass();
MyClass myClassInstance2 = new MyClass();

myClassInstance1 und myClassInstence2 sind Instanzen.
MyClass ist keine Instanz und schon gar nicht "alle Instanzen". MyClass liefert Eigenschaften der Klasse, mehr nicht.

 

[TFRLucky]

Meine nächste Idee wäre gewesen:

public static int numberOfElectronicDevicesSwitchedOn(ElectronicDevice... devices) {
        int x = 0;
        for(int i = 0; i < devices.length; i++) {
            if (devices.isOn) {
                x++;
            }

        }

        return x;
    }

weil devices steht ja für die Instanzen die ich hinzufüge. Aber jetzt kommt der Fehlercode "Cannot resolve symbol 'isOn'"

 

[KonradN]

Hast Du mal geschaut, wie Du auf die varargs Argumente zugreifen kannst?

Teilweise hast Du da ja schon einen Zugriff ... dieses devices.length ... was ist denn devices, dass es da ein length gibt?

 

[Neumi5694]

Meine nächste Idee wäre gewesen:

public static int numberOfElectronicDevicesSwitchedOn(ElectronicDevice... devices) {
        int x = 0;
        for(int i = 0; i < devices.length; i++) {
            if (devices.isOn) {
                x++;
            }

        }

        return x;
    }

weil devices steht ja für die Instanzen die ich hinzufüge. Aber jetzt kommt der Fehlercode "Cannot resolve symbol 'isOn'"

Handelt es sich bei devices um ein einzelnes Element? Nein, devices ist eine Auflistung von Elementen, oder genauer gesagt: ein Array. Hat eine Auflistung denn eine isOn-Eigenschaft? Ich glaube nicht, Tim.
Also ... wie greift man denn auf ein einzelnes Element eines Arrays zu?

Kleiner Tip:
Wenn du viele Zettel in einer Box hast und das Wort wissen willst, das auf Zettel Nr. 3 geschrieben steht, was musst du dann tun? Du nimmst den dritten Zettel aus der Box und liest ihn.
Mach das gleiche in Jave mit einem Array von ElectronicDevices.

 

[TFRLucky]

Ohh mein Gott, klar devices ist ein Array, also muss ich natürlich jedes Objekt im Array einzeln ansprechen🤦‍♂️😂 Danke für die Hilfe, ich stand echt aufm Schlauch, danke danke🙏

 

[mihe7]

Richtig, der Vollständigkeit halber sei hier noch die for-each-Schleife (offiziell "enhanced for statement") erwähnt, mit der Du das, was Du möchtest noch etwas prägnanter ausdrücken und Dir den Index zugleich auch noch sparen kannst:

int count = 0;
for (ElectronicDevice device : devices) { // für jedes device aus devices wiederhole...
    if (device.isOn) {
        count++;
    }
}
return count;

Die Stream-Variante erspare ich uns

 

[KonradN]

Die Stream-Variante erspare ich uns

Wenn man das schon erwähnt, dann muss man auch liefern:

return Arrays.stream(devices)
    .filter(d -> d.isOn)
    .count();

 

[mihe7]

Wenn man das schon erwähnt, dann muss man auch liefern:

Als hätte ich es gewusst...