execution interface graphique

execution interface graphique - Java - Programmation

Marsh Posté le 22-10-2005 à 20:29:29    

Bonjour,
J'ai créé une interface graphique sous java. Je résume rapidement mon inteface. Donc j'ai une fenetre, qd j'appuie sur le bouton entrez j'arrive sur une deuxieme page. Qd j'appuie sur annulez je reviens à la première page. A première vue mon interface marche correctemment. Mais en fait quand j'appuie plusieurs fois de suite sur entrez puis annulez mon application ralenti. J'ai remarqué qu'a force d'appuyer sur ces boutons mon code au lieu de s'executer une fois, il s'exécute plusieurs fois et ca va exponentiellement. Ca fait pas mal de temps que je suis dessus et je vois pas d'où ca peut venir. Pour etre plus explicit je vous joins mon code ( je sais pas si ca se fait  :ange: ).
En tout cas merci d' avance
 

Code :
  1. import javax.swing.*;
  2. import java.awt.*;
  3. import java.awt.event.*;
  5. public class test extends JFrame
  6. {
  7. private Frame f = new Frame();
  9. private JPanel promo = new JPanel();
  10. private JPanel logo = new JPanel();
  11. private JPanel ppal = new JPanel();
  12.    
  13. private JLabel acc = new JLabel ("Bienvenue dans un autre monde" );
  14. private JLabel sel = new JLabel ("Faites votre sélection" );
  16. private JButton entree= new JButton ("Entrez" );
  17. private JButton prepayee= new JButton ("Rouge" );
  18. private JButton recharge= new JButton ("Blanc" );
  19. private JButton annule= new JButton ("Annulez" );
  21. public void Fenetre()
  22. {       
  23.         f.setLayout(new BorderLayout());
  24.         promo.setLayout(new FlowLayout(FlowLayout.CENTER,1152,75));
  25.         promo.setBackground(new Color(8, 153, 0));
  26.         logo.setLayout(new FlowLayout(FlowLayout.CENTER,75,864));
  27.         logo.setBackground(new Color(8, 153, 0));
  29.         GridBagLayout repartiteur = new GridBagLayout();
  30.         GridBagConstraints contraintes;
  31.         ppal.setLayout(repartiteur);
  32.      ppal.setBackground(Color.BLACK);
  33.      contraintes = new GridBagConstraints();
  34.  contraintes.gridwidth = GridBagConstraints.REMAINDER;
  35.  contraintes.anchor = GridBagConstraints.NORTH;
  36.  contraintes.ipady = 50;
  37.  repartiteur.setConstraints(acc, contraintes);
  38.  acc.setFont(new java.awt.Font("Times", 0, 50));
  39.  acc.setForeground(Color.RED);
  40.  ppal.add(acc);
  41.    
  42.  contraintes = new GridBagConstraints();
  43.  contraintes.anchor = GridBagConstraints.CENTER;
  44.  repartiteur.setConstraints(entree, contraintes);
  45.  entree.setForeground(Color.RED);
  46.  ppal.add(entree);
  47.        
  48.         f.add(promo,BorderLayout.NORTH);
  49.         f.add(logo,BorderLayout.EAST);
  50.         f.add(ppal,BorderLayout.CENTER);
  52.         f.setSize(1152, 864);
  53.         f.setVisible(true);
  54.         entree.addActionListener(new ActionListener()
  55.         {
  56.          public void actionPerformed(ActionEvent event)
  57.             {
  58.              System.out.println("appuyé sur entrez" );
  59.              ppal.removeAll();
  60.              ppal.repaint();
  61.              Selection();
  62.             }
  63.         });
  64. }
  65. protected void Selection()
  66. {
  67.  ppal.setLayout(null);
  68.  sel.setFont(new java.awt.Font("Times", 0, 50));
  69.  sel.setForeground(Color.RED);
  70.  sel.setBounds(250,10,1000,60);
  71.  ppal.add(sel);
  72.  recharge.setBounds(300,300,150,50);
  73.  ppal.add(recharge);
  74.  prepayee.setBounds(550,300,150,50);
  75.  ppal.add(prepayee);
  76.  f.setVisible(true);
  77.  annule.setBounds(890,640,80,25);
  78.  ppal.add(annule);
  80.  annule.addActionListener(new ActionListener()
  81.  {
  82.   public void actionPerformed(ActionEvent event)
  83.      {
  84.       System.out.println("appuyé sur annule" );
  85.       ppal.removeAll();
  86.       ppal.repaint();
  87.       Fenetre();
  88.      }
  89.  });
  90. }
  91. public static void main(String[] argv)
  92. {
  93.  test f = new test();
  94.  f.Fenetre();
  95.  System.out.println("Démarre" );
  96. }
  97. }


 
Rayley!!!!

Reply

Marsh Posté le 22-10-2005 à 20:29:29   

Reply

Marsh Posté le 24-10-2005 à 01:08:00    

Salut,
 
Ton code est dégeu, mais bon, je le dis pas pour pas passer pour un gros connard. Pis l'important quand on débute, c'est que ça marche. :D
Alors justement pour que ça marche : le problème vient du fait que tu appelles une méthode Fenetre qui crée des trucs, et qui appelle Selection lors d'un appui sur un bouton, Selection crée d'autres trucs et appelle Fenetre... En gros tu crées un gros merdier qui prend de plus en plus de mémoire au fur et à mesure. Le mieux est que tu te bases sur le code suivant :
 

Code :
  1. import javax.swing.*;
  2. import java.awt.*;
  3. import java.awt.event.*;
  5. public class test extends JFrame
  6. { 
  8. // déclaration (et pas de construction, ie pas de "new" ) de tes boutons, labels, etc.
  9. private JButton entree;
  10. private JButton annule;
  12. public test() // constructeur, du nom de ta classe, et non pas une méthode
  13. {
  14.  super();
  16.  // création de TOUS tes boutons, de TOUS tes labels, etc.
  18.  entree = new JButton ("Entrez" );
  19.  entree.setVisible(true);
  20.  entree.addActionListener(new ActionListener() 
  21.  {
  22.   public void actionPerformed(ActionEvent event)
  23.   {
  24.    entree.setVisible(false);
  25.    annule.setVisible(true);
  26.   }
  27.  });
  29.  annule = new JButton ("Annule" );
  30.  annule.setVisible(false);
  31.  annule.addActionListener(new ActionListener() 
  32.  {
  33.   public void actionPerformed(ActionEvent event)
  34.   {
  35.    entree.setVisible(true);
  36.    annule.setVisible(false);
  37.   }
  38.  });
  40.  // mise en page de tout ton merdier
  41.  this.setLayout(new GridBagLayout());       
  42.  this.add(entree);
  43.  this.add(annule);
  44.  this.setSize(800,600);
  45.  this.setVisible(true);
  46. }
  48. public static void main(String[] argv)
  49. {
  50.  new test();
  51. }
  52. }


 
Bon, après, je suis pas le messie en Java, mais je pense que c'est un peu plus propre comme solution, à savoir de gérer ce que tu affiches dans les actionPerformed de chaque bouton. Il y a moyen de faire un truc un peu plus évolué avec une machine d'états (avec une variable dont la valeur correspond à un état, donc à une certaine configuration de boutons / couleurs / texte affichés).
 
Aussi, essaie de regrouper les choses ensemble pour rendre ton code plus clair (les "entree", "annule", "this", etc.).
 
Bon courage pour la suite en tout cas.


Message édité par rnoizet le 24-10-2005 à 01:20:12
Reply

Marsh Posté le 25-10-2005 à 17:54:25    

Je te remercie pour la mise en forme de mon code malgré ton manque de politesse!!!!!
 
Pour ceux que ca interresse j'ai trouvé une solution à mon problème alors au lieu d'utiliser removeall(), j'ai utiliser hide et show et éviter de faire des appels à différentes méthodes
 
Rayley!!!!

Message cité 2 fois
Reply

Marsh Posté le 25-10-2005 à 17:56:07    

rayley a écrit :

Je te remercie pour la mise en forme de mon code malgré ton manque de politesse!!!!!


:heink:


---------------
Now Playing: {SYNTAX ERROR AT LINE 1210}
Reply

Marsh Posté le 25-10-2005 à 21:37:58    

rayley a écrit :

Je te remercie pour la mise en forme de mon code malgré ton manque de politesse!!!!!


à quel moment a t'il manqué de politesse à ton égard ? il te fait une réponse nickel, et c'est comme ça que tu le remercies ?

Message cité 3 fois
Reply

Marsh Posté le 26-10-2005 à 08:38:25    

Harkonnen a écrit :

à quel moment a t'il manqué de politesse à ton égard ? il te fait une réponse nickel, et c'est comme ça que tu le remercies ?


[:lorilll]
 
Je propose 1 semaine de TT en vertu de ta nouvelle politique d'éradication des boulets et des malpolis sur la cat PHP qui viennent trainer sur la cat Java. [:dawa]

Message cité 1 fois

---------------
Now Playing: {SYNTAX ERROR AT LINE 1210}
Reply

Marsh Posté le 26-10-2005 à 13:50:07    

Harkonnen a écrit :

à quel moment a t'il manqué de politesse à ton égard ? il te fait une réponse nickel, et c'est comme ça que tu le remercies ?


Il a dit "dégueu", "connard" et "merdier".
Ces mots sont choquants, au même titre que le mot "cyprine".
(Exemple tout à fait fortuit, bien entendu)


---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?
Reply

Marsh Posté le 26-10-2005 à 13:55:57    

"Choquant" n'implique pas "malpoli".
 
[:moundir]


---------------
Now Playing: {SYNTAX ERROR AT LINE 1210}
Reply

Marsh Posté le 26-10-2005 à 14:00:56    

La politesse élementaire indique l'éviction de ces mots tant que faire se peut.


---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?
Reply

Marsh Posté le 26-10-2005 à 14:02:00    

Harkonnen a écrit :

à quel moment a t'il manqué de politesse à ton égard ? il te fait une réponse nickel, et c'est comme ça que tu le remercies ?


Si tu relis bien ma réponse je le remercie mais j'ai eu l'impression qu'il me prenait un peu de haut d'où ma réponse. Bon si ce n'était pas le cas je m'en excuse. Mais je pense que ca ne sert à rien d'épiloguer dessus.
 
Bisous les mecs
 
Rayley!!!!

Message cité 1 fois
Reply

Marsh Posté le 26-10-2005 à 14:02:00   

Reply

Marsh Posté le 26-10-2005 à 14:07:03    

rayley a écrit :

Bisous les mecs


Eh oh, et ta politesse ? [:bistou tt]

Message cité 1 fois

---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?
Reply

Marsh Posté le 26-10-2005 à 14:08:44    

the real moins moins a écrit :

Eh oh, et ta politesse ? [:bistou tt]


 
autant pour moi :jap:

Message cité 2 fois
Reply

Marsh Posté le 26-10-2005 à 14:17:39    

rayley a écrit :

autant pour moi :jap:


10 coups de fouet ou 1 semaine TT. [:dawa]


---------------
Now Playing: {SYNTAX ERROR AT LINE 1210}
Reply

Marsh Posté le 26-10-2005 à 14:19:19    

sircam a écrit :

[:lorilll]
 
Je propose 1 semaine de TT en vertu de ta nouvelle politique d'éradication des boulets et des malpolis sur la cat PHP qui viennent trainer sur la cat Java. [:dawa]


je vais l'envisager tiens [:figti]
mais est ce que la cat PHP est plus propre depuis ? sinon ça sert à rien :o

Reply

Marsh Posté le 26-10-2005 à 14:19:54    

rayley a écrit :

autant pour moi :jap:


la politesse et l'orthographe aussi :fou:
"au temps pour moi"

Reply

Marsh Posté le 26-10-2005 à 14:30:39    

Et les majuscules; et la ponctuation. :pfff:


---------------
Hey toi, tu veux acheter des minifigurines Lego, non ?
Reply

Marsh Posté le 26-10-2005 à 15:25:03    

Citation :

mais est ce que la cat PHP est plus propre depuis ?


Ca commence tout doucement à porter ses fruits, mais il faut rester vigilant. La cat PHP attire les boulets comme la m... attire les mouches.   [:pingouino]  
 
 
"au temps pour moi" => Les deux sont valables, hein.
 
 [:moundir]


---------------
Now Playing: {SYNTAX ERROR AT LINE 1210}
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed