조건부 인 경우 set.add ()의 부울 반환?

set 클래스의 add 연산자는 추가 할 요소가없는 경우 true를 반환하고 그렇지 않으면 false를 반환합니다. 기록하고있다

if (set.add(entry)) {
    //do some more stuff
}

깨끗한 코드 작성 측면에서 좋은 스타일로 간주됩니까? 한 번에 두 가지 일을했기 때문에 궁금합니다. 1) 요소 추가 및 2) 요소 존재 여부 확인

그렇습니다.

오퍼레이션이 부울 값을 리턴하도록하는 일반적인 지점은이를 사용하여 의사 결정을 수행 할 수 있다는 것 if입니다. 유일한 다른 방법이 잠재 변수에 반환 값을 저장하고 다음 즉시에 다시 사용할 것 실현할 수 if있는 방법 그냥 바보 확실히 쓰기에 바람직하지 않다 if(operation()) { ... }. 그냥 가서 그렇게하세요, 우리는 당신을 판단하지 않을 것입니다.

유지 보수 담당자가 이미 반환 값의 의미를 알고 있거나 찾아야하기 때문에 가능한 한 깨끗하지 않다고 말하고 싶습니다. 이미 존재하거나 존재하지 않았거나 값이 성공적으로 삽입되었다는 의미입니까? 많이 사용하지 않으면 알 수 없으며, 사용하더라도 훨씬 더 정신적 인 부담입니다.

나는 다음을 선호한다 :

boolean added = set.add(entry);

if (added) {
    //do some more stuff
}

예, 조금 더 장황하지만 컴파일러는 거의 동일한 바이트 코드를 생성해야하며 몇 년 동안 Java 세트를 사용하지 않은 사람들조차도 아무것도 보지 않고도 논리를 따를 수 있습니다.

참이 성공을 의미한다면, 그것은 분명하고 명확한 코드입니다.

함수 나 메소드가 성공하면 true (또는 true로 평가되는 것)를 반환한다는 광범위한 규칙이 있습니다. 귀하의 코드가 그에 따르는 한, 조건에 메소드를 넣는 것이 좋습니다.

이 같은 코드는 필자의 관점에서 불필요하게 어수선합니다.

boolean frobulate_succeeded = thing.frobulate();

if (frobulate_succeeded) {
    ...
}

자신을 반복하는 것처럼 느껴집니다.

그러나 반환 값의 의미에 대해서는 문제가 모호합니다. "추가 된 요소가 이미 존재하는지 여부를 나타내는 부울"이라고 말하면 true는 요소가 존재하고 추가가 발생하지 않았 음을 의미 할 수 있습니다. 이 경우 메소드의 리턴 동작을보다 일반적인 것으로 변경하는 것이 이상적입니다. 이것이 가능하지 않은 경우 코드에 반환 결과에 명확하게 레이블을 지정할 수있는 추가 중간 변수를 추가합니다 (다른 사람들이 제안한대로).

나는 그것이 C와 같다고 말하고 싶다. 대부분의 경우 돌연변이 결과에 대해 설명 적으로 명명 된 변수를 사용하고 if조건 에서 돌연변이가 발생하지 않는 것을 선호합니다 .

컴파일러는이 변수가 즉시 재사용되면이 변수를 제거합니다. 인간은 더 쉽게 시간을 읽을 수있을 것입니다. 나에게 그것은 더 중요하다.

누군가가 if 조건에 and/ or절을 추가하여 조건을 확장 해야하는 .add()경우 단락 평가로 인해 특정 경우 호출하지 않을 수 있습니다 . 단락이 구체적으로 예상되지 않는 한 이것은 버그로 끝날 수 있습니다.

코드가 Command Query Separation 을 위반하는 것 같습니다 . 이것은 Clean Code book 과 Function Structure 비디오 에서 논의됩니다 . 그래서 Clean Code 관점에서 나는 그것이 좋은 스타일로 간주되지 않는다고 생각합니다.

“클린 코드는 간단하고 직접적입니다. 깨끗한 코드는 잘 작성된 산문처럼 읽습니다. 깔끔한 코드는 디자이너의 의도를 모호하게하지 않으며, 선명한 추상화와 간단한 제어 라인으로 가득합니다.

-응용 프로그램을 사용한 객체 지향 분석 및 디자인의 Grady Booch 저자”

나에게 당신의 코드의 의도는 분명하지 않습니다. 항목이 성공적으로 추가되거나 이미 존재하는 경우 if가 모두 실행됩니까? 무엇을 add()반환합니까? 물건? 오류 코드?