Problème de lecture d'un répertoire

Version imprimable

Bonjour à tous,
sur un petit programme qui se propose d'afficher des images provenant d'un répertoire, j'ai installé deux boutons 'précédent' et 'suivant' dont l'action, si elle est évidente, revient au début ou à la fin du répertoire quand on atteint les limites. Le bouton suivant ne pose pas de problème, mais le 'précédent' bloque. voici le bout de code (c'est peut-être pas d'une grande ingéniosité, et c'est peut-être même "sale" mais bon..je suis autodidacte...mais aussi ouvert à toutes améliorations)
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 public void listeRepertoire (File repertoire) { // ces six premières lignes ont été chopées sur la faq, merci à Clément Cunin if (repertoire.isDirectory) { File [] list = repertoire.listFiles(extension); //'extension' est un filtre pour ne lire que des images if (list!=null) { for (int i =0; i<list.length;i++) { list.repertoire (list[i]); //c'est là que j'interviens : if (list[i].equals(f)) { if(i+1 ==list.length) { fichierSuivant=list[0]; break; } else { fichierSuivant=list[i+1]; } //ce bloc fonctionne //à partir de là, ça déconne : if (i==0) { fichierPrecedent=list[list.length-1]; break. } else { fichierPrecedent=list[i-1]; } } } } } }
En fait, j'arrive à afficher le dernier fichier en faisant 'precedent', mais après ça bloque, je ne peux plus remonter encore (mais je peux toujours avancer)..
Je pense que le problème est lié à la boucle 'for' et à l'incrémentation de 'i', mais je ne comprends pas ...
Si vous avez un peu de temps pour me répondre, merci beaucoup...

20/01/2012, 14h52
dinobogan

Il nous faut aussi le code qui utilise cette fonction.

ok,

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
 
class precedentListener implements ActionListener {
		@Override
		public void actionPerformed (ActionEvent pre) {
			if (pre.getSource() == prec) {
				listing liste=new listing();
				liste.listeRepertoire(dir);
 
				f=fichierPrecedent;
 
				try {
					img = ImageIO.read(f);
 
 
				} catch (IOException e) {
					e.printStackTrace();
				}effetImage eft = new effetImage();
				eft.redimensionner();
				pic.repaint(); 
 
			}
		}
	}
 
 
 
class listing{
 
		FileFilter ext = new FileFilter () {
			public boolean accept (File f) {
				if (f.isDirectory()) return true;
				else if ( f.getName().endsWith(".jpg")) return true;
				//etc...
				else return false;
			}
		};
		public  void listeRepertoire (File repertoire){
			texteCheck.append(repertoire.getAbsolutePath()+newLine );
			if (repertoire.isDirectory()) {
 
				File [] list = repertoire.listFiles(ext);
				if (list!=null) {
					for (int i = 0; i<list.length;i++){
 
						listeRepertoire(list[i]);	
						nombreFichier=list.length;
						 if (list[i].equals(f) ) { 
 
 
							if(i+1==list.length) {
								fichierSuivant =list[0];
								break;
							}else {
							fichierSuivant=list[i+1];
							}
 
							if (i==0) {
 
							fichierPrecedent=list[fichierActuel];
								break;
							}else {
 
 
								fichierPrecedent=list[i-1];
							}
 
 
						} 
 
 
 
						}

20/01/2012, 17h29
dinobogan

Quelle est cette variable "fichierActuel" ?
20/01/2012, 17h57
amatthieu
désolé... j'ai repiqué le code en supprimant des tests... j'ai oublié d'enlever cette variable...en fait c'est :
Code:

1 2 3 4 5 if (i==0) { fichierPrecedent=list[list.length-1]; break; }
la variable 'fichierActuel' était un doublon de 'i'
merci en tous cas à vous de répondre à mes petits problèmes...

(d'ailleurs, si jamais quelqu'un ici a la patience de regarder le code complet -400 lignes- et me dire les améliorations à faire, je serais ravi et infiniment reconnaissant).
24/01/2012, 11h14
dinobogan

C'est le bazar. Re-poste un code minimal compilable reproduisant le problème.