Discussion :

[Sergejack]

Je me suis remis au C++ depuis peu (quelques heures) et malgrès le fait de lire et relire les parties consacrés dans 3 des cours que j'ai trouvé ici même, je ne comprends pas ce qu'on me reproche (je suis inocent, monsieur l'agent).

Voici donc le header et le code.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
 
typedef unsigned char pixel[4];
 
class RasterLike {
private :
	pixel **data;
	const unsigned int width;
	const unsigned int height;
public :
	RasterLike(unsigned int _width, unsigned int _height);
	~RasterLike(void);
	void setPixel(unsigned int x, unsigned int y, pixel pix);
	pixel getPixel (unsigned int x, unsigned int y) const;
}

Code :

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

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

#include "cov001.h"

RasterLike::RasterLike(unsigned int _width, unsigned int _height) :
	width(_width), height(_height)
{
	this->data = new pixel* [this->width];
	for (unsigned int x = 0; x < this->height; x++) {
		this->data[x] = new pixel[this->height];
	}
	return;
}

RasterLike::~RasterLike() {
	for (unsigned int x = 0; x < this->height; x++) {
		delete[] this->data[x];
	}
	delete this->data;
}

void RasterLike::setPixel(unsigned int x, unsigned int y, pixel pix) {
	this->data[x][y] = &pix;
}

pixel RasterLike::getPixel(unsigned int x, unsigned int y) const {
	return this->data[x][y];
}

int main() {
	RasterLike a = new RasterLike(320,240);
}

(en fushia, les lignes 10 et 28)

Et je bute à la récetion des méssages d'erreurs suivant à la compilation :

cov001.cpp
f:\program files\avisynth 2.5\avisynth_256_src\avsdll\cov001.h(19) : error C2090: tableau retourné par la fonction
f:\program files\avisynth 2.5\avisynth_256_src\avsdll\cov001.cpp(10) : error C2533: 'RasterLike::{ctor}' : type de retour non autorisé pour les constructeurs
f:\program files\avisynth 2.5\avisynth_256_src\avsdll\cov001.cpp(28) : error C2440: '=' : impossible de convertir de 'unsigned char *__w64 []' en 'pixel'
Aucune conversion en types tableau, même s'il y a des conversions en références ou en pointeurs de tableau
[...]

Merci.

PS : compilé sous M$ Visual Studio 2005 Express.

[Matthieu Brucher]

Normal, tu fais un return dans un constructeur, alors qu'il ne peut rien retourner.

[Médinoc]

L'erreur n'est pas là : point-virgule oublié après la déclaration de classe.

Et tu dois remplacer "this->data[x][y] = &pix;" par "memcpy(this->data[x][y], pix, sizeof(pixel);" ou un truc du genre: Il n'y a pas d'opérateur d'affectation pour les tableaux.

[Sergejack]

Le return du constructeur, je ne l'ai mis qu'en désespoire de cause en suivant scrupuleusement la déclaration que l'on trouve dans le cours de C. Casteyde.
J'imagine donc que ce n'est pas une erreur en soit (une manie ?).

L'erreur n'est pas là : point-virgule oublié après la déclaration de classe.

Et tu dois remplacer "this->data[x][y] = &pix;" par "memcpy(this->data[x][y], pix, sizeof(pixel);" ou un truc du genre: Il n'y a pas d'opérateur d'affectation pour les tableaux.

Merci, je suis dans Java depuis tellement longtemps maintenant que je ne pouvais pas concevoir l'impossibilité d'assigner des tableaux.

[Matthieu Brucher]

En C, il y a un type de retour pour toutes les fonctions. En C++, il n'y en a pas pour les constructeurs, donc pas de return.

[Pragmateek]

En C, il y a un type de retour pour toutes les fonctions. En C++, il n'y en a pas pour les constructeurs, donc pas de return.

Le "return" sert à indiquer le rétablissement du contexte avant appel de la fonction et accessoirement renvoyer un résultat; même si il n'est pas obligatoire, l'accolade finale de la fonction étant suffisante pour indiquer le retour dans le cas où la fonction ne renvoie rien, le mettre reste cohérent; ainsi ceci compile:

Code :

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

1
2
3
4
class A
{
     public:A(){return;}     
};

[thewho]

Bonjour,

et après ceci

Code :

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

1
2
3
int main() {
	RasterLike a = new RasterLike(320,240);
}

n'oublie pas de libérer la mémoire

[roulious]

Et tu dois remplacer "this->data[x][y] = &pix;" par "memcpy(this->data[x][y], pix, sizeof(pixel);" ou un truc du genre: Il n'y a pas d'opérateur d'affectation pour les tableaux.

Dans ce genre de cas, mieux vaut définir une structure simple pour le pixel, ce qui évite de se trimbaler des tableaux et des pointeurs et ce qui permet d'utiliser un vrai operateur d'affectation.