Buchstaben zählen

[Hallo1234]

Hallo,
wieder mal eine Frage. Woran liegt es, dass bei diesem Code für jeden Buchstaben die Anzahl 0 ausgegeben wird?
Wie kann ich das beheben?

int anz =0;
        String wort = "aber";
        String[] let = {"a","b","c","d","e"};
        for(int i=0; i<let.length; i++) {
            for(int j=0; j<wort.length(); j++) {
                if(let[i] == wort.substring(j)) {
                    anz++;
                
            }
            
        }
            System.out.println("anzahl "+let[i]+ ": "+anz);
    }

Vielen Dank

 

[kneitzel]

a) Strings vergleicht man nicht mit ==
b) Füge da in der for Schleife mal ein System.out.println(wort.substring(j)); ein.

 

[Barista]

Du könntest im Array let statt Strings char verwenden.

Aus dem String wort kannst Du chars mit wort.charAt(i) abfragen.

 

[Hallo1234]

Danke für die Hilfe. Jetzt gibt es nur noch ein kleines Problem:

Wie kommt es, dass die Anzahl von "d" 2 ist, obwohl im Wort nicht mal ein d enthalten ist?

int anz =0;
        String wort = "aber";
        char[] let = {'a','b','c','d','e'};
        for(int i=0; i<let.length; i++) {
            for(int j=0; j<wort.length(); j++) {
                if(let[i] == wort.charAt(j)) {
                    anz++;

                }
            }
            System.out.println("anzahl "+let[i]+ ": "+anz);
        }

 

[thecain]

Weil du anz nie auf 0 zurücksetzt bzw den Scope zu gross gewählt hast

 

[Hallo1234]

Wenn ich auf 0 zurücksetze, wird überall 0 ausgegeben:

String wort = "aber";
        char[] let = {'a','b','c','d','e'};
        int anz = 0;
        for(int i=0; i<let.length; i++) {
            for(int j=0; j<wort.length(); j++) {
                if(let[i] == wort.charAt(j)) {
                    anz++;
                }else {
                    anz = 0;
                }
            }
            System.out.println("anzahl "+let[i]+ ": "+anz);
        }

Oder gibt es noch eine andere Möglichkeit, um auf 0 zurückzusetzen?
Stimmt das mit dem Scope jetzt?

 

[kneitzel]

Nein, das stimmt so nicht (Wie Du am Ergebnis ja erkennen kannst.

Wann soll denn die Anzahl auf 0 gesetzt werden? Doch immer, wenn Du das nächste Zeichen aus dem Array let auswählst.
Das ist ja die äußere for Schleife. Somit muss das anz=0 als erstes innerhalb der äußersten for schleife erfolgen.

Dazu kann ich Dir auch nur den Tipp geben, dass Du so Aufgaben mehr unterteilst. Generell ist eine Verschachtelungstiefe > 2 immer schlecht und schwer zu überblicken. Das kann man also vermeiden. Schon beim überlegen des Algorithmus kann man das direkt unterteilen:

Zählen alles Zeichen aus Zeichenmenge in Zeichenfolge folge:
- Für jedes Zeichen aus Zeichenmenge:
--- AnzahlDesZeichens := zähleZeichenInString(folge, zeichen)
--- Ausgabe AnzahDesZeichens

Also schon bei der Überlegung sowas unterteilen. Dann wird alles einfach und übersichtlich. Und auch die Bezeichner gleich deutlich wählen. i und j können leicht vertauscht werden und man sieht es nicht direkt.

 

[Hallo1234]

Danke dir für die ausführliche Erklärung : )
Es funktioniert jetzt endlich.

String wort = "aber";
        char[] let = {'a','b','c','d','e'};
        int anz = 0;
        for(int i=0; i<let.length; i++) {
            for(int j=0; j<wort.length(); j++) {
                if(let[i] == wort.charAt(j)) {
                    anz++;
            }
        }
            
            System.out.println("anzahl "+let[i]+ ": "+anz);
            anz = 0;
        }

Vielen Dank

 

[Barista]

Es hilft Dir sicher, wenn die Einrückung in Deinem Code korrekt ist.
Falls Du Eclipse verwendest Strg-A und Strg-I (alle Zeilen korrekt einrücken) oder nur Strg-F (Code formatieren).

 

[Hallo1234]

Das werde ich beachten, danke.