xero-hige/fiuba-algo3-tp2

Automatically exported from code.google.com/p/fiuba-algo3-tp2

fiuba-algo3-tp2's Issues

Duda acerca de traduccion / coherencia del codigo

Traduje:
(self obtenerEscenario) modificarPuntuacionEn: (penalizacion negated).
como:
this.getEscenario().modificarPuntuacionEn(penalizacion*(-1));
en NaveCivil, "Actuar Durante"

Ahora, de estar bien, ¿Cual es el punto en buscar el numero contrario, no se 
puede guardar como negativo?
De no ser asi,¿ que hace el negated ese ?, porque no es el "opuesto" de point 
evidentemente.

Original issue reported on code.google.com by [email protected] on 24 May 2011 at 5:11

SVN

La SVN estubo andando raro ultimamente y crea archivos en lados que no debe.

Original issue reported on code.google.com by [email protected] on 21 Jun 2011 at 7:17

El modelo no pasa las pruebas

Este problema seguramente viene del lado de que modificamos el modelo pero 
dejamos las  pruebas como estaban entonces se sigue evaluando sobre estados ya 
no existentes...

Original issue reported on code.google.com by [email protected] on 25 Jun 2011 at 2:23

Falta traducciones para VueloEnCirculosTest

Linea 30:  radioDeVuelo = posicionInicial r
Linea 35:  vuelo = new 
VueloEnCirculos(objetoVolador,(objetoVolador.getPosicion().negated()),true); 
//Nuevamente, que hace negated en smalltalk?

Original issue reported on code.google.com by [email protected] on 22 May 2011 at 3:22

Ant no compila correctamente.

El problema es que al compilar utilizando el ant, no se copian carpetas de 
imagenes y demás cosas que no sea de extención .java. Por lo tanto, al correr 
o al generar el JAR, no hay errores, pero las ventanas aparecen vacías por el 
hecho de no haber imágenes que leer.

Original issue reported on code.google.com by [email protected] on 12 Jun 2011 at 3:31

Clase Point

A la clase point le falta sobrecargar el metodo + 

Original issue reported on code.google.com by [email protected] on 22 May 2011 at 1:39

Do not run on X64

Anoche lo pude comprobar, el juego tiene fallos corriendo en windows 64bits. 
Probablemente sea un problema en la compilacion si se hizo con el Eclipse.

Original issue reported on code.google.com by [email protected] on 11 Jun 2011 at 5:52

Falta traducciones para VueloDireccionableTest

Linea 32:  Assert.assertTrue(desplazamiento.distance(new Point(2,4)) 
normalized() * 2)) <= 1E-10);

Original issue reported on code.google.com by [email protected] on 22 May 2011 at 4:18

Problemas de acoplamiento

Hay problemas al tener la vista metida adentro del modelo en ObjetoEspacial. 
Para solucionarlo de la mejor, deberíamos sacarla para afuera y que la maneje 
directamente el controlador. Además, hoy un chico que cursa a la noche me dijo 
(sin que yo le dijera nada de cómo habíamos hecho) que la onda era tener el 
modelo totalmente desacoplado de la vista.

Así que bueno, subo esta issue como próxima cuestión a resolver (igual es 
una boludez ya que para saber la vista de cierto objeto lo único que 
necesitamos es preguntarselo al parser). La cosa sería consultarlo 
directamente al parser desde el controlador y no obtenerlo a través del objeto 
del modelo.

Original issue reported on code.google.com by [email protected] on 17 Jun 2011 at 10:53

Pruebas con nomenclatura erronea

Me acabo de dar cuenta que las pruebas están escritas con la nomenclatura 
vieja (la que esta en smalltalk). Hay que cambiar los "cambiarAlgoA" por 
"setAlgo" o los "obtenerAlgo" por "getAlgo". Yo voy a tratar de empezar a 
modificar pero no creo que llegue con todas. Por favor revisen....

Original issue reported on code.google.com by [email protected] on 21 May 2011 at 2:50

Persistencia

Pongo esta issue para que podamos discutir sobre la persistencia.

Entre lo que vimos ayer y estuve pensando, creo que se puede hacer algo así:

Ejemplo de cómo se guardaría cada objeto en un archivo:

En algún archivo:
(supongamos que el objeto pueda instanciarse)
<algunObjetoEspacial>
    <posicion id=123 src="123.xml"></posicion>
    <tamanio>8</tamanio>
    //Todos los demás atributos
<algunObjetoEspacial>

En 123.xml:
<posicion>
    <x>723</x>
    <y>238</y>
</posicion>

(Notar que si ven el código fuente desde el Firefox, puede hacer click en los 
link e ir al archivo del atributo ;) ).

Después dentro del código va a quedar una especie de grafo para recorrer, y 
entonces al guardar/cargar, podemos hacer un recorrido en profundidad (usando 
una pila, o agarrando el cuaderno de Algoritmos 2 y copiando más o menos el 
código :P) e ir verificando si ya visitamos el próximo nodo que haya (o sea 
que el id aparezca en un Map<id, objetoCargadoOGuardado>).

Creo que así más o menos puede ir la cosa. Después habría que ver cómo 
hacemos con algunos objetos a los que no tenemos acceso para poner un método 
de guardar/cargar (por ejemplo las colecciones o el Rectangle del escenario).

Original issue reported on code.google.com by [email protected] on 10 Jun 2011 at 7:02

Assert poco claros

En varias pruebas se hace assertTrue de la forma:

abs(a - b) <= 1e-10

Debería ser:

assertEquals(a, b, 1e-10);

Original issue reported on code.google.com by [email protected] on 21 May 2011 at 7:59

Tamaño de las imágenes

Abro esto para discutir algo que noté de las imágenes.

Existe un método de las Image que permite agarrar una imagen a escala de la 
original. Así que podemos usar eso para que las imágenes de todo se ajusten a 
la relación entre el tamaño de la ventana y el modelo espacial que usemos, y 
entonces para lo único que deberíamos redimensionar una imagen es si queremos 
hacer una más para que no salga pixeleada.

¿Qué opinan?

Original issue reported on code.google.com by [email protected] on 16 Jun 2011 at 5:06

ObjetoVolador

Hay pruebas que usan objeto volador, Objeto volador no se puede instanciar 
porque es una clase abstracta. Hay que definir si esto se modifica o se elije 
una clase heredera para usar

Original issue reported on code.google.com by [email protected] on 22 May 2011 at 1:42