Type incompatible float* et float[3]

Bonjour à tous,

Je débute en C++ et j'essaye de créer la classe Triangle avec un constructeur à trois arguments (les coordonnées des points en 3D) :

Code:

---

1 2 3 4 5 6 7

class Triangle { public: Triangle(float x[3], float y[3], float z[3]);   private: float x[3], y[3], z[3]; };

---

Maintenant lorsque je définis mon constructeur :

Code:

---

1 2 3

Triangle::Triangle(float x[3], float y[3], float z[3]){ this->x = x; this->y = y; this->z = z; }

---

et que je compile, j'obtiens l'erreur de compilation suivante :

Code:

---

../src/shapes/triangle.cpp:11:12: error: incompatible types in assignment of ‘float*’ to ‘float [3]’

---

Quelle est l'erreur ici?

Merci d'avance

x[3] crée 3 variables x[0], x[1] & x[2].
Tu ne peux pas assigner 3 variables de cette façon.

:mrgreen: float x[3] c'est 1 tableau C - tu as 3 cases contiguës (accessible avec la syntaxe crochet [] et les indices commencent à 0 - "0-based indexing").
Comme c'est un POD ("Plain Old Data"), tu n'as pas les affectations :mrgreen:

Soit, il faut recopier case par case ( this->x[0] = x[0]; this->x[1] = x[1]; this->x[2] = x[2];) soit appeler la fonction memcpy (<- lien cplusplus.com en anglais)

Il avoir un objet avec l'opérateur d'affectation (operator =) Donc soit std::vector (ou 1 autre containeur STL) soit faire ta propre classe Vertex avec 3 membres flottants x, y, z.

Super, merci pour vos réponses.

Est-ce aussi envisageable de passer x, y et z par pointeur plutôt que par valeur dans le cas d'un tableau?

Si j'utilise ma propre classe Vertex, ce sera par pointeur comme ça (en ayant défini le type des attributs de Triangle en Vertex *) :

Code:

---

1 2 3

Triangle::Triangle(Vertex *x, Vertex *y, Vertex *z){ this->x = x; this->y = y; this->z = z; }

---

Citation:

---

Envoyé par johhry

Est-ce aussi envisageable de passer x, y et z par pointeur plutôt que par valeur dans le cas d'un tableau?

---

Si x, y, z sont des valeurs (entières ou flottantes) cela ne sert à rien de passer des pointeurs

Citation:

---

Envoyé par johhry

Si j'utilise ma propre classe Vertex, ce sera par pointeur comme ça (en ayant défini le type des attributs de Triangle en Vertex *)

---

Apprends le C++ et non pas le C :langue2:

Code:

---

1 2 3

Triangle::Triangle(const Vertex& input_x, const Vertex& input_y, const Vertex& input_z){ x = input_x; y = input_y; z = input_z; }

---

avec (à vérifier)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40

#include <iostream>     class Vertex { public:   Vertex(float init_x=0.0, float init_y=0.0, float init_z=0.0) : x(init_x), y(init_y), z(init_z) {}   Vertex(const Vertex& other) : x(other.x), y(other.y), z(other.z) {}     public:   Vertex& operator= (/*const*/ Vertex& rhs) { x = rhs.x; y = rhs.y; z = rhs.z;   return (*this); }     friend std::ostream& operator<< (std::ostream&, const Vertex&);     private: // or public   float x; float y; float z; };     std::ostream& operator<< (std::ostream& os, const Vertex& v) { os << "[x: " << v.x << ", y: " << v.y << ", z: " << v.z << "]";   return os; }

---

Bonjour,

Merci pour votre réponse!

C'est vrai que j'ai d'abord appris le C avant le C++.
Cependant, d'un point de vue de la création d'un objet, en redéfinissant l'opérateur d'égalité pour la classe Vertex, l'instruction suivante (dans Triangle) :

Code:

---

x = input_x;

---

créé un nouvel objet de type Triangle et lui recopie les valeurs de input_x.
En Java par exemple, on passe un objet par référence mais il n'y aurait pas de création d'un nouvel objet.

Pourquoi utiliser les références plutôt que les pointeurs?

Citation:

---

Envoyé par johhry

créé un nouvel objet de type Triangle et lui recopie les valeurs de input_x.

---

:nono: En C++ il y a 2 affectations : celles qui appellent le constructeur par copie (affectation lors d'1 définition) Vertex v(v1); // or Vertex v = v1 et l'affectation normale (et donc l'opérateur =) v = v1;.

J'ai mis à jour mon code pour montrer les 2 :zoubi:

Citation:

---

Envoyé par johhry

Pourquoi utiliser les références plutôt que les pointeurs?

---

La réponse :mrgreen: est dans ce fil de discussion "manières de retourner une image opencv" : ma réponse #4 et celle de @koala01 #5

Super merci!

Une dernière question : de quel type dois-je stocker des objets membres de la classe Triangle (les trois points de type Vertex)?

Comme référence ?

Code:

---

1 2 3 4 5

class Triangle{ public: Vertex & a, b, c; }

---

Citation:

---

Envoyé par foetus

:nono: En C++ il y a 2 affectations : celles qui appellent le constructeur par copie (affectation lors d'1 définition) Vertex v(v1); // or Vertex v = v1 et l'affectation normale (et donc l'opérateur =) v = v1;.

---

Mais on se retrouve bien avec deux instances de Vertex qui ont les mêmes coordonnées non (l'objet membre de Triangle et l'instance de Vertex en argument du constructeur)?

Toutes tes interrogations dépendent de ta conception de tes classes :mrgreen:

Si tu as affaire à une composition, alors la classe A est responsable de la création et de la destruction de la classe B : donc soit un objet B (sans pointeur ni référence) soit un pointeur pour éventuellement réduire la mémoire prise en n'allouant pas les classes B inutiles.

Si tu as affaire à une agrégation, alors la classe A est juste responsable de la vie de la classe B : donc soit une référence soit un pointeur, parce que l'objet lui sera transmis. La différence, c'est qu'avec une référence, il faut l'initialiser dans le constructeur de la classe A. Avec un pointeur, tu peux faire une initialisation paresseuse ("lazy initialization" en anglais) ou des injections de dépendances (<- lien wiki en anglais)

Si tu as affaire à une association, alors c'est majoritairement un pointeur, pour être modifié/ changé.


Et également, il y a aussi les pointeurs intelligents :ccool: