Comentarios sobre tu solución:
Debes de evitar "números mágicos", o sea números que no sabes de donde salieron, en este caso el 4. ¿De dónde salió ese 4?, ¿Qué significa ese 4?.
Define una constante, preferentemente Macro en este caso para saber que esa constante es el tamaño del arreglo
- Estudia más sobre arreglos
Tú mencionas que es una matriz 3x3, sin embargo declaras y recorres una matriz 4x4. Recuerda que los índices empiezan en 0
- Complejidad computacional
Si tu ves un programa que tiene muchos ciclos anidados significa que tiene una gran complejidad computacional, no siempre significa que sea malo, pero realmente uno debe de pensar el algoritmo lo más eficaz posible.
En este caso tienes 6 ciclos anidados, lo cual para la solución de tu problema es catastrófico. (El comentario es con buena intención y para ayudarte a mejorar)
El idioma estándar de la programación es el inglés, te recomendaría ir nombrando desde ahora a tus variables en inglés. Ojo, no el texto que se muestra al usuario, sino las variables que usas.
Solución
Te recomendaría fragmentar un poco tu código, no sé si hayas ya visto las funciones en C, pero puedes imaginarlas como un pedazo de código que puedes llamar y reutilizar cuando quieras.
En este caso se está implementando la función isRepeated
, la cual nos regresa un booleano (verdadero o falso | 1 o 0). La cual nos regresa verdadero si el elemento está repetido. ¿Dónde?
En un arreglo llamado elements
, el cual se va llenando al mismo tiempo que matrix
, La matriz que quieres llenar con elementos positivos no repetidos.
Esta función comparará si hay algún elemento repetido hasta el elemento i * MATRIX_SIZE + j
(es el número de elementos que has llenado hasta ahorita de la matriz)
Nota: Como verás, dos de las ventajas de no tener números mágicos es que sabes de dónde viene ese número y con sólo modificar tu constante, puedes modificar todo el comportamiento de tu programa.
#include <stdlib.h>
#include <stdio.h>
#include <stdbool.h>
#define MATRIX_SIZE 3
/**
* @param arr
* @param element
* @param size
* @return true if the element is repeated otherwise false
*/
bool isRepeated(const int * arr, int element, unsigned short size) {
for (unsigned short i = 0; i < size; ++i) {
if (arr[i] == element)
return true;
}
return false;
}
/**
* @param argc Number of arguments passed on command execution
* @param argv Array of string arguments
* @return program exit status
*/
int main(int argc, char ** argv) {
int matrix[MATRIX_SIZE][MATRIX_SIZE];
int elements[MATRIX_SIZE * MATRIX_SIZE];
unsigned short i = 0;
unsigned short j;
for(; i < MATRIX_SIZE; ++i) {
for(j = 0; j < MATRIX_SIZE; ++j) {
do
{
printf("\n\tIngrese el n%cmero de la posici%cn [%d][%d]: ", 163, 162, i+1, j+1);
scanf("%d", &matrix[i][j]);
elements[i * MATRIX_SIZE + j] = matrix[i][j];
}while(matrix[i][j] < 1 || isRepeated(elements, matrix[i][j], i * MATRIX_SIZE + j));
}
}
printf("\n\tMatriz capturada\n\t");
for(i = 0; i < MATRIX_SIZE; ++i) {
for(j = 0; j < MATRIX_SIZE; ++j)
printf("%4d", matrix[i][j]);
printf("\n\t");
}
return EXIT_SUCCESS;
}