Hay veces que me dice que un numero es repetido cuando no es repetido
Eso es porque no me hiciste caso en el hilo original y no has inicializado los cartones con un número que sea de por sí inválido... como por ejemplo el -1
Si el programa solo se va a ejecutar una vez es algo que puedes hacer de forma trivial al declarar el array
int carton[RENGLON][COLUMNA] = { -1 };
O si no puedes optar por la forma genérica, que consiste en recorrer todo el array y asignar valores de forma manual:
for (int x=0; x<RENGLON && repetido == 0; x++)
{
for (int y=0; y<COLUMNA && repetido == 0; y++)
{
m[x][y] = -1;
}
}
Y estos números no se pueden repetir ni superar el 90 ni ser inferior a 0
Estás metiendo código sin tener para nada claro dónde debería ir
- El detector de duplicados debe recorrer toda la matriz porque comprueba un valor con todos los que haya en la matriz
- El detector de rango solo debe validar el valor introducido por el usuario, luego no necesita bucles
Son dos algoritmos independientes con requisitos distintos y, por tanto, no deberían mezclarse.
int validoEnRango = 1;
int repetido = 0;
do
{
if (numero>90 || numero<0 )
{
validoEnRango = 0;
printf("El numero introducido es mayor a 90 o menor a 0, introduzca nuevamente el numero.\n ");
printf("Reescribe el numero ---> ");
scanf("%d",&m[i][k]);
}
else
{
repetido = 0;
for (int x=0; x<RENGLON && repetido == 0; x++)
{
for (int y=0; y<COLUMNA && repetido == 0; y++)
{
repetido = (m[x][y] == numero);
}
}
if( repetido )
printf("Numero repetido\n");
else
m[i][k] = numero; // El número no está repetido, lo guardamos en la matriz
}
} while( !validoEnRango || repetido );
E incluso sería factible moverlos a funciones independientes para añadir modularidad al código:
int estaDuplicado(int numero, int carton[RENGLON][COLUMNA])
{
int repetido = 0;
for (int x=0; x<RENGLON && repetido == 0; x++)
{
for (int y=0; y<COLUMNA && repetido == 0; y++)
{
repetido = (carton[x][y] == numero);
}
}
return repetido;
}
int numeroEnRango(int numero)
{
return numero >= 0 && numero <= 90;
}
// ...
int numero;
printf("Renglon %d columna %d AAA -> ",i,k);
int valido = 1;
do
{
scanf("%d",&numero);
int valido = numeroEnRango(numero);
if( !valido )
{
printf("El numero introducido es mayor a 90 o menor a 0, introduzca nuevamente el numero.\n ");
printf("Reescribe el numero ---> ");
}
else
{
valido = !estaDuplicado(numero, m);
if( !valido )
printf("Numero repetido\n");
}
} while( !valido );
m[i][k] = numero; // El número es válido, lo guardamos en la matriz
Por cierto, nota que un Array de Tamaño Fijo tal que
int matriz[5][10];
Tiene dos características principales:
- se declara en la pila del programa
- se almacena en memoria de forma secuencial
Es decir
int matriz[2][2] = { { 1, 2 }, { 3, 4 } };
quedaría en memoria así:
00 01 02 03 04 05 <-- posiciones de memoria
1 2 3 4 X X
^ ^
matriz memoria de otra variable
Es decir, es perfectamente posible linealizar el array para simplificar las iteraciones:
// Esto
for (int x=0; x<RENGLON && repetido == 0; x++)
{
for (int y=0; y<COLUMNA && repetido == 0; y++)
{
m[x][y] = -1;
}
}
// Se reduce a
for( int x=0; x<RENGLON*COLUMNA; x++ )
{
*(m + x) = -1;
}
// O, aplicando aritmética de punteros
for( int * ptr = m; ptr != m + RENGLON * COLUMNA; ++ptr )
{
*ptr = -1;
}
Juntando todos estos conceptos el código podría quedar así:
void inicializarCarton(int carton[RENGLON][COLUMNA])
{
for( int * ptr = m, end = m + RENGLON * COLUMNA; ptr != end; ++ptr )
{
*ptr = -1;
}
}
int estaDuplicado(int numero, int carton[RENGLON][COLUMNA])
{
int repetido = 0;
for( int * ptr = carton, end = carton + RENGLON * COLUMNA; ptr != end && !repetido; ++ptr )
{
repetido = (*ptr == numero);
}
return repetido;
}
int numeroEnRango(int numero)
{
return numero >= 0 && numero <= 90;
}
void generarCartonManual (int m[RENGLON][COLUMNA])
{
inicializarCarton(m);
for (i=0; i<RENGLON; i++) //Renglones 3
{
for (k=0; k<COLUMNA; k++) //columnas 5
{
int valido = 0;
do
{
int numero;
printf("Renglon %d columna %d AAA -> ",i,k);
int valido = 1;
do
{
scanf("%d",&numero);
int valido = numeroEnRango(numero);
if( !valido )
{
printf("El numero introducido es mayor a 90 o menor a 0, introduzca nuevamente el numero.\n ");
printf("Reescribe el numero ---> ");
}
else
{
valido = !estaDuplicado(numero, m);
if( !valido )
printf("Numero repetido\n");
}
} while( !valido );
m[i][k] = numero; // El número es válido, lo guardamos en la matriz
} while (!valido );
}
}
}