Discussion :

[polymorphisme]

Bonjour,

Suite à une analyse de code, Sonar me retourne le message suivant :

Déréférencement d'un pointeur null dans la méthode org.polymorphisme.swing.EssaisJButtonListener.actionPerformed(ActionEvent)"

en me précisant que le déréférencement est en rapport avec la ligne de code :

Code :

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

System.out.println(CLIQUER + str + ".");

Sauriez vous m'expliquer comment se fait-il qu'il y ait un déréférencement ?

Ma classe est simple :

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
public class EssaisJButtonListener extends JFrame implements ActionListener {
 
  private JButton jButtonIci;
  private JButton jButtonLa;
  private static final String CLIQUER = "Vous avez cliquer ";
 
  private JPanel buildContentPane() {
    JPanel jPanel = new JPanel();
    jPanel.setLayout(new FlowLayout());
    jButtonIci = new JButton(ICI);
    jButtonLa = new JButton(LA);    
    jButtonIci.addActionListener(this);
    jPanel.add(jButtonIci);
    jButtonLa.addActionListener(this);
    jPanel.add(jButtonLa);
 
    return jPanel;
  }
 
  public void actionPerformed(ActionEvent e) {
    Object source = e.getSource();
 
    String str = "";
 
    if (source.equals(jButtonIci)) {
      str = jButtonIci.getText();
    } else if (source.equals(jButtonLa)) {
      str = jButtonLa.getText();
    }
 
    System.out.println(CLIQUER + str + ".");    
  }

Merci d'avance pour votre aide.

[tchize_]

Je suis perplexe. A vue de pif, je suppose que sonar assume que JButton.getText() peux renvoyer null? Si tu retire le +str dans la ligne incriminée, le test passe?

[polymorphisme]

Bonjour,

moi aussi j'ai été surpris par cette remarque de Sonar,
d'autant plus qu'aucune information du compilateur ne l'indique,
j'ai vu écris un code similaire dans ces programmes que je crois bien écrit,
je pense à FOP par exemple.

Ceci étant, si j'enlève la chaine "+ str"

Code :

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

System.out.println(CLIQUER + ".");

Alors il n'y a plus de déréférencement.

A mieux regarder, j'ai remarqué que lorsque j'avais quelque chose tel que :

Code :

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

1
2
3
4
5
6
7
 UnObjet str = "";
 
    if (cond) {
      // affectation d'une valeur pour str
    }
 
    // instruction avec str.

Alors il y a déréférencement. Mais je ne vois pas pourquoi puisque mon objet a une valeur ?

[tchize_]

est-ce que, dans un des chemins, tu attribue une valeur qui viendrais d'un champ? Sonar pourrais dans un premier temps conclure que ce champ peux être null et, par conséquent, tout ce qui récupère une copie de ce champ peux devenir null...

[polymorphisme]

Non, ce n'est pas le cas.

Pas plus que dans mon petit exemple,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 public void actionPerformed(ActionEvent e) {
    Object source = e.getSource();
 
    String str = "";
 
    if (source.equals(jButtonIci)) {
      str = jButtonIci.getText();
    } else if (source.equals(jButtonLa)) {
      str = jButtonLa.getText();
    }
 
    System.out.println(CLIQUER + str + ".");    
  }

Je crois qu'il faudrait que j'aille fouiller la documentation de Sonar ou lire la classe correspondant à la détection d'un déréférencement.

[tchize_]

je viens de regarder le code de AbstractButton, et, effectivement, getText() peux renvoyer null.