error con array

[quilmes_05]

Este es mi programa en c++,el cual ingreso nombre y se guargan en un arrray y legos recorro ese array y mustro los nombres,no entiendo porque funciona mal.
cuando ingreso el tercer nombre me tira erro y no logro comprender por que.tambien no entiedno que sigifica el signo *.espero que me explique.gracias.

#include <iostream>
#include <string.h>
using namespace std;

int main ()
{
char *nombres[]={};
char nombre[8];
char s='s';
int n=0;
while(s!='n' && s=='s')
{
cout << "Nombre: ";cin >> nombre;
cout << "Deseas seguir: ";cin >> s;
nombres[n]=nombre;
n++;
}
cout <<"***liste de nombre*** \n\n";
for(int c=0;c<n;c++)
{
cout<< nombres[c];
}
}

[CapacProg]

Hola, primero el * define un puntero en C++, para saber más de ello aquí. En esencia un puntero guarda la dirección de memoria de una variable que es de un tipo de dato (para su implementación es de tipo char).

Hay un error en su código, la variable nombre tiene una dirección de memoria fija y este va cambiando de valor mientras c='s', lo malo está en que si nombre="abc", entonces nombres[0]="abc" en la primera iteración, luego si cambia a nombre="cde" y hace que nombres[1]=nombre, esto hará que nombres[0]="cde", puesto que nombres[0] y nombres[1] apuntan a la misma dirección de memoria.

El error que le tira es por falta de memoria en su vector de cadenas;

[sam90]

Para empezar, asi estas definiendo un doble puntero

char *nombres[]={};

Puntero o doble puntero, solo ocupa 32 bits (o 64) lo suficiente para guardar la direccion de memoria a donde apunta. La memoria en asi la tienes que reservar de otra forma. Hay dos metodos. Usando memoria statica o usando memoria dinamica.

Para memoria estatica simplementes defines asi tu variable:

char nombres[100][100];

Eso puede ser visto como 100 cadenas de 100 caracteres.

Para usar memoria dinamica tienes que usar el new...te recomiendo empezar con el otro metodo antes.


Luego cuando haces esta asignacion nombres[n]=nombre; solo estas asinando un puntero, asi que todos los nombres apuntan al mismo lugar, asi que en cada iteracion estarias pisando todos los nombres.

Yo eliminaria esa varianle temporaria y pasaria al cin directamente tu variable:

cout << "Nombre: ";cin >> nombres[n];

y para que sepas, si quieres copiar una cadena de caracteres a otras tienes que la funcion strcpy.

[quilmes_05]

gracias por sus respuestas,me quedo mas claro.ahora voy a leer mas sobre el tema.
Saludos