Функция, которая всегда возвращает тот же вектор

Question

Я пытаюсь написать функцию полезности, которая вернет вектор. Возвращаемый вектор всегда будет иметь одни и те же записи. Я использую его для фильтрации перечисления (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; 
} 

Answer 3

я бы не сделать это как функцию на всех, но как постоянной, как это в глобальном масштабе:

const std::vector<Direction> horizentalDirections = {Left, Right}; 

вот что константы созданы для.