Discussion :

[artemis93]

Bonjour,
Je bloque sur ma méthode qui me donne une erreur à la compilation,j'ai crée une class avec un tableau d'images, je cherche à récupérer le nom de mes images qui se trouve dans une autre class puis en faire une liste:
l'erreur est à ligne:
Collection.java:50: ')' expected
this.mesImagesLues[j]=new Bitmap(String n);

je suis pas sûr d'avoir bien fait pour récupérer le nom de l'image
Merci de votre aide.

ma collection d'images :

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
import java.io.*;
import java.util.Scanner;
public class Collection {	
	Scanner sc = new Scanner(System.in);
 
	Bitmap []mesImagesLues;
	int nbImagesLues=0;
 
	Bitmap []mesImagesCrees;
	int nbImagesCrees=0;
 
	Collection(){	//constructeur
		Bitmap []mesImagesLues=new Bitmap[0];
		this.nbImagesLues=0;
		Bitmap []mesImagesCrees=new Bitmap[0];
		this.nbImagesCrees=0;
	}
 
	void ajouterImageLues(Bitmap A){ 
		Bitmap []ImgLuesNv=new Bitmap[mesImagesLues.length + 1];
		for (int i=0; i<mesImagesLues.length; i++)
			ImgLuesNv[i]=mesImagesLues[i];
		ImgLuesNv[mesImagesLues.length]=A;
		mesImagesLues=ImgLuesNv;
 
	}
	void ajouterImageCrees(Bitmap X){ 
		Bitmap []ImgCreesNv=new Bitmap[mesImagesCrees.length + 1];
		for (int i=0; i<mesImagesCrees.length; i++)
			ImgCreesNv[i]=mesImagesCrees[i];
		ImgCreesNv[mesImagesCrees.length]=X;
		mesImagesCrees=ImgCreesNv;
	}
 
	void supprimerImageLues(){		
		System.out.println("Quelle image voulez-vous supprimer ?");
		int Ind=sc.nextInt();
		int IndSup=Ind-1;
		Bitmap []ImgLuesNv=new Bitmap[mesImagesLues.length - 1];
		for( int i=0;i<IndSup;i++)
			ImgLuesNv[i]=mesImagesLues[i];
		for( int i=0;i<ImgLuesNv.length-IndSup;i++)
			ImgLuesNv[i+IndSup]=mesImagesLues[i+(IndSup+1)];
		mesImagesLues=ImgLuesNv;
	}
 
	Collection saisirCollection(){ 
		for(int j=0;j<this.mesImagesLues.length;j++){	
			this.mesImagesLues[j]=new Bitmap(String n); //erreur à la compil pourtant je respecte le constructeur Bitmap
			this.mesImagesLues[j].ie=new ImageExterneLue(n);
		}
		return this;
	}
 
	void afficherCollection(){
		System.out.println("liste d'images");
		for(int i=0;i<mesImagesLues.length;i++){
			System.out.println( (i+1)+ "- ");
			mesImagesLues[i].saisiCollection();
		}
	}
}

ma class Images:

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
33
34
35
36
37
38
39
40
41
42
43
import java.io.*;
import java.util.Scanner;
 
public class Bitmap {
    Scanner sc = new Scanner(System.in);
    int[][][]data;
    String nom;
    ImageExterne ie;
    Bitmap(String n) throws IOException, FileNotFoundException { // constructeur pour les images lues
        this.ie=new ImageExterneLue(n);
        this.nom=n;
        this.data=((ImageExterneLue)ie).getTableau();
    }
    Bitmap(int largeur, int hauteur){		// constructeur pour les images créés
        this.data=new int[largeur][hauteur][3];
        for(int x=0;x<this.data.length;x++){  
                    for(int y=0;y<this.data[0].length;y++){
                            this.data[x][y][0]=250;
                            this.data[x][y][1]=250;
                            this.data[x][y][2]=250;
                    }
        }
        this.nom=null;
        this.ie=new ImageExterneCree(this.data);
    }
    Bitmap(int largeur, int hauteur,int R,int V, int B){
        this.data=new int[largeur][hauteur][3];
        for(int x=0;x<this.data.length;x++){
                    for(int y=0;y<this.data[0].length;y++){
                            this.data[x][y][0]=R;
                            this.data[x][y][1]=V;
                            this.data[x][y][2]=B;
                    }
        }
        this.nom=null;
        this.ie=new ImageExterneCree(this.data);
    }
   Bitmap creerImage(Bitmap ImgLue){		//Bitmap creerImage(Bitmap ImgLue){
    	    this.data=ImgLue.data;
    	    this.nom=null;
    	    this.ie=new ImageExterneCree(this.data);
    	    return this;
   }

et la dernière utilisée dans la class Bitmap:

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
import javax.imageio.*; 
import java.awt.image.*;
import java.io.*;
public class ImageExterneLue extends ImageExterne{
    public ImageExterneLue(String nomf) throws IOException, 
					      FileNotFoundException{
	img = ImageIO.read(new File(nomf));
	this.showImage();
    }
    public int[][][] getTableau(){
	int wid, hei;
	wid = img.getWidth();
	hei = img.getHeight();
	int[][][] res = new int[wid][hei][3];
	int[] rgb = img.getRGB(0,0,wid,hei,null,0,wid);
	for (int i=0; i<rgb.length; i++){
	    res[i%wid][i/wid][0]=rgb[i]>>16 & 0xFF;
	    res[i%wid][i/wid][1]=rgb[i]>>8 & 0xFF;
	    res[i%wid][i/wid][2]=rgb[i] & 0xFF;
	}
	return res;
    }

[thelvin]

Regarde sur la ligne suivante, comment le constructeur est appelé.

Sur la ligne suivante, il y a juste n, pas String n, et il n'y a pas d'erreur. Pourquoi "String" n, au juste ?

[artemis93]

le constructeur pour les images lues est comme ceci:

Code :

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

1
2
3
4
    Bitmap(String n) throws IOException, FileNotFoundException { // constructeur pour les images lues
        this.ie=new ImageExterneLue(n);
        this.nom=n;
        this.data=((ImageExterneLue)ie).getTableau();

c'est pour ça que j'ai mis un (String n) dans ma methode:

Code :

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

1
2
3
4
5
6
7
Collection saisirCollection(){ 
		for(int j=0;j<this.mesImagesLues.length;j++){	
			this.mesImagesLues[j]=new Bitmap(String n); //erreur à la compil pourtant je respecte le constructeur Bitmap
			this.mesImagesLues[j].ie=new ImageExterneLue(n);
		}
		return this;
	}

[thelvin]

Oui, tout comme le constructeur de ImageExterneLue

Pourtant, lui, quand tu l'appelles, tu ne fais pas String n, tu fais juste n. Alors pourquoi tu fais pas pareil avec Bitmap ?

[artemis93]

J'ai déjà essayé il me dit qu'il ne reconnait pas la variable n

[thelvin]

Bon, eh ben ce n'est pas la même erreur, ça avance .

Il se plaint que la variable n n'existe pas. Puisque ça n'existe pas, tu ne peux pas le passer en paramètre d'un constructeur ou d'autre chose : on ne peut faire des choses qu'avec ce qui existe.

Ces constructeurs veulent une String en paramètre, fais en sorte d'avoir une String à leur donner, et donne-leur.

[Nico02]

Salut,
J'avoue ne pas très bien comprendre la ligne

Code :

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

 this.mesImagesLues[j]=new Bitmap(String n);

String n ça veut rien dire c'est pas une chaîne de caractères, deplus à la ligne suivante tu écris

Code :

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

this.mesImagesLues[j].ie=new ImageExterneLue(n);

mais la variable n n'existe pas, elle n'est ni dans ta méthode, ni en attribut de ta classe

[artemis93]

Bonjour

Code :

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

this.mesImagesLues[j]=new Bitmap(String n);

mesImagesLuesest mon tableau de type Bitmap.Or dans la class Bitmap le constructeur prend en parametre un String qui represente le nom du fichier :
image.jpg par exemple. J'ai donc instancié chaque case de mon tableau en type Bitmap en donnant un String en parametre comme le constructeur( je ne suis pas sûr que c'est bon)
Puis

Code :

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

1
2
 
this.mesImagesLues[j].ie=new ImageExterneLue(n);

ImageExterneLue c'est une class qui permet d'ouvrir des images .jpg sur le disque.
Etant donné qu'il ya une réference ie de type ImageExterneLue dans la classBitmap , je l'instancie aussi.

J'avoue que ce n'est pas facile!
Merci

[tchize_]

Comme on l'a dit ce 'String' a rien à faire là, "String n" ca veux dire "déclarer une variable n de type String", on ne peux pas faire une déclaration à cet endroit du code. Remplace par

Code :

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

1
2
3
			String n = "blabla";
			this.mesImagesLues[j]=new Bitmap(n);
			this.mesImagesLues[j].ie=new ImageExterneLue(n);

Bon bien sur, a toi de mettre ce qui est correct dans n