ランダムに生成された色を値に割り当てる

Question

2ブロックのコードを結合する際に問題があります。 私は4つのランダムな色を生成するためにこのコードを持っています。

static Color[] colors = { Color.Red, Color.Green, Color.Yellow, Color.Purple}; // Escolher as cores disponiveis 
static Color GetRandomColor() 
{ 
    var random = new Random(); 
    return colors[random.Next(colors.Length)]; 
} 

私はコードが良いと思いますが、それが私に教えてください。

私はこのコードに関連付けする必要があります。

List<Utente> ListaUtente = new List<Utente>(); 
     ListaUtente.Add(new Utente(22, "Pedro", 555444333, "pedro@gmail.com")); 
     ListaUtente.Add(new Utente(22, "Lucas", 555444333, "lucas91@gmail.com")); 
     ListaUtente.Add(new Utente(22, "Rodrigo", 555444333, "rodrigo00@gmail.com")); 
     ListaUtente.Add(new Utente(22, "Gaspar", 555444333, "gaspar@gmail.com")); 
     ListaUtente.Add(new Utente(22, "Assis", 555444333, "assis@gmail.com")); 

は、私はこれらのランダムな色（紫、赤、黄、緑）の出力1にプログラムを必要とし、画面上に書きます。

のような何か：

Lucas - Green 
Rodrigo - Red 

は、どのように私は、これら2つのブロックをリンクすることができますか？私はリストと "Utente"クラスの属性に完全にアクセスできます。

Answer 1

ここであなたが探しているものがわかりません。クリス・ダナウェイとしても

foreach(Utente utente in ListaUtente) 
{ 
    Console.WriteLine($"{ GetRandomColor() } - { utente.Name }"); 
} 

：？あなたはこの枚この

ListaUtente.Add(new Utente(22, "Pedro", 555444333, "pedro@gmail.com", GetRandomColor())); 

その後

foreach(Utente utente in ListaUtente) 
{ 
    Console.WriteLine(utente.Name + " " + utente.Color); 
} 

Answer 2

を呼び出すことができますので、あなただけのUtenteクラスにColorプロパティを追加し、コンストラクタを更新することができますあなたがGetRandomColor（）を呼び出すたびにランダムの新しいインスタンスを作成するので、デフォルトのシード（システム時間）は（高速なマシンでは）同じであるため、同じ乱数が返される可能性が高いと指摘していますコールします。

したがって、より良い方法は、メソッドからランダムオブジェクトを抽出することです。 何かのように：

static Color[] colors = { Color.Red, Color.Green,Color.Yellow,Color.Purple}; 
static Random random = new Random(); 
static Color GetRandomColor() 
{ 
    return colors[random.Next(colors.Length)]; 
}