Não precisa fazer um loop dentro de outro, pois isso é bem ineficiente: para cada caractere da String
, você quer compará-lo com cada uma das 26 letras, então no final você estará fazendo N * 26 iterações (sendo N
o tamanho da String
).
Dá para fazer tudo isso percorrendo a String
uma única vez. Basta criar um array contendo a contagem de cada letra, e ir atualizando-o conforme percorre a String
.
Depois você faz outro loop para mostrar as quantidades de cada letra:
String frase = "Daniel Henrique";
int[] quantidades = new int[26];
for (char c : frase.toUpperCase().toCharArray()) {
int indice = c - 65;
if (indice >= 0 && indice < 26) {
quantidades[indice] += 1;
}
}
for (int i = 0; i < quantidades.length; i++) {
char letra = (char) (i + 65);
System.out.println(letra + "=" + quantidades[i]);
}
Primeiro eu crio um array contendo as quantidades de cada letra, por isso ele tem 26 posições. Eu me aproveito do fato de que em Java, um array de int
criado desta maneira já é inicializado com zero em todas as posições (o que é justamente o que precisamos, pois no início a quantidade de qualquer letra é zero, já que eu ainda não iniciei a contagem).
No primeiro for
eu percorro a String
. No seu código deu a entender que tanto faz se as letras forem maiúsculas ou minúsculas, então eu estou usando maiúsculas (toUpperCase()
).
Para cada caractere, eu verifico se ele é uma letra de A
a Z
(ao subtrair 65 de um char
, caso este seja uma letra de A
a Z
, o resultado estará entre zero e 25, que é uma posição válida no array de quantidades). Se for, eu somo 1 na respectiva quantidade.
No segundo for
eu imprimo as letras e suas respectivas quantidades. No fim, eu só percorri a String
e as quantidades uma vez, totalizando N + 26 iterações (sendo N
o tamanho da String
) - bem melhor que fazer um loop dentro de outro, o que como já foi dito, gera N * 26 iterações.