如何修复 c# 编译器错误“并非所有代码路径都返回值”Word.GetLetters

Question

并不是所有的路径都返回一个字符串值来返回我试图让所有的东西都返回一个字符串值，但仍然无法弄清楚。 我尝试了很多东西，但无法修复错误。 我尝试了 result.null，但仍然出现错误。 有人可以帮我吗，我不知道如何解决这个错误，我已经工作了一段时间试图解决这个错误，但还是想不通。

public letters GetLetters()
{
   var result = new letters();
   var Words = new List<string> {"start"};
   if (Words.Count == 0)
   {
      return null;
   }
   foreach (var word in Words)
   {
      if (word == null)
      {
         return null;
       }
       else
       {
          if(word.Length == 5)
          {
              string YourWord = word;
              char[] letters = YourWord.ToCharArray();
              char a = letters[0];
              char b = letters[1];
              char c = letters[2];
              char d = letters[3];
              char e = letters[4];
              result.First = a.ToString();
              result.Second = b.ToString();
              result.Third = c.ToString();
              result.Fourth = d.ToString();
              result.Fifth = e.ToString();
              return result;
            }
            else
            {
               return null;
            }
     }
  }
}

Answer 1

最简单的解决方案是删除：

if (Words.Count == 0)
{
    return null;
}

然后在循环之后：

return null;

这样，如果Words为空，它将有效地跳过循环，然后return null; . 因此无需专门检查Count是否为 0。此外，您还让编译器的工作变得更轻松 - 它可以看到每个代码路径都有一个return 。

话虽这么说-我会花一点时间测试代码，因为您的代码会忽略除Words中第一项以外的所有内容。 这很奇怪。

您的letters类型将字符存储为字符串也很奇怪。

Answer 2

我不确定你到底想做什么，但假设你有Letters class 作为：

    public class Letters
    {
        public string? First { get; set; }
        public string? Second { get; set; }
        public string? Third { get; set; }
        public string? Fourth { get; set; }
        public string? Fifth { get; set; }
    }

然后避免深度嵌套 `if/else`` 语句：

    public static Letters GetLetters()
    {
        var result = new Letters();
        var Words = new List<string> { "start" };
        if (Words.Count == 0)
        {
            return null;
        }

        foreach (var word in Words)
        {
            if (word == null)
            {
                return null;
            }
            if (word.Length == 5)
            {
                string YourWord = word;
                char[] letters = YourWord.ToCharArray();
                char a = letters[0];
                char b = letters[1];
                char c = letters[2];
                char d = letters[3];
                char e = letters[4];
                result.First = a.ToString();
                result.Second = b.ToString();
                result.Third = c.ToString();
                result.Fourth = d.ToString();
                result.Fifth = e.ToString();

                return result;
            }
        }

        return null;
    }