Как создать несколько элементов с одним классом на C++?

Question

У меня есть класс мебели с:

Furniture.h:

#include <iostream> 
#include <string> 
using namespace std; 

class Furniture { 
public: 
    Furniture(); 
    ~Furniture(); 
    void setname(string name); 
    void setprice(double price); 
    double getprice(); 
    string getname(); 
    virtual void printSpecs(); 
private: 
    string name; 
    double price; 
protected: 
    static int NumberOfItems; 
    int Id; 

}; 

furniture.cpp:

#include "furniture.h" 

Furniture::Furniture() { 
} 
Furniture::~Furniture() { 
} 
void Furniture::setname(string name) { 
    this->name = name; 
} 
string Furniture::getname() 
{ 
    return this->name; 
} 
void Furniture::setprice(double price) { 
    this->price = price; 
} 
double Furniture::getprice() { 
    return this->price; 
} 
void Furniture::printSpecs() { 
    cout<<"Price: "<<this->price<<endl; 
    cout<<"Name: "<<this->name<<endl; 
} 

int main() { 
    Furniture *model = new Furniture(); 
    model->setname("FinalDestiny"); 
    model->setprice(149.99); 
    model->printSpecs(); 
    delete model; 
} 

Все работает отлично, но я хочу добавить несколько предметов мебели, с тем же классом и просто обновите NumberOfItems. Есть ли способ сделать это?

Кроме того, мой код в порядке? Я имею в виду, как я могу его улучшить? Я совершенно новичок в ООП, и я хотел бы изучить некоторые хорошие практики.

Спасибо.

Answer 1

Идея концептуально сломана. Вы не можете сделать это; вам действительно нужны разные объекты.

В качестве альтернативы, если вы действительно хотите иметь несколько идентичных элементов, вы можете создать один элемент и создать для него несколько указателей и сохранить счет для количества активных элементов. A shared_ptr делает это, например.

При этом ваш код не должен использовать указатели вообще, это общий анти-шаблон в коде C++. Кроме того, ваш код, вероятно, не должен иметь сеттеров, обеспечивает надлежащий конструктор вместо:

int main() { 
    Furniture model("FinalDestiny", 149.99); 
    model.printSpecs(); 
} 

Гораздо короче, проще и не Possiblity утечки памяти.

Answer 2

Просто добавьте NumberOfItems в конструктор и уменьшите его в деструкторе.

Answer 3

Чтобы следить за количеством элементов, вы можете обновить количество элементов в конструкторе:

Furniture::Furniture() { 
    Id = NumberOfItems++; 
} 

и декремента в деструкторе, если вы хотите:

Furniture::~Furniture() { 
    NumberOfItems--; 
} 

Чтобы получить доступ к пункт по Id, вам нужно иметь дополнительный класс менеджера или использовать карту:

std::map<int,Furniture*> items; 

, который можно передать в качестве параметра конструктору и обновить его там:

Furniture::Furniture(std::map& items) { 
    Id = NumberOfItems++; 
    items[Id] = this; 
} 

И снаружи, вы можете просто получить элементы с:

Furniture* f = items[3]; 

Answer 4

Храните экземпляры мебели в массиве или лучше в vector. Вы можете получить к ним доступ с помощью индекса или итератора. Поле NumberOfItems не относится к классу мебели, экземпляр мебели не должен знать о том, сколько предметов мебели есть в системе. Используйте метод size() от vector, чтобы получить количество предметов мебели.

Answer 5

Я хотел бы написать таким образом

#include <iostream> 
#include <string> 
using namespace std; 

class Furniture { 
public: 
    Furniture(string name = "", double price = 0) 
     : name(name), price(price), Id(NumberOfItems++) 
    {} 
    Furniture(const Furniture &f) 
     : name(f.getname()), price(f.getprice()), Id(NumberOfItems++) 
    {} 

    void setname(string name) { this->name = name; } 
    void setprice(double price) { this->price = price; } 
    double getprice() const { return price; } 
    string getname() const { return name; } 

    virtual void printSpecs() {} 

private: 
    string name; 
    double price; 

protected: 
    static int NumberOfItems; 
    int Id; 
}; 
int Furniture::NumberOfItems; 

int main_furniture(int, char **) 
{ 
    Furniture a("product 1", 100); 
    Furniture x(a), y(a), z(a); 
} 

Я встраиваемый только упростить. Что интересно вы должны быть конструктор копирования реализации и (OT) вы забыли сопзЬ на добытчика ...