始终返回相同向量的函数

Question

我正在尝试编写一个返回向量的实用函数.返回的向量将始终具有相同的条目.我用它来过滤enum(Directions),以便客户端可以获得所需的子集.

以下是我希望如何解决问题的示例:

std::vector<Directions> horizontalDirections()
{
    static std::vector<Directions> toReturn;
    if (toReturn.size() == 0)
    {
        toReturn.push_back(Left);
        toReturn.push_back(Right);
    } 
    return toReturn;
}

这是正确的方法吗？

Answer 1

你这样做的方式.但我会返回一个const引用,以便在不需要时可以避免副本:

const std::vector<Directions>& horizontalDirections();

此外,如果您使用的是C++ 11,则您的实现可以缩写为:

const std::vector<Directions>& horizontalDirections()
{
    static std::vector<Directions> toReturn({Left, Right});
    return toReturn;
}

如果使用C++ 11,您可以更进一步,并将您声明horizontalDirections为全局const vector而不是函数:

const std::vector<Directions> horizontalDirections({Left, Right});

Answer 2

这基本上是正确的想法.我将通过引用返回以避免复制向量.并且您可以使用C++ 11初始化列表样式使身体更清晰:

const std::vector<Directions>& horizontalDirections() {
    static const std::vector<Directions> toReturn = {
        Left,
        Right
    };
    return toReturn;
}