Duda sobre Sudoku

[sundaysunday]

Buenas noches a todos. Andaba con un sudoku que tengo que hacer para clase, y me he quedado atascado en un punto.

El tema es que, a partir de una función "menú", el usuario decide si quiere jugar al Sudoku, o al Harakiri. Y, una vez elegida esa opción, el nivel de dificultad. Por lo que, dependiendo de las opciones elegidas, tendrá que cargarse un archivo u otro (hay 6 posibilidades) con los números que conforman el tablero. Mi idea sería que, en una función, se abriese el archivo, dependiendo del tipo de juego y la dificultad. Y usar ese archivo en otra función, la cual trataría con él para guardar su contenido en un array de char. El código me da error y me temo que es porque no se cómo pasar un argumento tipo FILE por valor (si es que se puede).

Código:

void leer_fichero(char opcion, char nivel, FILE *F){

	if (opcion == 's'){ //Si queremos jugar sl sudoku

		if (nivel == 'f'){	 //Y queremos jugar al nivel fácil		

			F = fopen ("SudokuFacil.txt", "r");

			if ((F = fopen("SudokuFacil.txt", "r")) == NULL) {
				printf ("Error en la apertura. Es posible que el fichero no exista \n ");
			}		
		} else if (nivel == 'm'){
			/*hacemos lo mismo para el nivel medio*/

		}else{
			
			/*Igual para difícil*/

		}

	} else {
	}


}

void leer_dificultad(FILE F, char tabl[TAMH][TAMH]){

	char num;
	int i = 0;
	int j = 0;

	
	do{
		
		if (num != EOF){
			
                        num = (char)getc(F); //Leo caracter del fichero
			tabl[i][j] = num; //Y lo guardo en la posición correspondiente
			j++; //pasamos a la siguiente columna del tablero
			
                        if (j == 9){ //El tablero solo tiene 9 casillas. Al llegar a la novena, incrementamos una fila y ponemos las columnas a 0.

				j = 0;
				i++;
			}
		}
	}while(num != EOF);
	mostrar_tablero(tabl);	//función que muestra el sudoku por pantalla		
			
}

Visto que daba fallo de segmentación (el problema venía de la función leer_dificultad, porque por leer_fichero si llegaba a entrar), decidí cambiar la primera función para que guardase el nombre del archivo en un array de String, para luego pasárselo como parámetro por valor a FILE en la segunda función. Tampoco funcionó.

Código:

void leer_fichero(char opcion, char nivel, char *nombreFichero[50]){

	if (opcion == 's'){ //Si queremos jugar sl sudoku

		if (nivel == 'f'){	 //Y queremos jugar al nivel fácil		

			strcpy(*nombreFichero, "SudokuFacil.txt");

		} else if (nivel == 'm'){
			/*hacemos lo mismo para el nivel medio*/

		}else{
			
			/*Igual para difícil*/

		}

	} else {
	}


}

void leer_dificultad(FILE F, char tabl[TAMH][TAMH], char nombreFichero[50]){

	char num;
	int i = 0;
	int j = 0;

        F = fopen (nombreFichero, "r"); //le pasamos la cadena de texto guardada, que es el nombre del fichero.

	if ((F = fopen(nombreFichero, "r")) == NULL) {
				printf ("Error en la apertura. Es posible que el fichero no exista \n ");
			}
	do{
		
		if (num != EOF){
			
                        num = (char)getc(F); //Leo caracter del fichero
			tabl[i][j] = num; //Y lo guardo en la posición correspondiente
			j++; //pasamos a la siguiente columna del tablero
			
                        if (j == 9){ //El tablero solo tiene 9 casillas. Al llegar a la novena, incrementamos una fila y ponemos las columnas a 0.

				j = 0;
				i++;
			}
		}
	}while(num != EOF);
	mostrar_tablero(tabl);	//función que muestra el sudoku por pantalla		
			
}

Un saludo

[Fw190]

La función fopen te devuelve un puntero a FILE, que es una estructura de datos definida en stdio.h. El error de segmentación te lo daba porque FILE no lo habías pasado como puntero pero estabas intentando utilizarlo en funciones que lo esperan como puntero.

Otra cosa que he visto es que cuando abres el fichero estás haciendo:

F = fopen(....);

y acto seguido haces

if ((F = fopen(....)) == NULL)

con lo cual estás abriendo el mismo fichero DOS veces sobre la misma estructura FILE. La segunda sobreescribe a la primera. Esto supone que, al finalizar el programa, aunque cierres el fichero (con fclose(F), que, por cierto, no veo que lo hayas hecho), aún se te puede quedar el fichero abierto, pudiendo pasar desde nada a encontrarte con un fichero inservible.

Y un mero detalle "estético", yo intercambiaría los nombres de las funciones. Estás llamando leer_fichero a la función donde estás leyendo la dificultad, y leer_dificultad a la función donde estás leyendo el fichero.

Yo dejaría algo así como:

Código C:

void leer_dificultad(void)
{
    char opcion, nombre_fichero[20];
 
    opcion = leer_opcion();  // Esta presentaría el menú de opciones disponibles
    switch (opcion)
    {
        case 'f':
            strcpy(nombre_fichero, "Facil");
            break;
        case 'm':
            strcpy(nombre_fichero, "Medio");
            break;
        case 'f':
            strcpy(nombre_fichero, "Dificil");
            break;
    }
    leer_fichero(nombre_fichero);
}
 
void leer_fichero(char *nombre)
{
    FILE *F;
    
    if ((F = fopen(nombreFichero, "r")) == NULL)
    {
        printf ("Error en la apertura\n");
        return;
    }
    // Aquí leo el contenido del fichero
    fclose(F);
}
 

Claro está, el tema de parámetros, declaraciones, etc, tendrás que ponérselo en función del resto de tu programa.

[yurl]

Hola, q tal?
tengo alguna q otra duda en mi devenir en este juego, y una de dos, o las comparto con alguien q me oueda echar una mano, o me quedo sin poder avanzar durante bastante tiempo.
Si alguna se puede permitir la molestia de contestar lo agradecería.