I tried to assign an array of pointer to nullptr.
class ToyBox
{
private:
Toy *toyBox[5];
int numberOfItems;
public:
ToyBox()
{
this->numberOfItems = 0;
this->toyBox = {}
}
}
An error throw at this in this->toyBox:
expression must be a modifiable lvalueC/C++(137)
Any suggestion to corrected?
You can only Initialize arrays in that way: Assign a single value to array. But in the constructor you could/must use Member Initialize List:
class ToyBox
{
private:
Toy *toyBox[5];
int numberOfItems;
public:
ToyBox() :
toyBox{nullptr}
, numberOfItems(0)
{
}
};
With C++, It's better to use std::array instead of raw C-Array:
related: CppCoreGuidlines: ES.27
class ToyBox
{
private:
std::array<Toy*, 5> toyBox;
int numberOfItems;
public:
ToyBox() :
toyBox({nullptr})
, numberOfItems(0)
{
}
};
Or (I think) better:
ToyBox() : numberOfItems(0)
{
std::fill(toyBox.begin(), toyBox.end(), nullptr);
}
ToyBox tracks the number of items, std::vector might be a better choice than std::array. In fact, an even better approach might be using ToyBox = std::vector<Toy*>.std::array as an improvement, then mention std::vector as the next step up.)
this->toyBox = {}What do you expect it to do? Also,Toy *toyBox[5];is not a pointer to an array, but array of pointers, is that intended?nullptrto the pointers in an array".