erreur java.lang.NullPointerException dont je ne comprends pas la provenance

**louisard** · 02/12/2015, 19h03

Bonsoir a tous

J'ai réaliser une petite appli et j'aiune erreur dont je ne comprend pas la provenance,

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
		/*** on determine si l'appel vient du tableau ***/
			if (!frame.appelTableau()) {
				this.dispose();
				frame = new MainFrame();
			} else {
				this.dispose();
				String titr=getTitre();
				Double chapLu=getDernierScanLu();
				System.out.println(getTitre());
				System.out.println(info.getAuteur());
				System.out.println(info.getGenre());
				System.out.println(info.getDernierChap());
				System.out.println(info.getDate());
				System.out.println(getDernierScanLu());
				System.out.println(info.getStatut());
				System.out.println(con.getDate());
 
				frame.recupInfo(titr, InfosManga.getGenre(), info.getAuteur(), info.getDernierChap(),
						info.getDate(), chapLu, con.getDate(), info.getStatut());
 
		/*		model.addManga(new Manga(getTitre(), InfosManga.getGenre(), info.getAuteur(), info.getDernierChap(),
						info.getDate(), getDernierScanLu(), con.getDate(), info.getStatut()));
			//	frame.traitement(); */
			}

a la ligne 18 ( frame.recup . . . ) il me renvoie une erreur de java.lang.NullPointerException hors j'ai tester juster juste avant toutes les valeurs et elles contiennes bien une valeur, et apres test sans parametres je me suis appercu que c'est l'appel sur cette methode qui renvoyait null ce qui tout aussi incomprehensible pour moi

voila la classe appeler ( ligne 54 )

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
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
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
 
@SuppressWarnings("serial")
public class MainFrame extends JFrame {
 
	public MainFrame() {
		// setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		// Code
		Dimension dimension = java.awt.Toolkit.getDefaultToolkit()
				.getScreenSize();
		int height = (int) dimension.getHeight();
		int width = (int) dimension.getWidth();
		setBounds(-10, 0, width, height);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		contentPane.setLayout(new BorderLayout(0, 0));
		setContentPane(contentPane);
 
		con.connection();
 
		ArrayList<Manga> mangaList = con.getAllMangas();
		con.close();
		model = new Affichage(mangaList);
 
		table = new JTable();
		table.setModel(model);
		contentPane.add(new JScrollPane(table), BorderLayout.CENTER);
 
		JPanel boutons = new JPanel();
 
		boutons.add(new JButton(new AddAction()));
		boutons.add(new JButton(new RemoveAction()));
 
		getContentPane().add(boutons, BorderLayout.WEST);
		this.setVisible(true);
	}
 
	private class AddAction extends AbstractAction {
		private AddAction() {
			super("Ajouter");
		}
 
		@SuppressWarnings("static-access")
		public void actionPerformed(ActionEvent e) {
			System.out.println("Test appel booleen");
			appel = true;
			new Formulaire();
 
/*			model.addManga(new Manga(titre, genre, auteur, dernierChap,
					dateDernierChap, dernierChapLu, dateDerniereLecture, statut));
*/		
		}
	}
 
	public void recupInfo(String titre, String genre, String auteur,
			String dernierChap, String dateDernierChap,
			double dernierChapLu, String dateDerniereLecture, String statut){
 
		model.addManga(new Manga(titre, genre, auteur, dernierChap,
				dateDernierChap, dernierChapLu, dateDerniereLecture, statut));
	}
	/*
	public void traitement(){
		String[] valeur= con.lectureInfosTableau().split(";");
		String d=null;
		for (int i = 0; i < valeur.length; i++) {
			titre=valeur[0];
			genre=valeur[1];
			auteur=valeur[2];
			dernierChap=valeur[3];
			dateDernierChap=valeur[4];
			d=valeur[5];
			dateDerniereLecture=valeur[6];
			statut=valeur[7];
		}
		dernierChapLu = Double.parseDouble(d);
 
	}
*/
 
	private class RemoveAction extends AbstractAction {
		private RemoveAction() {
			super("Supprimmer");
		}
 
		@SuppressWarnings("static-access")
		public void actionPerformed(ActionEvent e) {
			int[] selection = table.getSelectedRows();
 
			int ligne = table.getSelectedRow();// recupere le numero de la ligne
												// selectionner
			int colonne = 0;// recuperela 1ere collonne de la ligne selectionner
			Object cellule = table.getValueAt(ligne, colonne);
 
			con.connection();
			if (con.delManga(cellule.toString())) {
				con.close();
				for (int i = selection.length - 1; i >= 0; i--) {
					model.deleteMangaAt(selection[i]);
				}
			}
 
		}
	}
 
	public static boolean appelTableau() {
		return appel;
	}
}

je vous remercie de m'aider a comprendre ce merdier

**joel.drigo** · 03/12/2015, 00h33

Salut,

Avec un code partiel et une message d'erreur partiel (donne nous la stacktrace complète), je vois difficilement comment t'aider. Il n'y aucune déclaration ni initialisation des attributs dans la classe MainFrame.
En plus ton explication n'est par très claire : test sans paramètres ? la méthode renvoit null, alors qu'elle est void ?

Quand à la première méthode, elle est tellement hors contexte (pas de classe, on ne sait pas comment est initialisée frame...)...

**louisard** · 03/12/2015, 05h49

dsl, je ne pensais pas que se serait nécessaire, je parlais d'essayer d'appeler la methode dans MainFrame sans arguments passer en parametres

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
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
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
import java.awt.Frame;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.beans.PropertyChangeEvent;
import java.beans.PropertyChangeListener;
import java.text.NumberFormat;
import java.text.SimpleDateFormat;
import java.util.Calendar;
import java.util.Date;
 
import javax.swing.JButton;
import javax.swing.JCheckBox;
import javax.swing.JComboBox;
import javax.swing.JComponent;
import javax.swing.JFormattedTextField;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JTextField;
import javax.swing.event.DocumentEvent;
import javax.swing.event.DocumentListener;
 
public class Formulaire extends JFrame implements KeyListener, ActionListener,
		PropertyChangeListener {
 
	private static final long serialVersionUID = 1L;
 
	JLabel titre = new JLabel("Titre :");
	JLabel dernierScanLu = new JLabel("Dernier Scan Lu:");
 
	static JTextField textField = new JTextField(20);
	JTextField textField2 = new JTextField(20);
	SimpleDateFormat format;
 
	JFormattedTextField scanLu;
	NumberFormat scanFormat;
 
	JButton annuler = new JButton("Annuler");
	JButton ok = new JButton("Valider");
 
	String sel = null;
	MainFrame frame;
	Date now = new Date();
	InfosManga info;
	ConnexionMangaLu con = new ConnexionMangaLu();
	Affichage model;
 
	boolean champsValide = true;
	boolean rechercheEffectue = false;
 
	Object[] elements = new Object[] { "Shonen", "Shojo", "Seinen", "WebToons",
			"Ecchi" };
 
	static double chap = 0;
	JCheckBox dernierChap;
 
	@SuppressWarnings({ "rawtypes" })
	JComboBox choixType = new JComboBox();
 
	// implémentation de l'auto complétion
	// AutoCompleteSupport support =
	// AutoCompleteSupport.install(choixType,GlazedLists.eventListOf(elements));
 
	public Formulaire() {
		this.setTitle("Ajouter un Manga");
		this.setSize(500, 600);
		this.setLocationRelativeTo(null);
		this.setResizable(false);
		this.setLayout(new GridBagLayout());
 
		setUpFormats();
 
		annuler.setToolTipText("Annuler et revenir a la page precedente");
 
		textField.addKeyListener(this);
		textField2.addKeyListener(this);
 
		// Create the text fields and set them up.
		scanLu = new JFormattedTextField(scanFormat);
		scanLu.setValue(new Double(0));
		scanLu.setColumns(10);
		scanLu.addPropertyChangeListener("value", this);
 
		// Create the check boxes.
		dernierChap = new JCheckBox("Dernier Chap", false);
		dernierChap.setMnemonic(KeyEvent.VK_D);
 
		// dernierChap.addItemListener(this);
		dernierChap.addItemListener(new ItemListener() {
 
			public void itemStateChanged(ItemEvent e) {
 
				if (e.getStateChange() == ItemEvent.SELECTED) {
 
					if (lancementCheckBox())
						rechercheEffectue = true;
 
				}
			}
		});
 
		// Tell accessibility tools about label/textfield pairs.
		dernierScanLu.setLabelFor(scanLu);
 
		GridBagConstraints c = new GridBagConstraints();
 
		/********************* positionnement des objet en x et y ********************/
 
		c.gridx = 0;
		c.gridy = 0;
		c.gridheight = c.gridwidth = 1;
		c.anchor = GridBagConstraints.BASELINE_LEADING;
		c.insets = new Insets(40, 20, 0, 0);
		this.add(titre, c);
 
		c.gridx = 1;
		c.gridy = 0;
		c.gridwidth = GridBagConstraints.REMAINDER;
		c.fill = GridBagConstraints.HORIZONTAL;
		c.anchor = GridBagConstraints.BASELINE_LEADING;
		c.insets = new Insets(10, 10, 0, 20);
		this.add(textField, c);
 
		c.gridx = 0;
		c.gridy = 1;
		c.gridheight = c.gridwidth = 1;
		c.insets = new Insets(0, 20, 0, 0);
		this.add(dernierScanLu, c);
 
		c.gridx = 1;
		c.gridy = 1;
		c.insets = new Insets(10, 10, 0, 20);
		this.add(scanLu, c);
 
		c.gridx = 2;
		c.gridy = 1;
		c.gridwidth = GridBagConstraints.REMAINDER;
		c.fill = GridBagConstraints.HORIZONTAL;
		c.anchor = GridBagConstraints.BASELINE_LEADING;
		c.insets = new Insets(10, 10, 0, 20);
		this.add(dernierChap, c);
 
		c.gridx = 1;
		c.gridy = 8;
		c.gridwidth = GridBagConstraints.RELATIVE;
		c.fill = GridBagConstraints.NONE;
		c.anchor = GridBagConstraints.BASELINE_TRAILING;
		c.insets = new Insets(0, 5, 30, 0);
		this.add(ok, c);
 
		c.gridx = 2;
		c.gridy = 8;
		c.gridwidth = GridBagConstraints.REMAINDER;
		c.anchor = GridBagConstraints.BASELINE_LEADING;
		c.fill = GridBagConstraints.HORIZONTAL;
		c.insets = new Insets(10, 5, 30, 20);
		this.add(annuler, c);
 
		// Rcupération du choix sur la combobox
		sel = (String) choixType.getSelectedItem();
		ok.setEnabled(false);
 
		/* formatage de la date (jour Mois Année) */
		format = new SimpleDateFormat("dd MMM yyyy");
		/* utiliser le calendrier par défaut */
		Calendar calendar = Calendar.getInstance();
		String date = format.format(calendar.getTime());
		// action listener sur bouton Ok
		ok.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent event) {
				// System.out.println(support.getSelectedItem());
 
				if (!rechercheEffectue) {
					lancement();
				} else {
 
					if (con.enregistrement(getTitre().toUpperCase(), chap, date)
							&& con.infosTableau(getTitre().toUpperCase(), chap)) {
						JOptionPane.showMessageDialog(Formulaire.this,
								"Données enregistrées");
						con.close();
						choix();
					} else {
						close();
					}
				}
 
			}
		});
 
		annuler.addActionListener(new ActionListener() {
 
			public void actionPerformed(ActionEvent event) {
				close();
			}
		});
 
		scanLu.getDocument().addDocumentListener(new DocumentListener() {
			/**** Méthodes Héritées de la classe DocumentEvent ******/
			protected void update() {
				// Activation du bouton Ok si le champs Password contient des
				// données
				ok.setEnabled(scanLu.getDocument().getLength() > 0);
			}
 
			@Override
			public void insertUpdate(DocumentEvent e) {
				update();
			}
 
			@Override
			public void removeUpdate(DocumentEvent e) {
				update();
			}
 
			@Override
			public void changedUpdate(DocumentEvent e) {
				update();
			}
		});
 
		pack();
		this.setVisible(true);
	}
 
	/**
         * renvoie l'etat de la recherche ( a savoir si les infos du mangas ont deja
         * été reupérées )
         **/
	public boolean etatRecherche() {
		return rechercheEffectue;
	}
 
	/*** appel de la fonction permettant l'enregistrement des infos ***/
	private void lancement() {
		if (con.titreValide(getTitre())) {
 
			con.connection();
 
			if (con.addManga(getTitre(), getDernierScanLu())
					&& con.infosTableau(getTitre().toUpperCase(), chap)) {
				JOptionPane.showMessageDialog(Formulaire.this,
						"Données enregistrées");
				con.close();
				choix();
			} else {
				close();
			}
 
		} else {
			close();
			new Formulaire();
		}
 
	}
 
	/*** appel de la fonction permettant la recherche des infos ***/
	private boolean lancementCheckBox() {
		boolean lan = true;
		if (con.titreValide(getTitre())) {
 
			con.connection();
 
			String dernierChap = con.addMangaCheckBox(getTitre(),
					getDernierScanLu());
 
			if (dernierChap.equals("invalide")) {
				close();
				lan = false;
 
			} else {
				double Chap = Double.parseDouble(dernierChap);
				scanLu.setValue(new Double(Chap));
 
			}
 
		} else {
			lan = false;
			close();
			new Formulaire();
		}
		return lan;
	}
 
	private void setUpFormats() {
		scanFormat = NumberFormat.getNumberInstance();
 
	}
 
	private void close() {
		this.dispose();
	}
 
	public static String getTitre() {
		return textField.getText().toUpperCase();
	}
 
	public String getChoix() {
		return sel;
	}
 
	public static Double getDernierScanLu() {
		return chap;
	}
 
	/** choisir si l'on veut ajouter un nouveau mangas ou pas **/
	private void choix() {
 
		final JButton okButton = new JButton("OK");
		okButton.addActionListener(new ActionListener() {
 
			@Override
			public void actionPerformed(ActionEvent e) {
				JOptionPane pane = getOptionPane((JComponent) e.getSource());
				pane.setValue(okButton);
			}
		});
 
		final JButton cancelButton = new JButton("Annuler");
		cancelButton.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				JOptionPane pane = getOptionPane((JComponent) e.getSource());
				pane.setValue(cancelButton);
			}
		});
 
		// choix pour si oui ou non on veux inserer un nouveau manga
		if (JOptionPane.showConfirmDialog(null,
				"Voulez vous rentrer un nouveau manga ?", "Fais ton choix",
				JOptionPane.YES_NO_OPTION) == JOptionPane.OK_OPTION) {
 
			this.dispose();
			new Formulaire();
 
		} else {
 
			/*** on determine si l'appel vient du tableau ***/
			if (!frame.appelTableau()) {
				this.dispose();
				frame = new MainFrame();
			} else {
				this.dispose();
		//		frame.dispose();
		//		new MainFrame();
				String titr=getTitre();
				Double chapLu=getDernierScanLu();
				System.out.println(getTitre());
				System.out.println(info.getAuteur());
				System.out.println(info.getGenre());
				System.out.println(info.getDernierChap());
				System.out.println(info.getDate());
				System.out.println(getDernierScanLu());
				System.out.println(info.getStatut());
				System.out.println(con.getDate());
 
				frame.recupInfo(titr, InfosManga.getGenre(), info.getAuteur(), info.getDernierChap(),
						info.getDate(), chapLu, con.getDate(), info.getStatut());
 
		/*		model.addManga(new Manga(getTitre(), InfosManga.getGenre(), info.getAuteur(), info.getDernierChap(),
						info.getDate(), getDernierScanLu(), con.getDate(), info.getStatut()));
			//	frame.traitement(); 
				*/
			}
 
		}
 
	}
 
	private static JOptionPane getOptionPane(JComponent parent) {
		while (!(parent instanceof JOptionPane)) {
			parent = (JComponent) parent.getParent();
		}
		return (JOptionPane) parent;
	}
 
	@Override
	public void actionPerformed(ActionEvent arg0) {
		// TODO Auto-generated method stub
 
	}
 
	@Override
	public void keyPressed(KeyEvent e) {
		int key = e.getKeyCode();
 
		if (key == KeyEvent.VK_ENTER)
			ok.doClick();
	}
 
	@Override
	public void keyReleased(KeyEvent arg0) {
		// TODO Auto-generated method stub
 
	}
 
	@Override
	public void keyTyped(KeyEvent arg0) {
		// TODO Auto-generated method stub
 
	}
 
	/****
         * Fonction permettant l'élimination des lettres dans le champs Dernier chap
         ****/
	@Override
	public void propertyChange(PropertyChangeEvent e) {
		Object source = e.getSource();
		if (source == scanLu)
			/* on ne garde que la partie numérique */
			chap = ((Number) scanLu.getValue()).doubleValue();
 
	}
 
}

et la classe appeler

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
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
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.util.ArrayList;
 
import javax.swing.AbstractAction;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.border.EmptyBorder;
 
@SuppressWarnings("serial")
public class MainFrame extends JFrame {
 
	private JPanel contentPane;
	private JTable table;
	private Affichage model;
	Formulaire formulaire;
	Formulaire2 formulaire2;
	InfosManga info;
	static boolean appel = false;
 
	ConnexionMangaLu con = new ConnexionMangaLu();
 
	static String titre=null;
	static String auteur=null;
	static String genre=null;
	static String statut=null;
	static String dernierChap=null;
	static String dateDernierChap=null;
	static double dernierChapLu=0;
	static String dateDerniereLecture=null;
 
	public MainFrame() {
 
		Dimension dimension = java.awt.Toolkit.getDefaultToolkit()
				.getScreenSize();
		int height = (int) dimension.getHeight();
		int width = (int) dimension.getWidth();
		setBounds(-10, 0, width, height);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		contentPane.setLayout(new BorderLayout(0, 0));
		setContentPane(contentPane);
 
		con.connection();
 
		ArrayList<Manga> mangaList = con.getAllMangas();
		con.close();
		model = new Affichage(mangaList);
 
		table = new JTable();
		table.setModel(model);
		contentPane.add(new JScrollPane(table), BorderLayout.CENTER);
 
		JPanel boutons = new JPanel();
 
		boutons.add(new JButton(new AddAction()));
		boutons.add(new JButton(new RemoveAction()));
 
		getContentPane().add(boutons, BorderLayout.WEST);
		this.setVisible(true);
	}
 
	private class AddAction extends AbstractAction {
		private AddAction() {
			super("Ajouter");
		}
 
		@SuppressWarnings("static-access")
		public void actionPerformed(ActionEvent e) {
			appel = true;
			new Formulaire();
 
/*			model.addManga(new Manga(titre, genre, auteur, dernierChap,
					dateDernierChap, dernierChapLu, dateDerniereLecture, statut));
*/		
		}
	}
 
	public void recupInfo(String titre, String genre, String auteur,
			String dernierChap, String dateDernierChap,
			double dernierChapLu, String dateDerniereLecture, String statut){
 
 
		System.out.println("Test appel recup info \n");
		model.addManga(new Manga(titre, genre, auteur, dernierChap,
				dateDernierChap, dernierChapLu, dateDerniereLecture, statut));
	}
	/*
	public void traitement(){
		String[] valeur= con.lectureInfosTableau().split(";");
		String d=null;
		for (int i = 0; i < valeur.length; i++) {
			titre=valeur[0];
			genre=valeur[1];
			auteur=valeur[2];
			dernierChap=valeur[3];
			dateDernierChap=valeur[4];
			d=valeur[5];
			dateDerniereLecture=valeur[6];
			statut=valeur[7];
		}
		dernierChapLu = Double.parseDouble(d);
 
	}
*/
 
	private class RemoveAction extends AbstractAction {
		private RemoveAction() {
			super("Supprimmer");
		}
 
		@SuppressWarnings("static-access")
		public void actionPerformed(ActionEvent e) {
			int[] selection = table.getSelectedRows();
 
			int ligne = table.getSelectedRow();// recupere le numero de la ligne
												// selectionner
			int colonne = 0;// recuperela 1ere collonne de la ligne selectionner
			Object cellule = table.getValueAt(ligne, colonne);
 
			con.connection();
			if (con.delManga(cellule.toString())) {
				con.close();
				for (int i = selection.length - 1; i >= 0; i--) {
					model.deleteMangaAt(selection[i]);
				}
			}
 
		}
	}
 
	public static boolean appelTableau() {
		return appel;
	}
}

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
33
34
35
36
37
38
39
40
41
42
43
44
45
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
	at Formulaire.choix(Formulaire.java:365)
	at Formulaire.access$2(Formulaire.java:315)
	at Formulaire$2.actionPerformed(Formulaire.java:190)
	at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
	at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
	at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
	at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
	at javax.swing.plaf.basic.BasicButtonListener$Actions.actionPerformed(Unknown Source)
	at javax.swing.SwingUtilities.notifyAction(Unknown Source)
	at javax.swing.JComponent.processKeyBinding(Unknown Source)
	at javax.swing.JComponent.processKeyBindings(Unknown Source)
	at javax.swing.JComponent.processKeyEvent(Unknown Source)
	at java.awt.Component.processEvent(Unknown Source)
	at java.awt.Container.processEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.KeyboardFocusManager.redispatchEvent(Unknown Source)
	at java.awt.DefaultKeyboardFocusManager.dispatchKeyEvent(Unknown Source)
	at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(Unknown Source)
	at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(Unknown Source)
	at java.awt.DefaultKeyboardFocusManager.dispatchEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Window.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$500(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue$4.run(Unknown Source)
	at java.awt.EventQueue$4.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

**joel.drigo** · 03/12/2015, 07h50

Je trouve étrange que tu puisses dire qu'aucun des paramètres passés à recupInfo() soit null parce que tu les affiches : les lignes d'instruction pour les afficher sont juste avant, dans un bloc else. Comme je ne vois qu'une ligne capable d'initialiser la variable frame, et que cette ligne est dans un bloc if, (celui qui correspond au bloc else de l'appel recupInfo) qui teste la condiition en invoquant une méthode sur frame, je dirais qu'on ne passe même jamais dans le else, parce que le NullPointerException est sur la ligne if (!frame.appelTableau()) {. Affiche frame avant de if pour le vérifier. Tu as aussi le mode débug qui peux t'aider à trouver la variable qui vaut null.

**Kazh Du** · 03/12/2015, 10h06

Encore une fois, il manque une partie des informations : qu'est-ce que tu obtiens sur la sortie standard ? Rien ? Des données ? Si tu t'es dit que les afficher pouvait t'être utile, il y a de fortes chances que ça l'est pour nous aussi.

Personnellement, ce qui me chiffonne c'est ta variable chap (ou chapLu, je ne comprends pas grand chose à ton code) que tu convertis sans arrêt entre primitive et objet (et vice-versa). D'ailleurs, j'ai un doute sur le fait de faire

Code :

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

Double variable = 0;

C'est (en gros) ce que tu fais et je me demande si ça reviens pas à faire (comme en C++ au passage, d'où mon interrogation)

Code :

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

Double variable = null;

La conversion dans l'autre sens peut donner un NullPointerException est c'est peut être la cause de ton problème.

Au final, ça ne m'étonne pas que tu as besoin d'aides pour "comprendre ce merdier", ta classe Formulaire mélange données (avec des données statiques ?

), traitements et interface... MVC ?

**louisard** · 03/12/2015, 10h34

Le programme arrive bien dans le else, car je l'ai vérifier en faisant un System.out dans la fonction appeler .

Pour la convention du double , c bien fonctionnel car j'ai vérifier et lorsque je le met à 0 le programme l'enregistre sous la forme 0.0

**joel.drigo** · 03/12/2015, 10h54

Envoyé par louisard

Le programme arrive bien dans le else,

Donc tu nous montres pas tout. frame doit être initialisée par une bidouille, genre accès direct permis par la visibilité package. Il n'y a pas à tortiller : l'une des variables de la ligne 365 de la classe Formulaire est null. Il s'agit forcément d'une variable sur laquelle tu invoques une méthode ou tu accèdes à un attribut.

Envoyé par louisard

car je l'ai vérifier en faisant un System.out dans la fonction appeler .

La "fonction" appelée ? recupInfo ? La ligne 365 c'est bien la ligne frame.recupInfo(titr, InfosManga.getGenre(), info.getAuteur(), info.getDernierChap(),info.getDate(), chapLu, con.getDate(), info.getStatut()); ? Parce que s'il y a une trace dans cette méthode et qu'elle s'affiche, il y incohérence entre la stacktrace et le fait que la NullPointerException survient sur cette ligne (d'après ce que tu nous dit). Maintenant si tu parles de la méthode choix(), ça dépend où se trouve la trace (il faut qu'elle soit dans le bloc else).

Pour la convention du double , c bien fonctionnel car j'ai vérifier et lorsque je le met à 0 le programme l'enregistre sous la forme 0.0

Ce n'est pas ce que dit @Kazh Du. Il relève que si une variable est déclarer Double, et que tu ne l'initialise pas, si tu la passes en paramètre d'une méthode qui a un paramètre double, tu auras une NullPointerException, parce que si un double peut toujours être encapsulé (autoboxed) vers un Double, une variable Double ne peut pas toujours être désencapsulée (autounboxed) vers un double. Tu devrais effectivement simplifier ton code à commencer par ne pas utilise de Double quand tu n'as pas besoin de gérer une valeur null.

**louisard** · 03/12/2015, 15h40

Donc tu nous montres pas tout. frame doit être initialisée par une bidouille, genre accès direct permis par la visibilité package. Il n'y a pas à tortiller : l'une des variables de la ligne 365 de la classe Formulaire est null. Il s'agit forcément d'une variable sur laquelle tu invoques une méthode ou tu accèdes à un attribut.

j'ai verifier et non ce n'es pas le probleme, j'ai fait les tests

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
	/*** on determine si l'appel vient du tableau ***/
			if (!frame.appelTableau()) {
				this.dispose();
				frame = new MainFrame();
			} else {
				this.dispose();
			//	frame.close();
				frame = new MainFrame();
				System.out.println("titre :"+getTitre());
				System.out.println("auteru :"+info.getAuteur());
				System.out.println("genre :"+info.getGenre());
				System.out.println("dernier chap :"+info.getDernierChap());
				System.out.println("date :"+info.getDate());
				System.out.println("dernier scan lu :"+getDernierScanLu());
				System.out.println("staut :"+info.getStatut());
				System.out.println("date actuelle :"+con.getDate());
 
				frame.recupInfo(getTitre(), InfosManga.getGenre(), info.getAuteur(), info.getDernierChap(),
						info.getDate(), getDernierScanLu(), con.getDate(), info.getStatut());
 
		/*		model.addManga(new Manga(getTitre(), InfosManga.getGenre(), info.getAuteur(), info.getDernierChap(),
						info.getDate(), getDernierScanLu(), con.getDate(), info.getStatut()));
			//	frame.traitement(); 
				*/
			}

resultat des valeurs

Code :

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

1
2
3
4
5
6
7
8
9
 
titre :BLEACH
auteru :Kubo Tite
genre :Action, Adventure, Comedy, Drama, Fantasy, Horror,  Martial Arts, Shounen, Supernatural
dernier chap :653
date :Nov 27, 2015
dernier scan lu :0.0
staut :Ongoing
date actuelle :03 déc. 2015

donc ils contiennent tous une valeur

j'ai tester autrement en mettant la methode recupInfo() sans parametre d'appel et mm la j'ai la mm erreur, et j'ai fais une methode

Code :

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

1
2
3
4
5
public void close(){
		System.out.println("Mainframe: Test fermeture \n");
 
		this.dispose();
	}

appeler par

Code :

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

frame.close();

et meme cet appel ne passait pas, du coup le probleme vient de la classe MainFrame ( je suppose ) mais dans ce cas je ne comprend pas pk l'appel frame.appelTableau() fonctionne.

La "fonction" appelée ? recupInfo ? La ligne 365 c'est bien la ligne frame.recupInfo(titr, InfosManga.getGenre(), info.getAuteur(), info.getDernierChap(),info.getDate(), chapLu, con.getDate(), info.getStatut()); ? Parce que s'il y a une trace dans cette méthode et qu'elle s'affiche, il y incohérence entre la stacktrace et le fait que la NullPointerException survient sur cette ligne (d'après ce que tu nous dit). Maintenant si tu parles de la méthode choix(), ça dépend où se trouve la trace (il faut qu'elle soit dans le bloc else).

oui c'es bien sur cet ligne que j'ai une erreur

Ce n'est pas ce que dit @Kazh Du. Il relève que si une variable est déclarer Double, et que tu ne l'initialise pas, si tu la passes en paramètre d'une méthode qui a un paramètre double, tu auras une NullPointerException, parce que si un double peut toujours être encapsulé (autoboxed) vers un Double, une variable Double ne peut pas toujours être désencapsulée (autounboxed) vers un double. Tu devrais effectivement simplifier ton code à commencer par ne pas utilise de Double quand tu n'as pas besoin de gérer une valeur null.

donc si je met

Code :

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

Double ChapLu=0.0;

je ne devrais plus avoir ce probleme je suppose ?

**Kazh Du** · 03/12/2015, 16h29

Ton code est remplis d'erreur, si tu veux qu'on puisse t'aider, il va falloir faire du ménage :
1. Je n'aime pas ton this.dispose() une fenêtre ne se ferme pas toute seule, au pire elle peut se cacher (this.setVisible(false)). Je me demande, si tu ne détruis pas ton instance courante pendant son utilisation.
2. C'est quoi la construction croisées entre tes fenêtres ? Ton MainFrame créé un Formulaire et vice-versa. Il te faut une classe principale (un contrôleur serait pas mal).
3. Tes constructeurs sont trop complexes, font trop d'opérations et en plus des opérations risquées : un appel à une BDD dans un constructeur ?????

Un bon constructeur initialise juste ses attributs directs. Tout ce qui ne peut pas se résumer à this.attribut = new ClassAttribut();, ça dégage dans une méthode de type public void init();
4. Connais-tu le MVC ? Petit cours :
M(odel) : des classes avec juste des attributs, des getters et des setters. Pas de traitement d'information ici.
V(ue) : tes classes MainFrame et Formulaire, elles se contentent d'afficher le model, de fournir un nouveau model avec les valeurs insérées.
C(ontroleur) : elles font la transition entre les différentes parties du programme, font les calculs, gèrent les données...

Si tu utilises un Listener par exemple, tu peux par exemple le faire appeler une méthode de ton contrôleur, pas plus. Ou au pire des cas, un choix entre plusieurs méthodes.

On pourrait aller plus loin, mais déjà si tu mets ça en place on pourra t'aider.

Et tant qu'à faire, mets tout ton code en anglais. Moi le franglais, ça me file mal au crâne.

**tchize_** · 04/12/2015, 10h14

Mise à part le fait que le programme soit boderlique et qu'après 3 lectures, je n'ai toujours pas compris où certaines variables étaient initialisées, reprenons l'erreur, à cette ligne:

Code :

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

1
2
				frame.recupInfo(titr, InfosManga.getGenre(), info.getAuteur(), info.getDernierChap(),
						info.getDate(), chapLu, con.getDate(), info.getStatut());

Vu que l'erreur arrive là vu la stacktrace, il nous faut regarder tout ce qui se trouve à gauche d'un point '.' et tout ce qui subit de l'auto-deboxing. On peut donc avoir
frame = null
InfosManga = null
info = null
chapLu = null

On peut exclure InfosManga, c'est une classe pas une variable. On peut exclure info, tu l'utilise juste au dessus sans problème. On peut exlucre normalement frame, quoi qu'on ne sait pas avec ton code ce que tu bidouille dans dispose qui pourrait le mettre à null. Reste chapLu qui, etonnament est aussi la seule valeur que tu n'a pas testée avant l'appel.

**louisard** · 04/12/2015, 10h31

j'admet que mon code est un vrai bordel, j'essaierai le mvc conseiller par khaz du , mais je pense que ça ne pourra beaucoup de revoir la conception du programme.

La variable chapLu contient getDernierChapLu de la classe Formulaire , et je l'ai tester , elle n'es pas null, c d'ailleurs pour ça que j'arrive pas à savoir d'où viens l'erreur

**tchize_** · 04/12/2015, 10h40

Envoyé par louisard

La variable chapLu contient getDernierChapLu de la classe Formulaire , et je l'ai tester , elle n'es pas null,

Ha oui, mal vu dans ce bordel. Reste la possibilité de frame qui passe à null suite à l'appel de ton dispose. Comme il n'y a aucune indication sur comment frame fait pour être non null au départ, ce mystérieux truc extérieur au code que tu nous a donnée peut aussi bien être celui qui la refait passer à null. Tu initialize comment ton champ frame? A quoi sert ton appel à dispose?

**louisard** · 04/12/2015, 14h05

L'initialisation de frame .
Dans les déclarations : MainFrame frame;
Et à la ligne d'utilisation : frame = new MainFrame ;

Le this.dispose sers à fermer la frame de la classe Formulaire qui sert à récupéré les informations donner par l'utilisateur ( titre , dernierChapLu )

**Kazh Du** · 04/12/2015, 14h14

Es-tu sûr qu'il y a une donnée dans ta variable frame ?
Dans la première version du code que tu nous as donné, elle n'était pas initialisée, dans la seconde, l'initialisation était commentée et seulement dans la troisième tu lui as attribué une valeur.
Surtout que la ligne if (!frame.appelTableau()) { ne nous indique pas si ta variable n'est pas nulle, la méthode appelTableau() étant static.

**joel.drigo** · 04/12/2015, 14h45

Envoyé par Kazh Du

Ela méthode appelTableau() étant static.

Bien vu le coup de la méthode static ! Je n'avais jamais remarqué qu'appeler une méthode statique via une variable d'instance null (qu'on ne devrait pas faire) ne faisait pas de NPE (logique et pas logique à la fois

).
Donc appel doit être false (parce qu'on a pas déclenché "Ajouter", forcément possible que quand MainFrame est affichée), du coup on va dans le else et là, patatras frame est null.