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 조합이 먼저 나오냐의 차이만 있어 속도 상으로도 거의 같을 것으로 예상됩니다. ^^
정성태

... 121  122  123  124  125  126  127  128  129  130  [131]  132  133  134  135  ...
NoWriterDateCnt.TitleFile(s)
1780정성태10/15/201424206오류 유형: 249. The application-specific permission settings do not grant Local Activation permission for the COM Server application with CLSID
1779정성태10/15/201419744오류 유형: 248. Active Directory에서 OU가 지워지지 않는 경우
1778정성태10/10/201418197오류 유형: 247. The Netlogon service could not create server share C:\Windows\SYSVOL\sysvol\[도메인명]\SCRIPTS.
1777정성태10/10/201421315오류 유형: 246. The processing of Group Policy failed. Windows attempted to read the file \\[도메인]\sysvol\[도메인]\Policies\{...GUID...}\gpt.ini
1776정성태10/10/201418332오류 유형: 245. 이벤트 로그 - Name resolution for the name _ldap._tcp.dc._msdcs.[도메인명]. timed out after none of the configured DNS servers responded.
1775정성태10/9/201419449오류 유형: 244. Visual Studio 디버깅 (2) - Unable to break execution. This process is not currently executing the type of code that you selected to debug.
1774정성태10/9/201426646개발 환경 구성: 246. IIS 작업자 프로세스의 20분 자동 재생(Recycle)을 끄는 방법
1773정성태10/8/201429799.NET Framework: 471. 웹 브라우저로 다운로드가 되는 파일을 왜 C# 코드로 하면 안되는 걸까요? [1]
1772정성태10/3/201418606.NET Framework: 470. C# 3.0의 기본 인자(default parameter)가 .NET 1.1/2.0에서도 실행될까? [3]
1771정성태10/2/201428106개발 환경 구성: 245. 실행된 프로세스(EXE)의 명령행 인자를 확인하고 싶다면 - Sysmon [4]
1770정성태10/2/201421714개발 환경 구성: 244. 매크로 정의를 이용해 파일 하나로 C++과 C#에서 공유하는 방법 [1]파일 다운로드1
1769정성태10/1/201424132개발 환경 구성: 243. Scala 개발 환경 구성(JVM, 닷넷) [1]
1768정성태10/1/201419557개발 환경 구성: 242. 배치 파일에서 Thread.Sleep 효과를 주는 방법 [5]
1767정성태10/1/201424646VS.NET IDE: 94. Visual Studio 2012/2013에서의 매크로 구현 - Visual Commander [2]
1766정성태10/1/201422511개발 환경 구성: 241. 책 "프로그래밍 클로저: Lisp"을 읽고 나서. [1]
1765정성태9/30/201426062.NET Framework: 469. Unity3d에서 transform을 변수에 할당해 사용하는 특별한 이유가 있을까요?
1764정성태9/30/201422306오류 유형: 243. 파일 삭제가 안 되는 경우 - The action can't be comleted because the file is open in System
1763정성태9/30/201423876.NET Framework: 468. PDB 파일을 연동해 소스 코드 라인 정보를 알아내는 방법파일 다운로드1
1762정성태9/30/201424555.NET Framework: 467. 닷넷에서 EIP/RIP 레지스터 값을 구하는 방법 [1]파일 다운로드1
1761정성태9/29/201421604.NET Framework: 466. 윈도우 운영체제의 보안 그룹 이름 및 설명 문자열을 바꾸는 방법파일 다운로드1
1760정성태9/28/201419884.NET Framework: 465. ICorProfilerInfo::GetILToNativeMapping 메서드가 0x80131358을 반환하는 경우
1759정성태9/27/201430994개발 환경 구성: 240. Visual C++ / x64 환경에서 inline-assembly를 매크로 어셈블리로 대체하는 방법파일 다운로드1
1758정성태9/23/201437895개발 환경 구성: 239. 원격 데스크톱 접속(RDP)을 기존의 콘솔 모드처럼 사용하는 방법 [1]
1757정성태9/23/201418435오류 유형: 242. Lync로 모임 참여 시 소리만 들리지 않는 경우 - 두 번째 이야기
1756정성태9/23/201427463기타: 48. NVidia 제품의 과다한 디스크 사용 [2]
1755정성태9/22/201434238오류 유형: 241. Unity Web Player를 설치해도 여전히 설치하라는 화면이 나오는 경우 [4]
... 121  122  123  124  125  126  127  128  129  130  [131]  132  133  134  135  ...