fabivrp / TP2-TDA

0 stars 0 forks source link

Corrección TP2 #1

Open mbuchwald opened 2 days ago

mbuchwald commented 2 days ago

Hola! Dejo por acá las correcciones.

  1. Es bastante molesto que si yo paso una ruta de un archivo, me la rompan agregando un archivos/ adelante, haciendo que tenga que si o si poner el archivo allí. Por qué no puedo simplemente pasar una ruta y listo? Tuve que modificar el tp a mano.
  2. El código del tp está totalmente mezclado con código para visualizar cosas. O sea, tienen el código del algoritmo de reconstrucción totalmente entremezclado con código de cómo se va a mostrar. Esto es un error en Fundamentos (Algo1), en Algoritmos (Algo2), en Paradigmas (Algo3) y acá no va a dejar de serlo.
  3. Lo ejecuté para algunos casos extra, y en todos ganaba Mateo. Me resultaba extraño que hubieran cometido un error que no afecte a los casos que les dimos pero si a otros... y no, simplemente no les anda en ningún caso. Es decir, ¿el algoritmo no les funciona?

En función de esto, no hay mucho más para revisar de momento. Puedo pasarles otro escenario más si quieren donde no les funciona, pero el punto no va a cambiar. También revisé que no es que estuvieran dando las ganancias al revés, y no.

franmacke commented 2 days ago

Buen día, qué tal?

  1. Consideramos que era más fácil simplemente poner el número de monedas, pero lo vamos a tener en cuenta para la próxima, mil disculpas.
  2. Nos podrías pasar el/los casos que no andan por favor? Para los casos provistos, nos anduvo bien. Agregamos pruebas para corroborar que anden todos esos casos.

Desde ya, muchas gracias