Usando primary key como hashCode

Lembre-se que mesmo dois objetos com ids iguais podem ser diferentes se carregados duas vezes e alterados separadamente.

Mas a do hashcode faço exatamente igual.

O que deve ser considerado é o seguinte: Comparar os IDs é suficiente para estabelecer a igualdade entre os objetos? Isso faz sentido dentro do domínio que você está trabalhando ou foi feito só por economia de código?

Se a comparação apenas de ID for realmente o melhor a se fazer (e não o mais fácil, cuidado hein ), deixe desse jeito mesmo, está ótimo.
Senão, deve-se incluir os demais atributos na comparação do equals() e alterar o hashCode() para continuar respeitando o contrato.

Mark_Ameba:

Lembre-se que mesmo dois objetos com ids iguais podem ser diferentes se carregados duas vezes e alterados separadamente.

Mas a do hashcode faço exatamente igual.

A ideia do campo id na classe cliente é a mesma que o campo idCliente na tabela cliente: uma vez setado, não será mais alterado. Aliás só não marco o campo como final porque na hora de realizar a query não poderia setar o seu valor adequadamente.

Outra coisa. Se usar outras variáveis no equals é recomendável que as mesmas sejam usadas no hashCode.