”并非所有代码路径都会返回值”当返回枚举类型时

Question

我有枚举列表和方法，但收到错误：“并非所有代码路径都返回值”
我的方法有什么问题吗？我确信我总是返回 STANY 类型:/

感谢帮助:)

     private enum STANY { PATROL, CHAT, EAT, SEARCH, DIE };

     private STANY giveState(int id, List<Ludek> gracze, List<int> plansza)
    {
        // Sprawdz czy gracz stoi na polu z jedzeniem i nie ma 2000 jednostek jedzenia
        bool onTheFood = false;
        onTheFood = CzyPoleZjedzeniem(id, gracze, plansza, onTheFood);

        if (onTheFood && (gracze[id].IloscJedzenia < startFood / 2))
            return STANY.EAT;

        // Sprawdz czy gracz nie stoi na polu z innym graczem
        bool allKnowledge = true;
        allKnowledge = CzyPoleZInnymGraczem(id, gracze, allKnowledge);

        if (!allKnowledge)
            return STANY.CHAT;

        // Jesli ma ponad i rowna ilosc jedzenia patroluj
        if (gracze[id].IloscJedzenia >= startFood / 2)
            return STANY.PATROL;

        // Jesli ma mniej niz polowe jedzenia szukaj jedzenia
        if (gracze[id].IloscJedzenia > 0 && gracze[id].IloscJedzenia < startFood / 2)
            return STANY.SEARCH;

        // Jesli nie ma jedzenia umieraj
        if (gracze[id].IloscJedzenia <= 0)
            return STANY.DIE;
    }

Answer 1

如果这些条件都不满足，则不会返回。您需要使用 if...elseif...else
或者在所有 if 语句之后有一个 return，如果没有返回任何内容（没有满足任何 if 条件），则将返回一个值。

Answer 2

也许您确定总是会给出返回类型，但编译器不会[想象所有的 if 条件都失败 - 即：当您的代码由外部程序执行时变量被更改。那么会发生什么？]

只需在底部粘贴一个 return 作为“默认”值即可。如果你愿意的话，你也可以抛出异常，因为永远不应该到达底部，对吧？

Answer 3

这与您返回 ENUM 的事实无关，而与编译器检测到您从未使用任何值调用 RETURN 的情况有关。

您应该在函数底部添加一个 return 语句来返回一些默认值。

Answer 4

这是因为每个 return 语句前面都有一个 if 语句。因此，如果所有 if 语句都为 false，则不会返回任何值。如果不可能所有 if 语句都为 false，请删除 if (gracze[id].IloscJedzenia <= 0)，因为它是多余的。如果不是，请在末尾添加 return null; 或其他内容，或者抛出错误。

Answer 5

替换

// Jesli nie ma jedzenia umieraj
if (gracze[id].IloscJedzenia <= 0)
    return STANY.DIE;

为

// Jesli nie ma jedzenia umieraj
// (gracze[id].IloscJedzenia <= 0)
return STANY.DIE;

：将注释中的多余行保留为文档。

Answer 6

编译器的静态分析（即旨在检查此条件的代码）并不总是 100% 准确。但是，当它不是 100% 准确时，编译器往往会错误地给出误报（即，表示它不会从所有路径返回），而不是误报（即，不 说它不会从所有路径返回）。

因此在底部添加一个 return，使用一系列“if .. else if ... else”语句来代替，或者抛出异常来表明条件“不可能”。