Pattern MVC pour verification

Marsh Posté le 04-12-2013 à 15:40:36

Bonjour à tous !

Je débute et après quelques tutos je fait un exercice pour améliorer et mieux comprendre la logique des structures en Java

l'exercice est simple sur la base du jeu Pierre Feuille Ciseaux

1)Créer une interface qui permet de choisir pierre feuille ciseaux
2)comparer notre choix a celui de l'ordinateur(Random)
3)afficher l'image du coup jouer a chaque changement imgJ1 vs imgJ2.
4)attribuer 1 point au gagnant
5)le premier à 5pt gagne 1pt de set
6)le premier a 2pt de set avec 2pt d’écart gagne un point de manche
7)le premier a 2pt de manche gagne le jeu! (fiouuu)
8) a chaque point attribuer l'humeur du joueur (statut image) doit changer
8)statut initial=5, max=10, min=0. (ou initial=0 min=-5 et max=10)
8)celui-ci est égale à nbrAtteindrePoint(5)+scoreJ1-scoreJ2

j'ai réalisé l'exercice est je souhaite avoir vos critiques sur la conception du programme, le choix des variable, des I/O image, Pattern MVC, Pattern Observer, objet....... et toutes les chose qui vous dérange dans ce programme.

le code :
package Contrôle
Class Bouton :

Code :

package Controle;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Image;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.io.IOException;
import javax.imageio.ImageIO;
import javax.swing.JButton;
public class Bouton extends JButton implements MouseListener {
private String name;
private Image img;
private int number;
public Bouton(String str, int n, String imgJKP,int hight, int width) {
super(str);
this.name = str;
this.setNumber(n);
this.setPreferredSize(new Dimension(hight, width));
try {
img = ImageIO.read(getClass().getResourceAsStream(imgJKP));
} catch (IOException e) {
e.printStackTrace();
}
this.addMouseListener(this);
}
public void paintComponent(Graphics g){
Graphics2D g2d = (Graphics2D)g;
g2d.drawImage(img, 0, 0, this);
}
public void mouseClicked(MouseEvent event) {
//Inutile
}
@Override
public void mouseEntered(MouseEvent arg0) {
// TODO Auto-generated method stub
}
@Override
public void mouseExited(MouseEvent arg0) {
// TODO Auto-generated method stub
}
@Override
public void mousePressed(MouseEvent arg0) {
// TODO Auto-generated method stub
}
@Override
public void mouseReleased(MouseEvent arg0) {
// TODO Auto-generated method stub
}
public int getNumber() {
return number;
}
public void setNumber(int number) {
this.number = number;
}
}

Class JanKenPon

Code :

package Controle;
import vue.Windows;
public class JanKenPon {
public static void main(String[] args) {
// TODO Auto-generated method stub
Windows w = new Windows();
}
}

package Metier
Class Coups

Code :

package Metier;
import java.util.ArrayList;
import ObserverCoups.ObservableCoups;
import ObserverCoups.ObservateurCoups;
public class Coups implements ObservableCoups {
protected int jouer = 0;
private ArrayList<ObservateurCoups> listCoupsObservateur = new ArrayList<ObservateurCoups>();
public Coups() {
this.jouer = 0;
}
public void setCoupJouer(int c) {
this.jouer = c;
this.updateCoupObservateur();
}
public int getCoupJouer() {
return this.jouer;
}
@Override
public void addCoupObservateur(ObservateurCoups obs) {
this.listCoupsObservateur.add(obs);
}
@Override
public void delCoupObservateur() {
this.listCoupsObservateur = new ArrayList<ObservateurCoups>();
}
@Override
public void updateCoupObservateur() {
for(ObservateurCoups obs : this.listCoupsObservateur )
obs.updateCoups(this.jouer);
}
}

Class Joueur

Code :

package Metier;
public class Joueur {
// varaible
protected String name;
protected String type;
public Joueur() {
this.name = "anonyme";
this.type = "inconnu";
}
public Joueur(String name, String type) {
this.name = name;
this.type = type;
}
public String getName() {
return this.name;
}
public String getType() {
return this.type;
}
}

Class Partie

Code :

package Metier;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.Map;
import vue.Windows;
import ObserverPartie.ObservablePartie;
import ObserverPartie.ObservateurPartie;
public class Partie implements ObservablePartie {
// variable
protected int egaliteI = 0;
protected Thread reinitThread;
protected int nbrT;
protected int nbrS;
protected int nbrManche;
protected int nbrMatch;
protected int nbrSE;
protected int nbrJoueur;
protected String comment;
protected Joueur J1;
protected Joueur J2;
protected Score score;
protected Coups coup;
protected Personnage perso;
protected final Map<Joueur, Score> scores;
protected final Map<Joueur, Coups> coups;
protected final Map<Joueur, Personnage> persos;
private ArrayList<ObservateurPartie> listPartieObservateur = new ArrayList<ObservateurPartie>();
public Partie() {
this.nbrT = 5;
this.nbrS = 3;
this.nbrManche = 2;
this.nbrMatch = 1;
this.nbrSE = 2;
this.nbrJoueur = 2;
this.comment = "Jouer";
scores = new HashMap<Joueur, Score>();
getScores().put(J1 = new Joueur("J1","Humain" ), score = new Score(getJ1().getName()));
getScores().put(J2 = new Joueur("J2","COM" ), score = new Score(getJ2().getName()));
coups = new HashMap<Joueur, Coups>();
getCoups().put(J1, coup = new Coups());
getCoups().put(J2, coup = new Coups());
persos = new HashMap<Joueur, Personnage>();
getPersos().put(J1, perso = new Personnage("s_" ));
getPersos().put(J2, perso = new Personnage("u_" ));
}
public Partie(int nbrT, int nbrS, int nbrManche, int nbrMatch, int nbrSE, String persoJ1, String persoJ2) {
this.nbrT = nbrT;
this.nbrS = nbrS;
this.nbrManche = nbrManche;
this.nbrMatch = nbrMatch;
this.nbrSE = nbrSE;
this.nbrJoueur = 2;
this.comment = "Jouer";
scores = new HashMap<Joueur, Score>();
getScores().put(J1 = new Joueur("J1","Humain" ), score = new Score(getJ1().getName()));
getScores().put(J2 = new Joueur("J2","COM" ), score = new Score(getJ2().getName()));
coups = new HashMap<Joueur, Coups>();
getCoups().put(J1, coup = new Coups());
getCoups().put(J2, coup = new Coups());
persos = new HashMap<Joueur, Personnage>();
getPersos().put(J1, perso = new Personnage(persoJ1));
getPersos().put(J2, perso = new Personnage(persoJ2));
}
public void action(int coup) {
int hasard = (int) (1+3*Math.random());
this.getCoups().get(this.getJ1()).setCoupJouer(coup);
this.getCoups().get(this.getJ2()).setCoupJouer(hasard);
compareCoups(coup, hasard);
reinitThread = new Thread(new ReinitThread());
reinitThread.start();
}
protected void compareCoups(int coup, int hasard) {
if(coup == hasard) {
egalite();
} else if (coup == 1 && hasard == 2) {
J1Win();
} else if (coup == 2 && hasard == 3) {
J1Win();
} else if (coup == 3 && hasard == 1) {
J1Win();
} else {
J2Win();
}
setStatut();
}
protected void egalite() {
egaliteI++;
if (egaliteI <= 1)
setComment("Egalité !" );
else if (egaliteI == 2)
setComment("Woooh "+egaliteI+"eme Egalité !" );
else if (egaliteI < 5)
setComment("Enorme "+egaliteI+"eme Egalité !" );
else if (egaliteI >= 5)
setComment("Whouuu quel tension ! "+egaliteI+"eme Egalité !" );
}
/// win point
/// humain win
protected void J1Win() {
this.getScores().get(this.getJ1()).setPoint();
setComment("Joueur 1 Gagne le Point" );
if (this.getScores().get(this.getJ1()).getPoint() == this.getNbrT()) {
// methode set
Set(this.getJ1(), this.getJ2());
}
}
// fin humain win
/// COM gagne
protected void J2Win() {
this.getScores().get(this.getJ2()).setPoint();
setComment("Joueur 2 Gagne le Point" );
// test si partie.nbrT pour les points est atteint
}
// fin com gagne
// fin win point
protected void Set(Joueur a, Joueur b) {
this.getScores().get(a).setSet();
advantage();
/// verifie si le score set >= on nombre de set definit par nbrS avec l'ecart de x set
if (this.getScores().get(a).getSet() >= this.getNbrS()) {
if (this.getScores().get(a).getAdv() >= this.getNbrSE()) {
// methode manche
Manche(a, b);
}
}
}
protected void Manche(Joueur a, Joueur b){
this.getScores().get(a).setManche();
// si les manche == nbrManche definit
if (this.getScores().get(a).getManche() == this.getNbrManche()) {
Match(a, b);
}
}
protected void Match(Joueur a, Joueur b){
this.getScores().get(a).setMatch();
// si les manche == nbrManche definit
if (this.getScores().get(a).getMatch() == this.getNbrMatch()) {
Winner(a);
}
}
protected void Winner(Joueur a) {
setComment(a.getName()+" Gagne le match !" );
}
protected void advantage() {
if (this.getScores().get(this.getJ1()).getSet() > this.getScores().get(this.getJ2()).getSet()) {
this.getScores().get(this.getJ1()).setAdv(this.getScores().get(this.getJ1()).getSet()-this.getScores().get(this.getJ2()).getSet());
} else {
this.getScores().get(this.getJ2()).setAdv(this.getScores().get(this.getJ2()).getSet()-this.getScores().get(this.getJ1()).getSet());
}
}
// controle des score */
/// setstatut perso
protected void setStatut() {
int J1 = this.getScores().get(this.getJ1()).getPoint();
int J2 = this.getScores().get(this.getJ2()).getPoint();
this.getPersos().get(this.getJ1()).setPersoStatut(this.getNbrT()+(J1-J2));
this.getPersos().get(this.getJ2()).setPersoStatut(this.getNbrT()+(J2-J1));
}
// fin setstatut perso */
public void setComment(String comment) {
this.comment = comment;
updatePartieObservateur();
}
public String getComment() {
return this.comment;
}
public int getNbrT() {
return this.nbrT;
}
public int getNbrS() {
return this.nbrS;
}
public int getNbrManche() {
return this.nbrManche;
}
public int getNbrMatch() {
return this.nbrMatch;
}
public int getNbrSE() {
return this.nbrSE;
}
public Joueur getJ2() {
return J2;
}
public Joueur getJ1() {
return J1;
}
public Map<Joueur, Score> getScores() {
return scores;
}
public Map<Joueur, Coups> getCoups() {
return coups;
}
public Map<Joueur, Personnage> getPersos() {
return persos;
}
/// reinit des score
protected void reinit() {
if (getScores().get(getJ2()).getPoint() == getNbrT() || getScores().get(getJ1()).getPoint() == getNbrT()) {
getScores().get(getJ1()).resetPoint();
getScores().get(getJ2()).resetPoint();
// labComment
setComment("Jouer !" );
// reinit egalite
egaliteI = 0;
//reinit coups jouer
getCoups().get(getJ1()).setCoupJouer(0);
getCoups().get(getJ2()).setCoupJouer(0);
//reinit persos statut
getPersos().get(getJ1()).setPersoStatut(5);
getPersos().get(getJ2()).setPersoStatut(5);
}
if (getScores().get(getJ1()).getSet() >= getNbrS() || getScores().get(getJ2()).getSet() >= getNbrS()) {
if (getScores().get(getJ1()).getAdv() >= getNbrSE() || getScores().get(getJ2()).getAdv() >= getNbrSE()) {
getScores().get(getJ1()).resetSet();
getScores().get(getJ2()).resetSet();
}
}
if (getScores().get(getJ1()).getManche() == getNbrManche() || getScores().get(getJ2()).getManche() == getNbrManche()) {
getScores().get(getJ1()).resetManche();
getScores().get(getJ2()).resetManche();
}
if (getScores().get(getJ1()).getMatch() == getNbrMatch() || getScores().get(getJ2()).getMatch() == getNbrMatch()) {
getScores().get(getJ1()).resetMatch();
getScores().get(getJ2()).resetMatch();
}
}
protected class ReinitThread implements Runnable {
@Override
public void run() {
if (getScores().get(getJ2()).getPoint() == getNbrT() || getScores().get(getJ1()).getPoint() == getNbrT()) {
// TODO Auto-generated method stub
Windows.panBoutonsJeux.setVisible(false);
try {
Thread.sleep(3000);
} catch (InterruptedException e) {
// TODO Auto-generated catch block
e.printStackTrace();
}
reinit();
Windows.panBoutonsJeux.setVisible(true);
}
}
}
@Override
public void addPartieObservateur(ObservateurPartie obs) {
// TODO Auto-generated method stub
this.listPartieObservateur.add(obs);
}
@Override
public void delPartieObservateur() {
// TODO Auto-generated method stub
this.listPartieObservateur = new ArrayList<ObservateurPartie>();
}
@Override
public void updatePartieObservateur() {
// TODO Auto-generated method stub
for(ObservateurPartie obs : this.listPartieObservateur )
obs.updatePartie(this.comment);
}
}

Class Personnage

Code :

package Metier;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.Map;
import Observer.Observable;
import Observer.Observateur;
public class Personnage implements Observable {
protected int profil = 10;
protected int statut = 5;
protected Map<Integer, String> perso;
protected String pathIconPerso = "/";
private ArrayList<Observateur> listPersoObservateur = new ArrayList<Observateur>();
public Personnage(String pathImg) {
this.profil = 10;
this.statut = 5;
perso = new HashMap<Integer, String>();
for (int i = 0; i<=10; i++) {
perso.put(i, pathIconPerso+pathImg+i+".png" );
}
}
public void setPersoStatut(int c) {
this.statut = c;
this.updatePersoObservateur();
}
public int getPersoStatut() {
return this.statut;
}
public int getPersoProfil() {
return this.profil;
}
public int getPersoSize() {
return this.perso.size();
}
public String getPersoIcon() {
return this.perso.get(this.statut);
}
public String getPersoIcon(int c) {
return this.perso.get(c);
}
@Override
public void addPersoObservateur(Observateur obs) {
this.listPersoObservateur.add(obs);
}
@Override
public void delPersoObservateur() {
this.listPersoObservateur = new ArrayList<Observateur>();
}
@Override
public void updatePersoObservateur() {
for(Observateur obs : this.listPersoObservateur )
obs.updatePerso(this.statut);
}
}

Class Score

Code :

package Metier;
import java.util.ArrayList;
import ObserverScores.ObservableScores;
import ObserverScores.ObservateurScores;
public class Score implements ObservableScores {
/// variable de Score
protected String joueurName;
protected int point;
protected int set;
protected int manche;
protected int match;
protected int adv;
private ArrayList<ObservateurScores> listScoreObservateur = new ArrayList<ObservateurScores>();
/// constructeur
public Score(String name) {
this.joueurName = name;
this.point = 0;
this.set = 0;
this.manche = 0;
this.match = 0;
this.adv = 0;
}
public void setPoint() {
this.point += 1;
this.updateScoreObservateur();
}
public void setSet() {
this.set += 1;
this.updateScoreObservateur();
}
public void setManche() {
this.manche += 1;
this.updateScoreObservateur();
}
public void setMatch() {
this.match += 1;
this.updateScoreObservateur();
}
public void resetPoint() {
this.point = 0;
this.updateScoreObservateur();
}
public void resetSet() {
this.point = 0;
this.set = 0;
this.updateScoreObservateur();
}
public void resetManche() {
this.point = 0;
this.set = 0;
this.manche = 0;
this.updateScoreObservateur();
}
public void resetMatch() {
this.point = 0;
this.set = 0;
this.manche = 0;
this.match = 0;
this.updateScoreObservateur();
}
public int getPoint() {
return this.point;
}
public int getSet() {
return this.set;
}
public int getManche() {
return this.manche;
}
public int getMatch() {
return this.match;
}
public String getName() {
return this.joueurName;
}
public void setAdv(int adv) {
this.adv = adv;
}
public int getAdv() {
return this.adv;
}
@Override
public void addScoreObservateur(ObservateurScores obs) {
this.listScoreObservateur.add(obs);
}
@Override
public void delScoreObservateur() {
this.listScoreObservateur = new ArrayList<ObservateurScores>();
}
@Override
public void updateScoreObservateur() {
for(ObservateurScores obs : this.listScoreObservateur )
obs.updateScore(this.point, this.set, this.manche, this.match);
}
}

package Vue
Class Windows

Code :

package vue;
import java.awt.BorderLayout;
import java.awt.CardLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Font;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.ImageIcon;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.SwingConstants;
import Controle.Bouton;
import Metier.Partie;
import Observer.Observateur;
import ObserverCoups.ObservateurCoups;
import ObserverPartie.ObservateurPartie;
import ObserverScores.ObservateurScores;
public class Windows extends JFrame {
protected Partie partie = new Partie(5, 2, 2, 1, 1, "S_", "U_" );
/// Jpanel
protected JPanel panContainer = new JPanel();
protected JPanel panGridBagAction = new JPanel();
protected JPanel panGridBagScore = new JPanel();
protected JPanel panJ1 = new JPanel();
protected JPanel panJ2 = new JPanel();
protected JPanel panPoint = new JPanel();
protected JPanel panSet = new JPanel();
protected JPanel panManche = new JPanel();
protected JPanel panMatch = new JPanel();
protected JPanel panIconJ1 = new JPanel();
protected JPanel panIconJ2 = new JPanel();
protected JPanel panStart = new JPanel();
protected JPanel panComment = new JPanel();
protected JPanel panPersonnage = new JPanel();
protected JPanel panPersoJ1 = new JPanel();
protected JPanel panPersoJ2 = new JPanel();
protected JPanel panCoups = new JPanel();
protected JPanel panCoupsJ1 = new JPanel();
protected JPanel panCoupsJ2 = new JPanel();
protected JPanel panBoutons = new JPanel();
public static JPanel panBoutonsJeux = new JPanel();
protected JPanel panBoutonsOpt = new JPanel();
// CardLayout
protected CardLayout clayPersoJ1;
protected CardLayout clayPersoJ2;
protected CardLayout clayCoupsJ1;
protected CardLayout clayCoupsJ2;
/// JLabel
protected JLabel labJ1 = new JLabel("Joueur 1" );
protected JLabel labJ2 = new JLabel("Joueur 2" );
protected JLabel labPoint = new JLabel("Point" );
protected JLabel labSet = new JLabel("Set" );
protected JLabel labManche = new JLabel("Manche" );
protected JLabel labMatch = new JLabel("Match" );
protected JLabel labPointJ1 = new JLabel("0" );
protected JLabel labSetJ1 = new JLabel("0" );
protected JLabel labMancheJ1 = new JLabel("0" );
protected JLabel labMatchJ1 = new JLabel("0" );
protected JLabel labPointJ2 = new JLabel("0" );
protected JLabel labSetJ2 = new JLabel("0" );
protected JLabel labMancheJ2 = new JLabel("0" );
protected JLabel labMatchJ2 = new JLabel("0" );
protected JLabel labVersus = new JLabel("VS" );
protected JLabel labComment;
protected JLabel picLabCoupJ1;
protected JLabel picLabCoupJ2;
protected JLabel picLabIconJ1;
protected JLabel picLabIconJ2;
protected JLabel picLabSttPersoJ1;
protected JLabel picLabSttPersoJ2;
protected Bouton b1;
protected Bouton b2;
protected Bouton b3;
protected String imgPath = "/";
protected String[] tabCoups = {imgPath+"unknow.png",imgPath+"jan.png",imgPath+"ken.png",imgPath+"pon.png"};
public Windows() {
this.setTitle("Jan! Ken! Pon!" );
this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
this.setLocation(0, 0);
this.setResizable(false);
this.setSize(700, 650);
panJ1.setPreferredSize(new Dimension(300, 50));
panJ1.setBackground(Color.BLUE);
panJ2.setPreferredSize(new Dimension(300, 50));
panJ2.setBackground(Color.RED);
panPoint.setPreferredSize(new Dimension(200, 50));
panPoint.setBackground(Color.lightGray);
panSet.setPreferredSize(new Dimension(200, 50));
panSet.setBackground(Color.green);
panManche.setPreferredSize(new Dimension(200, 50));
panManche.setBackground(Color.lightGray);
panMatch.setPreferredSize(new Dimension(200, 50));
panMatch.setBackground(Color.green);
panIconJ1.setPreferredSize(new Dimension(300, 150));
panIconJ1.setBackground(Color.blue);
panIconJ2.setPreferredSize(new Dimension(300, 150));
panIconJ2.setBackground(Color.red);
panStart.setPreferredSize(new Dimension(200, 50));
panStart.setBackground(Color.lightGray);
panGridBagScore.setLayout(new GridBagLayout());
GridBagConstraints gbcScore = new GridBagConstraints();
//------------------------------------------------
gbcScore.gridx = 0;
gbcScore.gridy = 4;
gbcScore.gridheight = 1;
gbcScore.gridwidth = 1;
panGridBagScore.add(panJ1, gbcScore);
panJ1.add(labJ1, SwingConstants.CENTER);
Font Joueur = new Font("Arial", Font.BOLD, 25);
labJ1.setFont(Joueur);
//-----------------
gbcScore.gridy = 0;
gbcScore.gridx = 1;
panGridBagScore.add(panPoint, gbcScore);
//-----------------
gbcScore.gridwidth = GridBagConstraints.REMAINDER;
gbcScore.gridy = 4;
gbcScore.gridx = 2;
panGridBagScore.add(panJ2, gbcScore);
panJ2.add(labJ2, SwingConstants.CENTER);
labJ2.setFont(Joueur);
//------------------------------------------------
gbcScore.gridx = 0;
gbcScore.gridy = 0;
gbcScore.gridheight = 4;
gbcScore.gridwidth = 1;
gbcScore.fill = GridBagConstraints.VERTICAL;
panGridBagScore.add(panIconJ1, gbcScore);
panIconJ1.setLayout(new BorderLayout());
panIconJ1.add(panPersoJ1, BorderLayout.NORTH);
panPersoJ1.setBackground(Color.white);
panPersoJ1.setLayout(clayPersoJ1 = new CardLayout());
for (int i=0; i<partie.getPersos().get(partie.getJ1()).getPersoSize(); i++) {
picLabSttPersoJ1 = new JLabel(new ImageIcon(getClass().getResource(partie.getPersos().get(partie.getJ1()).getPersoIcon(i))));
panPersoJ1.add(picLabSttPersoJ1, "img"+i);
}
clayPersoJ1.show(panPersoJ1, "img5" );
//On place un écouteur sur persoJ1
partie.getPersos().get(partie.getJ1()).addPersoObservateur(
new Observateur() {
public void updatePerso(int nStatut) {
clayPersoJ1.show(panPersoJ1, "img"+nStatut);
}
});
//-----------------
gbcScore.gridy = 1;
gbcScore.gridx = 1;
gbcScore.gridheight = 1;
gbcScore.fill = GridBagConstraints.HORIZONTAL;
panGridBagScore.add(panSet, gbcScore);
gbcScore.gridx = 1;
gbcScore.gridy = 2;
gbcScore.gridheight = 1;
gbcScore.gridwidth = 1;
panGridBagScore.add(panManche, gbcScore);
gbcScore.gridx = 1;
gbcScore.gridy = 3;
gbcScore.gridheight = 1;
gbcScore.gridwidth = 1;
panGridBagScore.add(panMatch, gbcScore);
gbcScore.gridx = 1;
gbcScore.gridy = 4;
gbcScore.gridheight = 1;
gbcScore.gridwidth = 1;
panGridBagScore.add(panStart, gbcScore);
//-----------------
gbcScore.gridwidth = GridBagConstraints.REMAINDER;
gbcScore.gridheight = 4;
gbcScore.gridy = 0;
gbcScore.gridx = 2;
gbcScore.fill = GridBagConstraints.VERTICAL;
panGridBagScore.add(panIconJ2, gbcScore);
panIconJ2.setLayout(new BorderLayout());
panIconJ2.add(panPersoJ2, BorderLayout.NORTH);
panPersoJ2.setBackground(Color.white);
panPersoJ2.setLayout(clayPersoJ2 = new CardLayout());
for (int i=0; i<partie.getPersos().get(partie.getJ2()).getPersoSize(); i++) {
picLabSttPersoJ2 = new JLabel(new ImageIcon(getClass().getResource(partie.getPersos().get(partie.getJ2()).getPersoIcon(i))));
panPersoJ2.add(picLabSttPersoJ2, "img"+i);
}
clayPersoJ2.show(panPersoJ2, "img5" );
//On place un écouteur sur persoJ2
partie.getPersos().get(partie.getJ2()).addPersoObservateur(new Observateur(){
public void updatePerso(int nStatut) {
clayPersoJ2.show(panPersoJ2, "img"+nStatut);
}
});
//------------------------------------------------
panPoint.setLayout(new BorderLayout());
panSet.setLayout(new BorderLayout());
panManche.setLayout(new BorderLayout());
panMatch.setLayout(new BorderLayout());
Font scores = new Font("Arial", Font.BOLD, 15);
Font scoresNum = new Font("Arial", Font.BOLD, 25);
Font comment = new Font("Arial", Font.BOLD, 20);
labPoint.setFont(scores);
labSet.setFont(scores);
labManche.setFont(scores);
labMatch.setFont(scores);
labPointJ1.setFont(scoresNum);
labSetJ1.setFont(scoresNum);
labMancheJ1.setFont(scoresNum);
labMatchJ1.setFont(scoresNum);
labPointJ2.setFont(scoresNum);
labSetJ2.setFont(scoresNum);
labMancheJ2.setFont(scoresNum);
labMatchJ2.setFont(scoresNum);
labVersus.setFont(scores);
panPoint.add(labPoint, BorderLayout.CENTER);
labPoint.setHorizontalAlignment(SwingConstants.CENTER);
panSet.add(labSet, BorderLayout.CENTER);
labSet.setHorizontalAlignment(SwingConstants.CENTER);
panManche.add(labManche, BorderLayout.CENTER);
labManche.setHorizontalAlignment(SwingConstants.CENTER);
panMatch.add(labMatch, BorderLayout.CENTER);
labMatch.setHorizontalAlignment(SwingConstants.CENTER);
panPoint.add(labPointJ1, BorderLayout.WEST);
panSet.add(labSetJ1, BorderLayout.WEST);
panManche.add(labMancheJ1, BorderLayout.WEST);
panMatch.add(labMatchJ1, BorderLayout.WEST);
panPoint.add(labPointJ2, BorderLayout.EAST);
panSet.add(labSetJ2, BorderLayout.EAST);
panManche.add(labMancheJ2, BorderLayout.EAST);
panMatch.add(labMatchJ2, BorderLayout.EAST);
panStart.add(labVersus);
labVersus.setHorizontalAlignment(SwingConstants.CENTER);
panComment.setPreferredSize(new Dimension(695, 40));
panCoups.setPreferredSize(new Dimension(695, 200));
panCoups.setLayout(new BorderLayout());
panBoutons.setPreferredSize(new Dimension(695, 107));
panBoutons.setLayout(new BorderLayout());
panGridBagAction.setLayout(new GridBagLayout());
GridBagConstraints gbcCtnr = new GridBagConstraints();
panGridBagAction.setBackground(Color.black);
//On place un écouteur sur scoreJ2
partie.getScores().get(partie.getJ2()).addScoreObservateur(new ObservateurScores() {
public void updateScore(int point, int set, int manche, int match) {
labPointJ2.setText(""+point);
labSetJ2.setText(""+set);
labMancheJ2.setText(""+manche);
labMatchJ2.setText(""+match);
}
});
partie.getScores().get(partie.getJ1()).addScoreObservateur(new ObservateurScores() {
public void updateScore(int point, int set, int manche, int match) {
labPointJ1.setText(""+point);
labSetJ1.setText(""+set);
labMancheJ1.setText(""+manche);
labMatchJ1.setText(""+match);
}
});
//------------------------------------------------
gbcCtnr.gridx = 0;
gbcCtnr.gridy = 0;
gbcCtnr.gridheight = 1;
gbcCtnr.gridwidth = GridBagConstraints.REMAINDER;;
panGridBagAction.add(panComment, gbcCtnr);
panComment.add(labComment = new JLabel("Jouer !" ));
labComment.setHorizontalAlignment(SwingConstants.CENTER);
labComment.setFont(comment);
partie.addPartieObservateur(new ObservateurPartie(){
public void updatePartie(String comment) {
labComment.setText(comment);
}
});
gbcCtnr.gridy = 1;
panCoupsJ1.setBackground(Color.white);
panCoupsJ2.setBackground(Color.white);
panGridBagAction.add(panCoups, gbcCtnr);
panCoupsJ1.setPreferredSize(new Dimension(350, 200));
panCoups.add(panCoupsJ1, BorderLayout.WEST);
panCoupsJ2.setPreferredSize(new Dimension(350, 200));
panCoups.add(panCoupsJ2, BorderLayout.EAST);
panCoupsJ1.setLayout(clayCoupsJ1= new CardLayout());
panCoupsJ2.setLayout(clayCoupsJ2= new CardLayout());
for (int i=0; i<tabCoups.length; i++) {
picLabCoupJ1 = new JLabel(new ImageIcon(getClass().getResource(tabCoups[i])));
picLabCoupJ2 = new JLabel(new ImageIcon(getClass().getResource(tabCoups[i])));
panCoupsJ1.add(picLabCoupJ1, "img"+i);
panCoupsJ2.add(picLabCoupJ2, "img"+i);
}
clayCoupsJ1.show(panCoupsJ1, "img0" );
clayCoupsJ2.show(panCoupsJ2, "img0" );
//On place un écouteur sur persoJ1
partie.getCoups().get(partie.getJ1()).addCoupObservateur(new ObservateurCoups(){
public void updateCoups(int nCoup) {
clayCoupsJ1.show(panCoupsJ1, "img"+nCoup);
}
});
//On place un écouteur sur persoJ2
partie.getCoups().get(partie.getJ2()).addCoupObservateur(new ObservateurCoups(){
public void updateCoups(int nCoup) {
clayCoupsJ2.show(panCoupsJ2, "img"+nCoup);
}
});
gbcCtnr.gridy = 2;
panGridBagAction.add(panBoutons, gbcCtnr);
panBoutons.add(panBoutonsJeux, BorderLayout.NORTH);
b1 = new Bouton("Jan", 1,imgPath+"jan.png", 95, this.getWidth());
b2 = new Bouton("Ken", 2,imgPath+"ken.png", 95, this.getWidth());
b3 = new Bouton("Pon", 3,imgPath+"pon.png", 95, this.getWidth());
panBoutonsJeux.add(b1);
panBoutonsJeux.add(b2);
panBoutonsJeux.add(b3);
b1.addActionListener(new Jan());
b2.addActionListener(new Ken());
b3.addActionListener(new Pon());
panBoutons.add(panBoutonsOpt, BorderLayout.SOUTH);
//------------------------------------------------
panContainer.add(panGridBagScore);
panContainer.add(panGridBagAction);
this.add(panContainer);
this.setVisible(true);
}
public class Jan implements ActionListener {
@Override
public void actionPerformed(ActionEvent e) {
// TODO Auto-generated method stub
partie.action(1);
}
}
public class Ken implements ActionListener {
@Override
public void actionPerformed(ActionEvent e) {
// TODO Auto-generated method stub
partie.action(2);
}
}
public class Pon implements ActionListener {
@Override
public void actionPerformed(ActionEvent e) {
// TODO Auto-generated method stub
partie.action(3);
}
}
}

je ne maîtrise pas encore très bien java mais je souhaite mieux comprendre.
je m'en remet a vous, Merci d'avance pour votre temps.
Bonne journée

Marsh Posté le 04-12-2013 à 15:40:36

Marsh Posté le 04-12-2013 à 17:09:33

Le constructeur de la classe Windows fait 300 lignes. Il faut découper en méthodes ayant des noms évocateurs du boulot rendu.

Pour les patterns utilisés, je ne sais pas.
Pas le temps d'inspecter 1000 lignes de code. Parfois il faut s'en tenir à des choix de conception maison, surtout si c'est propre, maintenable et que cela fonctionne (c'est ton cas n'est ce pas ?)...

Message édité par willy le kid le 04-12-2013 à 17:10:25

Marsh Posté le 04-12-2013 à 17:12:08

Et descend tout ce qui est 'statique' layout, fontes et dispositions de la classe Windows dans des classes de GUI

Marsh Posté le 05-12-2013 à 11:56:18

Pas trop la flamme non plus pour tout lire, mais :
- pour ta classe Bouton, si tu ne défini le comportement que d'une seule méthode du MouseListener, utilise plutôt un MouseAdapter (classe abstraite qui défini toutes les méthodes du ML comme n'ayant aucune comportement, sauf celui que tu redéfini) ;
- le français dans le code, c'est illisible ("ObservableCoups" [:cerveau vomi] );
- dans ta classe Partie, tu accède aux membres par leurs getters (getCoups()) plutôt qu'au champ directement, pourquoi ?
- respecter les conventions de nommage : une variable commence par une minuscule (classe Partie, J1 et J2 par exemple) ;
- déjà dit, mais trop de taff dans les constructeurs.

C'est très général, mais ça rendra déjà les choses un peu plus claires.

Marsh Posté le 05-12-2013 à 13:29:56

Bonjour LeRiton, Bonjour willy le kid !

Merci pour votre réponse et pour votre temps !
Pour l'interface, je vais découper en méthode, a par en lisibilité cela apporte il autre chose ?
(peut être exécuter a nouveau une partie de ces méthode depuis une autre classe ?)

je comprend pas bien ce que tu veut dire Willy dans le second post, enfin je suis pas sur pourquoi dans des classes GUI je sépare mon label static comment tout seul(avec les autre static) dans une classe différente ?

LeMoutonLeRitonLeMoutonLeRitonLeMoutonLeRitonLeMoutonLeRitonLeMoutonLeRitonLeMoutonLeRitonLeMoutonLeRiton niarrc ! oups >_<"

-Bouton pour le moment j'ai pas fait de modification j'utiliserais les autres méthodes quand la fond sera bien construit.

-le français dans le code, ça cache, autre chose
j'ai créer un package ObserverCoups qui contient ObservableCoups et ObservateurCoups qui ecoute la classe coups, et upadte windows
mais j ai fait la mêmechose avec un package pour Score et les classe Obs...Scores, ...
peut être que je peut dire a tout les objet observé d'utilisé le même package -_-" ?

-les getter setter simplement un truc idiot... modification ok !
-conventions de nommage... modification ok !

question :
Dans ma classe windows j'instancie l'objet partie, si j'ai une interface par la suite qui permet de choisir le nombre de point/set/manche... devrais je créer une classe interne à la classe windows qui instancie l'objet partie avec des paramètres ou créer une classe extérieur ou encore le faire dans la classe partie directement ?

Pour les bouton l'actionPerformed est dans das classe interne a la classe(windows) qui contient l'interface, c'est approprié ?

Merci beaucoup a vous deux et très bonne journée

Sujets relatifs:

Leave a Replay

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

Name *

Email *

URL

Message *