string arraylist untersuchen frage

[maXXX12345]

Sehr geehrte Damen und Herren,

ich bitte sie, mir folgende Frage zu beantworten:

Ich möchte eine ArrayList<String> auf bestimmte Zeichenfolgen untersuchen und die Anzahl, wie oft diese vorkommen, zurückgeben.

Benutze dazu unten stehenden Code.

Jedoch bricht die Laufvaribale i viel zu früh ab.

Wieso?

(Anm.: Die Variable max ist normalerweise größer als 20000)

Ist in diesem Code nur ein kleine Fehler oder ist meine Methode
für mein Vorhaben grundsätzlich falsch?

Vielen dank für ihre Antworten im Vorraus!

Mit freundlichen Grüßen

-----------------------------------------------------------------

import java.util.*;

class zahelenlernen{

int zaehlen(ArrayList<String> quelltext) {

   int zaehlerderjpg = 0;

   int max = (quelltext.size()-2);

   for(int i = 0; i < max; i++){ 

   if(
quelltext.get(i+1) == "j"
      && quelltext.get(i+2) == "p" && 
quelltext.get(i+3) == "g") {

   zaehlerderjpg++;
   }
   }

   return zaehlerderjpg;

}

}

----------------------------------------------------------------

 

[SlaterB]

wie kann man einen vorzeitigen Abbruch einer Java-Schleife feststellen? das wäre ja mal interessant zu erfahren,
bei mir funktioniert der Code im Rahmen seiner Grenzen, jpg wird z.B. nur gefunden wenn im Quelltext direkt eingefügt,
besser wäre String-Vergleich mit equals

Leerzeilen zwischen jeder Zeile im Posting + Quellcode sowie fehlende Java-Tags sind übrigens störend,
kleingeschriebene Klassennamen auch nicht gut, besonder noch mit Tippfehlern..

public class Test {
    public static void main(String[] args)  {
        ArrayList<String> list = new ArrayList<String>();
        for (int i = 0; i < 10000; i++) {
            list.add("x");
        }
        list.add("j");
        list.add("p");
        list.add("g");
        System.out.println(new zahelenlernen().zaehlen(list));
    }
}


class zahelenlernen {

    int zaehlen(ArrayList<String> quelltext)   {
        int zaehlerderjpg = 0;
        int max = (quelltext.size() - 2);
        for (int i = 0; i < max; i++) {
            if (quelltext.get(i + 1) == "j" && quelltext.get(i + 2) == "p" && quelltext.get(i + 3) == "g")     {
                zaehlerderjpg++;
            }
        }
        return zaehlerderjpg;
    }
}

 

[HoaX]

Strings vergleicht man mit equals, auch wenn sie nur ein Zeichen enthalten!

 

[jpgbild]

Vereinfachen wir lieber:

        char[] ca = "hallojpgundjpg!".toCharArray();

        int count = 0;

        for (int i = 0; i < ca.length - 2; i++) {
            if (ca[i] == 'j' && ca[i + 1] == 'p' && ca[i + 2] == 'g') {
                count++;
                i += 2; // für p + g nicht prüfen
            }
        }

        System.out.println("count: " + count);

Und warum so formell

 

[jpgbild]

Oder so (sehr kryptisch):

        char[] ca = "hallojpgundjpg!".toCharArray();
        char[] ca2 = "jpg".toCharArray();

        int count = 0;

        for (int i = 0; i < ca.length - ca2.length + 1; i++) {
            int j = -1;
            while (++j < ca2.length && ca[i + j] == ca2[j])
                ;
            if (j == ca2.length) {
                count++;
                i += ca2.length - 1;
            }
        }

        System.out.println("count: " + count);

Die Vorteile liegen auf der Hand und mit dem Post-Inkrementoperator spart man sich fünf Zeichen ({j=j})

 

[Surras]

den inhalt eines Strings könnte man doch auch mit String.contains("jpg"); überprüfen. So muss er nicht komplett identisch (*edit*) sein und er überprüft es. Sollte er das erste gefunden haben geht man mit String.IndexOf(); an die Stelle nach dem 'jpg' gefunden wurde und sucht ab da weiter. oder ist dies zu umständlich?

 

[jpgbild]

den inhalt eines Strings könnte man doch auch mit String.contains("jpg"); überprüfen. So muss er nicht komplett identisch (*edit*) sein und er überprüft es. Sollte er das erste gefunden haben geht man mit String.IndexOf(); an die Stelle nach dem 'jpg' gefunden wurde und sucht ab da weiter. oder ist dies zu umständlich?

klar, es gibt auch andere Mechanismen. Aber die Frage ist halt, inwieweit man was implementieren soll, auf API zurückgreifen soll oder was wodurch lernen soll.
Let me google that for you