Hay varias fallas en el programa:
1.- Estás desbordando el búfer:
gets(&vector[i][30]);
La segunda dimensión solo admite indice de 0
a 29
. Lo que debes hacer es pasar la dirección de memoria del primer elemento de cada array. Ten en cuenta que un array bidimensional (o matriz) se conforma por arrays unidimensional.
Solución:
gets(vector[i]);
La expresión vector[i]
devuelve la dirección base de cada array que conforme la matriz.
2.- No estás detectando la longitud de cada string:
for(int i=0 ; i<tam ; i++)
{
for(int j=0 ; j<30 ; j++)
{
printf("%c",v[i][j]);
}
printf("\n");
}
Como verás, el bucle anidado recorre todo el array pero el problema es que puede que el array no ocupe todas las posiciones, así que posiblemente estarías imprimiendo caracteres extraños.
Podrías obtener la longitud de cada array por medio de la función strlen
:
for(int i=0 ; i<tam ; i++)
{
for(int j=0 ; j < strlen(v[i]); j++)
{
printf("%c",v[i][j]);
}
printf("\n");
}
De igual manera, no hace falta el bucle anidado (así evitamos usar la función strlen
):
for(int i=0 ; i < tam ; i++)
printf("%s\n", v[i]);
3.- Este código no tiene sentido:
v[tam][30];
No hace nada, deberías eliminarlo.
Siguiendo con las correcciones, el código quedaría así:
int main()
{
int dim;
printf("Ingrese el numero de elementos que tendra el vector: \n");
scanf("%d",&dim);
char vector[dim][30];
for(int i=0 ; i<dim; i++)
{
fflush(stdin);
printf("Para la posicion %d ingrese el valor: \n",i+1);
gets(vector[i]);
}
printf("\n\n");
Imprimir(vector,dim);
return 0;
}
void Imprimir(char v[][30],int tam)
{
for(int i=0 ; i < tam ; i++)
printf("%s\n", v[i]);
}
Observaciones:
1.- No uses la función gets para pedir cadenas. Es insegura, ya que puede provocar un desbordamiento de búfer. Su reemplazo es fgets:
fgets(vector[i], 30, stdin);
2.- No uses VLA:
char vector[dim][30];
No se suele usar los VLA por problemas de portabilidad, no todos los compiladores lo admiten y además es propenso a que ocurra desbordamiento de pila.
Para estar mas seguro, deberías usar una matriz dinámica.
3.- La función fflush no limpia el búfer stdin
, sino el stdout
:
fflush(stdin);
Por mas que funcione en algunas plataformas, no lo deberías usar. Como dice el viejo refrán: Más vale prevenir que lamentar.
En este hilo puedes ver como se limpia el búfer stdin
de una manera estándar.