O principal problema parece ser o incremento que não está ocorrendo. No momento está adicionando ao count
o próprio count
que inicialmente vale 0, então ele nunca vai sair deste valor. O operador de auto adição deve ser usado com valores que não sejam neutros, senão o valor não será mudado. Mesmo que começasse com 1, a contagem iria progredindo geometricamente, o que não é o desejado, cada passo deve somar sempre 1 e não a última contagem. É mais simples que isto, basta usar o operador de incremento que equivale a fazer count += 1
.
Outra mudança que eu faria por questões de performance é pegar o caractere ao invés da substring:
private int countVirgulas(String parametro) {
int count = 0;
for (int i = 0; i < parametro.length(); i++) {
if (parametro.charAt(i) == ',') {
count++;
}
}
return count;
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Ou se preferir:
private int countVirgulas(String parametro) {
int count = 0;
for (char caractere : parametro.toCharArray()) {
if (caractere == ',') {
count++;
}
}
return count;
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Se quiser usar o substring()
tem que acertar um erro, já que ele não está pegando um caractere de cada vez:
private int countVirgulas(String parametro) {
int count = 0;
for (int i = 0; i < parametro.length(); i++) {
if (parametro.substring(i, i + 1).equals(",")) {
count++;
}
}
return count;
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.