Skip to main content
se añadieron 120 caracteres en el cuerpo
Origen Enlace
Trauma
  • 26.9k
  • 5
  • 42
  • 68

Fíjate en lo que haces en tu constructor:

Cluster::Cluster( int pin[] ) {
  Storm led[6] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] )
  };
  _led = led;
  ...

Creas un arreglo local a la función, y usas su dirección para inicializar tu variable de instancia _led.

¿ Que pasará cuando salgas del constructor, y el espacio de pila ocupado por led[] se reutilice para otras cosas ? ¿ Adonde apuntará _led entonces ?

EDITO

Cambia tu constructor a esto; espero que Arduino soporte la memoria dinámica de C++ (debería):

Cluster::Cluster( int pin[] ){
  prevmillis = 0;
  counter = 0;

  _led[0] = new Storm( pin[0] );
  _led[1] = new Storm( pin[1] );
  _led[2] = new Storm( pin[2] );
  _led[3] = new Storm( pin[3] );
  _led[4] = new Storm( pin[4] );
  _led[5] = new Storm( pin[5] );
}

Eso tendría que funcionar sin necesitar cambios adicionales.Y un cambio menor en tu clase:

class Cluster {
  ...
private:
  ...
  Storm *_led[6];

};

Fíjate en lo que haces en tu constructor:

Cluster::Cluster( int pin[] ) {
  Storm led[6] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] )
  };
  _led = led;
  ...

Creas un arreglo local a la función, y usas su dirección para inicializar tu variable de instancia _led.

¿ Que pasará cuando salgas del constructor, y el espacio de pila ocupado por led[] se reutilice para otras cosas ? ¿ Adonde apuntará _led entonces ?

EDITO

Cambia tu constructor a esto; espero que Arduino soporte la memoria dinámica de C++ (debería):

Cluster::Cluster( int pin[] ){
  prevmillis = 0;
  counter = 0;

  _led[0] = new Storm( pin[0] );
  _led[1] = new Storm( pin[1] );
  _led[2] = new Storm( pin[2] );
  _led[3] = new Storm( pin[3] );
  _led[4] = new Storm( pin[4] );
  _led[5] = new Storm( pin[5] );
}

Eso tendría que funcionar sin necesitar cambios adicionales.

Fíjate en lo que haces en tu constructor:

Cluster::Cluster( int pin[] ) {
  Storm led[6] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] )
  };
  _led = led;
  ...

Creas un arreglo local a la función, y usas su dirección para inicializar tu variable de instancia _led.

¿ Que pasará cuando salgas del constructor, y el espacio de pila ocupado por led[] se reutilice para otras cosas ? ¿ Adonde apuntará _led entonces ?

EDITO

Cambia tu constructor a esto; espero que Arduino soporte la memoria dinámica de C++ (debería):

Cluster::Cluster( int pin[] ){
  prevmillis = 0;
  counter = 0;

  _led[0] = new Storm( pin[0] );
  _led[1] = new Storm( pin[1] );
  _led[2] = new Storm( pin[2] );
  _led[3] = new Storm( pin[3] );
  _led[4] = new Storm( pin[4] );
  _led[5] = new Storm( pin[5] );
}

Y un cambio menor en tu clase:

class Cluster {
  ...
private:
  ...
  Storm *_led[6];

};

se eliminaron 301 caracteres en el cuerpo
Origen Enlace
Trauma
  • 26.9k
  • 5
  • 42
  • 68

Fíjate en lo que haces en tu constructor:

Cluster::Cluster( int pin[] ) {
  Storm led[6] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] )
  };
  _led = led;
  ...

Creas un arreglo local a la función, y usas su dirección para inicializar tu variable de instancia _led.

¿ Que pasará cuando salgas del constructor, y el espacio de pila ocupado por led[] se reutilice para otras cosas ? ¿ Adonde apuntará _led entonces ?

No tengo experiencia conEDITO

Cambia tu constructor a esto; espero que Arduino, no se si en ese entorno está disponible soporte la memoria dinámica a la manera tradicional, ni si soporta todo el lenguaje C++ ( ¿ no sería mejor inicializar las variables de instancia en la lista de inicializadores del constructor ?C++ (debería), así que la solución que te ofrezco es que cambies tu clase, y hagas:

class Cluster {
  ...
private:
  ...
  Storm _led[6]; // arreglo, NO puntero.
};

y cambies también tu constructor, para adecuarlo a lo anterior:

Cluster::Cluster( int pin[] ) {
  prevmillis = 0;
  counter = 0; 

  _led[]_led[0] = {
   new Storm( pin[0] ),;
  _led[1] = new Storm( pin[1] ),;
  _led[2] = new Storm( pin[2] ),;
  _led[3] = new Storm( pin[3] ),;
  _led[4] = new Storm( pin[4] ),;
  _led[5] = new Storm( pin[5] ),
  };
}

Eso tendría que funcionar sin necesitar cambios adicionales.

Fíjate en lo que haces en tu constructor:

Cluster::Cluster( int pin[] ) {
  Storm led[6] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] )
  };
  _led = led;
  ...

Creas un arreglo local a la función, y usas su dirección para inicializar tu variable de instancia _led.

¿ Que pasará cuando salgas del constructor, y el espacio de pila ocupado por led[] se reutilice para otras cosas ? ¿ Adonde apuntará _led entonces ?

No tengo experiencia con Arduino, no se si en ese entorno está disponible la memoria dinámica a la manera tradicional, ni si soporta todo el lenguaje C++ ( ¿ no sería mejor inicializar las variables de instancia en la lista de inicializadores del constructor ? ), así que la solución que te ofrezco es que cambies tu clase, y hagas:

class Cluster {
  ...
private:
  ...
  Storm _led[6]; // arreglo, NO puntero.
};

y cambies también tu constructor, para adecuarlo a lo anterior:

Cluster::Cluster( int pin[] ) {
  prevmillis = 0;
  counter = 0;
  _led[] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] ),
  };
}

Fíjate en lo que haces en tu constructor:

Cluster::Cluster( int pin[] ) {
  Storm led[6] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] )
  };
  _led = led;
  ...

Creas un arreglo local a la función, y usas su dirección para inicializar tu variable de instancia _led.

¿ Que pasará cuando salgas del constructor, y el espacio de pila ocupado por led[] se reutilice para otras cosas ? ¿ Adonde apuntará _led entonces ?

EDITO

Cambia tu constructor a esto; espero que Arduino soporte la memoria dinámica de C++ (debería):

Cluster::Cluster( int pin[] ){
  prevmillis = 0;
  counter = 0; 

  _led[0] = new Storm( pin[0] );
  _led[1] = new Storm( pin[1] );
  _led[2] = new Storm( pin[2] );
  _led[3] = new Storm( pin[3] );
  _led[4] = new Storm( pin[4] );
  _led[5] = new Storm( pin[5] );
}

Eso tendría que funcionar sin necesitar cambios adicionales.

se añadieron 109 caracteres en el cuerpo
Origen Enlace
Trauma
  • 26.9k
  • 5
  • 42
  • 68

Fíjate en lo que haces en tu constructor:

Cluster::Cluster( int pin[] ) {
  Storm led[6] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] )
  };
  _led = led;
  ...

Creas un arreglo local a la función, y usas su dirección para inicializar tu variable de instancia _led.

¿ Que pasará cuando salgas del constructor, y el espacio de pila ocupado por led[] se reutilice para otras cosas ? ¿ Adonde apuntará _led entonces ?

No tengo experiencia con Arduino, no se si en ese entorno está disponible la memoria dinámica a la manera tradicional, ni si soporta todo el lenguaje C++ ( ¿ no sería mejor inicializar las variables de instancia en la lista de inicializadores del constructor ? ), así que la solución que te ofrezco es que cambies tu clase, y hagas:

class Cluster {
  ...
private:
  ...
  Storm _led[6]; // arreglo, NO puntero.
};

y cambies también tu constructor, para adecuarlo a lo anterior:

Cluster::Cluster( int pin[] ) {
  prevmillis = 0;
  counter = 0;
  _led[] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] ),
  };
}

Fíjate en lo que haces en tu constructor:

Cluster::Cluster( int pin[] ) {
  Storm led[6] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] )
  };
  _led = led;
  ...

Creas un arreglo local a la función, y usas su dirección para inicializar tu variable de instancia _led.

¿ Que pasará cuando salgas del constructor, y el espacio de pila ocupado por led[] se reutilice para otras cosas ? ¿ Adonde apuntará _led entonces ?

No tengo experiencia con Arduino, no se si en ese entorno está disponible la memoria dinámica a la manera tradicional, ni si soporta todo el lenguaje C++, así que la solución que te ofrezco es que cambies tu clase, y hagas:

class Cluster {
  ...
private:
  ...
  Storm _led[6]; // arreglo, NO puntero.
};

y cambies también tu constructor, para adecuarlo a lo anterior:

Cluster::Cluster( int pin[] ) {
  prevmillis = 0;
  counter = 0;
  _led[] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] ),
  };
}

Fíjate en lo que haces en tu constructor:

Cluster::Cluster( int pin[] ) {
  Storm led[6] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] )
  };
  _led = led;
  ...

Creas un arreglo local a la función, y usas su dirección para inicializar tu variable de instancia _led.

¿ Que pasará cuando salgas del constructor, y el espacio de pila ocupado por led[] se reutilice para otras cosas ? ¿ Adonde apuntará _led entonces ?

No tengo experiencia con Arduino, no se si en ese entorno está disponible la memoria dinámica a la manera tradicional, ni si soporta todo el lenguaje C++ ( ¿ no sería mejor inicializar las variables de instancia en la lista de inicializadores del constructor ? ), así que la solución que te ofrezco es que cambies tu clase, y hagas:

class Cluster {
  ...
private:
  ...
  Storm _led[6]; // arreglo, NO puntero.
};

y cambies también tu constructor, para adecuarlo a lo anterior:

Cluster::Cluster( int pin[] ) {
  prevmillis = 0;
  counter = 0;
  _led[] = {
    Storm( pin[0] ),
    Storm( pin[1] ),
    Storm( pin[2] ),
    Storm( pin[3] ),
    Storm( pin[4] ),
    Storm( pin[5] ),
  };
}
Origen Enlace
Trauma
  • 26.9k
  • 5
  • 42
  • 68
Loading