Discussion :

[win-pierre]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[koala01]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Médinoc]

Perdu, c'est:

Code C++ :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[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 )
Le bon code c'est:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Mais le mieux restant quand même new.

[Sylvain Togni]

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.

[Montag]

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 )
Le bon code c'est:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[Médinoc]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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) */

[r0d]

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

[Davidbrcz]

Alors vous m'en bouchez un coin.