Hola buenas, tengo un problema con una función recursiva, se mete pero nunca sale. No se si tengo un error en la sintaxis o en la condición de parada. Si alguien ve algún fallo y puede ayudarme...Gracias
void FncOpenPoint(Tablero Partida, int fil, int col)
{
if(fil>0&&fil<FIL&&col>0&&col<COL)
{
if(Partida[fil][col].mine==false)
if(Partida[fil][col].num=0&&Partida[fil][col].flag==false)
{
Partida[fil][col].visible=true;
/*for(int i=fil-1;i<fil+1;i++)
for(int j=col-1;j<col+1;j++)
FncOpenPoint( Partida, fil, col); */
FncOpenPoint( Partida, fil-1, col-1);
FncOpenPoint( Partida, fil-1, col);
FncOpenPoint( Partida, fil-1, col+1);
FncOpenPoint( Partida, fil, col-1);
FncOpenPoint( Partida, fil, col+1);
FncOpenPoint( Partida, fil+1, col-1);
FncOpenPoint( Partida, fil+1, col);
FncOpenPoint( Partida, fil+1, col+1);
}
else
if(Partida[fil][col].mine!=true)
Partida[fil][col].visible=true;
}
return;
}
}
Como es un buscaminas tiene que ir recorriendo la matriz de forma recursiva hasta que encuentre una mina o llegue al limite del tablero. Se exige que sea de forma recursiva. Gracias
Supongo que es que estas chequeando que flag es false para continuar, pero no lo cambias nunca.
Cambialo. O usa visible, que si lo cambias.
Se te ha colado una asignación (=0) donde debería haber una comparacion (==0):
if(Partida[fil][col].num=0&&Partida[fil][col].flag==false)
El resto no lo he mirado.
Cita de: do-while en 8 Enero 2018, 15:02 PM
Se te ha colado una asignación (=0) donde debería haber una comparacion (==0):
if(Partida[fil][col].num=0&&Partida[fil][col].flag==false)
El resto no lo he mirado.
Tienes razón pero sigue sin funcionar aún asi. La condición de parada es que encuentre los límites del tablero o que encuentre una mina. Pero hace dos cosas, solo me marca como visible la primera y no hace nada mas o se mete en bucle y no sale. Gracias!
Si nos pudieras decir qué es cada miembro de Partida, qué hace y cuáles son las condiciones de parada te podríamos ayudar mejor. Yo voy muy a ciegas.
Cita de: MAFUS en 8 Enero 2018, 19:15 PM
Si nos pudieras decir qué es cada miembro de Partida, qué hace y cuáles son las condiciones de parada te podríamos ayudar mejor. Yo voy muy a ciegas.
struct Estados
{
bool mine;
bool visible=false;
bool flag=false;
int num=0;
};
Las condiciones de parada de la funcón recursiva son que encuentre una casilla con un valor diferente a 0 o encuentre una mina.
Vale, ya entiendo. Verás: tal y como está no se detendrá nunca porque visible no lo tienes en cuenta a la hora de parar. Imagina que tienes dos casillas contiguas que no tienen nada. El jugador marca la de la izquierda y la hace visible y el código pasa después a la de la derecha. El código la hace visible y en algún momento volverá a mirar la de la izquierda. Como no tiene condición de parada el que una casilla sea visible este bucle se repetira (izquierda, derecha, izquierda, derecha, ...) hasta que la pila se llene de llamadas y el programa se detenga.
Cita de: MAFUS en 8 Enero 2018, 20:03 PM
Vale, ya entiendo. Verás: tal y como está no se detendrá nunca porque visible no lo tienes en cuenta a la hora de parar. Imagina que tienes dos casillas contiguas que no tienen nada. El jugador marca la de la izquierda y la hace visible y el código pasa después a la de la derecha. El código la hace visible y en algún momento volverá a mirar la de la izquierda. Como no tiene condición de parada el que una casilla sea visible este bucle se repetira (izquierda, derecha, izquierda, derecha, ...) hasta que la pila se llene de llamadas y el programa se detenga.
En eso creo que tienes toda la razón pero aun poniendo la condición de parada de visible sigue haciendo lo mismo de quitarme solo una si le asigno visible=true en el inicio o meterse en bucle si lo meto dentro de los ifs
void FncOpenPoint(Tablero Partida, int fil, int col)
{
Partida[fil][col].visible=true;
if(!Partida[fil][col].mine)
if(Partida[fil][col].num==0&&!Partida[fil][col].flag&&!Partida[fil][col].visible)
{
//cout<<"yeee"<<endl;
//Partida[fil][col].visible=true;
//system("PAUSE");
//do{
if(fil>0||col>0){
FncOpenPoint( Partida, fil-1, col-1);
//cout<<"yeeeffffffffffffffffffffffffffffffffffffffffff"<<endl;
FncOpenPoint( Partida, fil-1, col);
FncOpenPoint( Partida, fil-1, col+1);
FncOpenPoint( Partida, fil, col-1);
FncOpenPoint( Partida, fil, col+1);
FncOpenPoint( Partida, fil+1, col-1);
FncOpenPoint( Partida, fil+1, col);
FncOpenPoint( Partida, fil+1, col+1);
}//while (fil<0&&fil>FIL&&col<0&&col>COL);
/*for(int i=fil-1;i<fil+1;i++)
for(int j=col-1;j<col+1;j++){
FncOpenPoint( Partida, i, j);
//cout<<"yeee"<<endl;
}*/
//system("PAUSE");
}
else{
if(Partida[fil][col].mine!=true)
Partida[fil][col].visible=true; }
}void FncOpenPoint(Tablero Partida, int fil, int col)
{
if(!Partida[fil][col].mine)
if(Partida[fil][col].num==0&&!Partida[fil][col].flag&&!Partida[fil][col].visible)
{
//cout<<"yeee"<<endl;
Partida[fil][col].visible=true;
//system("PAUSE");
//do{
if(fil>0||col>0){
FncOpenPoint( Partida, fil-1, col-1);
//cout<<"yeeeffffffffffffffffffffffffffffffffffffffffff"<<endl;
FncOpenPoint( Partida, fil-1, col);
FncOpenPoint( Partida, fil-1, col+1);
FncOpenPoint( Partida, fil, col-1);
FncOpenPoint( Partida, fil, col+1);
FncOpenPoint( Partida, fil+1, col-1);
FncOpenPoint( Partida, fil+1, col);
FncOpenPoint( Partida, fil+1, col+1);
}//while (fil<0&&fil>FIL&&col<0&&col>COL);
/*for(int i=fil-1;i<fil+1;i++)
for(int j=col-1;j<col+1;j++){
FncOpenPoint( Partida, i, j);
//cout<<"yeee"<<endl;
}*/
//system("PAUSE");
}
else{
if(Partida[fil][col].mine!=true)
Partida[fil][col].visible=true; }
}
Cosillas:
Toda la parte del else te sobra ya que
if(!Partida[fil][col].mine)
es lo mismo que
if(Partida[fil][col].mine!=true)
El primer código está mal ideado ya que visible valdrá true y no podrá entrar en el segundo if:
Partida[fil][col].visible=true;
if(!Partida[fil][col].mine)
if(Partida[fil][col].num==0&&!Partida[fil][col].flag&&!Partida[fil][col].visible)
También veo que la función no está terminada pues buscas más allá de los límites de la tabla, tal vez te pierdes por allí. Como depuración haz que la función te diga en qué casilla se encuentra.
He intentado solucionar tu problema pero me he atascado en lo mismo que tú, aunque no este solucionado, te pongo el código que he creado por si te puede servir de algo.
int i,j;
Partida[fil][col].visible=true;
for (i=-1; i<2; i++)
for (j=-1; j<2; j++)
if (((fil+i>=0)&&(fil+i<FIL))&&((col+j>=0)&&(col+j<COL)))
if (Partida[fil][col].visible == true)
/* si la casilla tampoco tiene adyacentes se hace una llamada
recursiva */
if (Partida[fil][col].visible == false&&Partida[fil][col].num==0){
FncOpenPoint(Partida,fil+i,col+j);
}
else
Partida[fil][col].visible=true;
Partida[fil+i][col+j].visible = true;
}
}
Siguiendo tus premisas he hecho lo siguiente. Decir que no está probado y puede contener fallos.
void FncOpenPoint(Tablero Partida, int fil, int col) {
// Configuro los límites de búsqueda
int inicio_fil = fil==0? 0 : -1; // Si ya estoy arriba no debo mirar más arriba
int fin_fil = fil==FIL-1? 0 : 1; // Si ya estoy abajo no debo mirar más abajo
int inicio_col = col==0? 0 : -1; // Si ya estoy a la izquierda no debo mirar más a la izquierda
int fin_col = col==COL-1? 0 : 1; // Si ya estoy a la derecha no debo mirar más a la derecha
int i, j;
if( !Partida[fil][col].visible &&
!Partida[fil][col].mine &&
!Partida[fil][col].num &&
!Partida[fil][col].flag ) {
Partida[fil][col].visible = true;
for(i = inicio_fil; i <= fin_fil; ++i) {
for(j = inicio_col; j <= fin_col; ++j) {
if(i==0 && j==0) continue; // Para no volver a caer en la casilla actual. Aunque es redundante no gasto tantos ciclos de reloj como si hay que realizar una llamada a la función en la misma coordenada y esperar a que esta se detenga por si sola.
FncOpenPoint(Partida, fil+i, col+j);
}
}
}
}
A ver, vamos por partes.
En vez del buscaminas, hagamos una simplificación del problema para entenderlo mejor. Después solo lo adaptas al buscaminas (numeros, visible...)
De una matriz NxM de 0's y 1's, tenemos que dar una posición donde encontrar un 1. Nos vale cualquiera.
M=4, N=5
0 i M
+--+--+--+--+
| | | | |
+--+--+--+--+
|1 | | | |
+--+--+--+--+
j | | | | 1|
+--+--+--+--+
| | | | |
+--+--+--+--+
| | | | |
+--+--+--+--+
N
Sol: V[2][3]=1Lo primero que te tienes que dar cuenta es de que la complejidad en el caso peor va a ser Omega(n^2) y no puedes bajar de ahí (imagina que está en la última casilla en el orden de búsqueda).
Ahora viene la siguiente cuestión: se pide recursiva, de acuerdo, pero...
Por que no intenar dividiendo el tamaño del problema en vez de sustrayendo? Me explico, se divide la matriz en cuatro cuadrantes A,B,C,D dividiendo sus dimensiones N,M por la mitad, y por recursión damos por resuelto el problema. Hay dos casos base posible:
- matriz sin celdas, o vacía, donde nunca encuentras un 1. (significa que te has salido del rango)
matriz con UNA celda : si está, devolvemos cierto y las coordenadas de la celda. en otro caso falso y da igual las coordenadas
Imporante: En esta implementación siempre pensamos en el operador cortocircuitado OR... Es decir (true || a == 6 ) nunca pasa a evaluar la segunda expresi'on a==6. De otro modo, esto puede no funcionar. En la práctica, la mayoría de los compiladores proceden así, pero no se sabe si algun optimizador puede cambiar el orden.
Ahí va la solucion
#include <iostream>
using namespace std;
#define MAX 100
/*
M=4, N=5
0 i M
+--+--+--+--+
| | | | |
+--+--+--+--+
|1 | | | |
+--+--+--+--+
j | | | | 1|
+--+--+--+--+
| | | | |
+--+--+--+--+
| | | | |
+--+--+--+--+
N
Sol: V[2][3]=1
*/
/*
Global inmersion
P : V[j0..jn)[i0..in)
0<=i0<=in<=M, 0<=j0<=jn<=N
Q : found -> V[x][y]= 1
B1 : empty square
B2 : one-cell square
C(i0,in,j0,j0) = (in-i0)*(jn-j0) (Area)
O(n^2), since A=4,B=2, k=0 and 4 > 2
*/
bool RR(const int V[][MAX],
const int j0, const int jn, // N
const int i0, const int in, // M
int &x, int &y)
{
if (((in-i0)*(jn-j0)) == 0) // empty
return false;
if (((in-i0)*(jn-j0)) == 1) // 1 cell
{
x=j0;
y=i0;
return (V[x][y]==1);
}
else // 2 or more cells
{
const int hi = (in + i0) /2 ;
const int hj = (jn + j0) /2 ;
return
(
RR(V,j0,hj,i0,hi,x,y) || RR(V,j0,hj,hi,in,x,y) ||
RR(V,hj,jn,i0,hi,x,y) || RR(V,hj,jn,hi,in,x,y)
);
}
}
/*
P : \exists i : 0 <= i<N , 0 <=j<M : V[i][j]=1
Q : M[x][y]=1
*/
void R(const int V[][MAX],
const int N, const int M,
int &x, int &y)
{
RR(V,0,N,0,M,x,y);
return;
}
int main()
{
int V[MAX][MAX] ;
int n,N,m,M ;
int x,y;
cin >> N;
cin >> M;
for(int n=0; n<N; n++)
for(int m=0; m<M; m++) cin >> V[n][m];
R(V,N,M,x,y);
cout << "(" << x << "," << y << ")" << endl;
return 0;
}
Algunos casos de prueba
6 5
0 0 1 0 0
0 0 0 0 0
0 0 0 0 0
0 0 0 0 1
0 0 0 0 0
0 0 0 0 0Resultado
(0,2)
Otro
12 5
0 0 0 0 0
0 0 0 0 0
0 0 0 0 0
0 0 0 0 0
0 0 0 0 0
0 0 0 0 0
0 0 0 0 0
0 0 0 0 0
0 0 0 0 1
0 0 0 0 0
0 0 0 0 0
0 0 0 0 0
(8,4)