C: Lekarstwo na ostrzeżenie: przekroczenie liczby całkowitej w wyrażeniu?

Question

Próbuję uporządkować moją bibliotekę UART i nieco ją upiększyć, dodając kilka # definicji, dzięki czemu mogę ją później dostosować bez konieczności głębokiego zagłębiania się w kod, ale nie mogę uzyskać następującego fragmentu kodu pracy:

#define FOSC  8000000 
#define BAUDRATE 9600 
#define BRGVAL  (FOSC/2)/(16*BAUDRATE)-1 

void uart_init(){ 
    U1BRG = BRGVAL; 
} 

Po BRGVAL kalkulacja staje 25.0416667, a ponieważ nie jest liczbą całkowitą otrzymuję następujące ostrzeżenie za to, że kiedy przypisać do U1BRG:

UART.c: W funkcji ' uart_init ':

UART.c: 24: ostrzeżenie: liczba całkowita przepełnienie w wyrażeniu

... a kod po prostu nie działa na docelowym sprzęcie. (Jeśli ręcznie wstawiam U1BRG = 25 działa to jak zaklęcie)

Czy istnieje sposób na wpisanie tej stałej do liczby całkowitej, aby kompilator był szczęśliwy?

Wielkie dzięki, Hamza.

Answer 1

Integer przelewowy oznacza, że ​​przekroczył górną granicę wartości int, który jest prawdopodobnie 32767 jeśli otrzymujesz ten błąd. Nie ma nic wspólnego z zmiennoprzecinkową; Operacje, które określiłeś, są w rzeczywistości operacjami matematycznymi całkowitymi, więc część ułamkowa podziału jest i tak odrzucana.

spróbować czegoś takiego:

#define FOSC  8000000L 
#define BAUDRATE 9600L 
#define BRGVAL  ((unsigned int)((FOSC/2)/(16*BAUDRATE)-1)) 

void uart_init(){ 
    U1BRG = BRGVAL; 
} 

L przyrostek okazuje te stałe do long typu zamiast int typu. Przesłona (unsigned int) konwertuje na typ U1BRG i informuje kompilator, że rozumiesz, że wartość long zmieści się w unsigned int i ukryje w ten sposób ostrzeżenia, które może rzucić na ciebie.

Normalnie, to jest złe praktyki, aby wyciszyć ostrzeżenia kompilatora, ale w tym przypadku, to jasne, że choć trzeba long do przechowywania wartości pośrednich w obliczeniach, wynik końcowy będzie pasował do z unsigned int.

Answer 2

I prawdopodobnie to wykorzystać:

#define BRGVAL  ((int)(FOSC/2)/(16*BAUDRATE)-1) 

Answer 3

Mogłeś nie zwrócić na to uwagę, jaki jest typ danych dla U1BRG? Jeśli jest to int, rzucić go jak jak pokazano

 
#define FOSC  8000000 
#define BAUDRATE 9600 
#define BRGVAL  ((long)(FOSC/2)/(16*BAUDRATE)-1) 

void uart_init(){ 
    U1BRG = BRGVAL; 
} 

Edit: Zmieniony to wziąć pod uwagę z Adam Liss „s komentarz unsigned int jest zbyt mały, aby pomieścić wynikiem makro, zmieniłem to, aby to long ... Dzięki Adam za HeadsUp ...

Nadzieja to pomaga, poważaniem Toma.

Answer 4

Z twojego przykładu nie wynika jasno, czy U1BRG jest zmienną globalną czy stałą zdefiniowaną przez #. W każdym razie, po prostu rzucając na liczbę całkowitą powinno działać:

U1BRG = (int)BRGVAL; 

Answer 5

Lubię odpowiedź Filipa, ale myślę, że lepszym rozwiązaniem jest zmniejszenie formułę i zmienić swoje makra:

#define BRGVAL (FOSC/32/BAUDRATE-1) 

W ten sposób można wyeliminować obsadę więc kompilator może nadal ostrzec jeśli wybierasz niską szybkość transmisji, która spowoduje, że wartość dzielnika będzie zbyt duża dla 16-bitowej wartości int.