корректно ли передавать так адрес в memset? Опции

[Метценгерштейн]

Код

    
Element *tmpElement = (Element*)malloc(sizeof(Element));
memset (tmpElement, 0, sizeof(Element)); // записать 0 в структуру

я уже в tmpElement имею адрес. Его и передаю.

Корректно?

[skripach]

Корректно?

А в чем сомнения?
Только добавить проверку что malloc таки выделил память.

[johnshadow]

Корректно?

Вполне. Есть пару предложений:
malloc возвращает void* - т.е. приведение к типу "указатель на Element" не нужно

тип переменной на которую указывает tmpElement может в дальнейшем поменяться, я в таких случаях предпочитаю
в sizeof использовать сам указатель:

Код

Element *tmpElement = malloc(sizeof(*tmpElement));
memset (tmpElement, 0, sizeof(*tmpElement));

проверку на выделение памяти из кучи добавляю так (хотя возможно это и не совсем правильный метод):

Код

Element *tmpElement;
if ((tmpElement = malloc(sizeof(*tmpElement)))) {
memset (tmpElement, 0, sizeof(*tmpElement));
...
free(tmpElement); //если этот кусок памяти больше не нужен
}

Сообщение отредактировал johnshadow - Dec 13 2016, 07:35

[Метценгерштейн]

всем спасибо.

[jcxz]

Только добавить проверку что malloc таки выделил память.

Лучше задать обработчик события невыделения памяти malloc-у, чем в каждом месте выделения пихать проверку.

[megajohn]

а можно заюзать calloc