Arduino C++ Declare array of objects with parameters in constructor [duplicate] - c++

Question

This question already has answers here:
Object array initialization without default constructor
(14 answers)
Closed 3 years ago.
I only want to declare an array of objects of class control with size of 5. The contents of the specific objects will be filled afterwards.
class control {
public:
control(char* controlName) {
name = controlName;
}
private:
char* name;
};
void setup() {
control humidityControl("humidityControl");
// Problem: Declare an array controlArray with the size of 5 and the name "controlArray"
control controlArray[5]("controlArray"); // Error: no matching function for call to 'control::control()'
control controlArray("controlArray")[5]; // Error: expected ',' or ';' before '[' token
}
void loop() {
}
I'm using C++ on my Arduino. I'd appreciate every tipp how to fix this. Thanks!

Answer 1

Your problem is that you're mixing the declaration of an array (the square brackets) with a call to the constructor. This SO answer clarifies it: https://stackoverflow.com/a/1598409/2881667.
What you want to do is this:
void setup() {
control controlArray[5] = {"controlArray", "controlArray", "controlArray", "controlArray", "controlArray"};
}
Notice how you have to include the argument to the constructor 5 times. To avoid that, you have two options:
Use a default constructor:
control(char* controlName = "controlArray");
Might work for you, but only if you plan on initializing all arrays with the same value.
If using gcc, you can use its extension to initialize it like this:
control controlArray[5] = {[0 ... 4] = "controlArray"};
But this will only work with gcc, so not so good for portability.
Also, unrelated to your question - I think you probably meant to copy the string, and not store the pointer in your class, so you should use something like strncpy in the constructor, instead of just assigning to name, and you'll have to preallocate storage for name in that case (either declare it as a char array, like char name[255];, allocate space for it in the constructor, or use std::string to make it all easier).