search

pegimenez-ar / TSSE_TP2_leds_test

CESE - TSSE - TP2 - Tests de leds con ceedling
0 stars 0 forks source link

Se debe refactorizar el código de producción #1

Open evolentini opened 2 hours ago

evolentini commented 2 hours ago

Esta linea

https://github.com/pegimenez-ar/TSSE_TP2_leds_test/blob/e39b3379ee840b7f86d3caead137a198959f82bc/src/leds.c#L38

debería reemplazarse por una llamada a la función LedsSetAllOff() dado que al crear una función para apagar todos los leds lo más lógico es llamar directamente a esa función en lugar de duplicar el código, aun cuando se trate de una asignación. En un futuro apagar los leds podría ser mas complejo, pero esa complejidad quedaría encapsulada solo en la función correspondiente.

evolentini commented 2 hours ago

Estas lineas también deberían convertirse en una función privada o en un macro

https://github.com/pegimenez-ar/TSSE_TP2_leds_test/blob/e39b3379ee840b7f86d3caead137a198959f82bc/src/leds.c#L43

https://github.com/pegimenez-ar/TSSE_TP2_leds_test/blob/e39b3379ee840b7f86d3caead137a198959f82bc/src/leds.c#L51

https://github.com/pegimenez-ar/TSSE_TP2_leds_test/blob/e39b3379ee840b7f86d3caead137a198959f82bc/src/leds.c#L67