njara / mochadoom

Fork of Mochadoom from Sourceforge
GNU General Public License v3.0
0 stars 1 forks source link

RS39 - Disparo alternativo de armas #40

Closed elbraulio closed 8 years ago

elbraulio commented 8 years ago

Requisito de Usuario asociado RU17 - Modo alternativo Uso alternativo de armas

njara commented 8 years ago

Initialization

Se comenzó a buscar la ubicación de los modos de disparos primarios, para luego crear el secundario. Se detecto en el codigo, que el click Secundario es el numero 3.

masotrix commented 8 years ago

Concept Location:

Las clases directamente asociadas a disparar son /gamelogic/ActionFunctions.java /gamelogic/UnifiedGameMap.java /doom/DoomMain.java

masotrix commented 8 years ago

Impact Analysis:

Las clases en total afectadas son: /gamelogic/ActionFunctions.java /gamelogic/UnifiedGameMap.java /gamelogic/Actions.java /doom/DoomMain.java /doom/weaponinfo_t.java /doom/items.java /doom/think.java /defines/StateNum.java /data/info.java /data/Defines.java /finale/EndLevel.java

masotrix commented 8 years ago

Prefactoring y Change Propagation:

La cantidad de cambios necesarios es tal que se decidió realizar la menor cantidad de cambios adicionales posibles. Los cambios finalmente realizados involucran principalmente la adicion de funcionalidad, pues existía el "espacio" para la adición sin cambios que no estuvieran relación alguna con la acción de disparar, a pesar de que se considera que existe una gran cantidad de code duplication.

masotrix commented 8 years ago

Verification, Postfactoring y Conclusión:

Ya ha sido probado en una máquina, y pareciera no requerirse un postfactoring notorio. Finalmente, la issue ha concluido con dejar el código preparado para implementar sobre él disparos alternativos para cada arma.