C# : StartsWith와 두 개의 ToUpperInvariant 호출을 결합하는 것보다 더 좋은 방법

Question

내 머리 뒤쪽 어딘가에서 작은 음성이 "C# 코드가 냄새"라고 말합니다.

private const string STR_ConnectionString = "ConnectionString"; 
private readonly string upperCaseConnectionString = STR_ConnectionString.ToUpperInvariant(); 
// a lot further on 
string keyAttributeValue = keyAttribute.Value; 
if (keyAttributeValue.ToUpperInvariant().StartsWith(upperCaseConnectionString)) 
{ 
    // some C# code handling a key that starts with "ConnectionString" 
} 

상수 STR_ConnectionString은 코드의 다른 위치에도 사용됩니다.

냄새를 제거하는 방법은 무엇입니까?

Answer 1

당신은 overloaded StartsWith method는 StringComparison 열거 값 복용 사용할 수 있습니다 : 그것은 있기 때문에,

if (keyAttributeValue.StartsWith(STR_ConnectionString, 
           StringComparison.InvariantCultureIgnoreCase) 
{ 
    ... 
} 

그것은 또한 당신의 코드를 더 쉽게 읽을 :

keyAttributeValue.StartsWith(STR_ConnectionString, StringComparison.OrdinalIgnoreCase) // or use StringComparison.InvariantCultureIgnoreCase here 

Answer 2

는 대소 문자를 구별 매칭을 지원하는 StartsWith overload이를 당신의 의도를 표현합니다 : 당신이 정말로 원하는 것은 대소 문자를 구별하지 않는 비교이며, 여기에 쓰여 있습니다. 대/소문자를 구분하여 대문자로 변환 한 값을 비교하는 것을 원하지는 않습니다. 목표를 달성하는 데 사용하는 해결 방법 일뿐입니다.

Answer 3

StringComparison을 사용하여 내 프로젝트에서 대소 문자를 무시하는 확장 메서드를 만들었습니다.

public static string StartsWithIgnoreCase(this string value, string startsWith) 
{ 
    return value.StartsWith(value, StringComparison.InvariantCultureIgnoreCase); 
} 

원하는 경우 null 체크를 추가 할 수 있습니다. 당신이 당신의 자신의 방법을 압연하는 것처럼

STR_ConnectionString..StartsWith(upperCaseConnectionString, StringComparison.CurrentCultureIgnoreCase); 

그러나 보이는 : 당신의 ToUpper하고 있기 때문에 나쁜 냄새 경우

Answer 4

다음 문자열들은 startswith의 오버로드를 사용하여 결합 할 수 비교 응용 프로그램 구성을 처리하십시오. 이 항상

keyAttributeValue.StartsWith(STR_ConnectionString, StringComparison.InvariantCultureIgnoreCase) 

입니다하지만 당신이 무엇을뿐만 아니라 수행하지 않습니다 http://msdn.microsoft.com/en-us/library/system.configuration.configurationmanager.aspx

Answer 5

참조하십시오. 초당 수천 번해야하는 경우 보유하고있는 것에 충실하십시오. 그렇지 않은 경우 대소 문자를 구분하지 마십시오.

또한 빠르게 당신이 어떻게 든 원하는 것을 아마도

keyAttributeValue.StartsWith(STR_ConnectionString, StringComparison.OrdinalIgnoreCase) 

을 고려하십시오.