Iniciar sesiónCerrar
Correo:
Contraseña:
Entrar
Recordar sesión en este navegador
Iniciar sesiónIniciar sesiónCrear cuentaCrear cuenta
LWP » Foros » Java » Ayuda con un minijuego, no se como continuar.

Java - Ayuda con un minijuego, no se como continuar.

Volver
Nuevo Tema
<<>>
 
Vista:
sin imagen de perfil
Val: 82
Ha disminuido su posición en 6 puestos en Java (en relación al último mes)
Gráfica de Java

Ayuda con un minijuego, no se como continuar.

Publicado por System.out.println("Me llamo Carlos") (43 intervenciones) el 10/05/2020 14:35:17
Hola, tengo 14 años y este es mi primer programa con POO con mas de 100 lineas, uno de mis problemas es que no se me ocurre como poder modularizarlo es me hace bastante complicado, el otro es que si me podríais decir como hacer que la IA del minijuego intente ganar, estoy haciendo un método pero me parece que se me va alargar, ya que lo único que se me ocurre es escribir todas las posibilidades haciendo if anidados y else if por todas partes y se me da que me voy a liar mucho, os dejo el código tal como lo tengo para proponer ideas.

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
package tres_en_raya;
 
public class Tres_En_Raya {
 
	// Estos booleanos sirven para saber los turnos y las veces que uno gana.
 
	private boolean turnoRobot, turnoPersona, esTresEnRaya, NoEsTresEnRaya, HaGanado, NoHaGanado, esTurnoPrimero;
 
	// Para diferenciar las fichas del robot y de la persona
 
	private char fichaRobot;
 
	private char fichaJugador;
 
	// Para ir guardando los intentos
 
	private int intentos;
 
	// Asignar un nombre al jugador
 
	private String nombreJugador;
 
	// Para los getter y setter poder indicar en que columna escribir.
 
	private char[][] filasYcolumnas;
 
	// Para evaluar la respuesta de la jugada y poder utilizarla para devolverla con
	// setter otra vez.
 
	private int fila;
 
	private int columna;
 
	public Tres_En_Raya(String nombreJugador) {
 
		// Le damos valor a las diferentes fichas.
 
		fichaRobot = 'O';
		fichaJugador = 'X';
 
		// Nos Aseguramos de que el jugador siempre tenga nombre.
 
		this.nombreJugador = nombreJugador;
 
		// Le damos valor a filas, y columnas en un array.
 
		filasYcolumnas = new char[3][3];
 
	}
 
	// Metodo que nos devuelve los datos de la partida.
 
	public String GetDatosPartida() {
 
		if (HaGanado == true) {
 
			return "Nombre del Jugador " + nombreJugador + " \n Intentos: " + intentos + " \n Enhorabuena has ganado ";
 
		} else {
 
			return "Nombre del Jugador " + nombreJugador + " \n Intentos: " + intentos + " \n Que lastima has perdido ";
		}
 
	}
 
	// Metodo que nos permitira elegir donde queremos poner la fila y / o columna y
	// asi establecera un valor.
 
	public void setJugada(int fila, int columna) {
 
		// Si la jugada se pasa del numero tres que no la haga.
 
		if (turnoPersona == true && fila <= 3 && fila > 0 && columna <= 3 && columna > 0) {
 
			this.fila = fila;
 
			this.columna = columna;
 
 
			if (columna == 1 && fila == 1) {
 
				filasYcolumnas[0][0] = fichaJugador;
 
			} else if (columna == 2 && fila == 1) {
 
				filasYcolumnas[0][1] = fichaJugador;
 
			} else if (columna == 3 && fila == 1) {
 
				filasYcolumnas[0][2] = fichaJugador;
 
			} else if (columna == 1 && fila == 2) {
 
				filasYcolumnas[1][0] = fichaJugador;
 
			} else if (columna == 2 && fila == 2) {
 
				filasYcolumnas[1][1] = fichaJugador;
 
			} else if (columna == 2 && fila == 3) {
 
				filasYcolumnas[1][2] = fichaJugador;
 
			} else if (columna == 1 && fila == 3) {
 
				filasYcolumnas[2][0] = fichaJugador;
 
			} else if (columna == 2 && fila == 3) {
 
				filasYcolumnas[2][1] = fichaJugador;
 
			} else if (columna == 3 && fila == 3) {
 
				filasYcolumnas[2][2] = fichaJugador;
 
			}
		}
 
	}
 
	// Metodo que cambie el turno de el jugador al robot.
 
	public void setTurnoRobot () {
 
		turnoRobot = true;
 
		turnoPersona = false;
 
	}
 
      public void setTurnoJugador () {
 
		turnoPersona = true;
 
		turnoRobot = false;
	}
 
	// Metodo que imprima el valor en la tabla
 
	public char getImprimrTabla() {
 
		return fichaJugador;
 
 
	}
 
	// Metodo que nos permitira elegir quien comienza.
 
	public void setComienza(String respuesta) {
 
		if (respuesta.equalsIgnoreCase("Yo")) {
 
			turnoPersona = true;
			turnoRobot = false;
			esTurnoPrimero = true;
 
		} else if (respuesta.equalsIgnoreCase("IA")) {
 
			turnoRobot = true;
			turnoPersona = false;
			esTurnoPrimero = false;
 
		} else {
 
			// Que no salga del bucle
		}
	}
 
	// Metodo para saber si jugador ha hecho tres en raya
 
	public void es_tres_en_Raya() {
 
		if (filasYcolumnas[0][0] == fichaJugador && filasYcolumnas[0][1] == fichaJugador
				&& filasYcolumnas[0][2] == fichaJugador) {
 
			esTresEnRaya = true;
 
			HaGanado = true;
 
			NoHaGanado = false;
 
			NoEsTresEnRaya = false;
 
		} else if (filasYcolumnas[1][0] == fichaJugador && filasYcolumnas[1][1] == fichaJugador
				&& filasYcolumnas[1][2] == fichaJugador) {
 
			esTresEnRaya = true;
 
			HaGanado = true;
 
			NoHaGanado = false;
 
			NoEsTresEnRaya = false;
 
		} else if (filasYcolumnas[2][0] == fichaJugador && filasYcolumnas[2][1] == fichaJugador
				&& filasYcolumnas[2][2] == fichaJugador) {
 
			esTresEnRaya = true;
 
			HaGanado = true;
 
			NoHaGanado = false;
 
			NoEsTresEnRaya = false;
 
		} else if (filasYcolumnas[0][0] == fichaJugador && filasYcolumnas[1][1] == fichaJugador
				&& filasYcolumnas[2][2] == fichaJugador) {
 
			esTresEnRaya = true;
 
			HaGanado = true;
 
			NoHaGanado = false;
 
			NoEsTresEnRaya = false;
 
		} else if (filasYcolumnas[0][2] == fichaJugador && filasYcolumnas[1][1] == fichaJugador
				&& filasYcolumnas[2][0] == fichaJugador) {
 
			esTresEnRaya = true;
 
			HaGanado = true;
 
			NoHaGanado = false;
 
			NoEsTresEnRaya = false;
 
		} else if (filasYcolumnas[0][2] == fichaJugador && filasYcolumnas[1][2] == fichaJugador
				&& filasYcolumnas[2][2] == fichaJugador) {
 
			esTresEnRaya = true;
 
			HaGanado = true;
 
			NoHaGanado = false;
 
			NoEsTresEnRaya = false;
 
		} else if (filasYcolumnas[0][1] == fichaJugador && filasYcolumnas[1][1] == fichaJugador
				&& filasYcolumnas[2][1] == fichaJugador) {
 
			esTresEnRaya = true;
 
			HaGanado = true;
 
			NoHaGanado = false;
 
			NoEsTresEnRaya = false;
 
		} else if (filasYcolumnas[0][0] == fichaJugador && filasYcolumnas[1][0] == fichaJugador
				&& filasYcolumnas[2][0] == fichaJugador) {
 
			esTresEnRaya = true;
 
			HaGanado = true;
 
			NoHaGanado = false;
 
			NoEsTresEnRaya = false;
 
		} else {
 
			esTresEnRaya = false;
 
			HaGanado = false;
 
			NoHaGanado = true;
 
			NoEsTresEnRaya = true;
		}
	}
 
 
	public void getFichaIA() {
 
		if (turnoRobot == true && esTurnoPrimero == false) {
 
 
			// A partir de aqui todas las posibilidades.
 
			filasYcolumnas [0] [2] = fichaRobot;
 
			if (filasYcolumnas [1] [1] == fichaJugador) {
 
				filasYcolumnas [2] [0] = fichaRobot;
 
				 if (filasYcolumnas [2] [2] == fichaJugador ) {
 
					 filasYcolumnas [0] [0] = fichaRobot;
 
					 if (filasYcolumnas [0] [1] == fichaJugador ) {
 
						 filasYcolumnas [1] [0] = fichaRobot;
 
						 HaGanado = false;
					}
				 }
 
			} else if () {
 
			}
 
 
 
		} else if (turnoRobot == true) {
 
			if (filasYcolumnas [1] [1] == fichaJugador) {
 
				filasYcolumnas [1] [2] = fichaRobot;
 
 
			}
		}
 
	}
}
Valora esta pregunta
Me gusta: Está pregunta es útil y esta claraNo me gusta: Está pregunta no esta clara o no es útil
0
Responder
Imágen de perfil de Rodrigo
Val: 2.041
Plata
Ha mantenido su posición en Java (en relación al último mes)
Gráfica de Java

Ayuda con un minijuego, no se como continuar.

Publicado por Rodrigo (623 intervenciones) el 10/05/2020 16:09:10
Felicitaciones por tu programa,

Aqui sugiero algunas simplificaciones:
- En setJugada tienes varios if() que se pueden cambiar TODOS ellos por solo 1 instruccion, si usas lo que comparas como indice:
Sugiero eliminar desde la linea 80 a la 116 y usar esto en vez:
1
filasYcolumnas[fila-1][columna-1] = fichaJugador;

- Tienes 2 variables booleanas excluyentes entre si: turnoRobot y turnoPersona, no pueden ser ambas true o ambas false, Por lo que solo necesitas una. Elige una de ellas, elimina la otra, y deduce si el turno de la otra basado en el valor de esta unica variable (dicho de otra forma, si turnoRobot es true, es el turno del robot y no es el turno de la persona, si es false al reves)

- Parece que ocurre lo mismo con las variables haGanado y noHaGanado

- No es necesario comparar contra true
1
if( variableBooleana == true )
se puede hacer directamente
1
if( variableBooleana )
y para comparar si es false
1
if( !variableBooleana )

- Si varios if-elseif-elseif hacen exactamente la misma accion, puedes unir los if()s en 1 sola gran condicion uniendo las condiciones con ||

1
2
3
4
5
6
7
8
9
if( condicion1 ) {
   instrucciones
} else if( condicion2 ) {
   las mismas instrucciones
} else if( condicion3 ) {
   las mismas instrucciones
} else if( condicion4 ) {
   las mismas instrucciones
}
se puede cambiar por

1
2
3
if( condicion1 || condicion2 || condicion3 || condicion4 ) {
   instrucciones
}

Esto se puede aplicar a las lineas 173 a 261.
Como tienes condiciones complejas, vas a necesitar parentizar

1
2
3
if( (condicion1) || (condicion2) || (condicion3) || (condicion4) ) {
   instrucciones
}

- Se podria simplificar un poco mas la logica si cambias lo que guardas en tu programa.
Por ejemplo si filasYcolumnas tuviera valor enteros, no chars, y 1 fuera la persona y -1 el robot, 0 nadie.
Podrias sumar los valores para saber si hay 3 en raya en vez de comparar uno por uno por un cierto valor.

- Una sugerencia menor: reduce el espacio entre instrucciones, queda muy largo el programa asi con lineas blancas entre medio. Esta bien separar bloques grandes con 1 linea blanca, pero separar cada linea parece exagerado para mi.
Valora esta respuesta
Me gusta: Está respuesta es útil y esta claraNo me gusta: Está respuesta no esta clara o no es útil
2
Comentar