Wystąpienia podciągu w ciągu

Dlaczego poniższy algorytm nie zatrzymuje się dla mnie? (str to ciąg, w którym szukam, findStr to ciąg, który próbuję znaleźć)

String str = "helloslkhellodjladfjhello";
String findStr = "hello";
int lastIndex = 0;
int count = 0;

while (lastIndex != -1) {
    lastIndex = str.indexOf(findStr,lastIndex);

    if( lastIndex != -1)
        count++;

    lastIndex += findStr.length();
}

System.out.println(count);

Ostatnia linijka stwarzała problem. lastIndexnigdy nie byłby na -1, więc byłaby nieskończona pętla. Można to naprawić, przenosząc ostatnią linię kodu do bloku if.

String str = "helloslkhellodjladfjhello";
String findStr = "hello";
int lastIndex = 0;
int count = 0;

while(lastIndex != -1){

    lastIndex = str.indexOf(findStr,lastIndex);

    if(lastIndex != -1){
        count ++;
        lastIndex += findStr.length();
    }
}
System.out.println(count);

Co powiesz na użycie StringUtils.countMatches z Apache Commons Lang?

String str = "helloslkhellodjladfjhello";
String findStr = "hello";

System.out.println(StringUtils.countMatches(str, findStr));

To daje:

3

Twój lastIndex += findStr.length();został umieszczony poza nawiasami, powodując nieskończoną pętlę (gdy nie znaleziono żadnego wystąpienia, lastIndex zawsze miał findStr.length()).

Oto poprawiona wersja:

String str = "helloslkhellodjladfjhello";
String findStr = "hello";
int lastIndex = 0;
int count = 0;

while (lastIndex != -1) {

    lastIndex = str.indexOf(findStr, lastIndex);

    if (lastIndex != -1) {
        count++;
        lastIndex += findStr.length();
    }
}
System.out.println(count);

Krótsza wersja. ;)

String str = "helloslkhellodjladfjhello";
String findStr = "hello";
System.out.println(str.split(findStr, -1).length-1);

Czy naprawdę musisz sam poradzić sobie z dopasowywaniem? Zwłaszcza jeśli potrzebujesz tylko liczby wystąpień, wyrażenia regularne są bardziej uporządkowane:

String str = "helloslkhellodjladfjhello";
Pattern p = Pattern.compile("hello");
Matcher m = p.matcher(str);
int count = 0;
while (m.find()){
    count +=1;
}
System.out.println(count);     

Jestem bardzo zaskoczony, że nikt nie wspomniał o tej jednej wkładce. Jest prosty, zwięzły i działa nieco lepiej niżstr.split(target, -1).length-1

public static int count(String str, String target) {
    return (str.length() - str.replace(target, "").length()) / target.length();
}

Oto jest, opakowany w ładną metodę wielokrotnego użytku:

public static int count(String text, String find) {
        int index = 0, count = 0, length = find.length();
        while( (index = text.indexOf(find, index)) != -1 ) {                
                index += length; count++;
        }
        return count;
}

String str = "helloslkhellodjladfjhello";
String findStr = "hello";
int lastIndex = 0;
int count = 0;

while((lastIndex = str.indexOf(findStr, lastIndex)) != -1) {
     count++;
     lastIndex += findStr.length() - 1;
}
System.out.println(count);

na końcu liczby pętli wynosi 3; mam nadzieję, że to pomoże

Wiele z podanych odpowiedzi kończy się niepowodzeniem w jednym lub kilku z:

• Wzory o dowolnej długości
• Pokrywające się dopasowania (np. Liczenie „232” w „23232” lub „aa” w „aaa”)
• Metaznaki wyrażeń regularnych

Oto co napisałem:

static int countMatches(Pattern pattern, String string)
{
    Matcher matcher = pattern.matcher(string);

    int count = 0;
    int pos = 0;
    while (matcher.find(pos))
    {
        count++;
        pos = matcher.start() + 1;
    }

    return count;
}

Przykładowe połączenie:

Pattern pattern = Pattern.compile("232");
int count = countMatches(pattern, "23232"); // Returns 2

Jeśli chcesz wyszukiwać bez wyrażeń regularnych, po prostu skompiluj odpowiednio swój wzorzec z LITERALflagą:

Pattern pattern = Pattern.compile("1+1", Pattern.LITERAL);
int count = countMatches(pattern, "1+1+1"); // Returns 2

public int countOfOccurrences(String str, String subStr) {
  return (str.length() - str.replaceAll(Pattern.quote(subStr), "").length()) / subStr.length();
}

Zwiększaj lastIndexza każdym razem, gdy szukasz następnego wystąpienia.

W przeciwnym razie zawsze znajduje pierwszy podciąg (na pozycji 0).

public int indexOf(int ch,
                   int fromIndex)

Zwraca indeks w ramach tego ciągu pierwszego wystąpienia określonego znaku, rozpoczynając wyszukiwanie od określonego indeksu.

Więc twój lastindex wartość jest zawsze 0 i zawsze znajduje hello w ciągu.

Odpowiedź podana jako poprawna nie nadaje się do liczenia takich rzeczy, jak powroty linii i jest zbyt szczegółowa. Późniejsze odpowiedzi są lepsze, ale wszystko można osiągnąć po prostu za pomocą

str.split(findStr).length

Nie usuwa końcowych dopasowań, korzystając z przykładu w pytaniu.

Możesz liczbę wystąpień za pomocą wbudowanej funkcji bibliotecznej:

import org.springframework.util.StringUtils;
StringUtils.countOccurrencesOf(result, "R-")

spróbuj dodać lastIndex+=findStr.length()na końcu pętli, w przeciwnym razie skończysz w nieskończonej pętli, ponieważ po znalezieniu podciągu próbujesz go znaleźć ponownie i ponownie z tej samej ostatniej pozycji.

Spróbuj tego. Zastępuje wszystkie dopasowania rozszerzeniem -.

String str = "helloslkhellodjladfjhello";
String findStr = "hello";
int numberOfMatches = 0;
while (str.contains(findStr)){
    str = str.replaceFirst(findStr, "-");
    numberOfMatches++;
}

A jeśli nie chcesz niszczyć swojego str, możesz stworzyć nowy ciąg o tej samej zawartości:

String str = "helloslkhellodjladfjhello";
String strDestroy = str;
String findStr = "hello";
int numberOfMatches = 0;
while (strDestroy.contains(findStr)){
    strDestroy = strDestroy.replaceFirst(findStr, "-");
    numberOfMatches++;
}

Po wykonaniu tego bloku będą to twoje wartości:

str = "helloslkhellodjladfjhello"
strDestroy = "-slk-djladfj-"
findStr = "hello"
numberOfMatches = 3

Zgodnie z sugestią @Mr_and_Mrs_D:

String haystack = "hellolovelyworld";
String needle = "lo";
return haystack.split(Pattern.quote(needle), -1).length - 1;

Na podstawie istniejących odpowiedzi chciałbym dodać „krótszą” wersję bez warunku if:

String str = "helloslkhellodjladfjhello";
String findStr = "hello";

int count = 0, lastIndex = 0;
while((lastIndex = str.indexOf(findStr, lastIndex)) != -1) {
    lastIndex += findStr.length() - 1;
    count++;
}

System.out.println(count); // output: 3

Oto zaawansowana wersja do zliczania, ile razy token wystąpił w ciągu wprowadzonym przez użytkownika:

public class StringIndexOf {

    public static void main(String[] args) {

        Scanner scanner = new Scanner(System.in);

        System.out.println("Enter a sentence please: \n");
        String string = scanner.nextLine();

        int atIndex = 0;
        int count = 0;

        while (atIndex != -1)
        {
            atIndex = string.indexOf("hello", atIndex);

            if(atIndex != -1)
            {
                count++;
                atIndex += 5;
            }
        }

        System.out.println(count);
    }

}

Poniższa metoda pokazuje, ile razy podciąg powtórzył się w całym ciągu. Mam nadzieję, że wykorzystasz w pełni: -

    String searchPattern="aaa"; // search string
    String str="aaaaaababaaaaaa"; // whole string
    int searchLength = searchPattern.length(); 
    int totalLength = str.length(); 
    int k = 0;
    for (int i = 0; i < totalLength - searchLength + 1; i++) {
        String subStr = str.substring(i, searchLength + i);
        if (subStr.equals(searchPattern)) {
           k++;
        }

    }

tutaj jest inne rozwiązanie bez użycia regexp / patterns / matcherów lub nawet bez użycia StringUtils.

String str = "helloslkhellodjladfjhelloarunkumarhelloasdhelloaruhelloasrhello";
        String findStr = "hello";
        int count =0;
        int findStrLength = findStr.length();
        for(int i=0;i<str.length();i++){
            if(findStr.startsWith(Character.toString(str.charAt(i)))){
                if(str.substring(i).length() >= findStrLength){
                    if(str.substring(i, i+findStrLength).equals(findStr)){
                        count++;
                    }
                }
            }
        }
        System.out.println(count);

Jeśli potrzebujesz indeksu każdego podciągu w oryginalnym ciągu, możesz zrobić coś z indexOf w ten sposób:

 private static List<Integer> getAllIndexesOfSubstringInString(String fullString, String substring) {
    int pointIndex = 0;
    List<Integer> allOccurences = new ArrayList<Integer>();
    while(fullPdfText.indexOf(substring,pointIndex) >= 0){
       allOccurences.add(fullPdfText.indexOf(substring, pointIndex));
       pointIndex = fullPdfText.indexOf(substring, pointIndex) + substring.length();
    }
    return allOccurences;
}

public static int getCountSubString(String str , String sub){
int n = 0, m = 0, counter = 0, counterSub = 0;
while(n < str.length()){
  counter = 0;
  m = 0;
  while(m < sub.length() && str.charAt(n) == sub.charAt(m)){
    counter++;
    m++; n++;
  }
  if (counter == sub.length()){
    counterSub++;
    continue;
  }
  else if(counter > 0){
    continue;
  }
  n++;
}

return  counterSub;

}

To rozwiązanie wypisuje całkowitą liczbę wystąpień danego podciągu w całym ciągu, a także obejmuje przypadki, w których zachodzą na siebie dopasowania.

class SubstringMatch{
    public static void main(String []args){
        //String str = "aaaaabaabdcaa";
        //String sub = "aa";
        //String str = "caaab";
        //String sub = "aa";
        String str="abababababaabb";
        String sub = "bab";

        int n = str.length();
        int m = sub.length();

        // index=-1 in case of no match, otherwise >=0(first match position)
        int index=str.indexOf(sub), i=index+1, count=(index>=0)?1:0;
        System.out.println(i+" "+index+" "+count);

        // i will traverse up to only (m-n) position
        while(index!=-1 && i<=(n-m)){   
            index=str.substring(i, n).indexOf(sub);
            count=(index>=0)?count+1:count;
            i=i+index+1;  
            System.out.println(i+" "+index);
        }
        System.out.println("count: "+count);
    }
}