ランダムな名前ジェネレータを作成しようとしていて、割り当てられていないローカル変数を取得しようとしました

Question

誰かが間違っていると指摘できたら。

public string GenerateName(char gender) 
    { 
     string name; 
     Random rand = new Random(); 


     if (gender == 'm') 
     { 

      string[] maleNames = File.ReadAllLines(Directory.GetCurrentDirectory() + @"\ModifiedMalesNamesList.txt"); 
      string[] lastNames = File.ReadAllLines(Directory.GetCurrentDirectory() + @"\ModifiedLastNamesList.txt"); 
      name = maleNames[rand.Next(0, maleNames.Length - 1)] + " " + lastNames[rand.Next(0, lastNames.Length - 1)]; 

      return name; 
     } 
     else 
     { 

     } 
     return name; 
    } 

コンパイラは、割り当てられていないローカル変数を示します。あなたが値で初期化しない限り、あなたは文字列nameを使用することはできません

string name = string.Empty; 

：これに

string name; 

：このことから

：事前

Answer 1

で おかげでこれを編集してください。

詳細については、エラーを説明するには、文字列を宣言して値を返さないでください。 stringは参照型であるため、ヌル、空または任意の文字値のコレクションとして初期化する必要があります。それは「条件がを失敗する可能性がある場合」と、それはelse文に行くことを検出したため、コンパイラはあなたにエラーを教えてくれますので、この場合は

は、あなたのifとelse文に、else文に nameのない価値がありませんその後、 name変数を返します。

もしそうなら、if文に行くのではないなら、nameの値はどうなりますか？コンパイラはこれを検出するのに十分スマートです。あなたにもこれを行うことができます：

string name; 
    Random rand = new Random(); 
    if (gender == 'm') 
    { 
     string[] maleNames = File.ReadAllLines(Directory.GetCurrentDirectory() + @"\ModifiedMalesNamesList.txt"); 
     string[] lastNames = File.ReadAllLines(Directory.GetCurrentDirectory() + @"\ModifiedLastNamesList.txt"); 
     name = maleNames[rand.Next(0, maleNames.Length - 1)] + " " + lastNames[rand.Next(0, lastNames.Length - 1)]; 

     return name; 
    } 
    else 
    { 
     name = ""; 
    } 
    return name; 

をこのような場合には、あなたはすべての根拠をカバーし、それがエラーをトリガしないように、あなたは安全値で文字列nameを返すことができます。