2016-05-31 9 views
3

보다 구체적으로 말하자면, 사용자에게보기를 표시하거나 표시하지 않을 방법의 이름을 지정하는 방법은 입니까? 예를 들어무언가를 할 수도 있고하지 않을 수도있는 방법의 이름을 짓는 방법?

:

void maybeShowView() { 

    if (checkSomething()) { 
     presentViewToUser(); 
    } 
} 

그래서 maybeShowView는 단위 테스트 목적으로 제시 방법에서 아마 방법은 어쩌면 방법은 기기가 제대로 테스트 할 수 있습니다 이런 식으로 분리도 끔찍한 이름 임입니다. 우리 팀에서

우리는

  • maybeDoSomething
  • presentThingIfAvailable

같은 이름으로 올라와있다 그러나 이것은 정말 나쁜 이름

어떤 아이디어가 있습니까?

감사

참고 : 나는 부울 방법 shouldPresentView()를 만들 수 있지만 그것을 확인하기 위해 저를 필요로 매번 나는이 원하는 해달라고, 내 시야를 제공하고자합니다.

+1

'IfAvailable' 또는'IfApplicable' (및 기타)와 함께 메서드 이름을 끝내는 것이 나에게 합리적으로 들립니다. 그러나 나는 그것이 개인적인 의견의 문제라고 두려워합니다 ... – alfasin

+0

실제로 우리가 코코아를 사용한다면, 이것들은 다음과 같은 것들에 대한 접근입니다 : layoutIfNeeded – Hazneliel

답변

2

가장 좋은 방법은 조건의 이름과 방법 이름에 수행 할 작업을 모두 포함하는 것입니다.()가 true checkSomething 때보기가 표시됩니다 경우에 따라서, 당신은 지금처럼 부를 것이다 : 이것은 의심의 여지 방법이 실제로 무엇을 떠나지 않을 것입니다

void presentViewIfSomething() { 

    if (checkSomething()) { 
     presentViewToUser(); 
    } 
} 

. 이 접근 방식의 단점은 물론 조건을 변경하면 메소드의 이름을 변경해야한다는 것입니다.

+0

이것은 우리가 취할 접근법입니다 – Hazneliel