Является ли этот двойной литой действительный/правильный код?

Question

int compare_acd(const void *a, const void * b) 
{ 
    return ((int)(*((ADC_VALUE_TYPE*)a)) - (int)(*((ADC_VALUE_TYPE*)b))); 
} 

в моем случае ADC_VALUE_TYPE есть uint16_t, но это не правило.

Answer 1

Нет необходимости в внешней паре скобок, return не является функцией.

Кроме этого, я не вижу проблемы ... Вы бросаете указатели перед разыменованием, читаете правильное количество бит, затем «повышаете» до int.

ОБНОВЛЕНИЕ: на основании вашего комментария кажется, что вы хотите получить отрицательный результат, если, например, значение, на которое указывает a, меньше, чем указано на b, поэтому вам нужно преобразовать его в подписанный тип. Я думаю, это нормально, не на 100% уверен, что это необходимо или нет, но, по крайней мере, довольно ясно, что происходит.

Конечно, это может быть разбита на несколько заявлений для еще большей ясности:

int compare_acd(const void *a, const void * b) 
{ 
    const ADC_VALUE_TYPE va = *(ADC_VALUE_TYPE) a; 
    const ADC_VALUE_TYPE vb = *(ADC_VALUE_TYPE) b; 

    /* Cast to int to get signed response if a < b. */ 
    return (int) va - (int) vb; 
} 

Это, наверное, как я пишу это.