Boas práticas
Vou te dizer o que é mais importante sobre este assunto: inventaram uma praga na cabeça dos desenvolvedores chamada "boa prática".
O mundo do desenvolvimento ficará muito melhor quando as pessoas pararem de falar nisso. Essa é uma forma de dizer "faz aí do jeito que eu estou falando e não discuta". O mesmo vale pra "má prática" que é um "estou dizendo que é ruim e ponto". Assim não precisa explicar nada. Se fosse bom ou ruim mesmo a pessoa explicaria. Assim o desenvolvedor que está lendo aquilo pode decidir por conta própria se é bom ou ruim e quando deve ou não usar.
Quase tudo que existe pode ser bom ou ruim na maioria das situações, mas sem entender o porquê, serve pra nada. Tudo tem exceção, claro. Então falou em "boa/má prática", fuja da resposta. Ou a pessoa não sabe do que está falando ou é tendenciosa (claro que pode ser só preguiça/pressa em responder, mas como você vai saber?).
E o que está chamando de atributo na verdade se chama campo.
int
como "atributo"
Aquela resposta linkada especificamente está muito além do escopo da pergunta dela e aí cai no problema de não fazer compreender o que realmente importava.
É um completo absurdo dizer que um "atributo"/campo não deve ser int
. Nesse caso específico eu diria que não existe uma situação sequer que impeça, crie dificuldade, e principalmente seja uma "má prática" :). Um campo pode ser int
sempre que for necessário.
Claro que ele não deve ser int
se não é necessário ser. Mas aí é o que eu sempre falo: o certo é fazer o certo, não é fazer o que é "boa prática". E o certo só dá para saber no caso concreto. Toda vez que você faz o certo em algum lugar e tentar transpor isso para outra situação, está fazendo errado. Cada situação é única. Algumas coincidentemente devem ter a mesma solução.
Eu acho que a resposta queria dizer que não é bom ter um campo público exposto e se confundiu na escrita, só isso.
Sem entrar em muitos detalhes porque esta não é a questão aqui, isso é algo que pode ser ruim em várias situações, especialmente em Java, e recomenda-se usar um método getter/setter para acesso a ele. De qualquer forma a frase é bem infeliz. Não vou discutir isso aqui (mas discordo do que costuma-se dizer como verdade absoluta), o assunto é outro, mas linkei perguntas abaixo que falam sobre o assunto, se precisar de mais informações ainda pode fazer uma nova pergunta específica.
Seu exemplo
O seu caso específico está usando o contador
como privado. Ok. O problema do que está fazendo é que cria uma condição de corrida em ambientes concorrentes. Você pode ter o contador aumentando em 2 ou mais antes de ser usado em cada um, aí terá dois objetos com o mesmo índice. É bem raro acontecer, e justamente por isso o dia que acontecer será bem difícil reproduzir e descobrir o porquê.
O que está fazendo não é lá uma ideia das melhores. Eu tentaria fazer de outra forma, mas como eu não sei qual é o problema, os requisitos, qualquer coisa que eu fale soará como "boa prática".
###Getter/Setter
O indice
é público. Não há problema algum ser final
e public
. Isto em si não cria nenhum problema. Costuma ser melhor que seu acesso seja feito por um método acessador (veja nas perguntas abaixo o porquê), mas só costuma, se não tiver nenhum motivo para isso, pode deixar como público.
Eu concordo que em Java, que é uma linguagem que não gosta muito de syntatic sugar, costuma ser uma boa ideia acessar os campos por um método para garantir a melhor manutenibilidade. Se você precisar mudar a forma de acesso no futuro não precisará mudar todos os códigos que consumiram sua classe. Em outras linguagens isto pode ser mais conveniente e dar mais liberdade, se a pessoa souber o que está fazendo.
Em Java, pra expor um campo publicamente (qualquer tipo de campo) você deve ter certeza que nunca terá alterações na forma de acessá-lo. É raro ter tanta certeza assim. Mas tem casos.
O resto do que falam sobre o assunto é teoria boba.
A resposta do ramaral tem um exemplo de como fazer o acesso com um método auxiliar. Note que não tem um setter, afinal não faria sentido ter em um campo marcado como final
.
Conclusão
Nunca faça nada porque alguém disse pra você que é o certo, aprenda porque é o certo. Aí pode descobrir que a pessoa que te disse isto pode estar errada.
Fez certo em questionar aqui.
Pra falar a verdade acho que nenhuma das respostas nessas perguntas mostram bem essa questão por completo, mas lendo tudo já ajuda entender e formar uma opinião própria.
Coloquei no GitHub para referência futura.