search

crash7 / siiichess

Automatically exported from code.google.com/p/siiichess
0 stars 0 forks source link

6 #6

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
   Están hechos los DTOS, puede ser que al crear los métodos del controller salte que falte algún atributo. 
   Con respecto al BoardDTO me parece que si queda así en realidad es inncesario ya que solo tiene un array bidimensional de strings (que llevaría algo como "W,K" donde la W sería el color de la ficha y la K el tipo de ficha "king").

Original issue reported on code.google.com by mrenk...@gmail.com on 1 Jul 2011 at 11:21

GoogleCodeExporter commented 9 years ago 
Ahora que le mencionas, estoy de acuerdo, dejemos que el controller con 
getBoard devuelva simplemente una matriz (ojo! no un array list!) de strings.

public String[][] getBoard

Ya que estas Mariano, si cambiamos eso. Nos queda el DTO de Player y 
PlayerMove. Que te parece si movemos el paquete de DTOS a chess.dtos? osea, 
sacarlos del business

Original comment by Christia...@gmail.com on 2 Jul 2011 at 8:48

GoogleCodeExporter commented 9 years ago 
Suena lógica, ya que en realidad los DTO no son parte del business. Ahora los 
muevo.

Original comment by mrenk...@gmail.com on 2 Jul 2011 at 9:01

GoogleCodeExporter commented 9 years ago 
Dale, lo iba a cambiar yo, pero como vos estabas con eso no me quise meter :)

Original comment by Christia...@gmail.com on 2 Jul 2011 at 9:07

GoogleCodeExporter commented 9 years ago 
Con respecto a esto, agrego que te elimine el DTO de Board porque no es 
necesario.

Lo "cierro" y de ultima se reabre si salta algo con el controller, pero no creo 
que hay falta

Original comment by Christia...@gmail.com on 3 Jul 2011 at 4:33

GoogleCodeExporter commented 9 years ago 
Cerrada

Original comment by Christia...@gmail.com on 3 Jul 2011 at 8:28