Nhận chuỗi dài nhất và ngắn nhất theo cách giả tạo

Question

Tôi có các dòng sau trong mã của mình. Họ đang chiếm nhiều chỗ hơn họ nên làm. Bất kỳ đề xuất nào cho một mã nhỏ hơn.

string longestString; 
string shortestString; 
if (string1.Length > string2.Length) 
{ 
    longestString = string1; 
    shortestString = string2; 
} 
else 
{ 
    longestString = string2; 
    shortestString = string1; 
} 

Tôi biết, không phải là câu hỏi thực sự quan trọng, nhưng điều này chiếm 2/3 toàn bộ phương pháp chứ không phải là nội dung quan trọng.

Answer 1

lẽ:

int diff = string1.Length.CompareTo(string2.Length); 
string longestString = diff > 0 ? string1 : string2; 
string shortestString = diff > 0 ? string2 : string1; 

Nhưng nếu bạn có nhiều hơn hai chuỗi những điều này và bạn muốn có một giải pháp chung, bạn có thể sử dụng:

var lenLookup = new[] { string1, string2 }.OrderBy(s => s.Length).ToList(); 
string shortestString = lenLookup.First(); 
string longestString = lenLookup.Last(); 

Answer 2

Kể từ mã của bạn sẽ luôn thực hiện một trong hai nếu hoặc đường dẫn khác, chọn một là "mặc định" và hợp nhất nó với khai báo biến:

string longestString = string2 
string shortestString = string1; 
if (string1.Length > string2.Length) 
{ 
    longestString = string1; 
    shortestString = string2; 
} 

điểm thưởng cho thực tế là bạn sẽ thực sự khởi những biến đó.

Answer 3

Bạn có thể sử dụng ":" điều hành

string longestString = string1.Length > string2.Length ? string1 : string2; 
string shortestString = string1.Length > string2.Length ? string2 : string1; 

Answer 4

Vâng, bạn có thể làm điều này để xóa phương pháp của mình;

public string GetLongestString(string str1, string str2) 
{ 
    return str1.Length > str2.Length ? str1 : str2; 
} 

public string GetShortestString(string str1, string str2) 
{ 
    return str1.Length > str2.Length ? str2 : str1; 
} 

string longestString = GetLongestString(string1, string2); 
string shortestString = GetShortestString(string1, string2); 

Và tái sử dụng nó bất cứ khi nào bạn muốn!

Heck, làm cho nó thậm chí còn mát hơn (trong chế độ co-op với Tim);

public IEnumerable<string> GetLongestStrings(params string[] strings) 
{ 
    //returns first string with largest length out of given argumenst 
    int maxSize = strings.Max(str => str.Length); 
    return strings.Where(s => s.Length == maxSize); 
} 

public IEnumerable<string> GetShortestStrings(params string[] strings) 
{ 
    //returns first string with shortest length out of given arguments 
    int minSize = strings.Min(str => str.Length); 
    return strings.Where(s => s.Length == minSize); 
} 

Cách sử dụng;

string longestString = GetLongestStrings("str1", "str2", /*...*/ "strN").FirstOrDefault(); 

EDIT1: Triển khai đầu tiên của tôi không hiệu quả nhất. Như Tim đã đề nghị;

public string GetLongestString(params string[] strings) 
{ 
    return strings.OrderBy(s => s.Length).First(); 
} 

public string GetShortestString(params string[] strings) 
{ 
    return strings.OrderByDescending(s => s.Length).First(); 
} 

Cách sử dụng;

string longestString = GetLongestString("str1", "str2", /*...*/ "strN");