한 번만 호출되는 한 줄 함수

한 줄의 코드를 수행하고 프로그램에서 한 번만 호출 되는 매개 변수가없는 ( 편집 : 반드시 그런 것은 아님) 함수를 고려하십시오 (차후에 다시 필요하지는 않지만).

쿼리를 수행하고, 값을 확인하고, 정규 표현식과 관련된 작업을 수행 할 수 있습니다.

이에 대한 근거는 읽기 어려운 평가를 피하는 것입니다.

if (getCondition()) {
    // do stuff
}

getCondition()한 줄 함수는 어디에 있습니까 ?

내 질문은 간단합니다 : 이것은 좋은 습관입니까? 괜찮아 보이지만 장기에 대해서는 모른다 ...

한 줄에 따라 다릅니다. 행이 읽기 쉽고 간결한 경우 기능이 필요하지 않을 수 있습니다. 간단한 예 :

void printNewLine() {
  System.out.println();
}

OTOH, 함수가 복잡한, 읽기 어려운 표현을 포함하는 코드 줄에 좋은 이름을 주면 완벽하게 정당화됩니다 (나에게). 고안된 예 (여기에서 가독성을 위해 여러 줄로 나)) :

boolean isTaxPayerEligibleForTaxRefund() {
  return taxPayer.isFemale() 
        && (taxPayer.getNumberOfChildren() > 2 
        || (taxPayer.getAge() > 50 && taxPayer.getEmployer().isNonProfit()));
}

예, 모범 사례를 충족시키는 데 사용할 수 있습니다. 예를 들어, 분명한 이름의 함수가 더 큰 함수 내에 해당 코드 행을 가지고 있고 그 기능을 설명하는 한 줄 주석이 필요한 것보다 한 줄만 있어도 일부 작업을 수행하는 것이 좋습니다. 또한 인접한 코드 줄은 동일한 추상화 수준에서 작업을 수행해야합니다. 반례는 다음과 같습니다.

startIgnition();
petrolFlag |= 0x006A;
engageChoke();

이 경우 중간 줄을 의미있는 이름의 함수로 옮기는 것이 좋습니다.

많은 경우에 그러한 함수는 좋은 스타일이라고 생각하지만 다른 장소 에서이 조건을 사용할 필요가없는 경우 로컬 부울 변수를 대안으로 고려할 수 있습니다.

bool someConditionSatisfied = [complex expression];

이것은 코드 리더에 대한 힌트를 제공하고 새로운 기능을 도입하지 않아도됩니다.

Peter의 답변 외에도 미래의 특정 시점에서 해당 조건을 업데이트해야 할 경우 단일 편집 점 만 가질 것을 제안하는 방식으로 캡슐화하여 조건을 업데이트해야합니다.

베드로의 예에 따르면

boolean isTaxPayerEligibleForTaxRefund() {
  return taxPayer.isFemale() 
        && (taxPayer.getNumberOfChildren() > 2 
        || (taxPayer.getAge() > 50 && taxPayer.getEmployer().isNonProfit()));
}

이된다

boolean isTaxPayerEligibleForTaxRefund() {
  return taxPayer.isMutant() 
        && (taxPayer.getNumberOfThumbs() > 2 
        || (taxPayer.getAge() > 123 && taxPayer.getEmployer().isXMan()));
}

한 번만 편집하면 전체적으로 업데이트됩니다. 현명한 유지 관리 능력은 플러스입니다.

성능면에서 가장 최적화 된 컴파일러는 함수 호출을 제거하고 어쨌든 작은 코드 블록을 인라인합니다. 이와 같은 것을 최적화하면 실제로 함수 크기, 반환 등에 필요한 명령어를 삭제하여 블록 크기를 줄일 수 있으므로 인라인을 막을 수있는 상황에서도 일반적으로 안전합니다.

가독성 (또는 그것을 보완하는 것)에 추가하여 적절한 추상화 레벨에서 함수를 작성할 수 있습니다.

따라 다릅니다. 때로는 한 줄인 경우에도 함수 / 메서드에 식을 캡슐화하는 것이 좋습니다. 읽기가 복잡하거나 여러 곳에서 필요한 경우 좋은 방법으로 간주합니다. 단일 변경 지점과 더 나은 가독성을 도입 했으므로 장기적으로 유지 관리가 더 쉽습니다 .

그러나 때로는 필요하지 않은 것일 수도 있습니다. 어쨌든 표현이 읽기 쉽고 / 또는 한 곳에 나타나면 줄 바꿈하지 마십시오.

나는 당신이 그들 중 몇 가지를 가지고 있다면 괜찮다고 생각하지만 코드에 많은 것이있을 때 문제가 발생합니다. 그리고 컴파일러가 실행될 때 또는 사용하는 언어에 따라 인터 피터가 실행될 때 메모리에서 해당 기능을 수행합니다. 그래서 당신이 그들 중 3 명이라고 말하면 컴퓨터가 알 수 있다고 생각하지 않지만 100의 작은 것들을 시작하기 시작하면 시스템은 한 번만 호출 된 다음 파괴되지 않는 메모리에 함수를 등록해야합니다.

코드 리산 주석의 실제 의미를 명시하기 위해 리팩토링 한 응용 프로그램에서 최근 에이 정확한 작업을 수행했습니다.

protected void PaymentButton_Click(object sender, EventArgs e)
    Func<bool> HaveError = () => lblCreditCardError.Text == string.Empty && lblDisclaimer.Text == string.Empty;

    CheckInputs();

    if(HaveError())
        return;

    ...
}

한 줄을 이름이 지정된 메소드로 옮기면 코드를 더 쉽게 읽을 수 있습니다. 많은 사람들이 이미 그 자체를 언급했습니다 ( "자체 문서화 코드"). 메소드로 이동하는 다른 장점은 단위 테스트가 더 쉽다는 것입니다. 자체 메소드로 분리되고 단위 테스트를 수행하면 버그가 발견되면이 메소드에 없는지 확인할 수 있습니다.

이미 좋은 답변이 많이 있지만 언급할만한 특별한 경우가 있습니다 .

한 줄 문장 에 주석이 필요하고 그 목적을 명확하게 식별 할 수 있다면 (이름을 의미) API 주석으로 주석을 강화하면서 함수 추출을 고려하십시오. 이렇게하면 함수 호출을보다 쉽고 빠르게 이해할 수 있습니다.

흥미롭게도, 현재 할 일이 없다면 같은 작업을 수행 할 수 있지만 확장이 필요하다는 의견 (아주 가까운 미래 ¹⁾ )

def sophisticatedHello():
    # todo set up
    say("hello")
    # todo tear down

뿐만 아니라 이것으로 바뀔 수있었습니다

def sophisticatedHello():
    setUp()
    say("hello")
    tearDown()

¹⁾ 당신은 이것에 대해 정말로 확신해야합니다 ( YAGNI 원칙 참조 )

언어가 지원하는 경우 일반적으로 레이블이 지정된 익명 함수를 사용하여이를 수행합니다.

someCondition = lambda p: True if [complicated expression involving p] else False
#I explicitly write the function with a ternary to make it clear this is a a predicate
if (someCondition(p)):
    #do stuff...

IMHO 이것은 if레이블이 작은 전역 / 패키지 네임 스페이스를 어지럽히 지 않으면 서 복잡한식이 조건을 어지럽히 지 않는 가독성 이점을 제공하기 때문에 좋은 절충안 입니다. "정의"기능이 사용되는 곳에서 정의를 쉽게 수정하고 읽을 수 있도록하는 이점이 있습니다.

술어 함수일 필요는 없습니다. 나는 반복적 인 보일러 플레이트를 이와 같은 작은 기능으로 묶고 싶습니다. (괄호 구문을 어지럽히 지 않고 파이썬 목록 생성에 특히 효과적입니다). 예를 들어, 파이썬에서 PIL로 작업 할 때 다음과 같이 단순화 된 예

#goal - I have a list of PIL Image objects and I want them all as grayscale (uint8) numpy arrays
im_2_arr = lambda im: array(im.convert('L')) 
arr_list = [im_2_arr(image) for image in image_list]