BlueJ: was genau ist hier der Fehler?

[heheeee]

ich weiß nicht was ich ändern muss damit das richtig ist... ich bin eigentlich nicht so erfahren mit sowas, wir lernen grad in der Schule das Programmieren, vorerst mit BlueJ später dann auch mit Java, aber ich schreibe morgen eine Arbeit darüber und finde den Fehler nicht ;( es geht um die Bedingung if (oder so) und ich sollte eine Ampel programmieren die ihre Farbe ändert, klappt aber leider irgendwie nicht, das rot gefärbte ist rot unterringelt und dabei steht dann immer "incomparable types: RECHTECK and java.lang.String"

hier die Methode in der der Fehler liegt:

void orangeAmpelKasten()
{ if (ampelKasten=="schwarz")

{ ampelKasten.FarbeSetzen("rot");
}
else

{ ampelKasten.FarbeSetzen("gelb");

}
}

 

[BodyLAB]

incomparable types: RECHTECK and java.lang.String

Die Warnung bezieht sich sicherlich auf

ampelKasten=="schwarz"

Du benötigst einen Getter der dir die Farbe des RECHTECK in Stringform zurück gibt. So versuchst du ein Objekt vom Typ Rechteck mit einem Objekt vom Typ String zu vergleichen. So etwas geht nicht, das ist ähnlich wie Äpfel mit Birnen zu vergleichen.

class RECHTECK {
 String farbe;
   
    String getFarbe() { return farbe; }

}

Deinen Code solltest du in [CODE ] Kästchen Packen hier Ist besser zu lesen

 

[Jw456]

stringvergleich nicht mit "==" sondern mit equals()

 

[heheeee]

Die Warnung bezieht sich sicherlich auf

Du benötigst einen Getter der dir die Farbe des RECHTECK in Stringform zurück gibt. So versuchst du ein Objekt vom Typ Rechteck mit einem Objekt vom Typ String zu vergleichen. So etwas geht nicht, das ist ähnlich wie Äpfel mit Birnen zu vergleichen.

class RECHTECK {
 String farbe;
 
    String getFarbe() { return farbe; }

}

Deinen Code solltest du in [CODE ] Kästchen Packen hier Ist besser zu lesen

ich habe zwar nicht ganz verstanden wie du das gemeint hast, also wie gesagt ich habe keine Ahnung von sowas, ich denke das ist weil wir das mit diesem "Getter" noch nicht hatten, aber du hast mir einen Gedankenanstoß gegeben )

Du benötigst einen Getter der dir die Farbe des RECHTECK in Stringform zurück gibt. So versuchst du ein Objekt vom Typ Rechteck mit einem Objekt vom Typ String zu vergleichen. So etwas geht nicht, das ist ähnlich wie Äpfel mit Birnen zu vergleichen.

also vor allem das hier hat mir sehr geholfen, so bin ich dann irgendwie durch rumprobieren wieder auf das hier gekommen, und die Methode ist auch endlich richtig und meine Ampel funktioniert wieder xD

void orangeAmpelKasten(String ampelKastenFarbe)
{

if (ampelKastenFarbe=="schwarz")

{ ampelKasten.FarbeSetzen("rot");
}
else

{ ampelKasten.FarbeSetzen("gelb");

}
}

 

[Jw456]

das ist aber kein Stringvergleich

if (ampelKastenFarbe.equals("schwarz")){

 

[Oneixee5]

ampelKasten.FarbeSetzen("rot")

Diese Schreibweise ist sehr ungünstig. Der Compiler mag das akzeptieren aber die Welt hat sich auf Konventionen geeinigt, wie man so etwas schreibt. Der Grund ist ganz einfach, jeder soll Quellcode leicht verstehen können. Auch wenn dieser Wörter außerhalb der eigenen Muttersprache enthält. Informiere dich bitte über Java-Code-Konventionen, z.B. hier: https://google.github.io/styleguide/javaguide.html.
Normalerweise würde die Methode so heißen: ampelKasten.setFarbe("rot"); und das Gegenteil: String farbe = ampelKasten.getFarbe();

 

[heheeee]

das ist aber kein Stringvergleich

if (ampelKastenFarbe.equals("schwarz")){

dankee, ich habe das grad auch ausprobiert und es erscheint mir irgendwie auch einfacher, ich weiß aber nicht ob ich das so anwenden darf weil wir das in der Schule so noch nicht hatten.. aber vielen Dank )

 

[heheeee]

Diese Schreibweise ist sehr ungünstig. Der Compiler mag das akzeptieren aber die Welt hat sich auf Konventionen geeinigt, wie man so etwas schreibt. Der Grund ist ganz einfach, jeder soll Quellcode leicht verstehen können. Auch wenn dieser Wörter außerhalb der eigenen Muttersprache enthält. Informiere dich bitte über Java-Code-Konventionen, z.B. hier: https://google.github.io/styleguide/javaguide.html.
Normalerweise würde die Methode so heißen: ampelKasten.setFarbe("rot"); und das Gegenteil: String farbe = ampelKasten.getFarbe();

das mag sein, aber ich würde das gerne erst mal nur so machen wie unser Lehrer das sagt... ich lerne das hauptsächlich nur in der Schule (und auch nur dafür, ich habe nicht vor Informatikerin oder sowas in die Richtung zu werden..xD), und das was du da genannt hast haben wir noch nicht gehabt

 

[Jw456]

Teste doch mal deinen Code. Du wirst egal welche Farbe du an die Methode übergibst gelb setzen. Weil dein Vergleich mit == nie true sein wird.

PS mit == kannst du Zahlen zb "int" vergleichen aber nicht Strings

Und seit wann gibt es bei einer Ampel schwarz? Dann wäre die aus.

 

[BodyLAB]

PS mit == kannst du Zahlen zb "int" vergleichen aber nicht Strings

In Java Strings, the == operator is used to check the reference of both the string objects and equals() method used to check the value equality of both strings.

Der == Operator ist nur für Primitive Datentype korrekt. Bei Objekten wie z. B. String funktioniert dieser in Java so nicht. Er vergleicht ledeglich die Speicheradresse. Diese ist bei zwei unterschiedlichen Objekten nie die selbe.
Deswegen muss man noch die equals-Methode aus Object überschreiben um Objekte vergleichen zu können.

Lernt ihr Java oder JavaScript?

 

[heheeee]

Teste doch mal deinen Code. Du wirst egal welche Farbe du an die Methode übergibst gelb setzen. Weil dein Vergleich mit == nie true sein wird.

PS mit == kannst du Zahlen zb "int" vergleichen aber nicht Strings

Und seit wann gibt es bei einer Ampel schwarz? Dann wäre die aus.

habe ich.. wenn ich da schwarz eingebe bekomme ich eine rote Ampel (bzw Ampelkasten), jede andere Farbe ergibt eine gelbe Ampel (Ampelkasten)... also wie gesagt verurteilt mich bitte nicht, 1. ich habe keine Ahnung von sowas heißt ich kann auch nicht erklären wieso das hier in dem Beispiel so ist aber die Methode funktioniert so wie ich sie gelöst habe, und 2. die Farben sind doch wurscht, ich habe Ampel so verallgemeinert gesagt, ich meinte eigentlich den Ampelkasten, ich wollte den ursprünglich auch orange machen deswegen steht das oben im Kopf auch noch so, ich wollte eigentlich nur dass die Methode endlich läuft weil ich da seit gestern ohne Erfolg herumprobiert habe und das wichtig ist für meine Arbeit morgen. So wie ich das da gemacht habe, so haben wir das bis jetzt auch nur gelernt (wir machen das erst seit ca. 1 monat)
und danke für den Hinweis )

 

[heheeee]

Der == Operator ist nur für Primitive Datentype korrekt. Bei Objekten wie z. B. String funktioniert dieser in Java so nicht. Er vergleicht ledeglich die Speicheradresse. Diese ist bei zwei unterschiedlichen Objekten nie die selbe.
Deswegen muss man noch die equals-Methode aus Object überschreiben um Objekte vergleichen zu können.

Lernt ihr Java oder JavaScript?

ich weiß es nicht, wir üben vorerst mit BlueJ und später werden wir auf Java umsteigen, wurde uns so zumindest gesagt. Ich weiß nicht ob das dann Java oder JavaScript sein wird. Ich dachte mir aber ich frage trotzdem hier weil BlueJ ähnlich wie Java sein soll

 

[KonradN]

Nur zur Ergänzung. Da == die Referenzen vergleicht, funktioniert diese bei gewissen Gegebenheiten sehr wohl!
Bei Strings ist diese Nutzung ok, so lange nur Literalte (die dann auch in Variablen gespeichert wurden) verglichen werden. Der Vergleich mit z.B. "rot" ist also ok, wenn in der Variable auch immer nur so ein Literal genutzt wurde. (Hintergrund ist, dass hier immer auf einen und den gleichen String verwiesen wird! String.intern() wäre da die Methode, dessen Dokumentation man sich anschauen sollte.)
Die zweite Stelle sind Enums bzw. Konstanten. Wenn hier immer die gleiche Konstante zugewiesen wurde und nie eine neu erstellte Instanz (Sprich so, wie es bei Enums erzwungen wird), dann wäre es auch in Ordnung.

Da dies aber potenziell Fehleranfällig ist, sollte man es nicht machen und der Hinweis, dass man bitte equals nutzen soll, ist sehr sinnvoll.

verurteilt mich bitte nicht

Es geht hier um keine Verurteilung. Wenn dann sind wir unzufrieden mit dem, was ihr wie lernt. Wir wollen Dir hier nur helfen.

wir üben vorerst mit BlueJ und später werden wir auf Java umsteigen,

Also BlueJ ist eine Entwicklungsumgebung und da schreibt Ihr schon Java Programme. Es gibt nur gewisse Unterschiede, denn bei BlueJ habt ihr nicht die Klassischen Java Programme bei denen immer eine main Methode gestartet wird. Statt dessen wird der Java Code einfach übersetzt und BlueJ startet dann eine Methode, die ihr eben starten wollt.

 

[Jw456]

Ja Literale hatte ich vergesse das das geht mache ich nie so.

 

[M.L.]

Zusatz (unabhängig von BlueJ): es gibt mit Java zwecks Stringmanipulation auch andere (mehr oder weniger) empfehlenswerte Optionen, siehe https://www.javaguides.net/2018/06/guide-to-string-best-practices-in-java.html