Problema.

[sbcl]

Bueno, ayer empece con la idea de armar un simulador del Cubo magico de Rubik, e hice el set basico de clases.

Main.c:

Código c:

#include "classes.c"
 
int main()
{
    cube * test_cube;
}
 

classes.h:

Código c:

enum {RED, BLUE, YELLOW, GREEN, BLACK, WHITE};
class face{
      public:
      int colors[9];
      void fill_face_with_color(int);
      void set_right_column_colors(int*);
      face (int);
      };
 
class cube{
      public:
      face * faces[6];
      void initialize_cube();
      cube ();
      void display_cube_attributes();
      };
 

classes.c:

Código c:

#include "classes.h"
 
void face::fill_face_with_color(int color){
           int x;
           for (x = 0;x < 9; x++){
               colors[x] = color;
               cout << colors[x] << "  ";
               }
           for (x = 0;x < 9;x++){
               cout << colors[x] << "  ";}
           cout << endl; 
           }
           
face::face (int color){
           int x;
           for (x = 0;x < 9; x++){
               colors[x] = color;
               }
           }
                          
cube::cube (){
           int x;
           for (x = 0;x < 6;x++){
               face test_face(x); // Aca probe tambien face * test_face(x), face                  *test_face = new face(x), etc.
               cube::faces[x] = &test_face; 
               }
           }
 

Basicamente lo que quiero, es hacer un constructor de la clase Cube, que cuando se cree, llene su array de Faces(caras del cubo mágico), con todos los colores de cada cara iguales.

en resumen, seria para crear un nuevo cubo con todas las caras inicializadas, como si el cubo fuera nuevo.
Pero al tratar el método actual, la aplicación entra en un crash, o si nó, no asigna bien los integers en el array colores[].

Alguien es capaz de decirme que estoy haciendo mal? No le encuentro solución, es una pavada y sin embargo no me doy cuenta que hago mal.
Gracias.

[sbcl]

Disculpen, lo solucioné ya.
no termino de entender por que se solucionó asi, pero funcionó.
el "face test_face(x)" lo cambie por "face * test_face = new face(x)", y por lo visto agarró.
Disculpen la molestia, es que me estaba volviendo loco.

[estebanf93]

Iniciado por sbcl

Disculpen, lo solucioné ya.
no termino de entender por que se solucionó asi, pero funcionó.
el "face test_face(x)" lo cambie por "face * test_face = new face(x)", y por lo visto agarró.
Disculpen la molestia, es que me estaba volviendo loco.

Claro, porque usaste una variable puntera a face, lo que implica que debes reservar memoria (con new) para la variable y luego liberarla como es costumbre de hacer con delete

[alexg88]

Aunque ya has encontrado la solución, que efectivamente era esa, tienes que entender la razón por la cual funciona.

En lo que tenías antes:

Código C++:

cube::cube (){
           int x;
           for (x = 0;x < 6;x++){
               face test_face(x); 
               cube::faces[x] = &test_face; 
               }
           }
 

Al crear la variable test_face de manera no dinámica y luego asignarle la dirección de memoria de esa variable en la siguiente línea a cube::faces[x], lo que estás haciendo es que en todas las vueltas del bucle le estás asignando la misma dirección de memoria y,al final, todas las caras del cubo van a apuntar a la misma cara. Si lo vieses con el depurador, te darías cuenta que el valor del array colores es 5 (el del último valor del bucle).

Al utilizar memoria dinámica esto se soluciona porque ahora sí estás asignando diferentes objetos a cada posición del array faces.

A lo mejor ya te has dado cuenta, pero así viene bien si alguien entra y lo ve y no sabe porque es.

Saludos