Discussion :

[stc074]

Salut, je dispose d'un JFormattedTextField, lequel me donne une valeur que je récupère et test a chaque fois qu'une touche est appuyée
voici mon code :

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
private javax.swing.JFormattedTextField jFormattedTextField1;
[...]
jFormattedTextField1 = new javax.swing.JFormattedTextField(NumberFormat.getNumberInstance());
[...]
        jFormattedTextField1.addKeyListener(new java.awt.event.KeyAdapter() {
            public void keyReleased(java.awt.event.KeyEvent evt) {
                changeDejaPaye(evt);
            }
        });
[...]
    private void changeDejaPaye(java.awt.event.KeyEvent evt) {
        // TODO add your handling code here:
        try {
            DecimalFormat df=new DecimalFormat();
            df.setMaximumFractionDigits(2);
            double dejaPaye=((Number)jFormattedTextField1.getValue()).doubleValue();
            double netAPayer=facture.getTotal().getTotalTTC()-dejaPaye;
            facture.getTotal().setNetAPayer(netAPayer);
            jLabel7.setText(df.format(netAPayer));
        } catch(java.lang.NullPointerException e) {}
    }

le problème c'est que le catch est appelé au moment ou j'essaie de récupérer la variable dejaPaye, je me suis creusé la tête mais je ne vois pas d'où peu venir le problème.
Merci pour votre aide.

[thelvin]

Ben, affiche la stackTrace : il te dira quelle ligne pose problème, et donc où se trouve la variable null.

[tchize_]

d'une manière générale, c'est une très mauvaise idée de mettre un catch vide. Soit vous ne gérez pas l'exception -> alors faites la remonter, ne la bloquez pas. soit vous gérez l'exception -> alors faites en quelques chose d'utile (traitement d'erreur, trace dans les logs, etc)

[stc074]

Ben, affiche la stackTrace : il te dira quelle ligne pose problème, et donc où se trouve la variable null.

il semblerait que ce soit cette ligne qui pose problème :

Code :

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

double dejaPaye=((Number)jFormattedTextField1.getValue()).doubleValue();

Merci !

[stc074]

apparement

Code :

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

jFormattedTextField1.getValue()

a une valeur nulle (null)
je vois pas trop comment faire.

[ninlock]

Salut,

soit tu gère un message à retourner à l'utilisateur si la valeur est null,
soit tu empêche l'action si la valeur est null, mais dans tous les cas il faut que tu nous précise le type d'exception retourné et vérifie que le champ à bien une valeur avec un system.out ou en mode debug.

De plus est ce que tu est bien sur que ton champ contient bien une valeur et à quel moment tu la renseigne

[stc074]

Salut,

soit tu gère un message à retourner à l'utilisateur si la valeur est null,
soit tu empêche l'action si la valeur est null, mais dans tous les cas il faut que tu nous précise le type d'exception retourné et vérifie que le champ à bien une valeur avec un system.out ou en mode debug.

De plus est ce que tu est bien sur que ton champ contient bien une valeur et à quel moment tu la renseigne

Que me conseil tu de mettre dans mon catch pour arriver a faire cela?
Perso j'ai affiché la valeur jFormattedTextField1.getValue() car je me doutais que c'était la valeur null, et en effet elle vaut null.
Merci

[ninlock]

Soit sur ton action du bouton tu vérifie que la valeur de ton champs et null,
soit tu laisse remonter l'exception.
Et dans les 2 cas
soit tu écrit un message dans ta fenêtre pour préciser de remplir le champ
soit tu ouvre une dialog Box en utilisant par exemple:

Code :

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

1
2
JOptionPane.showMessageDialog(parentPanel,"Veuillez saisir une valeur avant de valider");

Une autre solution, plus longue a mettre en place serait de désactiver le bouton tant que le champ est vide, en ajoutant bien sûr un listener sur le champ pour réactiver le bouton de validation quand il contient quelque chose.

Par contre, tu ne l as pas précisé mais quand tu rempli le champ tu arrive bien à récupérer la valeur?

[bulbo]

Moi je conseille de ne pas mettre de catch mais de tester si la valeur est nulle avant de vouloir récupérer la doubleValue.

Une exception c'est long et couteux a générer alors qu'il suffirait de faire un test pour voir si getValue retourne null.

Et dans le cas ou ça retourne null, tu ne fais rien, vu que tu n'as rien à faire.. un return est parfait dans ce cas là. Si tu veux informer l'utilisateur du problème tu peux utiliser un JOptionPane juste avant mais je déconseille.

Ce qui est le plus élégant pour ce genre de chose c'est un petit label en rouge en dessous du TextField qui donne la raison du problème.

Bulbo

[stc074]

Moi je conseille de ne pas mettre de catch mais de tester si la valeur est nulle avant de vouloir récupérer la doubleValue.

Une exception c'est long et couteux a générer alors qu'il suffirait de faire un test pour voir si getValue retourne null.

Et dans le cas ou ça retourne null, tu ne fais rien, vu que tu n'as rien à faire.. un return est parfait dans ce cas là. Si tu veux informer l'utilisateur du problème tu peux utiliser un JOptionPane juste avant mais je déconseille.

Ce qui est le plus élégant pour ce genre de chose c'est un petit label en rouge en dessous du TextField qui donne la raison du problème.

Bulbo

Oui c'est ce que j'ai fait (tester si le jFormattedTextField est null) mais le probleme c'est que la valeur dans le champ elle est correcte (un double ou un entier) c'est pour ça que je demande de l'aide.
J'ai aussi remarqué que si je mettais une valeur valide et j'appuyais sur ENTER, la ça fonctionne.
Merci

[bulbo]

J'ai dit de tester si la valeur est nulle pas le TextField.

Code java :

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

1
2
3
4
5
6
7
8
9
Double val = (Double) jFormattedTextField1.getValue();
 
if (null == val)
{
   // JOptionPane ??
  return;
}
 
// Ici la suite du code du listener

[stc074]

J'ai dit de tester si la valeur est nulle pas le TextField.

Code java :

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

1
2
3
4
5
6
7
8
9
Double val = (Double) jFormattedTextField1.getValue();
 
if (null == val)
{
   // JOptionPane ??
  return;
}
 
// Ici la suite du code du listener

Oui c'est ce que je comprends pas justement, pourquoi la valeur est nulle alors que le champ lui contient une valeur valide.
Merci

[bulbo]

Tu as dit toi même qu'en cas de valeur valide tu n'as pas de problème.. faut savoir.

Je pense que le formattedTextField te retourne null justement quand ce que tu as rentré n'est pas un double..