Array de cadenas (punteros) solo guarda la última que introduzco

[windarin]

Hola a todos, tengo un problema detrás del que llevo días y no doy con la solución.
He creado un array de cadenas en C++, con las cadenas como punteros a caracter (no con ancho fijo).
Hago un bucle para que el usuario introduzca cada una de las cadenas, y he probado de varias maneras:
- cin >> array[i];
- cin.getline(array[i],10);
- con una variable intermedia y luego utilizando strcpy.
La única manera que me funciona, cuando muestro todos los valores resulta que ha introducido en todas las posiciones la última cadena que meto.
Por favor, si podéis echarme un cable os lo agradezco un montón.
¡Un saludo y gracias de antemano!
Aquí dejo el código simplificado (funciona, pero mal):

Código:

#include <iostream.h>
#include <string.h>
#include <fstream.h>

const int TAMANO=4;

main()
{
      char *familia[TAMANO];
      char cadena[10];
      for (int i=0; i<TAMANO; i++)
      {
              cout << "Introduce el nombre de la familia de la planta " << i+1 << " ";
              fflush(stdin);
              cin.getline(cadena,10);
              familia[i]=cadena;
              //strcmp(familia[i],cadena);
              //cin.getline(familia[i],10);
              //cin >> familia[i];
              cout << familia[i];
      }
      
      for (int k=0; k<TAMANO; k++)
      {
              cout << familia[k];
      }
      system("pause");
}

[AlvaroG]

El problema es que estás guardando "cadena" cada vez. getline irá escribiendo a cada posición, pero de la forma en la que estás asignando, en cada puntero del vector quedará copiado el puntero a la primera posición de "cadena".

Para que funcione, deberías hacer algo como esto luego del getline

Código C++:

familia[i] = new char[strlen(cadena)+1]
strcpy(familia[i], cadena);
 

Es decir, copiar la cadena explícitamente a la posición correspondiente en familia.

¡No olvides hacer el delete[] al terminar!


Saludos.

[Trovaz]

Recuerda que al usar punteros es necesario reservar espacio de memoria para lo que sea que vayas a escribir en ellos, por lo que por cada puntero que tienes debes hacer lo siguiente

Código c:

familia[i] = malloc((strlen(cadena)+1) * sizeof(char));
 

y ya despues puedes usar el strcpy para copiar el contenido de cadena en familia[i], por cierto no se si con cin.geline ya no es necesario borrar el contenido de cadena antes de leerla, pero por si las dudas podrias utilizar bzero para inicializar tu cadena en 0's antes de asignarle un valor.

Saludos++

[windarin]

Qué fallo, se me había olvidado por completo que tenía que reservar memoria :)

¡Muchísimas gracias por vuestra información, me ha funcionado perfectamente!

¡Un saludo!