文字列のリスト内の次の文字列を見つけよう

Question

私はUserNamesのリストをカンマ区切りの文字列で持っています。次の入力ユーザー名を探したい。例については ：

var s0 = "abc,deF,ghi,jkl"; 
var s1 = "abc"; 
var s2 = "def"; 
var s3 = "ghi"; 
var s4 = "jkl"; 

結果は次のようになります。ここでは

NextInString(s0,s1) == "def" 
NextInString(s0,s2) == "ghi" 
NextInString(s0,s3) == "jkl" 
NextInString(s0,s4) == "jkl" 

は私が持っているものです。

string NextInString(string listOfNames, string userName) 
{ 
    if(listOfNames == string.Empty || userName == string.Empty) 
     return string.Empty; 

    var s = listOfNames.Split(','); 
    var count = 0; 
    foreach (var element in s) 
    { 
     if (element == userName)break; 
     count++; 
    } 

    if (s.Length -1 == count) 
    { 
     return s[count]; 
    } 
    else return s[ count + 1 ]; 
} 

私の質問は、これにアプローチするより良い/簡単な方法はありますか？

Answer 1

あなたはこのようにLINQを使用することができます：ここで

string userName = "abc"; 
string listOfNames = "abc,xyz,123"; 

var names = listOfNames 
    .Split(',') 
    .Select((n, i) => new {name = n, index =i}) 
    .ToArray(); 

var firstMatch = names.FirstOrDefault(n => n.name == userName); 
var result = firstMatch == null 
    ? string.Empty 
    : firstMatch.index == names.Length - 1 
     ? string.Empty 
     : names[firstMatch.index + 1].name; 

Answer 2

は、LINQのアプローチです：このような何かについて

string NextInString(string listOfNames, string userName) 
{ 
    if(listOfNames == string.Empty || userName == string.Empty) return string.Empty; 

    var names = listOfNames.Split(','); 
    return names 
     .SkipWhile(x => x != userName) 
     .Skip(1) 
     .FirstOrDefault() ?? names.Last(); 
} 

Answer 3

どのように？

var s = listOfNames.Split(','); 
for (var i = 0; i < s.count; i++) 
{ 
    if (i == s.count - 1) 
    { 
     return string.Format("No user after {0} was found", userName); 
    } 
    else if (s[i] == userName) 
    { 
     return s[i + 1]; 
    } 
} 

Answer 4

あなたは、文字列のリストがトリミングされていることを確認するために余分なステップを取る場合は、あなただけのList<T>のIndexOf()方法を使用することができます。

string csv = "test1, test2, test3, test4";  
List<string> names = csv.Split(',').Select(x => x.Trim()).ToList(); 

次に、あなたのNextInString()方法（私はこれが悪いと命名されたと思います）メソッドは、次のようになります。私はフィドルを作っ

private static string NextInString(List<string> names, string userName) 
{  
    int index = names.IndexOf(userName); 

    if(names.Count - 1 == index || index == -1) 
    { 
     return "No result"; 
    } 
    else 
    { 
     return names[index + 1]; 
    } 
} 

here

static void Main(string[] args) 
{ 
    var list = "cat,dog,rat".Split(','); 
    Console.WriteLine(list.FindItemAfter("cat")); 
    Console.WriteLine(list.FindItemAfter("dog")); 
    Console.ReadLine(); 
} 

それが返されます：

static class IListExtensions 
{ 
    public static T FindItemAfter<T>(this IList<T> list, T targetItem) 
    { 
     return list[list.IndexOf(targetItem)+ 1]; 
    } 
} 

あなたはこのようにそれを使用することができます：あなたは、文字列の後にこれを行うためのちょっといい拡張メソッドを作ることができます

Answer 5

は、スプリット、とても似ています

犬
ラット

---

このオーバーロードにより、要求されたアイテムが見つからない場合、または次のアイテムがリストの外にある場合に返されるデフォルト値を指定できます。

public static T FindItemAfter<T>(this IList<T> list, T targetItem, T defaultValue) 
{ 
    var index = list.IndexOf(targetItem); 
    if (index == -1 || index >= list.Count - 1) 
    { 
     return defaultValue; 
    } 
    return list[index + 1]; 
}