Há um terremoto no Canadá :)
Se for necessário e souber o que está fazendo não tem problema. É claro que este exemplo pode ser um problema porque um saldo não deveria ser manipulado diretamente. Não sei nem se um cliente deveria ter um saldo e um método Sacar()
. Eu sei que pode ser só um exemplo, mas ele induz a algo errado.
Mas não pense que ele dá alguma segurança. Se um programador quiser mudar um valor ele vai mudar. O encapsulamento só protege do programador bem intencionado que não quer correr o risco de cometer um erro acessando o campo de forma direta. É algo que o compilador "impede" o acesso, mas daí pra frente tudo pode acontecer.
É mais uma questão de organização do código, algo que ajuda gerenciar a complexidade dos objetos e passar a intenção correta. Tem a ver com encapsulamento.
Desta forma ajuda a obter a coesão e evita o acoplamento não intencional.
Na verdade tem um problema pior nesse código que é armazenar valor monetário em um double
.
Não sou fã do termo atributo para isto, prefiro campo.
Veja mais: