Skip to content

Todas las variables globales de una biblioteca deberían ser static #10

@evolentini

Description

@evolentini

Es una fuerte recomendación evitar las variables gobales compartidas, por lo cual todas las variables globales deberían ser marcadas con la directiva static

uint8_t hora [3];
uint8_t fecha [3];
int estado = RTC_inactivo;
char i2c_msg[20];
static uint8_t opcion = 0;
uint8_t buff[20];

Esta claro que esto afectaría a las pruebas, pero eso es justamente porque las pruebas no están bien planteadas: las pruebas deberían llamar a las funciones RTC_send_hora(), RTC_send_fecha(), RTC_leer_hora() y RTC_leer_fecha() y utilizando mocks de las funciones I2C_Read() y I2C_Write inspeccionar lo que se envía y simular lo que se recibe por el puerto I2C.

@marianofino @rafaeloliva

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions