erreur de compilation avec malloc

Bonjour,
Je ne comprend pas mon erreur lorsque je compile la partie qui suit:
(client est une structure).

Code:

---

1 2 3 4 5

client *x;   x = malloc(sizeof(*x));

---

lorsque je compile, j'ai comme erreur (pour la dernière ligne ci-dessus):
invalid conversion from 'void*' to 'client*'

Merci pour vos réponces.

Salut,

Déjà, malloc devrait etre évité en C++

Ensuite, s'il est utilisé, il faut prendre l'habitude de "transtyper" le retour de cette fontion dans le type souhaité (alors que ce n'est pas requis en C).

Il faut donc, idéalement, utiliser new sous la forme de

Code:

---

1 2	client *x; x = new client(/*parametres éventuels */);

---

(utiliser delete au lieu de free ;)) ou, si tu tiens à utiliser malloc et free, l'utiliser sous la forme de

Code:

---

1 2	client *x; x=(client)malloc(sizeof(client));

---

Perdu, c'est:

Code:

---

1 2	client *x; x = static_cast<client*>(malloc(sizeof *x));

---

En C++, il faut éviter les casts C autant que possible, parce qu'ils permettent n'importe quoi.

Medinoc > Sauf que ton code à un comportement indéfini puisque tu déférence un pointeur non initialisé.
Ca peut donner n'importe quoi (crash de l'appli, du PC, ou pire, renversement de NS ou encore hiver nucléaire :aie:)
Le bon code c'est:

Code:

---

1 2	client *x; x = static_cast<client*>(malloc(sizeof client));

---

Mais le mieux restant quand même new.

Citation:

---

Envoyé par Davidbrcz

Medinoc > Sauf que ton code à un comportement indéfini puisque tu déférence un pointeur non initialisé.

---

Perdu, l'expression à droite d'un sizeof n'est pas évaluée, car sa valeur est inutile, seul son type compte.

Citation:

---

Envoyé par Davidbrcz

Medinoc > Sauf que ton code à un comportement indéfini puisque tu déférence un pointeur non initialisé.
Ca peut donner n'importe quoi (crash de l'appli, du PC, ou pire, renversement de NS ou encore hiver nucléaire :aie:)
Le bon code c'est:

Code:

---

1 2	client *x; x = static_cast<client*>(malloc(sizeof client));

---

Mais le mieux restant quand même new.

---

Euh, je ne pense pas que le *p soit évalué, j'en veux pour preuve :

Code:

---

1 2 3 4 5 6 7 8

#include <iostream>   int main(void) { char *p=NULL; std::cout << sizeof(*p) << std::endl; return 0; }

---

Ce code devrait provoquer un plantage si *p était évalué or ce n'est pas le cas. Le code de Médinoc me semble donc tout à fait correct.

Et j'enfonce le clou, le pointeur n'est pas déréférencé lors d'un sizeof.

On peut même mettre un appel de fonction dans un sizeof, et la fonction ne sera pas appelée:

Code:

---

1 2 3 4 5 6

struct a { ... };   struct a * fonc(int);   size_t cb = sizeof *fonc(42); /* fonc() ne sera pas appelée,   et cb sera toujours égal à sizeof(struct a) */

---

Oui enfin bref, un malloc en C++ c'est comme une doudoune en été:
c'est lourd, inutile, inapproprié et ça pose plein de problèmes :mrgreen:

Alors vous m'en bouchez un coin.