Regra de negócio dentro do construtor

Boa noite, Douglas! Como vai?

Nesse caso específico, acho que não faz sentido existir o setAgencia(), afinal de contas, uma vez que o cliente tenha criado uma conta a agência e o número não mudam! Portanto, o melhor nesse caso seria remover o setAgencia() e o setNumero() e deixando no construtor a validação dos valores informados quando da criação do objeto!

Agora, em situações onde tanto o setter quanto o construtor precisam existir, eu centralizaria a lógica no getter como vc sugeriu e, além disso, o modificaria com o final de modo que ele não pudesse ser sobrescrito.

public class MyClass {

    public static void main(String args[]) {
        Child child = new Child();

        System.out.println(child.getName());
    }
}

class Person {

    public final String getName() {
        return "Person";
    }
}

class Child extends Person {

    @Override
    public String getName() {
        return "Child";
    }
}

Esse código irá gerar um erro avisando que getName() não pode ser sobrescrito por ser final!

Pegou a ideia? Qualquer coisa é só falar!

Grande abraço e bons estudos, meu aluno!

Compreendi um pouco mais! Num contexto em que eu só poderia receber números positivos tanto no construtor quanto no setter, poderia ser uma solução eu pegar seu valor absoluto (simplesmente retirar o sinal do número) com um getter que não pode ser sobrescrito?

EX: construtor

public Turma(int sala) {
        this.sala= this.getNumeroAbsoluto(sala);    
    }

EX: setter

public void setSala(int sala) {
        this.sala= this.getNumeroAbsoluto(sala);
    }

EX: getter

public final int getNumeroAbsoluto(int numero) {
    return Math.abs(numero);
}

E também, há alguma contra indicação para o uso do final? Por exemplo, se alguém criar todos os métodos como final, não teria como ninguém alterar um pouco a lógica para os que estendesse tal classe.

Valeu pelas respostas!

Opa, Douglas!

Num contexto em que eu só poderia receber números positivos tanto no construtor quanto no setter, poderia ser uma solução eu pegar seu valor absoluto (simplesmente retirar o sinal do número) com um getter que não pode ser sobrescrito?

Nesse contexto, o correto seria lançar uma exceção caso o número recebido fosse negativo. Poderia ser feito assim:

public Pessoa(int idade) {
     this.idade = setIdade(idade);
}

public final void setIdade(int idade) {
     if (idade < 0) throw new IllegalArgumentException("A idade não pode ser negativa!");

     this.idade = idade;
}

E também, há alguma contra indicação para o uso do final? Por exemplo, se alguém criar todos os métodos como final, não teria como ninguém alterar um pouco a lógica para os que estendesse tal classe.

O que vc precisa ficar atento é que ao colocar o final num método vc não irá conseguir sobrescrevê-lo!

Pegou a ideia? Qualquer coisa é só falar!

Grande abraço e bons estudos, meu aluno!

Opa, foi mal pela demora! Entendi a lógica. Valeu!