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 at outlook.com

비밀번호

댓글 작성자
 



2020-06-29 09시55분
[dimohy] 특별히 속도에 영향있는 코드가 아니라면 가독성이 좋은 코드에 한표입니다.
특히나 개발환경에서 가이드 해줄 정도면 컴파일러에서 알아서 최적화는 해줄 수 있겠네요 ^^;
[guest]
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 조합이 먼저 나오냐의 차이만 있어 속도 상으로도 거의 같을 것으로 예상됩니다. ^^
정성태

... 151  152  153  154  155  156  157  [158]  159  160  161  162  163  164  165  ...
NoWriterDateCnt.TitleFile(s)
1099정성태8/15/201128253오류 유형: 132. 어느 순간 갑자기 접속이 안 되는 TFS 서버
1098정성태8/15/201150335웹: 24. 네이버는 어떻게 로그인 처리를 할까요? [2]
1097정성태8/15/201121631.NET Framework: 235. 메서드의 메타 데이터 토큰 값으로 클래스를 찾아내는 방법
1096정성태8/15/201125747디버깅 기술: 42. Watson Bucket 정보를 이용한 CLR 응용 프로그램 예외 분석 - (2)
1095정성태8/14/201126146디버깅 기술: 41. Windbg - 비정상 종료된 닷넷 프로그램의 StackTrace에서 보이는 offset 값 의미
1094정성태8/14/201130581오류 유형: 131. Fiddler가 강제 종료된 경우, 웹 사이트 방문이 안되는 현상
1093정성태7/27/201124154오류 유형: 130. Unable to connect to the Microsoft Visual Studio Remote Debugging Monitor ... Access is denied.
1092정성태7/22/201126610Team Foundation Server: 46. 코드 이외의 파일에 대해 소스 제어에서 제외시키는 방법
1091정성태7/21/201125651개발 환경 구성: 128. WP7 Emulator 실행 시 audiodg.exe의 CPU 소모율 증가 [2]
1089정성태7/18/201131201.NET Framework: 234. 왜? Button 컨트롤에는 MouseDown/MouseUp 이벤트가 발생하지 않을까요?파일 다운로드1
1088정성태7/16/201124246.NET Framework: 233. Entity Framework 4.1 - 윈도우 폰 7에서의 CodeFirst 순환 참조 문제파일 다운로드1
1087정성태7/15/201126969.NET Framework: 232. Entity Framework 4.1 - CodeFirst 개체의 직렬화 시 순환 참조 해결하는 방법 - 두 번째 이야기파일 다운로드1
1086정성태7/14/201128363.NET Framework: 231. Entity Framework 4.1 - CodeFirst 개체의 직렬화 시 순환 참조 해결하는 방법 [1]파일 다운로드1
1085정성태7/14/201128884.NET Framework: 230. Entity Framework 4.1 - Code First + WCF 서비스 시 EndpointNotFoundException 오류 - 두 번째 이야기파일 다운로드1
1084정성태7/11/201134154.NET Framework: 229. SQL 서버 - DB 테이블의 데이터 변경에 대한 알림 처리 [4]파일 다운로드1
1083정성태7/11/201128183.NET Framework: 228. Entity Framework 4.1 - Code First + WCF 서비스 시 EndpointNotFoundException 오류
1082정성태7/10/201127730.NET Framework: 227. basicHttpBinding + 사용자 정의 인증 구현 [2]파일 다운로드1
1081정성태7/9/201127055VC++: 53. Windows 7에서 gcc.exe 실행 시 Access denied 오류 [2]
1080정성태7/8/201125539웹: 23. Sysnet 웹 사이트의 HTML5 변환 기록 - 두 번째 이야기파일 다운로드1
1079정성태7/6/201129983오류 유형: 129. Hyper-V + Realtek 랜카드가 설치된 시스템의 BSOD 현상 [2]
1078정성태7/5/201137470VC++: 52. Chromium 컴파일하는 방법 [2]
1077정성태6/24/201135096.NET Framework: 226. HttpWebRequest 타입의 HaveResponse 속성 이야기파일 다운로드1
1076정성태6/23/201129268오류 유형: 128. SQL Express - User Instance 옵션을 사용한 경우 발생하는 오류 메시지 유형 2가지
1075정성태6/21/201124858VS.NET IDE: 69. 윈폰 프로젝트에서 WCF 서비스 참조할 때 Reference.cs 파일이 비어있는 경우
1074정성태6/20/201124974.NET Framework: 225. 닷넷 네트워크 라이브러리의 트레이스 기능파일 다운로드1
1073정성태6/20/201127195오류 유형: 127. Visual Studio에서 WCF 서비스의 이름 변경 시 발생할 수 있는 오류
... 151  152  153  154  155  156  157  [158]  159  160  161  162  163  164  165  ...