¿Por qué no se muestra los datos del array bidimensional?

Question

Estoy teniendo un problema raro en mi código, hice esto repetidas veces pero ahora no se que pasa y no me deja imprimir el array.

El código es el siguiente

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <string.h>
#define MESES 12
    
void cargarNombreYApellido (char nombre[100]);
void crearContrasenia (char nombre[100], char password[200]);
int elegirCantidadDeEmpresas(int cantEmpresas);
void cargarMatrizAleatoria(int cantEmpresas, float empresas[][MESES]);
void mostrarVector(int cantEmpresas, float empresas[][MESES]);
    
//#############################################################
int main()
{
    srand(time(NULL));
     
    int cantEmpresas = elegirCantidadDeEmpresas(cantEmpresas);
    int empresas[cantEmpresas][MESES];
    cargarMatrizAleatoria(cantEmpresas, empresas[cantEmpresas][MESES]);
    return 0;
}
//##############################################################
    
    
   
/*Elegir con cuantas empresas quiere trabajar el usuario, menor a 3*/
    
int elegirCantidadDeEmpresas(int cantEmpresas)
{
    printf("\n\n\nCon cuantas empresa deseas trabajar? (Menos de tres) \n");
    scanf("%d", &cantEmpresas);
    return cantEmpresas;
    //agregar problema al añadir 3 empresas o mas
}
    
void cargarMatrizAleatoria(int cantEmpresas, float empresas[][MESES])
{
    int i,k;
    
    for (i=0; i<cantEmpresas; i++)
    {
        for (k=0; k<MESES; k++)
        {
            empresas[i][k]=rand()%11;
        }
    }
}
    
void mostrarVector(int cantEmpresas, float empresas[][MESES])
{
    int i;
    int k;
    for (i=0; i<cantEmpresas; i++)
    {
        for (k=0; k<MESES; k++)
        {
            printf("%f  - ", empresas[i][k]);
        }
        printf("\n");
    }
}

Si yo borro el printf donde intento imprimir el array, la consola retorna en valor 0, pero claramente no me muestra mi array. Cuando yo pongo:

printf("%f  - ", empresas[i][k]);

La consola no solo que no imprime nada, si no que retorna en un valor muy extraño.

Si te esta imprimiendo valores sin sentido, entoces estas tocando memoria que no es tuya, revisa los tamaños — Pablochaches, Aug 04 '20 at 22:01
@HatoriHanso al momento de intentar pasar esto: `empresas[cantEmpresas][MESES]` ya estás desbordando el búfer, porque las columnas de un array 2D va de 0 a 11 (en este caso claro) — MrDave1999, Aug 04 '20 at 23:08
Hola dave, lamentablemente no, esto me esta carcomiendo la cabeza, no entiendo que esta mal en el codigo como para que este no pueda imprimir los numeros — Hatori Hanso, Aug 05 '20 at 00:58

MrDave1999 · Accepted Answer · 2020-08-05T02:53:46.437

Observaciones principales:

1.- No necesitas especificar el nombre del parámetro cuando declaras una función. Por lo tanto, lo puedes hacer así:

void cargarNombreYApellido (char[]);
void crearContrasenia (char[], char[]);
int elegirCantidadDeEmpresas(void); //El "void" indica que esta función no recibe ningún parámetro.
void cargarMatrizAleatoria(int, float[][MESES]);
void mostrarVector(int, float [][MESES]);

2.- La función elegirCantidadDeEmpresas no necesita de un parámetro, debido a que retorna el número de empresas, con eso basta:

int elegirCantidadDeEmpresas()
{
    int cantEmpresas; //--> Variable local.
    printf("\n\n\nCon cuantas empresa deseas trabajar? (Menos de tres) \n");
    scanf("%d", &cantEmpresas);
    return cantEmpresas;
}

3.- Estás desbordando el búfer y esto genera un comportamiento indefinido (puede resultar cualquier cosa en tiempo de ejecución):

cargarMatrizAleatoria(cantEmpresas,empresas[cantEmpresas][MESES]);

La primera dimensión representa el tamaño de filas y solo puede recibir índices de 0 a cantEmpresas - 1 y la segunda dimensión son el tamaño de columnas que va desde 0 a 11. Entonces no podemos pasarnos de esos límites, de lo contrario, tendrás errores.

Realmente lo que quieres hacer es pasar la dirección de memoria del primer elemento de la matriz para que la función se encargue de rellenar de datos dicha matriz. Por lo tanto, deberías hacer esto:

cargarMatrizAleatoria(cantEmpresas, empresas);

El identificador empresas no es más que un alias que representa la dirección del primer elemento de la matriz.

4.- Sin embargo, aquí hay otro problema, ya que el segundo parámetro solo puede recibir una "matriz" de tipo float y usted le está pasando una matriz de tipo int, ahí estamos mal, tanto el argumento (en este caso el argumento sería el identificador empresas) como el parámetro (en este caso sería float empresas[][MESES]) deben ser del mismo tipo.

Así que debemos cambiar la declaración del identificador empresas a:

float empresas[cantEmpresas][MESES];

5.- Según el estándar la función main debe ser definida de esta forma:

int main(void) //--> void indica: "NO ACEPTO PARÁMETROS".
{
     //... code
     return 0;
}

Siguiendo estas observaciones podríamos cambiar el código a esto:

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <string.h>
#define MESES 12

/* Declaraciones: */
void cargarNombreYApellido (char[]);
void crearContrasenia (char[], char[]);
int elegirCantidadDeEmpresas(void); //Esta función NO ACEPTA PARÁMETROS.
void cargarMatrizAleatoria(int, float[][MESES]);
void mostrarVector(int, float [][MESES]);

int main(void)
{
    srand(time(NULL));
    int cantEmpresas = elegirCantidadDeEmpresas();
    float empresas[cantEmpresas][MESES];
    cargarMatrizAleatoria(cantEmpresas, empresas);
    mostrarVector(cantEmpresas, empresas);
    return 0;
}

int elegirCantidadDeEmpresas()
{
    int cantEmpresas;
    printf("\n\n\nCon cuantas empresa deseas trabajar? (Menos de tres) \n");
    scanf("%d", &cantEmpresas);
    return cantEmpresas;
}

void cargarMatrizAleatoria(int cantEmpresas, float empresas[][MESES])
{
    int i,k;
    for (i=0; i<cantEmpresas; i++)
    {
        for (k = 0; k < MESES; k++)
            empresas[i][k]= rand()%11;
    }
}

void mostrarVector(int cantEmpresas, float empresas[][MESES])
{
    int i;
    int k;
    for (i=0; i<cantEmpresas; i++)
    {
        for (k=0; k<MESES; k++)
            printf("%f  - ", empresas[i][k]);
        printf("\n");
    }
}

Observaciones extras:

1.- Esta sentencia:

float empresas[cantEmpresas][MESES];

Se denomina VLA (Variable-length-Arrays), que en español significa un arreglo con longitud variable. En el código de arriba la variable cantEmpresas vendría ser esa longitud variable que se obtiene en tiempo de ejecución.

El problema radica que no todos los compiladores incluyen esta característica, debido a que, es propenso a que ocurra desbordamiento de pila. Así que por cuestión de portabilidad, no se lo debería usar.

2.- La forma segura de hacer esto, sería crear una matriz dinámica. Ahí tocaría crear un array de punteros (representaría las filas) y luego reservar memoria otra vez para asignarle la dirección a cada puntero del array (todo esto representaría las columnas y así formando una especie de array bidimensional). Para más información recomiendo leer este hilo.

EDIT:

Para generar números flotantes aleatorios lo podrías hacer de esta manera:

void cargarMatrizAleatoria(int cantEmpresas, float empresas[][MESES])
{
    int i,k;
    float n;
    for (i=0; i<cantEmpresas; i++)
    {
        for (k = 0; k < MESES; k++)
        {
            n = rand() / (float) RAND_MAX; //Genera un rango de (0.0 a 1.0)
            empresas[i][k]= (n * 300) - 100; //Genera un rango de (-100 a 200)
        }
    }
}

Fuente: How to generate random float number in C.

Entonces, simplemente mi error estaba en como utilizo los parametros? Nuestro profesor nunca nos explico desbordamiento de pilas, VDA y tampoco nos explico como definir parametros utilizando solamente int nombre _(int, float, char[])_ — Hatori Hanso, Aug 05 '20 at 02:13
Tenías dos errores: En el primero estabas declarando una matriz de tipo `int` cuando debería ser de tipo `float` y el segundo es por la forma como pasabas el valor al parámetro. Debías pasar únicamente el identificador `empresas`. — MrDave1999, Aug 05 '20 at 02:18
Gracias dave, y ahora que necesito darle valor de -100 hasta 200, con decimal incluido, como tengo que hacer? Estaba leyendo y vi la palabra reservada "drand48" pero es solo para linux, hay alguna formula especial para asignarles valor con decimal entre esos parametros? — Hatori Hanso, Aug 05 '20 at 02:25

Pablochaches · Answer 2 · 2020-08-04T22:46:15.047

2

En el código que publicaste hice los siguientes cambios

El primer error esta en esta linea

cargarMatrizAleatoria(cantEmpresas, empresas[cantEmpresas][MESES]);

Tienes que pasar un float*[12], pero pasas el valor que este en empresas[cantEmpresas][MESES], por eso falla. Tienes que activar los avisos del compilador, asi te enteraras de estos problemas antes de que caigas en errores.

Luego el problema que comentabas, no llamas la función mostrarVector, por eso no imprime nada.

Ya aplicando los cambios el y limpiando el codigo queda asi:

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <string.h>
#define MESES 12

int elegirCantidadDeEmpresas(void);
void cargarMatrizAleatoria(int cantEmpresas, float empresas[][MESES]);
void mostrarVector(int cantEmpresas, float empresas[][MESES]);


int main()
{
    srand(time(NULL));
    int cantEmpresas = elegirCantidadDeEmpresas();
    float empresas[cantEmpresas][MESES];
    cargarMatrizAleatoria(cantEmpresas, empresas);
    mostrarVector(cantEmpresas, empresas);
    return 0;
}

int elegirCantidadDeEmpresas(void){
    int cantEmpresas;
    printf("\n\n\nCon cuantas empresa deseas trabajar? (Menos de tres) \n");
    scanf("%d", &cantEmpresas);
    return cantEmpresas;
}

void cargarMatrizAleatoria(int cantEmpresas, float empresas[][MESES]){
    int i,k;
    for (i=0; i<cantEmpresas; i++) {
        for (k=0; k<MESES; k++) {
            empresas[i][k]=rand()%11;
        }
    }
}

void mostrarVector(int cantEmpresas, float empresas[][MESES]){
    int i;
    int k;
    for (i=0; i<cantEmpresas; i++) {
        for (k=0; k<MESES; k++) {
            printf("%f  - ", empresas[i][k]);
        }
        printf("\n");
    }
}

edited Aug 04 '20 at 22:46

answered Aug 04 '20 at 22:14

Pablochaches

2,505
1
5
21

Pero necesito que los datos sean flotantes, es lo que me pide el enunciado "Una vez que ya sabemos la cantidad de empresas (y de meses, 12, obvio), generar una matriz de números flotantes aleatorios, entre -100 y 500, estos flotantes tienen que ser con coma, no puede ser todos coma 00. " – Hatori Hanso Aug 04 '20 at 22:15
Dejame mejor edito el codigo que puse – Pablochaches Aug 04 '20 at 22:19
Usa la solución de @Pablochaches que es muy buena y antes de pasar los valores les sumas un valor aleatorio entre 0 y 1 sin incluir los extremos – quevedo Aug 04 '20 at 22:22
Pero necesito que los valores aleatorios sean flotantes, no enteros – Hatori Hanso Aug 04 '20 at 22:31
@HatoriHanso Ya le hice el cambio para que sea con `float`. Ahora, que sea entre -100 y 500, ademas de que también sean decimales al azar es tu tarea. Yo no soy tu maestro ni me pagas para que sea tu tutor. Corregi el problema de la pregunta, otros erroes pequeños, y limpie el codigo. Y me vas a pedir mas?.Los cambios para eso hazlos en la funcion `cargarMatrizAleatoria`, lo demas queda a tu creatividad – Pablochaches Aug 04 '20 at 22:33
Yo nunca pedi ayuda para generar los valores dentro de esos parametros, simplemente necesitaba ayuda para que me muestre la matriz, muchas gracias – Hatori Hanso Aug 04 '20 at 22:34

¿Por qué no se muestra los datos del array bidimensional?

2 Answers2