Vejo alguns problemas neste código, todos conceituais.
O que você chama de atributo na verdade chama-se campo. Ei sei, tem monte de lugar ensinando errado, não é culpa sua.
Este código provavelmente deveria ter um construtor. Tudo leva a crer que é mais adequado. E aí tem que pensar se deveria ter getters e setters, esse é o mecanismo da moda mas que as pessoas usam sem pensar se deveriam ter isso no seu código, se ele está atendendo um motivo real e não apenas seguindo regras inventadas por aí. A coisa tá tão grave que que as pessoas dão mais valor para ele que o construtor.
Especialmente deveria pensar se um setter é necessário, em muitos casos não deveria poder mudar o estado depois da sua construção. Inclusive em muitos caso isso nem deveria ser uma classe, mas por enquanto Java não tem uma forma de criar um tipo sem ser desta forma, o que sempre foi uma deficiência da linguagem que agora será consertada.
Mas o getter pode ser errado porque ele pode não ser necessário para o que pretende fazer, pode ser que só queira acessar o dado direto e nunca de outra forma, então não tem porque seguir regra boaba se é isso que quer fazer. Adote a semântica correta. Se fosse proibido acessar o campo de forma direta a linguagem nem deixaria fazê-lo. Algumas pessoas se apropriaram da verdade absoluta e dizem que nunca deveria fazer isso, mesmo a linguagem, mais pragmática, permitindo. Mas em outros casos até deveria ter uma forma de acessar o dado sem o campo direto, mas seria um método que faz algo mais semântico e não um getter que tem uma semântica muito simples e alguns dizem que isso é conceitualmente errado.
Os nomes todos são dizem nada e isso por si só é um erros conceitual, dar nomes significativos pra tudo é fundamental para o código ser legível e dar um entendimento do que ele deveria fazer.
Este é um caso aparentemente artificial que provavelmente nem deveria ser uma classe. Modelar classes é sempre fazer algo que faça sentido.
Mantendo a ideia geral o seu código parece fazer o que está perguntando na pergunta, mas falha em executar corretamente, por um erro conceitual. Se um objeto depende de uma operação anterior para funcionar corretamente (o que mostra o comentário do MauroAlmeida) então tem algo errado nele, não é um objeto coeso, e se código faz isso. Se você não executar o método setR()
a soma não estará disponível. Na verdade isso nem é um setter, embora use o mesmo nome. Um padrão de projeto chamado setter sempre tem um método chamado setXXX(valor)
ou algo parecido (tem linguagens que tem uma sintaxe melhor para isto e chama isto de propriedade), portanto tem sempre que receber um valor, o seu método não recebe um e não é um setter (tenho certeza que acha que é um). Por isso que eu falo que precisa entender o que está usando.
Ainda que insista nesse erro, o getter (getR()
), deveria dar erro se a soma não foi realizada. E sim, você precisaria criar um mecanismo para controlar isto, o que me parece um absurdo (tenho uma pergunta sobre o assunto, em outros tempos alguns usuários ativos do site já teriam respondido). Não estou sugerindo fazer isto, o mais correto é esse getter só dar o resultado e nem guardar algo em outro campo, assim:
public int getR() { return n1 + n2; }
Conforme o comentário do hkotsubo acima. A única questão é que isto eliminaria o campo r
e a pergunta diz que quer isto. Claro, acho que é um problema XY, está querendo resolver um problema causado pelo problema criado por uma decisão errada anterior.
Não estou dizendo que tudo isso que eu disse que está errado não pode ser usado corretamente em outro caso ou em algum caso que seja mais claro o que quer fazer.
De nada adianta tentar programar orientado a objeto quando o objeto dessa técnica é fazer o código ser mais organizado e semântico. Se não entender a semântica do problema ou a semântica da linguagem ou a semântica dos padrões que resolveu adotar, já está fazendo errado por definição.
Por isso que eu sempre falo que os exemplo que tem por aí desensinam a fazer correto, porque eles são receitas de bolo, em geral receitas de programa de televisão, não são feitas para produzir comidas deliciosas que qualquer um consegue fazer, é só para dar água na boca nas pessoas. Se a pessoa não dominar muito aquilo a receita não serve para nada.