Лучшая практика Java Access Modifier

Question

Это может показаться основным вопросом, но я хотел бы получить это право.

У меня есть класс «AWorld». Внутри этого класса у меня есть метод, который рисует границу, в зависимости от размера карты, заданного пользователем.

Если переменная 'mapSize' является частной, но я хочу получить доступ к ее значению из одного класса, то более целесообразно ссылаться на нее напрямую или использовать метод getter.

Код, приведенный ниже, должен объяснить, что я хочу знать.

package javaFX; 

public class AWorld { 
    //initialized later 
    AWorld newWorld; 

    private int mapSize = 20; 

    public int getMapSize() 
    { 
     return mapSize; 
    } 

    public void someMethod() 
    { 
     int var = newWorld.mapSize; //Do I reference 'mapSize' using this... 
    } 
    // Or... 

    public void someOtherMethod() 
    { 
     int var = newWorld.getMapSize(); //Or this? 
    } 
    public static void main(String[] args) {} 

} 

Answer 1

Любой из них в порядке, так как вы получаете примитивное поле. Если метод get выполняет другую операцию перед возвратом данных, например. выполняя математическую операцию над значением, тогда было бы лучше использовать ее, а не напрямую обращаться к полю. Это особенно важно при использовании шаблона proxy/decorator для ваших классов.

Вот пример второго утверждения сверху:

//base class to be decorated 
abstract class Foo { 
    private int x; 
    protected Foo foo; 
    public int getX() { return this.x; } 
    public void setX(int x) { this.x = x; } 
    public Foo getFoo() { return this.foo; } 

    //method to prove the difference between using getter and simple value 
    public final void printInternalX() { 
     if (foo != null) { 
      System.out.println(foo.x); 
      System.out.println(foo.getX()); 
     } 
    } 
} 

//specific class implementation to be decorated 
class Bar extends Foo { 
    @Override 
    public int getX() { 
     return super.getX() * 10; 
    } 
} 

//decorator 
class Baz extends Foo { 
    public Baz(Foo foo) { 
     this.foo = foo; 
    } 
} 

public class Main { 
    public static void main(String[] args) { 
     Foo foo1 = new Bar(); 
     foo1.setX(10); 
     Foo foo2 = new Bar(foo1); 
     //here you see the difference 
     foo2.printInternalX(); 
    } 
} 

Выход:

10 
100 

Answer 2

Вам лучше разыменовать него непосредственно.

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

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

Answer 3

IMHO, если вы ссылаетесь на поле текущего экземпляра, общим правилом является доступ к поле непосредственно с mapSize или this.mapSize.

Если вы ссылаетесь на значение из другого экземпляра (будь то того же класса или другого класса, я бы использовал метод getter). Я считаю, что это приведет к более простому рефакторингу. Он также поддерживает контракт, что любой другой экземпляр получает значение поля через геттер, что позволяет использовать дополнительные функции в геттере.