Блог

всем привет, у меня есть этот фрагмент кода:

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

это безопасный код или нет, параметр имеет тип param_t *, но мне надоело выполнять каждый раз приведение, когда я использую его в функции, у кого-нибудь есть другое элегантное решение? заранее спасибо

Это странный сбой.

Когда вы определяете функцию, вы говорите, что параметр имеет тип void* .
Затем с помощью приведения вы явно преобразуете его в param_t*
И компилятор неявно преобразует это param_t* в void* с присваиванием.

Вам нужна другая переменная

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

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

Вам не нужно приводить значение void * к другому типу указателя в C.

Так что просто сделайте:

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

(Но почему вы все равно используете void * для аргумента?)

(Неявное) приведение в присваивании безопасно, даже если значение указателя равно NULL, поэтому нет необходимости откладывать его. Вы могли бы сделать это:

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

  if (myparam == NULL)
    return;

  ...
}
  

Очевидное решение — избегать использования void * и использовать param_t * вместо этого, как в:

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

  ...
}
  

Вы могли бы отказаться от нулевого теста, если знаете, что он никогда не вызывается с нулевым указателем. В качестве альтернативы вы могли бы заменить его на assert(param != NULL) .