C# - Il modo più semplice per rimuovere la prima occorrenza di una sottostringa da un'altra stringa

Question

Ho bisogno di rimuovere la prima (e SOLO la prima) occorrenza di una stringa da un'altra stringa.

Ecco un esempio che sostituisce la stringa "\\Iteration". Questo:

 
ProjectName\\Iteration\\Release1\\Iteration1 

sarebbe diventato questo:

 
ProjectName\\Release1\\Iteration1 

Ecco un codice che fa questo:

const string removeString = "\\Iteration"; 
int index = sourceString.IndexOf(removeString); 
int length = removeString.Length; 
String startOfString = sourceString.Substring(0, index); 
String endOfString = sourceString.Substring(index + length); 
String cleanPath = startOfString + endOfString; 

che sembra un sacco di codice.

Quindi la mia domanda è questa: c'è un modo più pulito/più leggibile/più conciso per farlo?

Answer 1

int index = sourceString.IndexOf(removeString); 
string cleanPath = (index < 0) 
    ? sourceString 
    : sourceString.Remove(index, removeString.Length); 

Answer 2

string myString = sourceString.Remove(sourceString.IndexOf(removeString),removeString.Length); 

MODIFICA: @OregonGhost ha ragione. Io stesso avrei infranto la sceneggiatura con i condizionali per verificare la presenza di un tale evento, ma stavo operando partendo dal presupposto che le stringhe fossero date per appartenere l'una all'altra da qualche requisito. È possibile che le regole di gestione delle eccezioni richieste dal business prevedano questa possibilità. Io stesso userei un paio di linee extra per eseguire i controlli condizionali e anche per renderlo un po 'più leggibile per gli sviluppatori junior che potrebbero non prendersi il tempo per leggerlo abbastanza a fondo.

Answer 3

Ha scritto un test TDD rapida per questo

[TestMethod] 
    public void Test() 
    { 
     var input = @"ProjectName\Iteration\Release1\Iteration1"; 
     var pattern = @"\\Iteration"; 

     var rgx = new Regex(pattern); 
     var result = rgx.Replace(input, "", 1); 

     Assert.IsTrue(result.Equals(@"ProjectName\Release1\Iteration1")); 
    } 

rgx.Replace (ingresso, "", 1); dice di cercare input per qualsiasi cosa corrisponda al modello, con "", 1 volta.

Answer 4

È possibile utilizzare un metodo di estensione per divertimento. In genere, non è consigliabile collegare i metodi di estensione a una classe generica come la stringa, ma come ho detto è divertente. Ho preso in prestito la risposta di Luke poiché non ha senso reinventare la ruota.

[Test] 
public void Should_remove_first_occurrance_of_string() { 

    var source = "ProjectName\\Iteration\\Release1\\Iteration1"; 

    Assert.That(
     source.RemoveFirst("\\Iteration"), 
     Is.EqualTo("ProjectName\\Release1\\Iteration1")); 
} 

public static class StringExtensions { 
    public static string RemoveFirst(this string source, string remove) { 
     int index = source.IndexOf(remove); 
     return (index < 0) 
      ? source 
      : source.Remove(index, remove.Length); 
    } 
} 

Answer 5

Sono assolutamente d'accordo che questo è perfetto per un metodo di estensione, ma penso che può essere migliorato un po '.

public static string Remove(this string source, string remove, int firstN) 
    { 
     if(firstN <= 0 || string.IsNullOrEmpty(source) || string.IsNullOrEmpty(remove)) 
     { 
      return source; 
     } 
     int index = source.IndexOf(remove); 
     return index < 0 ? source : source.Remove(index, remove.Length).Remove(remove, --firstN); 
    } 

Questo fa un po 'di ricorsione che è sempre divertente.

Ecco un semplice test di unità così:

[TestMethod()] 
    public void RemoveTwiceTest() 
    { 
     string source = "look up look up look it up"; 
     string remove = "look"; 
     int firstN = 2; 
     string expected = " up up look it up"; 
     string actual; 
     actual = source.Remove(remove, firstN); 
     Assert.AreEqual(expected, actual); 

    } 

Answer 6

sourceString.Replace(removeString, "");