[File][JFileChooser]Ajouter une extension pour "Save as

Version imprimable

Bonjour à tous.

Je suis en train d'implenter une fonction "Save as..." sur mon logiciel, et quand le JFileChooser s'ouvre, je laisse à l'utilisateur le choix de choisir entre plusieurs type de fichiers (.xml, .txt...).

J'aimerais bien que l'utilisateur ne soit pas obligé de mettre une extension au nom qu'il a choisit, quand il chercher à enregistrer son fichier.
En gros, si il choisit le format ".xml" et qu'il tape le nom "monFichier", je voudrais lui enregistrer son fichier sous le nom "monFichier.xml".
Or, une fois que j'ai récupèré mon fichier depuis le JFileChooser (.getSelectedFile())
je ne peux plus modifier son nom.

Voici mon code, si quelqu'un y voit une erreur...
Code:

1 2 3 4 5 6 7 8 9 10 11 12 exptFile=exportFileChooser.getSelectedFile(); String fileName = new String(exptFile.getAbsolutePath()+cff.getExtension()); System.out.println("fileName "+fileName); File tempFile = new File(fileName); File tempFile2 = new File(fileName.concat(".coucou")); if(!tempFile.renameTo(tempFile2)) System.out.println("renameTo qui ne marche pas !!!"); System.out.println("tempFile.getAbsolutePath "+tempFile.getAbsolutePath()); System.out.println("tempFile2.getAbsolutePath "+tempFile2.getAbsolutePath()); System.out.println("exptFile.getAbsolutePath "+exptFile.getAbsolutePath()); if(!exptFile.renameTo(tempFile)
Avec ce genre de déclaration

Code:

File exptFile=null; JFileChooser exportFileChooser;

et cff une extension de FileFilter.

Dans mon code ci-dessus, tempFile2 N'est PAS renommé. Il a donc le même nom que tempFile auquel est ajouté l'extension ".coucou".
De même, quand j'essaie de renommer le fichier qui m'interresse vraiment (exptFile), ça ne marche pas...

27/01/2006, 01h16
Gfx
renameTo() permet de renommer un fichier qui existe sur le disque dur. Pourquoi ne pas simplement faire cela :
Code:

1 2 3 4 5 exptFile=exportFileChooser.getSelectedFile(); if (!exptFile.endsWith(".xml")) { exptFile += ".xml"; }
Et c'est tout !
27/01/2006, 10h08
leminipouce

Ben parceque je me choppe une petite erreur :

Code:

The operator += is undefined for the argument type(s) File, String

Laisse moi deviner, j'aurais dû préciser que je suis sous Java 1.4.2_06, n'est-ce pas ?

PS: L'erreur m'est annoncée par Eclipse 3.1 mon EDI.

Non rien a voir avec Java 1.4 (file vite sous 1.5 gredin !). C'est moi qui me suis trompe. Donc on recommence :
Code:

1 2 3 4 5 6 String selectedFile = exportFileChooser.getSelectedFile().getAbsolutePath(); if (!selectedFile.endsWith(".xml")) { selectedFile+= ".xml"; } exptFile = new File(selectedFile);

Salut,

Juste un détail : je te conseille juste d'ajouter un toLowerCase() dans la condition afin de ne pas rajouter une extension si elle est en majuscule :
Code:

1 2 3 4 5 String selectedFile = exportFileChooser.getSelectedFile().getAbsolutePath(); if (!selectedFile.toLowerCase().endsWith(".xml")) { selectedFile+= ".xml"; } exptFile = new File(selectedFile);
a++

27/01/2006, 10h46
leminipouce

Citation:

Envoyé par Gfx

file vite sous 1.5 gredin !

Diantre, j'aimerais bien, par contre ça va pas être facile à faire accepter au client. (En fait je dis ça alors que c'est complètement impossible... leur déploiement est un brin long. J'avais déjà fait le code en 1.5, et j'ai été obligé de faire marche arrière ! J'espère qu'il feront l'effort de migrer, dans quelques mois, sur la 1.6 :D)

Bon sinon pour le code, c'est super !

Merci beaucoup à vous deux.

En fait, mon problème c'est que je récupérais mon fichier avant de le renommer. Cela dit, si vous avez une minute (m'enfin il y en a peut-être qui veulent dormir ;)) je veux bien que vous m'expliquiez pourquoi le renameTo ne marche pas dans mon cas (n i pour le fichier à sauver, ni même pour les deux fichiers temporaires que je créais en test.
27/01/2006, 11h11
Gfx

Et dire qu'on planche deja sur la 1.7 nous :)

Faire un new File(...) ne cree pas le fichier sur disque dur, le renameTo() ne peut donc pas marcher. File() peut representer un fichier physique mais aussi virtuel.
27/01/2006, 11h25
leminipouce

J'y pensais... Il m'aurait donc fallu, je suppose, utiliser File.createFile(...) ?
27/01/2006, 11h32
Gfx

File.createNewFile() plus precisement, mais la tu risques d'ecraser un fichier de l'utilisateur s'il a le meme nom. Il vaut mieux se contenter de manipuler des chaines de caracteres et de creer le fichier a la fin (s'il n'existe pas deja :).
27/01/2006, 11h40
leminipouce

Exact. Comme quoi, il ne faut pas que je dise des choses de tête.... j'ai pas une bonne tête !

Par contre, tu fais bien de me dire ça, car ça me fait penser qu'il faut que je test si le fichier n'existe pasa déjà, quand je lui ajoute une extension...

Merci beaucoup.

Je pofine mon code, et je le resoumets...

Voilà, je crois que mon code c'est un peu embelli.

En tout cas, il marche comme je le veux !

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
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
	if (tmsFile == null) {	
		//Get selected input file and create window for it.
		int returnVal = exportFileChooser.showSaveDialog(this);
		if (returnVal != JFileChooser.APPROVE_OPTION) return null;
 
		//Retreive file types allowed for the export function
		FileFilter[] fileFilters = exportFileChooser.getChoosableFileFilters();
 
		//Retrieve file name typed in by the user, in order to verify that it has a correct extension
		String selectedFile = exportFileChooser.getSelectedFile().getAbsolutePath();
		String[] extensions = new String[fileFilters.length-1];	//Save allowed extension for further display
		int nbExt = 0;
		boolean hasExtension=false;
		//Check, if the file name is one of the file type allowed, making comparison between extensions. 
		for(int i=0;i<fileFilters.length && !hasExtension && selectedFile!=null;i++){
			//Do not compare extension with the "All file" filter.
			if(!fileFilters[i].equals(exportFileChooser.getAcceptAllFileFilter())){
				extensions[nbExt++]=((CommonFileFilter)fileFilters[i]).getExtension();
				if(selectedFile.toLowerCase().endsWith(((CommonFileFilter)fileFilters[i]).getExtension()))
					hasExtension=true;
			}
		}
 
		//If the file does not have the correct extension, or has no extension...
		if (!hasExtension) {
			//...add the one defined by the filter type selected.
			if(!exportFileChooser.getFileFilter().equals(exportFileChooser.getAcceptAllFileFilter())){
				CommonFileFilter cff = (CommonFileFilter)exportFileChooser.getFileFilter();
				selectedFile+= cff.getExtension();
			}else{
				//If the filter selected is "All files", then inform that the user
				//have to enter an extension.
				StringBuffer allowedExtensions=new StringBuffer(extensions[0]);
				for(int i=1; i<extensions.length;i++){
					allowedExtensions.append(" or "+extensions[i]);
				}
 
				JOptionPane.showMessageDialog(
						this,
						"MySoft could not proceed with EXPORT...."
							+ "\nPlease make sure that the file name ends with"
							+ allowedExtensions,
						"Notice", JOptionPane.ERROR_MESSAGE);
					return null;
			}
		}
		exptFile = new File(selectedFile);
 
		//Check if the file does not already exist
		File parentFile = exptFile.getParentFile();	//Work directory
		File[] otherFiles = null;							//List all the files of the work directory
 
		boolean alreadyExist = false;
		boolean overwriteIt = true;
 
		if(parentFile!=null){ 
			otherFiles = parentFile.listFiles();	//Retrieve the list of all files contained into the work directory
 
			for(int i=0;i<otherFiles.length && !alreadyExist;i++){
				//If the file already exist in the work directory,
				//ask the user if he wants to overwrite or not the file.
				if(otherFiles[i].equals(exptFile)){
					int chosen = JOptionPane.showOptionDialog(null, 
							"The file "+exptFile.getName()+" already exists."+
							"\nAre you sure you want to overwrite it?",
							"File already exists", 
							JOptionPane.YES_NO_OPTION, JOptionPane.WARNING_MESSAGE,
							null, null, null);
 
					switch(chosen) {
						case JOptionPane.YES_OPTION:
							overwriteIt=true;
							break;	
 
						case JOptionPane.NO_OPTION:
							overwriteIt=false;
							break;
 
						case JOptionPane.CLOSED_OPTION:
							overwriteIt=false;
							break;
					}
 
					alreadyExist=true;
				}
			}
		}
 
		//Exit the export function if the user does not want to overwrite
		//the file, or if it is impossible to create the file.
		if (exptFile == null || !overwriteIt) return null;
	}

et après bien sûr, je fais mon traitement habituel, avec remplissage du fichier...

Une fois de plus, Merci beaucoup.