Müssen doppelte Werte aus C# -Programm entfernen

Question

Ich brauche Hilfe mit einem C# -Programm, das ich erstelle. In diesem Szenario gebe ich doppelte Werte in das Programm ein. Zum Beispiel a, b, b, c, c.
Die Übung ist, dass wenn doppelte Buchstaben eingegeben werden (keine Zahlen), sollte ich einen Fehler erhalten, der besagt "Duplicate Value. Bitte versuchen Sie es erneut!" und akzeptiert den doppelten Wert nicht und sollte die Werte als a, b, c, d, e anzeigen.

class Program 
{ 
    static void Main(string[] args) 
    { 
    char[] arr = new char[5]; 

    //User input 
    Console.WriteLine("Please Enter 5 Letters only: "); 

    for (int i = 0; i < arr.Length; i++) 
    { 
     arr[i] = Convert.ToChar(Console.ReadLine()); 
    } 
    //display 
    for(int i = 0; i<arr.Length; i++) 
    { 
     Console.WriteLine("You have entered the following inputs: "); 
     Console.WriteLine(arrArray[i]); 
    } 
} 
} 

Answer 1

Verwenden Sie Any linq Ausdruck, um Duplikate zu überprüfen. char.TryParse validiert Eingabe und gibt true zurück, wenn erfolgreich.

public static void Main() 
{ 
    char[] arr = new char[5]; 

    //User input 
    Console.WriteLine("Please Enter 5 Letters only: "); 

    for (int i = 0; i < arr.Length; i++) 
    { 
     char input; 

     if(char.TryParse(Console.ReadLine(), out input) && !arr.Any(c=>c == input)) 
     {   

      arr[i] = input; 
     } 
     else 
     { 
      Console.WriteLine("Error : Either invalid input or a duplicate entry."); 
      i--; 
     } 
    } 

    Console.WriteLine("You have entered the following inputs: "); 
    //display 
    for(int i = 0; i<arr.Length; i++) 
    { 

     Console.WriteLine(arr[i]); 
    } 

} 

Arbeiten Code

Answer 2

Mit Hilfe eines Hash-Tabelle (Generic Dictionary) ist ein effizienter Weg, um zu bestimmen, ob eine eingegebene Zeichen bereits aufgetreten.

Auch die Char.IsLetter-Methode im .NET-Framework ist eine gute Möglichkeit, nach fehlerhaften Daten zu suchen.

static void Main(string[] args) { 
    Dictionary<char, bool> charsEntered = new Dictionary<char, bool>(); 

    Console.WriteLine("Please enter 5 characters, each on a separate line."); 
    while (charsEntered.Count() < 5) { 
     Console.WriteLine("Enter a character:"); 
     char[] resultChars = Console.ReadLine().ToCharArray(); 

     if(resultChars.Length != 1 || !Char.IsLetter(resultChars[0])) { 
      Console.WriteLine("Bad Entry. Try again."); 
     } else { 
      char charEntered = resultChars[0]; 
      if (charsEntered.ContainsKey(charEntered)) 
       Console.WriteLine("Character already encountered. Try again."); 
      else 
       charsEntered[charEntered] = true; 
     } 
    } 

    Console.WriteLine("The following inputs were entered:"); 
    Console.WriteLine(String.Join(", ", charsEntered.Keys)); 
    Console.ReadLine(); 
} 

Answer 3

richtige Datenstruktur zu Beginn wählen, verwenden HashSet statt Array, da die Operationen werden hauptsächlich & Einfügen aufzublicken.

Answer 4

auf Antwort der Shelvin Ausarbeiten von HashSet mit

HashSet<char> chars = new HashSet<char>(); 

//User input 
Console.WriteLine("Please Enter 5 Letters only: "); 
for (int i = 0; i < 5;) 
{ 
    char c = Convert.ToChar(Console.ReadLine()); 
    if(!("abcdefghijklmnopqrstuvwxyz".Contains(c.ToString().ToLower()))) 
    { 
      Console.WriteLine("Please enter an alphabet"); 
      continue; 
    } 
    else if (!chars.Contains(c)) 
    { 
      chars.Add(c); 
      i++; 
    } 
    else 
    { 
      Console.WriteLine("Duplicate value please try again"); 
      continue; 
    } 
} 
//display 
Console.WriteLine("You have entered the following inputs: "); 
foreach(char c in chars) 
    Console.WriteLine(c.ToString()); 

Console.Read(); 

Answer 5

Keep it simple, und obwohl ein HashSet schön semantisch ist, ist es nicht für 5 Elemente benötigt (es ist tatsächlich langsamer als ein List in diesem Fall). Schlimmer noch, es erfordert eine parallele Struktur, um die Zeichen zu verfolgen (vorausgesetzt, Sie kümmern sich um die Reihenfolge).

eindeutig keine dieser Überlegungen wichtig für so ein kleines Beispiel, aber es ist gut, sie Front zu lernen und nicht springt immer Big-O-Notation, wenn tatsächlich die Leistung und Speicherverbrauch gemessen sollte Ihr Führer für die meisten praktisch Anwendungen.

Stattdessen können Sie einfach tun: -

List<char> chars = new List<char>(5); 
while (chars.Count < 5) 
{ 
    char c = Console.ReadKey().KeyChar; 
    if (!char.IsLetter(c)) continue; 
    if (chars.Contains(char)) continue; 
    chars.Add(char); 
} 

plus unabhängig von Fehlermeldungen Sie hinzufügen möchten.