Prüfen, ob string vorhanden ist in der Liste C#

Ich habe die Codierung ein Programm, dass speichert die Daten über Studierende (name und Alter) in einem .txt-Datei. Ich bin jetzt dabei die delete-Methode. Allerdings, wenn der Benutzer eine Zeichenfolge eingibt, soll ich ihn vergleichen, den Eingang zu den strings in meiner List<string>, das ist voll von Namen. Code:

    string tempFileName;
    string inputSel; //Selection string for delete
    Console.WriteLine(" -- Deleting Grade {0} -- ", grade);
    Console.WriteLine("- Enter a student name to delete: ");
    foreach (string file in fileNames)
    {
        tempFileName = file.Replace(".txt", "");
        studentNames.Add(tempFileName);
    }
    foreach (string name in studentNames)
    {
        Console.Write("{0}\n", name);
    }
    Console.WriteLine();
    Console.Write("> ");
    inputSel = Console.ReadLine();
    string input = inputSel.ToLower();
    string tempString;
    bool foundString = false;
    foreach (string file in studentNames)
    {
        tempString = file.ToLower();
        if (inputSel == tempString)
        {
            foundString = true;
        }
    }
    if (!foundString)
    {
        Console.WriteLine("Wrong name entered!");
        Console.WriteLine("Returning to grades menu..");
        Console.WriteLine("Press any key to continue.");
        Console.ReadKey();
        return;
    }

Wie Sie sehen können, das Programm speichert inputSel in input (ToLower()), dann vergleicht jeder string in studentNames List<string>, und wenn es eine übereinstimmung findet, es spiegelt das foundString bool aber selbst wenn ich einen Namen eingeben, der übereinstimmungen (zum Beispiel, es sagt JacobMusterson, geben Sie mir JacobMusterson, sollte es überspringen, der "student nicht gefunden", aber es funktioniert nicht.

vergessen Sie nicht zu akzeptieren hilfreiche Antworten

InformationsquelleAutor Ilan321 | 2013-10-30

c#foreach list

2

sollten Sie verwenden, input nicht inputSel
```
if (input == tempString)
{
    foundString = true;
}
```
wegen der Zeile:
```
string input = inputSel.ToLower();
```
wo Sie sind assinging Eingabe niedrigeren version inputSel

Ich schlage vor, Sie verwenden IngonreCase in der Zeichenkette.Vergleichen Sie für nicht machen ToLower()
```
var b =  string.Compare("a","A",StringComparison.OrdinalIgnoreCase);
```
es gibt 0 zurück, wenn gleich sehen hier

EDIT:

persönlich würde ich verwenden:
```
var exists = studentNames.Any(x=>string.Compare(x,inputSel,StringComparison.OrdinalIgnoreCase)==0);
```
- Wow, ich kann nicht glauben, dass ich verpasst, dass. Vielen Dank! (Warum kann ich nicht akzeptieren Antworten?)
- Sie können, aber Sie müssen einige Minuten warten, popup sagt, dass
InformationsquelleAutor Kamil Budziewski

können Sie einfach tun, es auf diese Weise:

Boolean foundString = studentNames.Exists(name => name.ToLower().Equals(input));

InformationsquelleAutor Marco Forberg

Wird es besser lesbar und wirksam, wenn Sie verwenden Contains Methode der List:

foreach (string file in studentNames)
    {
        tempString = file.ToLower();
        if (inputSel == tempString)
        {
            foundString = true;
        }
    }
if (!foundString)
    {
        Console.WriteLine("Wrong name entered!");
        Console.WriteLine("Returning to grades menu..");
        Console.WriteLine("Press any key to continue.");
        Console.ReadKey();
        return;
    }

werden kann umschreiben:

if(!studentNames.Contains(inputSel, StringComparer.Create(CultureInfo.InvariantCulture, true)))
{
            Console.WriteLine("Wrong name entered!");
            Console.WriteLine("Returning to grades menu..");
            Console.WriteLine("Press any key to continue.");
            Console.ReadKey();
            return;
}

InformationsquelleAutor Kirill Bestemyanov

0

Einfach einen kleinen Kommentar auf deiner foreach-Schleife. Sie immer Durchlaufen alle Einträge in der Schleife, auch wenn Sie bereits herausgefunden, dass die Zeichenfolge in der Sammlung.

Verbessern Sie die performance Ihres Codes durch den Austausch der letzten foreach.

Versuchen:
```
bool foundString = studentNames.Select(file => file.ToLower()).Any(tempString => inputSel == tempString);
```
InformationsquelleAutor Tomtom

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.