Ocorrências de substring em uma string

Por que o seguinte algoritmo não está sendo interrompido para mim? (str é a string em que estou procurando, findStr é a string que estou tentando encontrar)

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);

A última linha estava criando um problema. lastIndexnunca seria -1, então haveria um loop infinito. Isso pode ser corrigido movendo a última linha de código para o bloco 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);

Que tal usar o StringUtils.countMatches do Apache Commons Lang?

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

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

Isso gera:

3

Seu lastIndex += findStr.length();foi colocado fora dos colchetes, causando um loop infinito (quando nenhuma ocorrência foi encontrada, lastIndex sempre foifindStr.length() ).

Aqui está a versão fixa:

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);

Uma versão mais curta. ;)

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

Você realmente tem que lidar com a correspondência? Especialmente se tudo o que você precisa é o número de ocorrências, as expressões regulares são mais organizadas:

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);     

Estou muito surpreso que ninguém tenha mencionado esse forro. É simples, conciso e tem um desempenho um pouco melhor do questr.split(target, -1).length-1

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

Aqui está, embrulhado em um método agradável e reutilizável:

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);

no final da contagem de loop é 3; espero que ajude

Muitas das respostas fornecidas falham em um ou mais dos seguintes:

• Padrões de comprimento arbitrário
• Correspondências sobrepostas (como contar "232" em "23232" ou "aa" em "aaa")
• Meta-caracteres de expressão regular

Aqui está o que eu escrevi:

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;
}

Chamada de exemplo:

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

Se você deseja uma pesquisa sem expressão regular, compile seu padrão adequadamente com o LITERALsinalizador:

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();
}

Incremente lastIndexsempre que procurar a próxima ocorrência.

Caso contrário, ele sempre encontrará a primeira substring (na posição 0).

public int indexOf(int ch,
                   int fromIndex)

Retorna o índice dentro dessa cadeia de caracteres da primeira ocorrência do caractere especificado, iniciando a pesquisa no índice especificado.

Portanto, seu lastindexvalor é sempre 0 e sempre encontra olá na string.

A resposta dada como correta não é boa para contar coisas como retornos de linha e é muito detalhada. Respostas posteriores são melhores, mas tudo pode ser alcançado simplesmente com

str.split(findStr).length

Não descarta correspondências finais usando o exemplo na pergunta.

Você pode numerar ocorrências usando a função de biblioteca embutida:

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

tente adicionar lastIndex+=findStr.length()ao final do seu loop, caso contrário, você terminará em um loop sem fim, porque depois de encontrar a substring, você está tentando encontrá-lo repetidamente da mesma última posição.

Tente este. Ele substitui todos os jogos por um -.

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

E se você não quer destruir o seu, strvocê pode criar uma nova string com o mesmo conteúdo:

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

Depois de executar este bloco, estes serão seus valores:

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

Como @Mr_and_Mrs_D sugeriu:

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

Com base nas respostas existentes, gostaria de adicionar uma versão "mais curta" sem o 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

Aqui está a versão avançada para contar quantas vezes o token ocorreu em uma sequência inserida pelo usuário:

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);
    }

}

Este método abaixo mostra quantas vezes a substring se repete em toda a sua string. Espero usar completo para você: -

    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++;
        }

    }

aqui está a outra solução sem o uso de regexp / patterns / matchers ou mesmo sem o uso de 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);

Se você precisar do índice de cada substring na string original, poderá fazer algo com o indexOf assim:

 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;

}

Esta solução imprime o número total de ocorrências de uma determinada substring em toda a cadeia, incluindo também os casos em que existem correspondências sobrepostas.

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);
    }
}