, где должна идти эта логика проверки?

Question

У меня есть функция, которая нарисовать изображение на графике:

private void DrawSmallImage(Graphics g) 
{ 
      if (this.SmallImage == null) return; 

      var smallPicHeight = this.Height/5; 

      var x = this.ClientSize.Width - smallPicHeight; 
      var y = this.ClientSize.Height - smallPicHeight; 

      g.DrawImage(this.SmallImage, x, y, smallPicHeight, smallPicHeight); 
} 

чек if (this.SmallImage == null) return; должен быть в функции DrawSmallImage или должны быть в вызывающем? что лучше?

Answer 1

Лучшее место, чтобы поставить проверку логики будет в вызывающем методе.

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

Answer 2

Я бы поместил его в вызывающий. Метод DrawSmallImage не должен знать ничего о состоянии класса. Это цель DrawSmallImage.

Answer 3

Я оставил там чек и передал SmallImage в качестве параметра в метод DrawSmallImage.

Этот способ может быть использован для любого изображения.

Что-то вроде

private void DrawSmallImage(Graphics g, Image smallImage) 
{ 
      if (smallImage == null) return; 

      var smallPicHeight = this.Height/5; 

      var x = this.ClientSize.Width - smallPicHeight; 
      var y = this.ClientSize.Height - smallPicHeight; 

      g.DrawImage(smallImage, x, y, smallPicHeight, smallPicHeight); 
} 

Answer 4

Вы можете бросить исключение, когда это произойдет:

if (this.SmallImage == null) 
    throw new NullReferenceException("Small image can't be null"); 

Затем ожидать вызывающий проверить свойство, как:

bool HasSmallImage 
{ 
    get { return this.SmallImage!=null; } 
} 

Answer 5

SmallImage является полем член. На мой взгляд, лучший дизайн - тот, который не позволяет ему когда-либо быть нулевым (например, он инициализируется в каждом конструкторе и бросает исключение ArgumentException, если он должен быть установлен в null). Конечно, вы не уделяли много внимания, поэтому это может быть невозможно. Но это определенно нужно рассмотреть.

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