Java, Scanner i hermetyzacja

Question

Jak zaimplementować w kodzie wprowadzanie danych, żeby było to zgodne z zasadami hermetyzacji? Czy umieszczenie Scanner w "setterze" jest poprawne?

 

private int liczba  
public void setWaga(int jakaLiczba)
    {
        System.out.println("Podaj liczbe ");
        Scanner sc = new Scanner(System.in);
        liczba = sc.nextInt();
        jakaLiczba = liczba;
    }

 

Answer 1

Nie jest to poprawne. Powinieneś w jakimś mainie czy gdzie tam używasz setter'a, najpierw pobrać tego inta przez scanner, a potem go podać do settera. Setter powinien tylko i wyłącznie przypisywać podaną wartość do zmiennej.

Comments

Tak dla pewności dopytam: czy tak jest poprawnie?

import java.util.Scanner;
class Baza
{
    private int liczba;

    public int inputLiczba()
    {
        System.out.println("Jaka liczba? ");
        Scanner sc = new Scanner(System.in);
        liczba = sc.nextInt();
        return liczba;
    }

    public void setLiczba(int jakaLiczba)
    {
        jakaLiczba = liczba;
    }

    public int getLiczba()
    { 
        return liczba;
    }

    void odlicz()
    {
            System.out.println(liczba);
        if(liczba > 10)
            System.out.println("Powyzej 10");
        else
            System.out.println("Ponizej 10");
    }
} //koniec baza

class BazaTester
{
    public static void main (String[] args)
    {
        Baza liczby [] = new Baza[3];
        for (int i = 0; i < liczby.length; i++)
        {
            liczby[i] = new Baza();
            liczby[i].inputLiczba();
            liczby[i].setLiczba(5);
            liczby[i].odlicz();
        }
    } //koniec main
} //koniec BazaTester

---

W setterze przypisujesz do argumentu settera wartość zmiennej liczba, powinno być na odwrót.

setLiczba(int liczba){
this.liczba = liczba;
}