Discussion :

[Tora21]

Salut à tous, j'ai un petit souci. J'ai un gouffre dans mon code et mes données se perdent. J'ai du oublié de faire un lien ou peut-être autre chose. Et mes variable ne se remplissent pas.

voici ma classe de traitement :

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
 
public class ConvertTime
{
	private double		instantT;
	private double		resultatTemps;
	private double		taux				= 10;
	private PanelLabel	panelLabelrecupere	= new PanelLabel ();
	private String		text;
	private String		result;
 
	public ConvertTime ()
	{
	}
 
	public ConvertTime (PanelLabel aPanelLabel)
	{
		this.panelLabelrecupere = aPanelLabel;
	}
 
	public void calculMoiCa ()
	{
		this.text = this.panelLabelrecupere.getChampDeSaisie ().getText ();
		if (this.text.length () < 1)
		{
			System.out.println ("variable vide");
			this.panelLabelrecupere.getLabelResult ().setText ("Pas de donnée en entrée");
		}
		else
		{
			try
			{
 
				this.instantT = Double.parseDouble (this.text);
				this.resultatTemps = this.instantT * this.taux;
				this.result = String.valueOf (this.resultatTemps);
				this.panelLabelrecupere.getLabelResult ().setText (this.result);
			}
			catch (NumberFormatException aNFE)
			{
				System.out.println ("Erreur de format: " + aNFE.getMessage ());
				System.out.println ("Code -2: Sortie pour NumberFormatException");
				System.exit ( - 2);
			}
			return;
		}
	}
}

et voici ma classe ou ce situe les champs à remplir:

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
 
public class PanelLabel extends JPanel
{
	private JTextField	champDeSaisie	= new JTextField (20);
	private JLabel		labelResult		= new JLabel (" ");
 
	public PanelLabel ()
	{
		this.add (this.champDeSaisie);
		this.add (this.labelResult);
		this.setLayout (new GridLayout (0, 1));
 
		Tools.faitBordureEtTexte (this.champDeSaisie, "Element à convertir");
		Tools.faitBordureEtTexte (this.labelResult, "Résultat de la conversion", Color.BLUE);
	}
 
	public JTextField getChampDeSaisie ()
	{
		return this.champDeSaisie;
	}
 
	public JLabel getLabelResult ()
	{
		return this.labelResult;
	}
}

le but est de multiplier la donnée du champDeSaisie par le taux. Mais le champDeSaisie et vide lors de son arrivé dans ConvertTime. Et lorsque j'essaye de remplir le LabelResult dans ConvertTime, au niveau du "if (this.text.length () < 1)", la sortie console fonctionne mais la sortie sur l'interface non.

Donc je suppose que mon souci est dans la non-transmission des données entre ces deux classe. D'où le gouffre à données.

Quelqu'un aurait-il une tite idée?
Un œil extérieur est souvent utile.

[Le_novice_de_l'info]

Bonjour Tora21,

Pour savoir un peu comment vous procèdez,pourriez vous poster l'extrait de code où vous faites appel à votre classe convertTime, ainsi que votre frame

@+

[Tora21]

Voici la "FenetrePrincipale":

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
 
public class FenetrePrincipale extends JFrame
{
	private PanelLabel	panelLabelRecup		= new PanelLabel ();
	private PanelBouton	panelBoutonRecup	= new PanelBouton ();
 
	public FenetrePrincipale (PanelLabel aPanelLabel, PanelBouton aPanelBouton)
	{
		this.panelLabelRecup = aPanelLabel;
		this.panelBoutonRecup = aPanelBouton;
	}
 
	public void initialiseInterface ()
	{
		this.add (this.panelLabelRecup, BorderLayout.NORTH);
		this.add (this.panelBoutonRecup, BorderLayout.CENTER);
		this.getRootPane ().setDefaultButton (this.panelBoutonRecup.getBoutConver ());
	}
 
	public FenetrePrincipale ()
	{
		setDefaultLookAndFeelDecorated (true);
		setDefaultCloseOperation (JFrame.EXIT_ON_CLOSE);
		this.setLocationRelativeTo (null);
		this.setTitle ("CONVERT-BOX");
 
		this.initialiseInterface ();
 
		this.pack ();
		this.repaint ();
		this.setVisible (true);
	}
}

et voici le "GestionnaireAction" :

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
 
public class GestionnaireAction implements ActionListener
{
	private ConvertTime	convertTimeRecupere	= new ConvertTime ();
 
	public GestionnaireAction ()
	{
	}
 
	public GestionnaireAction (ConvertTime aConvertTime)
	{
		this.convertTimeRecupere = aConvertTime;
	}
 
	public void actionPerformed (ActionEvent aAE)
	{
		String cause = aAE.getActionCommand ();
 
		if (cause.equals ("Convertir"))
		{
			this.convertTimeRecupere.calculMoiCa ();
			return;
		}
		if (cause.equals ("Quitter"))
		{
			System.exit ( - 1);
			return;
		}
	}
}

.

[Le_novice_de_l'info]

Merci mais désolé des données fourni l'erreur viens de votre classe ConvertTime.

En effet il manque quelque chose à votre méthode CalculMoiCa()

Code :

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

1
2
3
4
5
public void CalculMoiCa(PanelLabel apanelLabel){
     this.text = this.panelLabelrecupere.getChampDeSaisie ().getText ();
 
     // le reste de votre code me semble correct
}

à remplacer par

Code :

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

1
2
3
4
5
public void CalculMoiCa(PanelLabel apanelLabel){
     this.text = apanelLabel.getChampDeSaisie ().getText ();
 
     // le reste de votre code me semble correct
}

Le problème était que vous travailliez sur un objet que vous aviez créée dans la classe elle-même est donc les valeurs étaient celles que vous aviez définit dans le constructeur de PanelLabel

Je pense que comme ca ca doit marcher. J'attends de vos nouvelles

@+

[Tora21]

Ah mais non en fait j'ai oublié de vous donner un classe.

Voici la classe panelBouton:

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
 
public class PanelBouton extends JPanel
{
	private JButton				boutConver		= new JButton ("Convertir");
	private JButton				boutQuit		= new JButton ("Quitter");
 
	private String				helpTextConv	= "Permet de lancer la convertion";
	private String				helpTextQuit	= "Permet de quitter le programme";
 
	private GestionnaireAction	gestionAction	= new GestionnaireAction ();
 
	public PanelBouton (GestionnaireAction aGestionAction)
	{
		this.gestionAction = aGestionAction;
	}
 
	public PanelBouton ()
	{
		this.add (this.boutConver);
		this.add (this.boutQuit);
 
		Tools.bouton ("Convertir", this.boutConver, this.gestionAction, this.helpTextConv);
		Tools.bouton ("Quitter", this.boutQuit, this.gestionAction, this.helpTextQuit);
	}
 
	public JButton getBoutConver ()
	{
		return this.boutConver;
	}
 
}

Le reste n'est pas essentiel.

[Le_novice_de_l'info]

OU sinon par rapport à ce que vous avez écrit

Code :

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

1
2
3
4
5
6
7
 
public void calculMoiCa(PanelLabel unPanelLabel){
      ConvertTime unConvertTime = new ConvertTime(unPanelLabel);
      this.text = unConvertTime.panelLabelrecupere.getChampSaisi().getText();
 
      // le reste de votre code me semble correct
}

Cette version me semble plus près de se que vous avez réalisé jusque là

[Tora21]

hum.. je ne comprend pas bien ce que vous me donnez comme solution, donc si vous pouviez me l'explique svp.

De plus je l'ai ajouté au code et cela m'a obligé à modifier ma classe Gestionnaire Action, et à y importer la classe PanelLabel ce qui ajoute au 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
 
private PanelLabel unPanelLabel = new PanelLabel ();
 
public GestionnaireAction(PanelLabel aPanelLabel)
	{
		this.unPanelLabel = aPanelLabel;
	}
 
// à l'interieur de actionPerformed
if (cause.equals ("Convertir"))
		{
			this.convertTimeRecupere.calculMoiCa (this.unPanelLabel);
			return;
		}

Malgré cela ma variable this.text reste toujours vide.
Me serais-je tromper en apliquant votre solution?

[Le_novice_de_l'info]

Je suis en train de m'arracher les cheveux. Es tu obliger d'avoir les deux Classes étendant JPanel ou tu peux les regrouper ?

Voire même carrément le faire sur la JFrame

[Tora21]

Ne te rend pas chauve pour si peu !!!

L'intérêt que je vois à la division des classes en plusieurs parties vient du fait, que lorsque je souhaiterais modifier quelque chose, il me sera plus facile de le faire sans trop abîmer le programme.

De plus, ce n'est que le début du programme, je n'ai pas encore implémenté tous les composants, ce n'est que l'architecture. Mais déjà là.... Gros blêm...

[Le_novice_de_l'info]

Beh écoute, je suis en train de bosser dessus : voilà mes avancements

Déjà j'ai regroupé tous ce qui est graphique dans une seule classe. J'ai mis les gestionnaire d'évênement en classe interne à cette classe graphique. Et de l'autre je garde la classe ConvertTime que j'ai allégé sans pour autant détourner son utilisation première

J'ai procédé à ce regroupement pour un soucis de lisibilité car j'ai mis du temps à naviguer dans les différent bout de code que tu m'a envoyé.

Perso., j'opte plus souvent pour mettre les gestionnaire d'évênement en classe interne, toujours dans un soucis de lisibilités.

[Le_novice_de_l'info]

Je vois aussi que tu mélange graphique et console. Quelle est la finalité de ton travail ?
Graphique ou console ?

Car pour tes messages d'erreurs, je peut te conseiller une JOptionPane (pop-up)

[Tora21]

Hum classe interne je crois pas connaitre le principe mais je suppose que ca ressemble a un truc du genre:

Code :

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

1
2
3
4
5
6
7
 
public class ...
{
   public class ...
   {
    }
}

et le mélange graphique/console ne servait qu'a une seule chose me prouvé que ce n'était pas ma méthode calculMoiCa() qui buggai, mais que c'était bien la transmission des donnée qui foiraient.

Je suis intéressée par les différentes façon de procéder, ca me permet d'élargir mes connaissance et de découvrir de nouvelles logiques.

[Le_novice_de_l'info]

Voilà je te monter ce que j'ai fait :

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
 
public class ConvertTime
{
	private double		instantT;
	private double		resultatTemps;
	private double		taux				= 10;
 
	public ConvertTime (){}
 
	public double calculMoiCa (String valeur)
	{
		try
		{ 
			// un petit test (expression régulière pour tester si la chaîne est vraiment un nombre).
			this.instantT = Double.parseDouble (valeur);
			this.resultatTemps = this.instantT * this.taux;
 
		}
		catch (NumberFormatException aNFE)
		{
			JOptionPane.showMessageDialog(null, "Erreur de format: " + aNFE.getMessage (), "une erreur de saisie", JOptionPane.ERROR_MESSAGE);
		}
		return resultatTemps;
	}
}

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
 
public class FenetrePrincipale extends JFrame {
 
	/**
         * 
         */
	private static final long serialVersionUID = 1L;
 
	private JTextField champDeSaisie;
	private JLabel labelResult;
	private JButton boutConver;
	private JButton	boutQuit;
 
	private JPanel panelBouton;
	private JPanel panelLabel;
 
	private ConvertTime	convertTimeRecupere;	
 
	public FenetrePrincipale() {
 
		panelBouton = new JPanel();
		panelLabel = new JPanel();
 
		champDeSaisie = new JTextField(25);
		champDeSaisie.setText("");
		labelResult = new JLabel("En attente du resultat");
		boutConver = new JButton("Convertir");
		boutQuit = new JButton("Quitter");
 
		panelBouton.add(boutConver);
		panelBouton.add(boutQuit);
 
		panelLabel.add(champDeSaisie);
		panelLabel.add(labelResult);
 
		this.add(panelLabel, BorderLayout.NORTH);
		this.add(panelBouton, BorderLayout.CENTER);
 
		boutConver.addActionListener(new GestionnaireAction());
		boutQuit.addActionListener(new GestionnaireAction());
 
	}
 
 
 
	public class GestionnaireAction implements ActionListener
	{
		public void actionPerformed (ActionEvent aAE)
		{					
			if (aAE.getSource() == boutConver)
			{
				if(champDeSaisie.getText().equals("")){
					JOptionPane.showMessageDialog(null, "Veuillez saisir une valeur à convertir" , "Erreur de saisie", JOptionPane.OK_OPTION);				
				}
				else{
					convertTimeRecupere = new ConvertTime ();
					String aConvertir = champDeSaisie.getText();
					Double resultat = convertTimeRecupere.calculMoiCa(aConvertir);
					labelResult.setText(resultat.toString());				
				}
			}
			if (aAE.getSource() == boutQuit)
			{
				System.exit(0);
			}
		}
	}

Voilà dis moi si ca répond a peu près a ce que tu recherches.

@+

[Tora21]

Alors, j'ai compilé et ça fonctionne c'est déjà ca. Ça me permettra d'avancer sur mon programme, et par la même je découvre une nouvelle logique qui n'est pas si mal, mis à part que ça me force à mélanger des éléments qui vont devenir de plus en plus nombreux.

pour le moment je vais avancé sur ta solution pour continuer le développement de mon application, et je vais en m^me temps continuer de plancher sur le pourquoi du comment que la communication ne passe pas entre mes deux classe .

Merci bien Le_novice_de_l'info pour le temps que tu as dépensé.

et à la prochaine.