1

Tengo una consulta: En lenguaje C, estoy tratando de pasar una matriz(array bidimensional) que contiene strings como parámetro a una función la cual es void y manda a imprimir lo que tiene dicha matriz.

La cuestión es que lleno la matriz y al momento de querer imprimir no me funciona (o sea no me aparece la información que llene).

Aquí les dejo el código:

#include <stdio.h>
#include <stdlib.h>

//Prototipo de la funcion
void Imprimir(char [][30],int);

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][30]);
    }
    printf("\n\n");
    Imprimir(vector,dim);
    return 0;
}

//FUNCION IMPRIMIR 
void Imprimir(char v[][30],int tam)
{
    v[tam][30];
    for(int i=0 ; i<tam ; i++)
    {
        for(int j=0 ; j<30 ; j++)
        {

            printf("%c",v[i][j]);
        }
        printf("\n");
    }

}

1 respuesta 1

0

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.

2
  • Muchas gracias por tu ayuda muy valiosa ! me hice lio al pepe. Gracias !
    – Juan J
    Commented el 27 oct. 2020 a las 20:53
  • @JuanJ Puedes agradecer aceptando la respuesta que resolvio tu problema.
    – MrDave1999
    Commented el 27 oct. 2020 a las 20:57

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.