Vector в классе?

Question

[avion]

Почему не работает вектор в private? Почему нельзя заранее выделить память?

class Stack {
    public:
        bool is_empty() {
            return s_top == -1;
        }

        void push(int element) {
            s_top++;
            arr[s_top] = element;
        }

        int pop() {
            s_top--;
            return arr[s_top + 1];
        }
    private:
        int s_top = -1;
        vector<int> arr(10);
};

Comments

[Евгений Шатунов]

Данил , твой вопрос требует конкретизации.
Какого поведения ты ожидаешь от конструкции vector<int> arr(10)?
Какой стандарт C++ ты используешь для сборки этого кода?
Сколько элементов ты хочешь видеть в векторе после конструирования экземпляра Stack?

Answer 1

[WinPooh32]

Потому что нужно вызывать конструктор явно:

private:
    vector<int> arr = vector<int>(10);

Comments

[avion]

Хорошо, а почему в main работает просто vector arr(10);?

[Евгений Шатунов]

Данил, видимо, потому что ты собираешь код в стандарте C++03 или более раннем.
Что означает "работает просто vector arr(10)"? Работать код может самым различным способом, даже неправильным. Чтобы понять тебя, от тебя нужны детали и пояснения твоей терминологии.

[avion]

Евгений Шатунов, C++14

[Евгений Шатунов]

Данил, я готов поспорить что не C++14 точно.
Рекомендую дать ответы на все мои вопросы.

[avion]

[avion]

Под "работает", я имею ввиду выделение определенного количества память при использование arr(10), но когда я использую это же в классе, то не компилируется.

[avion]

[Roman]

Для начала

Почему не работает

non-static member function declaration
https://en.cppreference.com/w/cpp/language/member_...

Данил, про https://en.cppreference.com/w/cpp/language/data_members
и про конструктор и зачем он нужен слышал?

class Stack {
    public:
  	explicit Stack(size_t size = 10) : arr(size) {}
        bool is_empty() {
            return s_top == -1;
        }

        void push(int element) {
            s_top++;
            arr[s_top] = element;
        }

        int pop() {
            s_top--;
            return arr[s_top + 1];
        }
    private:
        int s_top = -1;
        vector<int> arr;
};

public: 
  inline explicit Stack(size_t size)
  : s_top{-1}
  , arr{std::vector<int, std::allocator<int> >(size, std::allocator<int>())}
  {
  }

А если тебе нужен массив фиксированного размера, то чем обусловлен выбор для этого вектора ака динамического массива? Ведь его интерфейс у тебя нигде не задействован, ну за исключением доступа по индексу.

Почему не https://en.cppreference.com/w/cpp/container/array ???

class Stack {
    public:
        bool is_empty() {
            return s_top == -1;
        }

        void push(int element) {
            s_top++;
            arr[s_top] = element;
        }

        int pop() {
            s_top--;
            return arr[s_top + 1];
        }
    private:
        int s_top = -1;
        array<int, 10> arr;
};

вот посмотри https://en.cppreference.com/w/cpp/container/stack/stack

[Евгений Шатунов]

Данил, ты все еще никак не ответил на мои вопросы.
Даже в примере WinPooh32 происходит инициализация вектора. Важно понимать, что именно ты хочешь получить от std::vector<int>{10}: вектор с одним элементом или с десятью?

Если у тебя имеются ошибки компиляции, тебе стоит первыми выписать именно их. Желательно весть твой код оформить так, чтобы в комментариях к строчкам было видно, какие у тебя на этих строчках ошибки.

Пункт 3.8 правил Тостера запрещает использовать изображения для демонстрации кода. Используй тег <code> для демонстрации своего кода.