Почему мой конструктор классов не инициализирует его переменные-члены?

Question

Простите меня, если информация немного отсутствует, поскольку я стараюсь следовать рекомендуемым «Рекомендациям».

Я создал класс под названием «Item» для хранения информации элемента и определили мой конструктор следующим образом:

Item::Item(std::string number, std::string name, int quantity, double price) { 
    itemNumber = number; 
    itemName = name; 
    quantity = quantity; 
    unitPrice = price; 
} 

и я инициализировать его в своей основной функции, как так:

Item temp("string", "string", 0, 0); 

но когда я печатал значения элемента, напечатано только itemNumber и itemName. Все, что напечатано, было случайным мусором. Тогда я понял, что проблема заключается в количестве, и цена не получает инициализации. Почему это?

Answer 1

Попробуйте

this->quantity = quantity; 

или (ИМХО лучше):

Item::Item(const std::string& number, const std::string& name, int quantity_, double price) { 
                  //^
    // ... 
    quantity = quantity_; 
    // ... 
} 

или (ИМХО даже лучше) использовать список конструктор инициализатора:

Item::Item(const std::string& number, const std::string name&, int quantity, double price) 
: itemNumber(number), itemName(name), quantity(quantity), unitPrice(price) {} 

"Почему что?"

В настоящее время имя параметра изменяет имя переменной вашего члена в области тела конструктора.

---

Лично я, как правило, объявление классов нравится следующий

class Item { 
public: 
    Item(const std::string& number, const std::string& name, int quantity, double price) 
    : itemNumber_(number) 
    , itemName_(name) 
    , quantity_(quantity) 
    , unitPrice_(price) {} 
private: 
    std::string itemNumber_; 
    std::string itemName_; 
    int quantity_; 
    double unitPrice_ 
}; 

Answer 2

quantity = quantity; И здесь относятся к параметру, а не переменный член. Чтобы обратиться к переменной-члена внутри функции, которая имеет параметр с тем же именем, вы должны использовать этот указатель, например this->quantity = quantity; или же квалифицировать его с именем класса, например Item::quantity = quantity;

Также можно использовать список инициализации, чтобы избежать этого проблемы и, возможно, более эффективно инициализировать ваш класс.

Item::Item(const std::string& number, const std::string& name, int quantity, double price) 
: itemNumber(number) 
, itemName(name) 
, quantity(quantity) 
, price(price) 
{ 
} 

Я взял на себя смелость сделать ваши строковые параметры const ссылкой, чтобы избежать копирования.