C# içinde bir dizgiyi ayrıştırma; daha temiz bir yol var mı?

Question

C#, NET 3.5

Bu sadece bana çirkin kokuyor, ama başka bir şekilde düşünemiyorum.

"Joe Smith (jsmith)" (sans quotes) biçimindeki bir dize verildiğinde, yalnızca parantez içindeki "jsmith" dizesini ayrıştırmak istiyorum. RegEx'in benim (me) sağlıklı kaçınma daha

private static string DecipherUserName(string user) 
{ 
    if(!user.Contains("(")) 
     return user; 

    int start = user.IndexOf("("); 

    return user.Substring(start).Replace("(", string.Empty).Replace(")", string.Empty); 
} 

Diğer, alt dizeyi ayrıştırmak için daha basit bir yolu var

: Ben bu ile geldim?

Düzenleme: "Joe Smith (gozdemir)" (sans tırnak): dize her zaman olacaktır ayrıştırmak için, netleştirmek için. IndexOf işlevi dönecektir beri değeri yok olduğunda

Answer 1

bulmaktır sadece "(" konumuna 1 ekleyebilir. bir kesmek

private static string DecipherUserName (string user) {   
    int start = user.IndexOf("("); 
    if (start == -1) 
     return user; 
    return user.Substring (start+1).Replace(")", string.Empty); 
} 

Answer 2

-1, sen

private static string DecipherUserName(string user) 
{   
    int start = user.IndexOf("("); 

    if (start > -1) 
    { 
     return user.Substring(start).Replace("(", string.Empty).Replace(")", string.Empty); 
    } 
    else 
    { 
     return user; 
    } 
} 

Answer 3

Regexes kendine gönül yarası bir ton tasarruf edeceğiz böylece yararlıdır ... biraz farklı şeyler yapabilir mermiyi ısırmak ve onları öğrenmek. Bütün Shebang değil, sadece temeller.

Çalışacağınız bir regex "\ w + \ ((*. *)") - jsmith Match.Groups [1] dizininde olurdu.

Regexes almaya

kolay yollarından biri, ilk sizin beri yerine Sen gerekmez sonra eşleşmeleri tükürmek bir regex ve bazı metin yazın bildireceğiz bir web sitesi ...

Answer 4

Kind ... ^^

return user.Substring(user.IndexOf('(') + 1).TrimEnd(')'); 

user hiçbir parantez açma içeriyorsa, IndexOf()-1 döndürür, bir tane ekleriz, sıfır alır ve SubString() tüm dizeyi döndürür. Kullanıcının adı kapanış parantezi ile bitmediği sürece TrimEnd()'un hiçbir etkisi olmayacaktır. user endeksini verir IndexOf() bir parantez açma içeriyorsa

, birini ekleyerek Açma parantezi Skipp ve Substring() ile dize kalan kısmını çıkartmak. Son olarak, kapanış parantezini TrimEnd() ile kaldırıyoruz.

Answer 5

Ben

int start=user.IndexOf('('); 
if (start != -1) { 
    end = user.IndexOf(')', start); 
    return user.Substring(start+1,end-start-1); 
} else 
    return user; 

kullanmayı tercih ediyorum Ama bu sadece kozmetik bir değişimdir: IndexOf'un karakterleri kullanarak biraz daha hızlı olduğunu ve alt dize yöntemini kullanarak daha tam olarak ne yapılması gerektiği ifade etmek görünüyor (ve yöntemi) ... (basit olabilir) String.Split kullanılarak

söyledi

, Daniel L 'ın yöntem parantez birden çiftleri varsa daha sağlamdır (ama hatalı biçimlendirilmiş dizeleri ile çok iyi uğraşmaz ve zorundadır dize dizisi oluşturmak.

Her şeyden önce, normal ifadelere olan ilginizden kurtulmanızı öneririm, çünkü bu durum tam olarak istedikleri şeydir :-)

Answer 6

Kullanıcı dizgisi her zaman "Joe Smith (jsmith)" biçimindeyse, bu işe yaramalıdır.

private static string DecipherUserName(string user) 
{ 
    string[] names = user.Split(new char[] {'(', ')'}); 
    return names.Length > 2 ? names[1] : user; 
} 

Ve kullanıcı dize her zaman ise "Joe Smith (gozdemir)", bu her zaman çalışacaktır.

Sadece mizah amaçlı ikinci giriş.