fernandodaniele / cese-tsse Goto Github PK

View Code? Open in Web Editor NEW

0.0 3.0 0.0 442 KB

C 96.30% C++ 1.13% Shell 0.16% Makefile 2.42%

cese-tsse's Introduction

CESE-TSSE

cese-tsse's People

Contributors

Watchers

cese-tsse's Issues

Estas declaraciones no deberían ser publicas

Estas definiciones no son necesarias para utilizar la librería, por lo tanto deberían estar en el archivo .c y no en el archivo .h

CESE-TSSE/TPLeds/leds/src/led.h

Lines 6 to 9 in 0fe7942

 #define LED_OFF 0 

 #define LED_ON 1 

 #define LEDS_ALL_OFF 0x0000 

 #define LEDS_ALL_ON 0xFFFF

Las constantes están mal utilizadas

La prueba verifica que el valor de TODOS los leds es 0x0000 o 0x0001 que son los valores que corresponden a todos los leds apagados o al primer led encendido y todo el resto apagado. Esto no es lo mismo que LED_ON y LED_OFF que determinan si un led esta apagado o encendido, pero solo uno. Escribir la prueba de esta forma se presta a una mala interpretación del significado de las constantes.

CESE-TSSE/TPLeds/leds/test/test_led.c

Lines 37 to 49 in 0fe7942

 void test_TurnOnfIndividualLed(void) 

 { 

 LedOn(1); 

 TEST_ASSERT_EQUAL_HEX16 (LED_ON,vLeds); 

 } 

 //Se puede apagar un led individual 

 void test_TurnOffIndividualLed(void) 

 { 

 LedOn(1); 

 LedOff(1); 

 TEST_ASSERT_EQUAL_HEX16 (LED_OFF,vLeds); 

 }

Se deben eliminar las contantes mágicas

CESE-TSSE/TPLeds/leds/src/led.c

Lines 24 to 28 in 0fe7942

 if (led >16) 

 { 

 ErrorRecord(); 

 return 0; 

 }

No se sube código binario al repositorio

Se debe ajustar el archivo .gitignorepara excluir a la carpeta build y todo su contenido

fernandodaniele / cese-tsse Goto Github PK

cese-tsse's Introduction

CESE-TSSE

cese-tsse's People

Contributors

Watchers

cese-tsse's Issues

Estas declaraciones no deberían ser publicas

Las constantes están mal utilizadas

Se deben eliminar las contantes mágicas

No se sube código binario al repositorio

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	#define LED_OFF 0
	#define LED_ON 1
	#define LEDS_ALL_OFF 0x0000
	#define LEDS_ALL_ON 0xFFFF

	void test_TurnOnfIndividualLed(void)
	{
	LedOn(1);
	TEST_ASSERT_EQUAL_HEX16 (LED_ON,vLeds);
	}

	//Se puede apagar un led individual
	void test_TurnOffIndividualLed(void)
	{
	LedOn(1);
	LedOff(1);
	TEST_ASSERT_EQUAL_HEX16 (LED_OFF,vLeds);
	}