配列内に同じ番号がありません

Question

配列に値を受け入れるプログラムを作っていますが、同じ値を2回入力しようとすると、値が一意であれば拒否されます。

using System; 

public class Program 
{ 
    public static void Main() 
    { 

     char[] charray = new char[7]; 

     Console.WriteLine("Enter 7 unique alphabetic characters: "); 

     for (int i = 0; i < charray.Length; i++) 
     { 
      charray[i] = Convert.ToChar(Console.ReadLine()); 

      for (int j = 0; j < charray.Length; j++) 
      { 
       if (charray[i] == charray[j]) 
       { 
        Console.WriteLine("Please enter a unique alphabetic character."); 
       } 
      } 
     } 
     Console.WriteLine(charray); 
    } 
} 

誰かが間違っていることを教えてもらえますか？

Answer 1

あなたはあなたの配列の要素にあなたが割り当てたものとあなたの配列のすべての要素を比較していますので、もちろん、あなたは今入力した項目で常に複製を見つけることができます。

あなたが実際にしたいことは次のとおりです。

void Main() 
{ 
    char[] charray = new char[7]; 

    Console.WriteLine("Enter 7 unique alphabetic characters: "); 

    for (int i = 0; i < charray.Length; i++) 
    { 
     var x = Convert.ToChar(Console.ReadLine()); 
     if (charray.Contains(x)) 
     { 
      Console.WriteLine("Please enter a unique alphabetic character."); 
      i--; 
     } 
     else 
     { 
      charray[i] = x; 
     } 
    } 
    Console.WriteLine(charray); 
} 

Answer 2

このアプローチは、特別に、非常に遅いところであなたは、アレイを拡張する場合、7は結構ですが、最適化のレベルが非常に悪く、あなたが見たいかもしれませんハッシュマップのコンセプト。そして、他の人が言ったように、あなたが最初にあなたの配列の中に入力を挿入するので、forはあなたの入力を常に見つけ出し、重複して返すので、挿入操作の前に重複をチェックしたいかもしれません。私は次の実装を提案したいと思います

Answer 3

：

char[] charArray = new char[7]; 
Console.WriteLine("Enter {0} unique alphabetic characters: ", charArray.Length); 

int i = 0; 
while (i < charArray.Length) 
{ 
    char inputChar = Console.ReadKey().KeyChar; 
    Console.WriteLine(); 

    if (charArray.Contains(inputChar)) 
    { 
     Console.WriteLine("Please enter a unique alphabetic character."); 
    } 
    else 
    { 
     charArray[i] = inputChar; 
     ++i; 
    } 
} 

Console.WriteLine(charArray);