安全鑄造

Question

大家好我有這樣的片斷代碼：

void writer(void* param){ 
    if(NULL == param){ 
     return; 
    } 
    param = (param_t*)param; 
... 
} 

是安全的代碼，還是不行，參數是從類型param_t *，但我生病做每施法時間，當我在功能上使用它，有人有另一個優雅的解決方案嗎？在此先感謝

Answer 1

這是一個奇怪的無操作。

當你定義你說param是void*類型的函數。
然後，通過演員表將其明確轉換爲param_t*
然後編譯器將該分配隱式轉換爲param_t*至void*。

你需要另一個變量

void writer(void *param) { 
    param_t *internal_param; 

    if (NULL == param) return; 
    internal_param = param; 
    /* ... */ 
} 

Answer 2

你沒有投一個void *到另一個指針類型C.

所以只是做：

void writer(void* param){ 
    param_t *myparam; 
    if(NULL == param){ 
     return; 
    } 
    myparam = param; 
... 
} 

（但爲什麼您使用的參數爲void *無論如何？）

Answer 3

顯而易見的解決方案是避免使用void *和使用param_t *代替，如：

void writer(param_t * param) 
{ 
    if (param == NULL) 
    { 
    return; 
    } 

    ... 
} 

您可以刪除空測試，如果你知道那它從來沒有用NULL指針調用。或者，您可以用assert(param != NULL)替換它。

Answer 4

即使指針值爲NULL，賦值中的（隱式）轉換也是安全的，所以不需要推遲它。你可以這樣做：

void writer(void* param) 
{ 
    param_t* myparam = param; 

    if (myparam == NULL) 
    return; 

    ... 
}