препроцессор буэээ (проблема решена)

[wizzard]

#define R_PU_33 17890UL
#define R_PD_33 988UL
#define ADC_V_REF 2560UL

#define MV_TO_ADC(MV,  R_PU, R_PD) (((MV)*(unsigned long)1024UL*(R_PD))/((ADC_V_REF)*((R_PU)+(R_PD))))

int _tmain(int argc, _TCHAR* argv[])
{
    unsigned long i=0;
    i=MV_TO_ADC(33000UL, R_PU_33, R_PD_33);
    printf("%l", i);
    return 0;
}

Этот код печатает “68” из-за integer overflow в числителе, несмотря на то, что везде где можно воткнуты unsigned long, и правильный ответ 690.

Что делать? Вручную раскрывать макросы не хочется, их подобных много. Проверено на msvc, gcc и avr-gcc (ATMEL).

Comments

[sashman.livejournal.com]

Что интересно (и очевидно),
#define MV_TO_ADC(MV, R_PU, R_PD) (((((MV)*1024)/(ADC_V_REF))*(R_PD))/((R_PU)+(R_PD)))
работает нормально.

т.е. в cc не выполняются базовые законы математики. И никаких ворнингов, что особо противно.

Тысячи желчи на разработчиков.

[faceted-jacinth.livejournal.com]

Проверил на msvc 2008, в обоих режимах (х64 и х32).

Во-первых, он тебе выдаёт варнинг.

Во-вторых, запомни: типы long, unsigned long, и соответствующие константы использовать нельзя, вместо них следует использовать long long, unsigned long long и LL/ULL суффиксы, а ещё лучше - подключить types.h или как там её, где объявлены int64_t сотоварищи. Ну или накрайняк самому себе написать. Я к тому, что так всё работает правильно вощемта.