Hilfe bei Zählen von Farben?

[Maawe]

Hallo
Habe ein Problem. Ich habe ein Arrey (verwaltung) in dem es Autos in Grün, Rot und Gelb gibt. Nun soll einfach ausgegeben werden wieviel Autos es mit dieser Farbe gibt. Hab auch schon einen Ansatz:

 public void wievieleFarben()
  {
    int grn = 0;
    int glb = 0;
    int rt  = 0;
    for(int i=0;i<verwaltung.length;i++)
    {
     if (verwaltung[i].getFarbe() == "Grün")
     grn=grn+1;
     if (verwaltung[i].getFarbe() == "Gelb")
     glb=glb+1;
     if (verwaltung[i].getFarbe() == "Rot")
     rt=rt+1;
    }
    System.out.println(grn);
    System.out.println(glb);
    System.out.println(rt);
  }

Allerdings zeigt er mir am ende immer nur Nullen an, hab auch keine Ahnung was der Fehler ist, vielleicht kann mir da jemand weiterhelfen ^^.

 

[Kiri]

 ...
     if (verwaltung[i].getFarbe() == "Grün")
...

[WR]Strings vergleicht man mit equals nicht mit ==[/WR]

if (verwaltung[i].getFarbe().equals("Grün"))

 

[Maawe]

Danke für den Hinweis, wusste ich noch nicht da ich noch relativ unwissend bin ^^.
Leider liefert er mir immernoch 3x 0 als ergebnis :/

 

[Kiri]

Dann zeig den ganzen Code!

 

[Maawe]

Achherrje , habe den Fehler gerade selber gefunden :bloed: ich sollte natürlich auf Groß und Kleinschreibung achten, hab es oben im programm kleingeschrieben :bloed: Trotzdem danke für deine Hilfe
'close

 

[Kiri]

Du solltest aber noch deine If-Anweisungen anpassen, denn du prüfst immer alle drei Farben, auch wenn die erste schon richtig ist!

 

[Maawe]

Jo danke, schon geschehen

 

[VfL_Freak]

Moin,

außerdem gleich noch ein Tipp:

Gewöhn' Dir von vornherein an, "sprechende Variablennamen" zu verwenden !!

Ist ein Projekt erstmal größer und Du hast vlt. auch länger nicht mehr reingeschaut, dann garantiere ich dir, dass Du irgendwann nicht mehr weißt, wofür "grn", "glb" oder "rt" stehen ....

Oder sind auf Deiner Tastatur die Vokale kaputt ???:L :lol:

Gruß
Klaus