¿Porque mi matriz imprime números no aleatorios?

Question

    #include <stdio.h>
    #include <windows.h>
    #include <time.h> 
        int semillitaRandom(int _nombres,int* a) {
        time_t seconds;
        time(&seconds);
        *a=(rand()%_nombres);
        }
    

       int main () {
        int _nombres,i,j,k;
        printf("\nIngrese el tamaño del vector para cargar nombres:\n");
        scanf("%i",&_nombres);
        char vector[_nombres][10];
        int* _al;
        printf("\n Ingrese los nombres para el vector:\n");
        for (k=0;k<_nombres;k++)
        {
            fflush(stdin);
            gets(vector[k]);
        }
        int _filas,_columnas;
        printf("\nIngrese la cantidad de filas:\n");
        scanf("%i",&_filas);
        printf("\nIngrese la cantidad de columnas:\n");
        scanf("%i",&_columnas);
        char* punteroMatriz[_filas][_columnas];
        printf("\n Matriz nombres:\n");
        for (i=0;i<_filas;i++)
        {
            for (j=0;j<_columnas;j++)
            {
                semillitaRandom(_nombres,&_al);
                punteroMatriz[i][j]=vector[(int)_al];
                printf("%s | ",punteroMatriz[i][j]);
            }
            printf("\n");
        } 
        }

En este caso el problema se basa en crear un vector que contenga nombres ingresados por el usuario, luego ese vector transformarlo en matriz y que dentro de la matriz se llene con números aleatorios (del tamaño de la cantidad de nombres ingresados) para que con los números aleatorios apunte directamente a posiciones del vector y los imprima como matriz.

Este es el código que tengo creado, por alguna razón la matriz me brinda siempre el mismo resultado, creo que hay un problema en mis punteros para el aleatorio.

Answer 1

Tienes muchos errores en tu código, desde un indentado confuso hasta fallos lógicos pasando por errores de concepto.

1. La función semillitaRandom no establece ninguna semilla de números pseudo-aleatorios, las rutinas de c incluyen una función para establecer la semilla del motor de números pseudo-aleatorios: srand.

2. La función semillitaRandom tiene establecido un retorno de tipo int pero no devuelve nada, así que es incorrecta.

3. No puedes usar variables cuyo nombre empiece por guion bajo (_), el estándar de C reserva ese tipo de nombres para los compiladores (traducción y resaltado míos):

   7.1.3 Identificadores reservados

   • Todos los identificadores que empiecen por un guion bajo y una letra mayúscula o minúscula están siempre reservados para cualquier uso, excepto esos identificadores que sean léxicamente idénticos a palabras clave.
   • Todos los identificadores que empiecen por un guion bajo están siempre reservados para ser usados como identificadores en el ámbito de un archivo tanto en el espacio de nombres ordinario como en el de etiquetas.

   Por lo que tus variables _nombres, _al, _filas y _columnas no cumplen con los criterios del estándar. Lee este hilo para saber más.

4. Las variables deben tener un nombre auto-explicativo, nombres como i, j o k no dan ninguna pista de cuál es la misión de esas variables.

5. No es necesario pre-declarar las variables de indizado de bucles for, estas variables se pueden declarar en el propio bucle.

6. Favorece el pre-incremento frente al post-incremento, lee este artículo para saber más.

7. Las formaciones¹ deben tener un tamaño conocido en tiempo de compilación, si usas una variable como tamaño (una variable es un valor conocido en tiempo de ejecución) estás usando una formación de tamaño variable (FTV, en inglés: VLA: Variable Lenght Array) que es una extensión no estándar de compilador que puede dar lugar a problemas o ejecuciones erráticas, así que tus formaciones vector y punteroMatriz son incorrectas. Lee estos hilos para saber más.

8. Si quieres almacenar números, no uses formaciones de caracteres, la formación bidimensional punteroMatriz debería ser de enteros (int).

9. En c las funciones que no reciben parámetros deben llevar void en la lista de argumentos, así que la función main es incorrecta.

10. La función main debe devolver un valor, lee este hilo para saber más.

Los puntos anteriores son lo que has hecho mal a nivel conceptual y a nivel de programación. Si corriges el primer punto tendrás números aleatorios, si corriges el resto de puntos tu código funcionará en lugar de explotar.

---

• ¹También conocidas como arreglos o en inglés array.