Microsoft MVP성태의 닷넷 이야기
글쓴 사람
정성태 (techsharer at outlook.com)
홈페이지
첨부 파일

C# - 불린 형 상수를 반환값으로 포함하는 3항 연산자 사용 시 단축 표현 권장(IDE0075)

예를 들어 다음과 같은 코드를 사용하면,

static bool Test(int id)
{
    return (id == 0) ? _valueOnConf : false;
}

Visual Studio가 IDE0075 - Conditional expression can be simplified 메시지를 출력합니다. 오호~~~ 저 식을 좀 더 단축할 수 있다는 건데요. 검색해 보니 ReSharper 리팩터링 도구에 관련 자료가 나옵니다. ^^

Code Inspection: Simplify conditional ternary expression
; https://www.jetbrains.com/help/resharper/SimplifyConditionalTernaryExpression.html

그러니까, bool을 반환하는 3항 연산자를 사용할 때 2항 또는 3항에 불린 형 상수가 포함된 경우라면 &&, || 연산자로 단축시킬 수 있는 것입니다.

가령, 위에서 예로 든 코드와 같이 "false"를 포함하고 있다면 &&로 단축할 수 있습니다.

static bool Test(int id)
{
    return (id == 0) && _valueOnConf;
}

마찬가지로, 포함하고 있는 불린 형 상수가 true라면, 조건식을 역으로 뒤집고 OR(||) 연산을 사용하도록 바꿔 표현할 수 있습니다.

static bool Test(int id)
{
    return (id == 0) ? _valueOnConf : true;
}

static bool Test(int id)
{
    return (id != 0) || _valueOnConf;
}

그렇긴 한데... '가독성' 측면에서 보면 과히 좋아 보이지 않습니다. 사실 3항 연산자 자체도 초보 개발자들은 헷갈려 하는 경우가 많죠. ^^ 회사 동료와도 저것에 대해 이야기를 해보니, 역시 저 표현은 쓰지 않는 것이 좋겠다는 의견입니다. 여러분들은... 어떻게 생각하시나요? ^^




[이 글에 대해서 여러분들과 의견을 공유하고 싶습니다. 틀리거나 미흡한 부분 또는 의문 사항이 있으시면 언제든 댓글 남겨주십시오.]





[최초 등록일: ]
[최종 수정일: 6/29/2020 ]

Creative Commons License
이 저작물은 크리에이티브 커먼즈 코리아 저작자표시-비영리-변경금지 2.0 대한민국 라이센스에 따라 이용하실 수 있습니다.
by SeongTae Jeong, mailto:techsharer@outlook.com

비밀번호

댓글 쓴 사람
 



2020-06-29 09시55분
[dimohy] 특별히 속도에 영향있는 코드가 아니라면 가독성이 좋은 코드에 한표입니다.
특히나 개발환경에서 가이드 해줄 정도면 컴파일러에서 알아서 최적화는 해줄 수 있겠네요 ^^;
[손님]
2020-06-29 10시44분
@dimohy 좋은 의견 감사합니다. ^^ 확인해 보니, 둘의 IL 코드가 동일하지는 않지만 다음과 같이,

// 3항 연산자
    /* 0x00000332 02 */ IL_0000: ldarg.0
    /* 0x00000327 2C02 */ IL_0001: brfalse.s IL_0005

    /* 0x00000329 16 */ IL_0003: ldc.i4.0
    /* 0x0000032A 2A */ IL_0004: ret

    /* 0x0000032B 7E01000004 */ IL_0005: ldsfld bool ConsoleApp1.Program::_valueOnConf
    /* 0x00000330 2A */ IL_000A: ret

// 단축
    /* 0x00000332 02 */ IL_0000: ldarg.0
    /* 0x00000333 2D06 */ IL_0001: brtrue.s IL_0009

    /* 0x00000335 7E01000004 */ IL_0003: ldsfld bool ConsoleApp1.Program::_valueOnConf
    /* 0x0000033A 2A */ IL_0008: ret

    /* 0x0000033B 16 */ IL_0009: ldc.i4.0
    /* 0x0000033C 2A */ IL_000A: ret

바이트 수도 같고, 패턴도 brfalse/brtrue에 따라 ldc/ldsfld가 나오냐 ldsfld/ldc 조합이 먼저 나오냐의 차이만 있어 속도 상으로도 거의 같을 것으로 예상됩니다. ^^
정성태

[1]  2  3  4  5  6  7  8  9  10  11  12  13  14  15  ...
NoWriterDateCnt.TitleFile(s)
12290정성태8/11/2020121.NET Framework: 931. C# - IP 주소에 따른 국가별 위치 확인파일 다운로드1
12289정성태8/6/2020107개발 환경 구성: 502. Portainer에 윈도우 컨테이너를 등록하는 방법
12288정성태8/5/202043오류 유형: 637. WCF - The protocol 'net.tcp' does not have an implementation of HostedTransportConfiguration type registered.
12287정성태8/5/202034오류 유형: 636. C# - libdl.so를 DllImport로 연결 시 docker container 내에서 System.DllNotFoundException 예외 발생
12286정성태8/5/202033개발 환경 구성: 501. .NET Core 용 container 이미지 만들 때 unzip이 필요한 경우
12285정성태8/4/202065오류 유형: 635. 윈도우 10 업데이트 - 0xc1900209 [2]
12284정성태8/4/202094디버깅 기술: 169. Hyper-V의 VM에 대한 메모리 덤프를 뜨는 방법
12283정성태8/3/202039디버깅 기술: 168. windbg - 필터 드라이버 확인하는 확장 명령어(!fltkd)
12282정성태8/2/202048디버깅 기술: 167. windbg 디버깅 사례: AppDomain 간의 static 변수 사용으로 인한 crash (2)
12281정성태8/2/2020159개발 환경 구성: 500. (PDB 연결이 없는) DLL의 소스 코드 디버깅을 dotPeek 도구로 해결하는 방법
12280정성태8/2/202054오류 유형: 634. 오라클 (평생) 무료 클라우드 VM 생성 후 SSH 접속 시 키 오류 발생
12279정성태7/29/202063개발 환경 구성: 499. 닷넷에서 접근해보는 InterSystems의 Cache 데이터베이스파일 다운로드1
12278정성태8/2/202065VS.NET IDE: 149. ("Binary was not built with debug information" 상태로) 소스 코드 디버깅이 안되는 경우
12277정성태8/2/2020101개발 환경 구성: 498. DEVPATH 환경 변수의 사용 예 - .NET Reflector의 (PDB 연결이 없는) DLL의 소스 코드 디버깅
12276정성태7/23/2020119.NET Framework: 930. 개발자를 위한 닷넷 어셈블리 바인딩 - DEVPATH 환경 변수
12275정성태7/28/2020101개발 환경 구성: 497. 닷넷에서 접근해보는 InterSystems의 IRIS Data Platform 데이터베이스파일 다운로드1
12274정성태7/21/202079개발 환경 구성: 496. Azure - Blob Storage Account의 Location 이전 방법파일 다운로드1
12273정성태7/20/2020208개발 환경 구성: 495. Azure - Location이 다른 웹/DB 서버의 경우 발생하는 성능 하락
12272정성태7/16/2020100.NET Framework: 929. (StrongName의 버전 구분이 필요 없는) .NET Core 어셈블리 바인딩 규칙파일 다운로드1
12271정성태7/16/202081.NET Framework: 928. .NET Framework의 Strong-named 어셈블리 바인딩 (2) - 런타임에 바인딩 리디렉션파일 다운로드1
12270정성태7/16/202075오류 유형: 633. SSL_CTX_use_certificate_file - error:140AB18F:SSL routines:SSL_CTX_use_certificate:ee key too small
12269정성태7/16/202070오류 유형: 632. .NET Core 웹 응용 프로그램 - The process was terminated due to an unhandled exception.
12268정성태7/15/202074오류 유형: 631. .NET Core 웹 응용 프로그램 오류 - HTTP Error 500.35 - ANCM Multiple In-Process Applications in same Process
12267정성태7/15/2020141.NET Framework: 927. C# - 윈도우 프로그램에서 Credential Manager를 이용한 보안 정보 저장파일 다운로드1
12266정성태7/14/2020159오류 유형: 630. 사용자 계정을 지정해 CreateService API로 서비스를 등록한 경우 "Error 1069: The service did not start due to a logon failure." 오류발생
[1]  2  3  4  5  6  7  8  9  10  11  12  13  14  15  ...