Почему не пустой вектор?

Question

Мне интересно, почему это не признает, что вектор пуст и снабжен вектором pStack с двойным запуском? Объект программы просто просто поставить вектор стека с 50 в первый раз, когда он пуст. Затем, как только он будет отправлен с начальной суммой, он должен вычесть с помощью пользовательского ввода (ставка). Тогда, так как его вектор переносит старую сумму во второй раз, поэтому ее можно вычесть с помощью пользовательского ввода ставки снова.

#include <iostream> 
#include <string> 
#include <vector> 

using namespace std; 


class Bet 
{ 
public: 
    void set_stack(double n); 
    void set_bet(double n); 
    double analyze(); 
    double get_stack(); 
    double get_bet(); 

private: 
    double pCount; 
    double bet; 
}; 

double Bet::analyze() 
{ 
    double p = pCount; 
    double b = bet; 
    double z = p - b; 
    return z; 
} 

void Bet::set_bet(double n) 
{ 
    bet = z; 
} 

double Bet::get_bet() 
{ 
    return bet; 
} 

void Bet::set_stack(double n) 
{ 
    pCount = n; 
} 

double Bet::get_stack() 
{ 
    return pCount; 
} 


double start = 50; // Start needs to go inside of pStack 
double bbet; 
vector<double> pStack(1); 


int main() 
{ 
    bool con = true; 
    while(con){ 
     if(pStack.empty()){ 
     pStack.push_back(start); // pStack should be supplied with start when empty 
     } 

     double adjst = pStack[0]; 

     Bet rr; 
     rr.set_stack(adjst); 
     pStack.clear(); 
     cout << "Enter min 1 Bet: "; 
     cin >> bbet; 
     rr.set_bet(bbet); 

     double aStack = rr.analyze(); 
     pStack.push_back(aStack); 
     rr.set_stack(aStack); 
     double newStack = rr.get_stack(); 

     cout << "Stack: " << newStack << endl; 
     cout << "Bet: " << bbet << endl; 
    } 

    system("pause"); 
    return 0; 
} 

Answer 1

Вы используете конструктор заполнения. Эта строка инициализирует вектор для 1 элемента по умолчанию.

vector<double> pStack(1); 

Если вы хотите, чтобы вектор начинался с пустого, вы должны просто использовать конструктор по умолчанию.

vector<double> pStack; 

Answer 2

vector<double> pStack(1); 

Вы инициализирует свой вектор с его имея начальный размер 1, поэтому ваш pStack.empty() проверка возвращает false.

Сделайте это вместо этого, чтобы его исходное состояние было пустым.

vector<double> pStack; 

Кроме того, удалите empty() чек и поднять ваш push_back за пределами цикла.

bool con = true; 
pStack.push_back(start); 

while(con){ 
    ... 

Возможно, вы также захотите пересмотреть свое использование глобальных переменных. Насколько я вижу, вы можете просто положить start, bbet и pStack внутри main().

Answer 3

Когда вы определяете вектор pStack, вы используете конструктор, который принимает целое число, которое представляет количество элементов для выделения.

vector<double> pStack(1); 

В результате ваш вектор будет иметь 1 инициализированный по умолчанию элемент.

Чтобы создать пустой вектор это сделать:

vector<double> pStack;