Primeiro, acho que você esqueceu um *
antes do peso++
, que aliás acho que deveria ser ++peso
no seu caso.
As variáveis soma
e peso
deveriam ser inicializadas. O valor inicial delas é zero.
Sugiro separar o uso dos operadores ++
e --
em suas próprias instruções isoladas. Senão fica muito confuso. Como são incrementadas, decrementadas antes de serem usadas em soma += ...
, então elas ficariam antes do soma += ...
:
while (qtos)
{
qtos--;
peso++;
soma += (*(num + qtos) - '0') * peso;
}
No lugar do while( qtos )
basta usar while (qtos >= 0)
.
Acho que você queria que num
fosse uma String
e não um char
. Se no seu código original, o char num
fosse char *num
, decerto seria isso. Se for esse o caso, o *(num + qtos)
se tornaria em Java num.charAt(qtos)
. Aliás, a variável qtos
é o tamanho da String
, e portanto não precisa ser passada como parâmetro, devendo ser definida com o valor inicial de num.length()
.
Isso daqui pode ser simplificado:
remainder = soma % 11;
return ((char) ((remainder == 0 || remainder == 1) ? '0' : 59 - remainder));
O (remainder == 0 || remainder == 1)
pode ser simplificado para remainder <= 1
:
remainder = soma % 11;
return ((char) (remainder <= 1 ? '0' : 59 - remainder));
A ideia é retornar 11 - remainder
, exceto nos casos onde isso resultaria em 10 ou 11. Então, é possível fazer isso:
remainder = 11 - (soma % 11);
return (char) ('0' + (remainder >= 10 ? 0 : remainder));
Também é boa ideia declarar a variável apenas quando você for precisar dela.
Seu código ficaria assim:
char metodoDigito(String num) {
short soma = 0, peso = 0, qtos = num.length();
while (qtos >= 0) {
qtos--;
peso++;
soma += (num.charAt(qtos) - '0') * peso;
}
short remainder = 11 - (soma % 11);
return (char) ('0' + (remainder >= 10 ? 0 : remainder));
}
Mais três observações:
Tenho dúvidas se o melhor é retornar char
ou retornar int
, acho que int
seria melhor.
Também há pouca razão para se usar short
em Java já que o tipo de inteiro padrão no Java é o int
.
O seu while
também pode ser convertido em um laço for
.
Não faz muito sentido declarar o nome metodo
em um método. É totalmente redundante e desnecessário. O nome digitoVerificador
(ou mesmo apenas dv
) seria melhor.
Provavelmente esse método seria estático e público, pois não utiliza instâncias da classe onde está inserido e não aparenta ser uma funcionalidade interna de alguma classe.
É possível ver que há uma relação entre o tamanho de num
, peso
e qtos
. O valor de qtos
começa em num.length() - 1
e diminui até zero. O valor de peso
vai de 1 até num.length()
. Com isso, podemos usar a relação qtos == num.length() - peso
e então eliminar a variável peso
e também podemos iterar os caracteres de num
em qualquer ordem.
Se iterarmos os caracteres de num
na ordem que aparecem, podemos usar a sintaxe do enhanced-for.
Com isso, o código seria esse:
public static int digitoVerificador(String num) {
int soma = 0, peso = 0;
for (char c : num.toCharArray()) {
peso++;
soma += (c - '0') * peso;
}
int remainder = 11 - (soma % 11);
return remainder >= 10 ? 0 : remainder;
}