La division d'une chaîne sans utiliser la méthode String.Split ne renvoie pas le résultat correct

Question

Je souhaite diviser une chaîne sans utiliser la méthode String.Split.
J'ai trouvé une solution possible here. Le code que j'utilise provient de la deuxième réponse.
Ceci est mon code:

public string[] SplitString(string input, char delimiter) 
{ 
    List<String> parts = new List<String>(); 
    StringBuilder buff = new StringBuilder(); 

    for (int i = 0; i < input.Length; i++) 
    { 
     if (input[i] == delimiter) 
     { 
      parts.Add(buff.ToString()); 
      buff.Clear(); 
     } 
     else 
     { 
      buff.Append(input[i]); 
     } 
    } 
    return parts.ToArray(); 
} 

Mon problème est que lorsque je tente de diviser une chaîne comme celui-ci
chien personne voiture chat par « », le résultat contient les mots sans la dernière un (dans cet exemple - personne).
Si après le dernier mot il y a un espace blanc, le résultat est correct.

J'ai essayé d'ajouter quelque chose comme i == input.Length lorsque la boucle est 0-i < = input.Length. Mais le résultat était toujours sans le dernier mot.
Ai-je raté quelque chose?

Answer 1

Ajouter un autre parts.Add(buff.ToString()); après avoir quitté la boucle - pour vider le dernier mot dans la collection. Vous pouvez vérifier la longueur avant de le faire ou non comme commenté et expliqué pourquoi par @hvd.

if(buff.Length != 0) 
{ 
    parts.Add(buff.ToString()); 
} 
return parts.ToArray(); 

---

Une autre approche, au lieu d'utiliser un StringBuilder serait:

public static string[] SplitString(string input, char delimiter) 
{ 
    List<String> parts = new List<String>(); 
    int start = 0; 
    for(int i = 0; i < input.Length; i++) 
    { 
     if(input[i] == delimiter) 
     { 
      parts.Add(input.Substring(start, i - start)); 
      start = i + 1; 
     } 
    } 
    parts.Add(input.Substring(start, input.Length - start)); 
    return parts.ToArray(); 
} 

Ou utilisez yield return et retour IEnumerable<string>:

public static IEnumerable<string> SplitString(string input, char delimiter) 
{ 
    int start = 0; 
    for(int i = 0; i < input.Length; i++) 
    { 
     if(input[i] == delimiter) 
     { 
      yield return input.Substring(start, i - start); 
      start = i + 1; 
     } 
    } 
    yield return input.Substring(start, input.Length - start); 
} 

Answer 2

Voici ce qui vous manque dans votre code après pour la boucle:

for (int i = 0; i < input.Length; i++) 
    { 
     if (input[i] == delimiter) 
     { 
      parts.Add(buff.ToString()); 
      buff.Clear(); 
     } 
     else 
     { 
      buff.Append(input[i]); 
     } 
    } 

    // This you need to add 
    if (!string.IsNullOrEmpty(buff.ToString())) 
    { 
     parts.Add(buff.ToString()); 
    } 

    return parts.ToArray();