Discussion :

[raph.g]

bonjour a tous forumer, je viens a vous apres de longues recherches sur le web en vain.
Je vous explique le probleme : je desire realiser une application de retouche d'images élémentaires avec une interface graphique en visual studio et une dll en c++. La partie visual no problem par contre en c++ j'ai choisi d'utiliser du code managé (apparement plus simple...)

J'arrive a creer mon image depuis vb en utilisant ma dll jarrive a appeler mes methodes c++ depuis vb MAIS je n'arrive pas a modifier l'image en c++ car il me sort une erreur enorme du style : memoire protégée en acces.

Je m'explique :

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
 
//class pixel
public class pixel
{
public:
	int r;
	int g;
	int b;
 
	pixel(void);
	pixel(int,int,int);
	void affichage();
 
};
 
//class monimage
public ref class monimage
{
private:
	pixel*** matrice;
	int x;
	int y;
	int nbcouleurs;
	int type;
public:
	monimage(void);
	monimage(System::String^);
	~monimage(void);
	void affichage();
};

le probleme vient de l'acces a matrice dans la fonction affichage qui est la suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void monimage::affichage(){
for (int i=0;i<x;i++){
		cout<<endl;
		for (int j=0;j<y;j++){
			matrice[i][j]->affichage();
 
		}
	}
}
void pixel::affichage(){
	cout<<this->r<<" "<<this->g<<" "<<this->b<<" ";
}

je peux faire cet affichage si je cree une matrice dans un bloc et que jappel dans ce meme bloc la fonction affichage()
Mais des que j'appel la fonction affichage() avec une image cree en dehors du meme bloc.
PAF
dans pixel::affichage il me dit que this->r ou g ou b est protégé...
Hors grace au debuger vb j'arrive a voir letat de la matrice juste avant lappel et elle est bien construite c est juste que il ne veut pas acceder a ses attributs...
et je construit la matrice comme ceci :

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
 
char* p=(char*)malloc(sizeof(char)*3);
	i=0;
	j=0;
	int* val=(int*)malloc(sizeof(int)*3);
	matrice=new pixel**[x];
	if(type==6){
		for(int dx=0;dx<x;dx++){
			matrice[dx] = new pixel*[y];
			for(int dy=0;dy<y;dy++){
 
				while(monstream.eof()!=true){
					val[i]=monstream.get();
					if(i==2){
						//cout<<val[0]<<" "<<val[1]<<" "<<val[2]<<" ";
						matrice[dx][dy]=new pixel(val[0],val[1],val[2]);
						//matrice[dx][dy]->affichage();
						i=0;
					}
					i++;
				}
			}
		}
	}

quelqu'un aurait une idee??
merci beaucoup!

[Invité]

Bonjour,
Je n'ai pas tout regardé en détail, mais il y a des choses que je ne comprend pas.
Votre tableau "matrice" est-il à 2 ou 3 dimensions?
En fait vous le déclarez à 3 dimensions et l'utilisez à 2 dimensions. D'ailleurs, une image est à 2 dimensions.
Vous faites une classe "pixel", pourquoi pas, mais je n'ai pas vu où étaient développés les 2 constructeurs de cette classe

Code :

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

1
2
3
void pixel::affichage(){
	cout<<this->r<<" "<<this->g<<" "<<this->b<<" ";
}

Pourquoi n'écrivez-vous pas tout simplement

Code :

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

1
2
3
void pixel::affichage(){
	cout<<r<<" "<<g<<" "<<b<<" ";
}

Et c'est pas tout

[Steph_ng8]

Votre tableau "matrice" est-il à 2 ou 3 dimensions?

C'est un tableau à 2 dimensions de pointeurs sur des « pixel ».
Par contre, le troisième niveau de pointeurs ne me paraît pas indispensable…

Vous faites une classe "pixel", pourquoi pas, mais je n'ai pas vu où étaient développés les 2 constructeurs de cette classe

Sans doute ne sont-ils pas indispensables pour comprendre le problème…

Code :

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

1
2
3
4
5
pixel::pixel(void) : r(), g(), b()
{}
 
pixel::pixel(int r, int g, int b) : r(r), g(g), b(b)
{}

Code :

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

1
2
3
void pixel::affichage(){
	cout<<this->r<<" "<<this->g<<" "<<this->b<<" ";
}

Pourquoi n'écrivez-vous pas tout simplement

Code :

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

1
2
3
void pixel::affichage(){
	cout<<r<<" "<<g<<" "<<b<<" ";
}

Et pourquoi pas ?
L'utilisation explicite du pointeur « this » n'est certes pas (toujours) obligatoire, mais elle permet de repérer visuellement et rapidement que l'on travaille avec des données ou des fonctions membres.
Ça peut être pratique lorsque l'on travaille à plusieurs, sur un gros projet, ou que l'on reprend un ancien code…

[Steph_ng8]

Question:
Les « public class … » et « public ref class … », c'est une syntaxe particulière à visual studio/vb ?
Quelle est sa sémantique ?
Pareil pour « System::String^ » (l'accent circonflexe à la fin du type).

@raph.g Pas sûr que ce soit une bonne idée de mélanger les « malloc » et les « new »…
C'est du C++, que diable !
Vive le « new » !

Sinon, désolé mais je ne vois pas d'où vient le problème.

[Guulh]

Question:
Les « public class … » et « public ref class … », c'est une syntaxe particulière à visual studio/vb ?

C'est du C++/CLI, un langage d'interop managé / non managé.

Pour en revenir à la question initiale : c'est un peu dommage d'utiliser le framework .Net et de ne pas l'utiliser... La namespace System.Drawing, contient plein de classe de gestion d'images (Image, Bitmap, ...) Que tu as intérêt à utiliser plutôt que tout te recarrer à la main. Et à moins d'une contrainte particulière (utilisation d'un lib C++ existante par exemple), mieux vaut éviter l'interop...

[Steph_ng8]

Ok, merci Guulh.