Правильно ли реализована инкапсуляция в данном коде и в чем ошибки?

Question

[Сергей Сергей]

class Fox {
    public static void main(String[] args) {
        FoxTest f = new FoxTest();
        f.seta(0);
        f.setb(9);
        f.setc(8);
        f.setd(5);
        int a;
        for(a=1; a<3; a++){
            int b;
        for(b=1; b<3; b++){
                int c;
            for(c=1; c<2; c++){
                    int d;
                for(d=1; d<2; d++){
                  System.out.println("   " + f.geta() + f.getb() + f.getc() + f.getd() /*+ a + b + c + d*/);
                }   
            }
        }   
    }
}
}

class FoxTest {

    private int a;
    void seta(int i) {
        a = i;
    }
    private int b;
    void setb(int i) {
        b = i;
    }
    private int c;
    void setc(int i) {
        c = i;
    }
    private int d;
    void setd(int i) {
        d = i;
    }
    int geta(){
        return a;
        }
    int getb(){
        return b;
        }
    int getc(){
        return c;
        }
    int getd(){
        return d;
    }
}

Comments

[tukreb]

Тут даже класса как такого и нет, просто структура ака самая примитивная реализация класса.

Answer 1

[Денис Загаевский]

Тебе же уже сказали, что этот код слишком примитивен. Никакой инкапсуляции здесь нет. Этот твой класс с геттерами и сеттерами можно выкинуть, ничего не изменится.

Comments

[Орхан Гасанлы]

И еще один совет:

private int a;
    void seta(int i) {
        a = i;
    }
    private int b;
    void setb(int i) {
        b = i;
    }
    private int c;
    void setc(int i) {
        c = i;
    }
    private int d;
    void setd(int i) {
        d = i;
    }

Обратите внимание на рекомендации к написанию кода.
Например, сперва определяются:
финализированные статические публичные переменные
публичные переменные
приватные переменные
конструкторы
публичные методы
приватные методы и т.д.