Analisi di una stringa in C#; c'è un modo più pulito?

Question

C#, .NET 3.5

Questo mi sa solo di brutto, ma non riesco a pensare a un altro modo.

Dato una stringa con il formato di "Joe Smith (jsmith)" (senza virgolette), vorrei analizzare solo la stringa 'jsmith' all'interno della parentesi. Mi è venuta in mente questo:

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); 
} 

Altro che la mia (dis) l'avversione sano per RegEx, c'è un modo più semplice per analizzare la stringa?

Edit: Per chiarire, la stringa da analizzare sarà sempre di: "Joe Smith (jsmith)" (sans virgolette).

Answer 1

Non dovrebbe essere necessario sostituire il primo dato che può solo aggiungere 1 in posizione "(".

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

Poiché la funzione IndexOf restituisce -1 quando il valore non esiste, si potrebbe fare le cose un po 'diverse ...

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

regex sono così utili che ti risparmiare un sacco di mal di cuore mordere il proiettile e impararli. Non l'intero shebang, solo le basi.

Un'espressione regolare che funziona è "\ w + \ ((. *) \)" - jsmith si trova in Match.Groups [1].

Un modo semplice per raccogliere espressioni regolari è quello di trovare un sito web che ti consentono di digitare una regex e del testo poi sputare le partite ...

Answer 4

una specie di hack ... ^^

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

Se user non contiene parentesi aperta, IndexOf() restituisce -1, aggiungiamo uno, ottieni zero e SubString() restituisce l'intera stringa. TrimEnd() non avrà alcun effetto a meno che il nome dell'utente non termini con una parentesi chiusa.

Se user contiene una parentesi aperta, IndexOf() ritorna il suo indice, abbiamo Skipp la parentesi di apertura con l'aggiunta di uno, ed estrarre il resto della stringa con Substring(). Infine rimuoviamo la parentesi chiusa con TrimEnd().

Answer 5

userei

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

ma questo è solo un cambiamento cosmetico: l'utilizzo di caratteri in IndexOf è un po 'più veloce, e utilizzando il metodo Substring sembra esprimere più esattamente ciò che dovrebbe essere fatto (e la metodo è più robusto se si dispone di più coppie di parentesi ...)

detto questo, Daniel metodo L 's (utilizzando String.Split) potrebbe essere più semplice (ma non fare molto bene con le stringhe malformate e deve costruire un array di stringhe).

Tutto sommato, ti suggerisco di superare la tua avversione per le espressioni regolari, dal momento che quella situazione è esattamente ciò a cui sono destinate :-) ...

Answer 6

Se la stringa utente è sempre nel formato "Joe Smith (jsmith)", questo dovrebbe funzionare.

E se la stringa utente è sempre "Joe Smith (jsmith)", funzionerà sempre.

private static string DecipherUserName(string user) 
{ 
    return "jsmith"; 
} 

La seconda voce solo a scopo di umorismo.