Discussion :

[armoirapizza]

Bonjour,
j'ai écrit une petite méthode pour retourner horizontalement une image mais elle me renvoie une image toute noire...
Si quelqu'un a une idée...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public static int[][][] inverserImage(int[][][]img, int [][][] tmp){
   tmp= new int[img.length][img[0].length][3];
       for(int i=img.length; i>0; i--){
	   for(int j=0; j<img[0].length; j++){
		if(i<img.length  && j<img[0].length && (img[i][j][0]>0 ||img[i][j][1]>0 ||img[i][j][2]>0)){
		     tmp[i][j][0]=img[img.length-1][j][0];
		     tmp[i][j][1]=img[img.length-1][j][1];
		     tmp[i][j][2]=img[img.length-1][j][2];
		}
	   }
      }
}

[supersnail]

Bonjour,

T'as pas oublié un "return tmp" quelque part?

[armoirapizza]

dans le copie coller oui
pas dans mon code...

[thelvin]

Tu dois avoir un ArrayIndexOutOfBoundsException quelque part, puisque ton i commence à image.length et que tu l'utilises direct comme index de tableau. Il faut commencer à image.length - 1. (Et ne pas exclure le zéro, je pense.)

Comme il y a une Exception tout de suite, la méthode ne fait presque rien et lance une Exception.

Tu dois sûrement avoir un catch(Exception e) quelque part qui n'affiche pas ton Exception, et à cause de ça tu ne vois pas qu'il y a eu un problème. C'est mal.

[armoirapizza]

lol.."c'est mal" on dirait mon prof!
j'avais vu pour le length-1.
ça renvoie une image toute noire...mais de la même taille!

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
public static int[][][] inverserImage(int[][][]img, int [][][] tmp){
		tmp= new int[img.length][img[0].length][3];
		for(int i=img.length; i>0; i--){
			for(int j=0; j<img[0].length; j++){
				if(i<img.length  && j<img[0].length && (img[i][j][0]>0 ||img[i][j][1]>0 ||img[i][j][2]>0)){
					tmp[i][j][0]=img[img.length-1][j][0];
					tmp[i][j][1]=img[img.length-1][j][1];
					tmp[i][j][2]=img[img.length-1][j][2];
				}
			}
		}
		//for(int i=tmp.length; i>=0; i--){
		//	for(int j=0; j<img[0].length; j++){
		//		if(i<img.length  && j<img[0].length && (tmp[i][j][0]>0 ||tmp[i][j][1]>0 ||tmp[i][j][2]>0)){
		//			img[i][j][0]=tmp[i][j][0];
		//			img[i][j][1]=tmp[i][j][1];
		//			img[i][j][2]=tmp[i][j][2];
		//		}
		//	}
		//}
		return tmp;
	}

[thelvin]

Comment ça tu "avais vu" ? L'erreur est toujours Là. Si tu la vois, enlève-la.

[armoirapizza]

Bonjour Thelvin,
j'ai fait la modif pour le départ de la boucle à length-1 et >=0.
le programme ne plantait pas, il créait bien un image et l'enregistre mais elle est toute noire.
En tout cas merci.

[thelvin]

Ah, effectivement, pardon, en bricolant ton code j'avais enlevé le test dans ta boucle. Au passage, il n'a rien à faire ici. La boucle devrait commencer directement à la bonne valeur.

Dans ta boucle, tu ne copies que des valeurs dont l'abscisse est img.length-1, c'est-à-dire la dernière colonne de ton image. Suffit qu'elle soit toute noire, et voili-voilà.

[armoirapizza]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public static int[][][] inverserImage(int[][][]img, int [][][] tmp){
		tmp= new int[img.length][img[0].length][3];
		for(int i=0,  j=img.length-1; i<j; i++, j--){
			for( int k=0; k<img[0].length; k++){
				if(i<img.length  && j<img[0].length && (img[i][j][0]>=0 ||img[i][j][1]>=0 ||img[i][j][2]>=0)){
					tmp[i][k][0]=img[i][k][0];
					tmp[i][k][1]=img[i][k][1];
					tmp[i][k][2]=img[i][k][2];
				}
			}
		}
		return tmp;
	}

Bonjour,
effectivement, le test ici ne sert à rien, c'est juste que si j'arrive à faire marcher cette méthode, je voulais proposer au passage de retirer une couleur pour la copier sur une autre. (Pourquoi faire simple quand on peut faire compliqué...)
En tout cas, l'image que je retourne est blanche sur la colonne concernée.
(j'ai essayé avec plusieurs)
Je ne comprends vraiment pas.