ce code est juste?

Version imprimable

ce code est juste?

est ce que ce boue de code est juste ou non. sachant que je veux enregistrer le contenu d'un JTextArea sur un fichier dont le non et le chemin sont donnés par JFileChooser. voici mon code sur une action performed:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
    private void jButton3_actionPerformed(ActionEvent e) {
        JFileChooser jfilechooser1=new JFileChooser();
        if (jfilechooser1.showSaveDialog(this) == jfilechooser1.APPROVE_OPTION){
            try{
            File file = new File("jfilechooser1.getAbsolutePath()\\jfilechooser1.getName().txt");
 
                BufferedWriter buffWrit = new BufferedWriter(new FileWriter(file));
                buffWrit.write( jTextArea1.getText());
                buffWrit.flush();
                buffWrit.close();}
            catch(IOException f){
 
            }
            }
    }

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
private void jButton3_actionPerformed(ActionEvent e) {
        JFileChooser jfilechooser1=new JFileChooser();
        if (jfilechooser1.showSaveDialog(this) == jfilechooser1.APPROVE_OPTION){
            try{
 
                BufferedWriter buffWrit = new BufferedWriter(new FileWriter(jfilechooser1.getAbsolutePath()));
                buffWrit.write( jTextArea1.getText());
                buffWrit.flush();
               }
            catch(IOException f){
 
            }
finally{ buffWrit.close();}
            }
    }

attention au finally si adiGuba passe par là ;) (dédicasse)

24/04/2008, 17h07
natha
Pour savoir si le code est juste tu peux le tester.
Par contre j'adooooore ce genre de chose :
Code:

1 2 3 catch(IOException f){ }
Et s'il y a une erreur ? il se passe quoi ?
Bah rien... et donc si ça marche pas, on sait pas pourquoi !

Ne JAMAIS laisser un bloc d'exception tout vide comme ça ! Il faut au moins logger l'erreur.
24/04/2008, 17h42
Tommy31

En plus la méthode close() peut jetter une exception, qu'il faudrait catcher.
24/04/2008, 18h32
soujava

ce code ne contient aucune erreur syntaxique mais sémantique c'est sur. plus précisément quand j'écris sur un JTextArea et je clique sur le bouton3 puis je choisis le chemin et le non du fichier, normalement un fichier est enregistré avec le contenu du JTextArea alors pour ce code aucun fichier n'a été enregistré?? ma question est a votre avis pourquoi ?
24/04/2008, 18h42
natha

Parce que ça ça ne risque pas de fonctionner :

Code:

new File("jfilechooser1.getAbsolutePath()\\jfilechooser1.getName().txt");
24/04/2008, 19h11
soujava

précisez s'il vous plais
24/04/2008, 19h25
Icefire

car tu mets en string ton contenu de Jfilechooser...

ça serait plutôt :

new File(jfilechooser1.getAbsolutePath()+"\\"+jfilechooser1.getName()+".txt");

à noter que le plus sert de combinaision pour les strings.

Là il aurait essayé de faire un fichier au répertoire \jfilechooser1.getAbsoluthePath....... de la manière que tu l'as fait !

J'espère que ça t'aide
24/04/2008, 19h28
natha

Citation:

Envoyé par soujava

précisez s'il vous plais

Tu pourrais réfléchir un peu aussi.
25/04/2008, 06h37
Icefire

j'ai dû regarder vraiment vite... parceque ce que je t'ai écrit avait des erreurs dedans.

je crois qu' absoluthe path a déjà tout le path dedans... pourquoi rajouter le nom + "txt" ? En plus, je crois que getName() va te donner l'extension du fichier ...

new File(jfilechooser1.getAbsolutePath());

mais... c'est le genre de test que tu devrais tester toi même.

Exemple:

String path = jfileChooser1.getAbsolutePath();

System.out.println(path);

..... new File(path);

Tu vas voir si ton path a de l'allure et si c'est vraiment ce que la eprsonne a saisi.

Puis côté variables... je te conseille tout suite de mettre des noms de variables appropriés... car je te jure que quand tu feras des classes de 1000 lignes et plus, ça va te fatiguer des button3 et jfilechooser1
25/04/2008, 17h30
Maeuh
normalement jfilechooser1.getText() te donne directement le chemin complet pointant sur le fichier sélectionné

tu peut faire par exemple :
Code:

1 2 3 4 5 6 String pathFile= jfilechooser1.getText(); File model= new File(pathFile); if (model.isFile() && model.canRead()) { //traitement adéquat }
25/04/2008, 17h43
19cmos83
Citation:
Citation:

Envoyé par natha

Pour savoir si le code est juste tu peux le tester.
Par contre j'adooooore ce genre de chose :

Code:

1 2 3 catch(IOException f){ }

Et s'il y a une erreur ? il se passe quoi ?
Bah rien... et donc si ça marche pas, on sait pas pourquoi !

Ne JAMAIS laisser un bloc d'exception tout vide comme ça ! Il faut au moins logger l'erreur.
bref ...
Code:

1 2 File file = new File("jfilechooser1.getAbsolutePath()\\jfilechooser1.getName().txt");
heu ? tu essaies de faire quoi au juste à ce moment précis ?
25/04/2008, 22h15
soujava

je sais ou est l'erreur mais comment faire pour récupérer le chemin et le non du fichier ou je dois enregistrer le contenu du JTextArea??
26/04/2008, 11h12
jojodu31

je te l'ai dis moult fois...

Code:

jfilechooser1.getAbsolutePath()

ceci te donne le chemin COMPLET teste le donc ...
26/04/2008, 12h54
19cmos83

La méthode : getAbsolutePath()
Returns the absolute pathname string of this abstract pathname.
26/04/2008, 18h31
soujava

mais java n'a pas accepté cela jfilechooser1.getAbsoluPath()
26/04/2008, 19h30
19cmos83

Citation:

java n'a pas accepté ...

demande lui pourquoi, et dis nous ensuite pourquoi, pour qu'on puisse essayer de t'aider ...