這是雙重鑄造有效/正確的代碼？

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

這可能是我怎麼會寫。