Wo ist mein Fehler in der Methode DRINGEND

[Ru$h]

Hallo,
ich habe folgende Methode:

 public void auswertenMerkmal (Schmetterling Kinder[], String merkmal){
                 
                // Lege Teilliste an, welche nur diejenigen Schmetterlinge beinhaltet, die das gewünschte Merkmal besitzen.
              
                LinkedList<Schmetterling> KinderMerkmal = new LinkedList<Schmetterling>();
               
                 for (int i=0; i<Kinder.length; i++){
                     if ((Kinder[i].getFuehlerform().equals(merkmal)) || (Kinder[i].getMusterung().equals(merkmal)) || (Kinder[i].getFluegelfarbe().equals(merkmal))){
                         KinderMerkmal.add(Kinder[i]);
                     }
                }
                  System.out.println(KinderMerkmal);
            }

Dazu eine Erklärung: die Methode bekommt eine "Liste" Schmetterling Kinder[] von ca. 1500 Schmetterlingen mit jeweils 3 unterschiedlichen Merkmalen (Musterung(entweder keins, gepunktet oder streifen), Farbe(rot, gelb, grün, blau), Fühlerform(gerade oder gekrümmt)). Es kommen daher in der "Liste" Schmetterlinge mehrfach vor.
Wenn ich der Methode noch ein Merkmal mit hinzu gebe, dann soll diese Methode mir eine Teilliste erstellen, mit der prozentualen Anzahl der Schmetterlinge mit dem hereingegeben Merkmal.
Meine Methode ist aber nicht richtig. Die Konsole spuckt mir folgendes aus: []

den Scanner habe ich in einer anderen Klasse so prgrammiert:

String merkmal = "";
        System.out.println (" ");
        System.out.println ("Welches Merkmal wollen Sie von allen Kinderschmetterlingen genauer untersuchen?");
        System.out.println ("Geben Sie '1' für 'keine Musterung', '2' für 'schwarze Punkte', '3' für 'schwarze Streifen',");
        System.out.println("'4' für 'rot', '5' für 'gelb', '6' für 'grün', '7' für 'blau', '8' für 'gerade Fühler', '9' für 'gekrümmte Fühler' ein");       
           while(merkmal == ""){
           String EingabeV1 = scanner.nextLine();
            switch (EingabeV1){
                case "1": merkmal = "keine Musterung";
                break;
                case "2": merkmal = "schwarze Punkte";
                break;
                case "3": merkmal = "schwarze Streifen";
                break;
                case "4": merkmal = "rot";
                break;
                case "5": merkmal = "gelb";
                break;
                case "6": merkmal = "grün";
                break;
                case "7": merkmal = "blau";
                break;
                case "8": merkmal = "gerade Fühler";
                break;
                case "9": merkmal = "gekrümmte Fühler";
                break;
                default: System.out.println("Eingabe ungültig!");
            }
        }
           Auswertung auswertung2 = new Auswertung();
           auswertung2.auswertenMerkmal(Kinder, merkmal);

Kann jemand meine Methode korrigieren. Ich sehe den Fehler nicht

 

[Robat]

1. Du hast in deiner while-Schleife merkmal == "" stehen .. Strings vergleicht man inhaltlich aber mit merkmal.equals("")
2. Sind die Merkmale in der switch-case-Anweisung wirklich genau so geschrieben wie bei den Schmetterlingsobjekten?

 

[Ru$h]

Ja die Merkmale in der switch-case-Anweisung sind wirklich genau so geschrieben wie bei den Schmetterlingsobjekten
und wenn ich in der while-schleife die Vergleiche auf merkmal.equals setze,

String merkmal = "";
        System.out.println (" ");
        System.out.println ("Welches Merkmal wollen Sie von allen Kinderschmetterlingen genauer untersuchen?");
        System.out.println ("Geben Sie '1' für 'keine Musterung', '2' für 'schwarze Punkte', '3' für 'schwarze Streifen',");
        System.out.println("'4' für 'rot', '5' für 'gelb', '6' für 'grün', '7' für 'blau', '8' für 'gerade Fühler', '9' für 'gekrümmte Fühler' ein");       
           while(merkmal == ""){
           String EingabeV1 = scanner.nextLine();
            switch (EingabeV1){
                case "1": merkmal.equals("keine Musterung");
                break;
                case "2": merkmal = "schwarze Punkte";
                break;
                case "3": merkmal = "schwarze Streifen";

bringt er mir für die 1 und für die 2 folgendes zurück

Welches Merkmal wollen Sie von allen Kinderschmetterlingen genauer untersuchen?
Geben Sie '1' für 'keine Musterung', '2' für 'schwarze Punkte', '3' für 'schwarze Streifen',
'4' für 'rot', '5' für 'gelb', '6' für 'grün', '7' für 'blau', '8' für 'gerade Fühler', '9' für 'gekrümmte Fühler' ein
1
2
[MendelsLand.Schmetterling@4554617c, MendelsLand.Schmetterling@74a14482,

 

[Robat]

Der Hinweis von 1. galt wirklich nur der Bedingung deiner while-Schleife. Die switch-case war okay.
Du kannst dir in deiner auswerten-Methode mal die Merkmale der Schmetterlinge + dein übergebenes Merkmal ausgeben lassen. So kannst du zumindest feststellen, ob da eventuell doch Unterschiede bestehen.

 

[Ru$h]

Also das funktioniert:

1   Keine Musterung, rot, gerade  12.792792792792792 Prozent
2   Keine Musterung, rot, gekrümmt  10.18018018018018 Prozent
3   Keine Musterung, gelb, gerade  12.702702702702704 Prozent
4   Keine Musterung, gelb, gekrümmt  12.432432432432433 Prozent
5   Keine Musterung, grün, gerade  0.0 Prozent
6   Keine Musterung, grün, gekrümmt  0.0 Prozent
7   Keine Musterung, blau, gerade  0.0 Prozent
8   Keine Musterung, blau, gekrümmt  0.0 Prozent
9   schwarze Punkte, rot, gerade  12.252252252252251 Prozent
10   schwarze Punkte, rot, gekrümmt  14.414414414414415 Prozent
11   schwarze Punkte, gelb, gerade  12.522522522522522 Prozent
12   schwarze Punkte, gelb, gekrümmt  12.702702702702704 Prozent
13   schwarze Punkte, grün, gerade  0.0 Prozent
14   schwarze Punkte, grün, gekrümmt  0.0 Prozent
15   schwarze Punkte, blau, gerade  0.0 Prozent
16   schwarze Punkte, blau, gekrümmt  0.0 Prozent
17   schwarze Streifen, rot, gerade  0.0 Prozent
18   schwarze Streifen, rot, gekrümmt  0.0 Prozent
19   schwarze Streifen, gelb, gerade  0.0 Prozent
20   schwarze Streifen, gelb, gekrümmt  0.0 Prozent
21   schwarze Streifen, grün, gerade  0.0 Prozent
22   schwarze Streifen, grün, gekrümmt  0.0 Prozent
23   schwarze Streifen, blau, gerade  0.0 Prozent
24   schwarze Streifen, blau, gekrümmt  0.0 Prozent
Welches Merkmal wollen Sie von allen Kinderschmetterlingen genauer untersuchen?
Geben Sie '1' für 'keine Musterung', '2' für 'schwarze Punkte', '3' für 'schwarze Streifen',
'4' für 'rot', '5' für 'gelb', '6' für 'grün', '7' für 'blau', '8' für 'gerade Fühler', '9' für 'gekrümmte Fühler' ein
1
keine Musterung

mit

public void auswertenMerkmal (Schmetterling Kinder[], String merkmal){
          
            System.out.println(merkmal);
                 
              /*  // Lege Teilliste an, welche nur diejenigen Schmetterlinge beinhaltet, die das gewünschte Merkmal besitzen.
              
                LinkedList<Schmetterling> KinderMerkmal = new LinkedList<Schmetterling>();
               
                 for (int i=0; i<Kinder.length; i++){
                     if ((Kinder[i].getFuehlerform().equals(merkmal)) || (Kinder[i].getMusterung().equals(merkmal)) || (Kinder[i].getFluegelfarbe().equals(merkmal))){
                         KinderMerkmal.add(Kinder[i]);
                     }
                }
                 System.out.println(KinderMerkmal);
           */  }

Aber dadurch wird mein Problem nicht besser
Es liegt also wirklich nur an meiner auswerten - Methode
Hast du eine Idee?

 

[Robat]

Wenn mich nicht alles täusch ist dein Merkmal am Anfang klein geschrieben aber das Merkmal der Schmetterlinge beginnt mit einem großen k.
keine Musterung
Keine Musterung

 

[Ru$h]

Hups gar nicht gesehen. Vielen Dank!
Aber es ändert trotz alle dem nichts an der Ausführung.
Ich habe die Methode nun so angepasst:

public double[] auswertenMerkmal (Schmetterling Kinder[], String merkmal){
           
            System.out.println("Sie haben "+merkmal+" gewählt");
       
           
            int[] zaehler = new int[9];
            for(int i=0;i<9;i++){
                zaehler[i] = 0;
            }
           
            for (int i=0; i<Kinder.length; i++){
                if (Kinder[i].getMerkmal() == "Keine Musterung"){
                    zaehler[0]++;
                    }
                     else if (Kinder[i].getMerkmal() == "schwarze Punkte"){
                            zaehler[1]++;
                     }
                     else if (Kinder[i].getMerkmal() == "schwarze Streifen"){
                            zaehler[2]++;
                     }
                     else if (Kinder[i].getMerkmal() == "rot"){
                            zaehler[3]++;
                     }
                     else if (Kinder[i].getMerkmal() == "gelb"){
                            zaehler[4]++;
                     }
                     else if (Kinder[i].getMerkmal() == "grün"){
                            zaehler[5]++;
                     }
                     else if (Kinder[i].getMerkmal() == "blau"){
                            zaehler[6]++;
                     }
                     else if (Kinder[i].getMerkmal() == "gerade"){
                            zaehler[7]++;
                     }
                     else if (Kinder[i].getMerkmal() == "gekrümmt"){
                            zaehler[8]++;
                     }
               
                }
                double[] Anteile2 = new double[9];
                for(int i =0;i<9;i++){
                    Anteile2[i] = (double)(zaehler[i]/Kinder.length) * 100;
                   
                    System.out.print(i+1);
                    System.out.print("  ");
                    switch(i){
                        case 0: System.out.print(" "+ "Keine Musterung"+" ");
                        break;
                        case 1: System.out.print(" "+ "schwarze Punkte"+" ");
                        break;
                        case 2: System.out.print(" "+ "schwarze Streifen"+" ");
                        break;
                        case 3: System.out.print(" "+ "rot"+" ");
                        break;
                        case 4: System.out.print(" "+ "gelb"+" ");
                        break;
                        case 5: System.out.print(" "+ "grün"+" ");
                        break;
                        case 6: System.out.print(" "+ "blau"+" ");
                        break;
                        case 7: System.out.print(" "+ "gerade"+" ");
                        break;
                        case 8: System.out.print(" "+ "gekrümmt"+" ");
                        break;
                    }
                      
                   // System.out.println(zaehler[i]);
                    System.out.println(" "+(double)zaehler[i]/Kinder.length*100+" "+"Prozent");
                }
            return Anteile2;
}

Die Frage ist nun noch, dass weder der zaéhler zählt, noch die Prozentzahl errecnet wird.
In der Schmetterlingsklasse habe ich noch einen Getter für das Merkmal geschrieben.

 

[Flown]

Kannst du hier mal das ganze Programm posten?

 

[Robat]

Auch hier wieder: Strings vergleicht man inhaltlich mit equals.

zaehler[i]/Kinder.length

Hier wird immer 0 rauskommen, wenn zaehler[i] < als Kinder.length ist, da es sich um eine Integerdivision handelt.

 

[MoxxiManagarm]

Kinder[i].getMerkmal() == "schwarze Punkte"
else if (Kinder[i].getMerkmal() == "rot")
 else if (Kinder[i].getMerkmal() == "gerade"){

Unabhängig von dem ==, ich dachte du hast 3 verschiedene Merkmalstypen. Mit getMerkmal() kannst du doch aber theoretisch nur einen bekommen außer du machst einen append, aber dann ist der direkte Vergleich nicht richtig.

PS.: Ich stell mir gerade vor du willst mehr Merkmale hinzufügen wie anders farbige Musterungen. Dann musst du deinen Code komplett neu anpassen!

Meiner einer würde ja eventuell den Weg über Enum gehen.

Spoiler

package javaforum.org.schmetterlinge;

import java.util.ArrayList;

public class Schmetterlingsfarm {

private ArrayList<Schmetterling> schmetterlinge;

public Schmetterlingsfarm() {
schmetterlinge = new ArrayList<>();
}

public boolean addSchmetterling(Schmetterling schmetterling) {
return schmetterlinge.add(schmetterling);
}

public int zaehleMerkmal(Merkmal merkmal) {
int zaehler = 0;

for(Schmetterling schmetterling : schmetterlinge) {
if(schmetterling.hatMerkmal(merkmal)) zaehler++;
}

return zaehler;
}

public static void main(String[] args) {
Schmetterlingsfarm farm = new Schmetterlingsfarm();

farm.addSchmetterling(new Schmetterling(Musterung.GEPUNKTED, Fuehler.GERADE));
farm.addSchmetterling(new Schmetterling(Musterung.GESTREIFT, Fuehler.GERADE));
farm.addSchmetterling(new Schmetterling(Musterung.GESTREIFT, Fuehler.GEKRUEMMT));

System.out.println("### MUSTERUNGEN ###");
for(Musterung musterung : Musterung.values()) {
System.out.println(musterung.ordinal() + " " + musterung.name() + " : " + farm.zaehleMerkmal(musterung));
}

System.out.println("\n### FUEHLER ###");
for(Fuehler fuehler : Fuehler.values()) {
System.out.println(fuehler.ordinal() + " " + fuehler.name() + " : " + farm.zaehleMerkmal(fuehler));
}
}
}

Spoiler

package javaforum.org.schmetterlinge;

public class Schmetterling {
private Musterung musterung;
private Fuehler fuehler;

public Schmetterling(Musterung musterung, Fuehler fuehler) {
this.musterung = musterung;
this.fuehler = fuehler;
}

public boolean hatMerkmal(Merkmal merkmal) {
return musterung.equals(merkmal) || fuehler.equals(merkmal);
}
}

Spoiler

package javaforum.org.schmetterlinge;

public enum Musterung implements Merkmal {
KEINE, GESTREIFT, GEPUNKTED;
}

Spoiler

package javaforum.org.schmetterlinge;

public enum Fuehler implements Merkmal {
GEKRUEMMT, GERADE;
}

Spoiler

package javaforum.org.schmetterlinge;

public interface Merkmal {

}