このダブルキャスティングは有効なコードですか？

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; 
} 

。