search

RungeKutta2 / Khoeli

Repo del motor de juegos.
2 stars 0 forks source link

Objetos concretos antes que Strings #14

Closed Federico-G closed 4 years ago

Federico-G commented 4 years ago

Voy a tomar el siguiente código como ejemplo, pero va para varios casos donde ocurre lo mismo

https://github.com/RungeKutta2/Khoeli/blob/86703e0f52b4fc2132bdd61ab2bcfc8b47cb19b4/src/models/Location.java#L80

Muchos de estos métodos son realizados después de procesar de alguna manera el texto. Por lo tanto, en teoría, deberían tener (en caso correcto) el objeto concreto del que se trata, y no un string.

En el ejemplo de más arriba, el item ya deberían saber cual es, así que se podría directamente mandar el objeto

lsantosferreiro commented 4 years ago

Link al issue original: https://github.com/RungeKutta2/Khoeli/issues/6

Corregido en: https://github.com/RungeKutta2/Khoeli/commit/8a6b8865af46ba8e88c3ef58c3f145687119b6cc @Federico-G

Federico-G commented 4 years ago

👍