부울을 더 명확하게 호출하려면 어떻게해야합니까? 부울 트랩

@ benjamin-gruenbaum의 의견에서 언급했듯이 이것을 부울 트랩이라고합니다.

이런 기능이 있다고 해

UpdateRow(var item, bool externalCall);

내 컨트롤러에서 해당 값 externalCall은 항상 TRUE입니다. 이 함수를 호출하는 가장 좋은 방법은 무엇입니까? 나는 보통

UpdateRow(item, true);

그러나 나는 '진정한'가치가 무엇을 의미하는지 나타 내기 위해 부울을 선언해야합니까? 함수 선언을 보면 알 수 있지만 다음과 같은 것을 본다면 분명히 더 빠르고 명확합니다.

bool externalCall = true;
UpdateRow(item, externalCall);

PD :이 질문이 여기에 실제로 맞는지 확실하지 않은 경우, 그 질문에 대한 자세한 정보를 어디서 얻을 수 있습니까?

PD2 : 매우 일반적인 문제라고 생각했기 때문에 어떤 언어에도 태그를 달지 않았습니다. 어쨌든, 나는 C #으로 일하고 C #에서 허용되는 대답은 작동합니다.

항상 완벽한 솔루션은 아니지만 다음 중에서 선택할 수있는 많은 대안이 있습니다.

• 귀하의 언어로 제공되는 경우 명명 된 인수를 사용하십시오 . 이것은 매우 잘 작동하며 특별한 단점이 없습니다. 일부 언어에서는 모든 인수를 명명 된 인수로 전달할 수 있습니다 updateRow(item, externalCall: true)( 예 : ( C # ) 또는 update_row(item, external_call=True)(Python)).

  별도의 변수를 사용하라는 제안은 명명 된 인수를 시뮬레이션하는 한 가지 방법이지만 관련 안전 이점이 없습니다 (해당 인수에 대해 올바른 변수 이름을 사용했다고 보장 할 수는 없습니다).

• 더 나은 이름으로 공용 인터페이스에 고유 한 기능 을 사용하십시오 . 이것은 이름에 paremeter 값을 넣어 명명 된 매개 변수를 시뮬레이션하는 또 다른 방법입니다.

  이것은 매우 읽기 쉽지만 이러한 기능을 작성하는 많은 상용구로 이어집니다. 또한 부울 인수가 여러 개인 경우 조합 폭발을 잘 처리 할 수 ​​없습니다. 중요한 단점은 클라이언트가이 값을 동적으로 설정할 수 없지만 올바른 함수를 호출하려면 if / else를 사용해야한다는 것입니다.

• 열거 형을 사용하십시오 . 부울의 문제점은 "true"및 "false"입니다. 따라서 더 나은 이름을 가진 유형을 소개하십시오 (예 :) enum CallType { INTERNAL, EXTERNAL }. 추가 혜택으로, 프로그램 의 형식 안전성 이 향상 됩니다 (언어가 열거 형을 고유 한 형식으로 구현하는 경우). 열거 형의 단점은 공개적으로 보이는 API에 유형을 추가한다는 것입니다. 순전히 내부 함수의 경우 이것은 중요하지 않으며 열거 형에는 큰 단점이 없습니다.

  열거 형이없는 언어에서는 짧은 문자열 이 대신 사용되는 경우가 있습니다. 이것은 작동하며 원시 부울보다 낫지 만 오타에 매우 취약합니다. 그런 다음 함수는 인수가 가능한 값 세트와 일치한다고 즉시 주장해야합니다.

이러한 솔루션 중 어느 것도 엄청나게 성능에 영향을 미치지 않습니다. 명명 된 매개 변수와 열거 형은 컴파일 타임에 컴파일 할 수 있습니다 (컴파일 된 언어의 경우). 문자열을 사용하면 문자열을 비교할 수 있지만 작은 문자열 리터럴과 대부분의 응용 프로그램에서는 비용이 무시할 수 있습니다.

올바른 해결책은 제안한대로하는 것이지만 미니 외관으로 패키지하는 것입니다.

void updateRowExternally() {
  bool externalCall = true;
  UpdateRow(item, externalCall);
}

가독성은 미세 최적화보다 우선합니다. 부울 플래그의 의미를 한 번만 찾아야하는 개발자의 노력을 감당할 수있는 것보다 확실히 함수 호출을 추가 할 수 있습니다.

UpdateRow (var item, bool externalCall)와 같은 기능이 있다고 가정 해보십시오.

왜 이런 기능이 있습니까?

어떤 상황 에서 externalCall 인수를 다른 값으로 설정하여 호출합니까?

하나는 외부 클라이언트 응용 프로그램에서 왔고 다른 하나는 동일한 프로그램 (예 : 다른 코드 모듈) 내에있는 경우 두 가지 다른 방법 을 사용해야한다고 주장합니다. 인터페이스.

그러나 비 프로그램 소스에서 가져온 일부 데이텀 (예 : 구성 파일 또는 데이터베이스 읽기)을 기반으로 호출 한 경우 부울 전달 방법이 더 적합합니다.

가독성과 가치 안전을 모두 구현하기 위해 언어 기능을 사용하는 것이 이상적 이지만 , 실제 접근 방식 인 통화 시간 설명을 선택할 수도 있습니다 . 처럼:

UpdateRow(item, true /* row is an external call */);

또는:

UpdateRow(item, true); // true means call is external

또는 (Frax가 제안한대로) :

UpdateRow(item, /* externalCall */true);

1. 당신은 당신의 바보를 '이름'할 수 있습니다. 다음은 OO 언어 (제공 클래스에서 표현할 수있는 UpdateRow())의 예이지만 개념 자체는 모든 언어로 적용될 수 있습니다.

   class Table
   {
   public:
       static const bool WithExternalCall = true;
       static const bool WithoutExternalCall = false;
   

   그리고 전화 사이트에서 :

   UpdateRow(item, Table::WithExternalCall);
   

2. 아이템 # 1이 더 낫다고 생각하지만 함수를 사용할 때 사용자가 새로운 변수를 사용하도록 강요하지는 않습니다. 유형의 안전이 당신을 위해 중요하다면, 당신은 만들 수 있습니다 enum종류와 만드는 UpdateRow()대신 이것을 받아 들일 bool:

   UpdateRow(var item, ExternalCallAvailability ec);

3. 어떻게 든 bool매개 변수 의 의미를 더 잘 반영하도록 함수 이름을 수정할 수 있습니다 . 확실하지 않지만 어쩌면 :

   UpdateRowWithExternalCall(var item, bool externalCall)

아직 읽지 않은 또 다른 옵션은 다음과 같습니다. 최신 IDE를 사용하십시오.

예를 들어 IntelliJ IDEA는 true또는 null또는 같은 리터럴을 전달하는 경우 호출하는 메소드에서 변수의 변수 이름을 인쇄합니다 username + “@company.com. 이것은 작은 글꼴로 이루어 지므로 화면의 공간을 너무 많이 차지하지 않으며 실제 코드와 매우 다르게 보입니다.

나는 여전히 어디서나 부울을 던지는 것이 좋은 생각이라고 말하지 않습니다. 작성하는 것보다 훨씬 더 자주 코드를 읽는다는 주장은 매우 강력하지만,이 경우에는 귀하 (및 동료)가 귀하의 업무를 지원하는 데 사용하는 기술에 크게 의존합니다. IDE를 사용하면 예를 들어 vim보다 문제가 훨씬 적습니다.

테스트 설정의 일부를 수정 한 예 :

2 일 동안 아무도 다형성을 언급하지 않았습니까?

target.UpdateRow(item);

항목으로 행을 업데이트하려는 클라이언트 인 경우 데이터베이스 이름, 마이크로 서비스 URL, 통신에 사용되는 프로토콜 또는 외부 호출 여부에 대해 생각하고 싶지 않습니다. 그렇게하기 위해 사용되어야합니다. 나에게 이러한 세부 사항을 밀어 넣지 마십시오. 내 항목을 가져 와서 이미 어딘가에 행을 업데이트하십시오.

그렇게하면 건설 문제의 일부가됩니다. 여러 가지 방법으로 해결할 수 있습니다. 여기 하나가 있습니다 :

Target xyzTargetFactory(TargetBuilder targetBuilder) {
    return targetBuilder
        .connectionString("some connection string")
        .table("table_name")
        .external()
        .build()
    ; 
}

당신이 그것을보고 "내 언어가 논쟁을 명명했다면, 나는 이것을 필요로하지 않는다"고 생각하고 있다면. 좋아요, 좋아요 무슨 말을하는지조차 모르는 발신 클라이언트로부터이 말이되지 않도록하십시오.

이것은 의미상의 문제 이상이라는 점에 유의해야합니다. 디자인 문제이기도합니다. 부울을 전달하면 분기를 사용하여 처리해야합니다. 객체 지향적이지 않습니다. 둘 이상의 부울을 전달해야합니까? 언어가 여러 번 발송되기를 바랍니다. 컬렉션을 중첩 할 때 수행 할 수있는 작업을 살펴보십시오. 올바른 데이터 구조는 당신의 인생을 훨씬 쉽게 만들어 줄 수 있습니다.

updateRow함수 를 수정할 수 있다면 아마도 두 가지 함수로 리팩토링 할 수 있습니다. 함수가 부울 매개 변수를 사용한다고 가정하면 다음과 같이 보입니다.

function updateRow(var item, bool externalCall) {
  Database.update(item);

  if (externalCall) {
    Service.call();
  }
}

코드 냄새가 약간납니다. externalCall변수 설정 에 따라 기능이 크게 다를 수 있으며 ,이 경우 두 가지 다른 책임이 있습니다. 한 가지 책임 만있는 두 가지 기능으로 리팩토링하면 가독성이 향상 될 수 있습니다.

function updateRowAndService(var item) {
  updateRow(item);
  Service.call();
}

function updateRow(var item) {
  Database.update(item);
}

이제이 함수를 호출하는 모든 곳에서 외부 서비스가 호출되는지 한 눈에 알 수 있습니다.

물론 항상 그런 것은 아닙니다. 상황과 맛의 문제입니다. 부울 매개 변수를 두 함수로 리팩토링하는 것은 일반적으로 고려할 가치가 있지만 항상 최선의 선택은 아닙니다.

UpdateRow가 제어 코드베이스에 있다면 전략 패턴을 고려할 것입니다.

public delegate void Request(string query);    
public void UpdateRow(Item item, Request request);

여기서 Request는 일종의 DAO (사소한 경우 콜백)를 나타냅니다.

실제 사례 :

UpdateRow(item, query =>  queryDatabase(query) ); // perform remote call

틀린 경우 :

UpdateRow(item, query => readLocalCache(query) ); // skip remote call

일반적으로 엔드 포인트 구현은 더 높은 수준의 추상화로 구성되며 여기서 사용합니다. 유용한 무료 부작용으로, 코드를 변경하지 않고도 원격 데이터에 액세스하는 다른 방법을 구현할 수있는 옵션을 제공합니다.

UpdateRow(item, query => {
  var data = readLocalCache(query);
  if (data == null) {
    data = queryDatabase(query);
  }
  return data;
} );

일반적으로, 이러한 제어 역전은 데이터 스토리지와 모델 간의 결합을 줄입니다.