Расчет членов класса

Question

Во время кодирования, я столкнулся с следующей ситуацией:

У меня есть класс, давайте назовем его C. Он имеет атрибут A произвольного типа, который должен быть рассчитан первым:

public class C { 
    private int A; 

    public C(...) { 
     ... 
    } 

    public void calculateA() { 
     A = 42; 
    } 

    public int getA() { 
    } 
} 

Мой вопрос теперь в том, как реализовать сорбент getA правильно. Должен ли я проверить, определено ли A и иначе ли исключение?

Или я должен просто позвонить calculateA в getA, если он не установлен?

Что является лучшим выбором для такого рода проблем?

---

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

Answer 1

Это вопрос дизайна, несколько неуместный для этого форума.

Если клиент должен позвонить calculateA() до getA(), то getA() может передать исключение, если оно не было рассчитано. Тот факт, что a является примитивным, не имеет значения, вы можете либо инициализировать недопустимое значение - например. если это тип длины объекта, то инициализировать его до -1 - или использовать другую переменную boolean или использовать Integer, которая будет нулевой, независимо.

Понять, что это создает временную зависимость - клиенты должны вызывать методы в фиксированном порядке. Обычно это недоверчиво, но могут быть обстоятельства, чтобы оправдать это, вы должны позвонить Car.start() перед вызовом Car.stop().

С другой стороны, если C «владеет» A, то это его обязанность рассчитать. Он может сделать это несколькими способами:

• Жадно, в свою <init>, если вся необходимая информация avaialable
• Лениво/по требованию, но в кэше, вызовите calculateA() первый раз A нужен.
• Рассчитывать каждый раз, когда он вызывается, хотя getA() может быть вводящим в заблуждение именем в этом случае.

Answer 2

Это зависит. Если вычисление статично, вы также можете вызвать функцию calculateA() из конструктора.

public C(...) { 
     ... 
     calculateA() 
    } 

public void calculateA() { 
    A = 42; 
} 

Если договор геттер() предписывает, что только действительные и расчетные значения A будут возвращены, вычислить A перед его возвращением.

Если A не может быть рассчитан по какой-либо причине, и если имеет смысл в вашей программе прекратить выполнение, прежде чем вычислять что-либо еще с недопустимым A, выбросьте исключение.

Answer 3

Я думаю -

public class C { 
    private int A = Integer.MIN_VALUE; 
    ... 
    public int getA() { 
     if(A==Integer.MIN_VALUE)// Assume Integer.MIN_VALUE when it is not calculated 
      throw new RuntimeException(); 
     return A; 
    } 
} 

Это не может быть 100% доказательство, как предполагается, Integer.MIN_VALUE не может быть вычислено значение.

Сделайте переменную экземпляра A как целочисленный объект вместо примитива int, поэтому вы можете установить значение null, которое может быть проще определить состояние.

Answer 4

Это плохой пример, потому что int является примитивным типом. Это никогда не имеет значения. Он инициализируется до нуля.

Но ваша точка действительна для ссылочных типов.

Я считаю это плохой дизайн, позволяющий ссылаться на частный член. Конструктор должен инициализировать объект, чтобы он был на 100% готов к работе.