IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

avec Java Discussion :

Modele M/VC mon code y resemble t-il ?


Sujet :

avec Java

  1. #1
    Membre régulier
    Profil pro
    Inscrit en
    Août 2007
    Messages
    98
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Août 2007
    Messages : 98
    Points : 87
    Points
    87
    Par défaut Modele M/VC mon code y resemble t-il ?
    Bonjour j'ai tenté de réaliser un solveur de sudoku en respectant partiellement l'architecture MVC. Je n'ai en effet pas trop vu l'intérêt du controleur donc en gros j'ai deux classes dont je vous met les grandes lignes ici :

    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
     
    package com.vue;
     
    import java.awt.Color;
    import java.awt.GridLayout;
    import java.awt.Point;
    import java.awt.event.ActionEvent;
    import java.awt.event.ActionListener;
    import java.util.Observable;
    import java.util.Observer;
    import javax.swing.*;
     
    import com.modele;
     
    public class Ihm implements Observer {
     
    	private static int rows = 9;
    	private static int columns = 9;
     
    	JWindow pave = new JWindow();
    	JFrame window;
     
    	BouttonGrille buttonPressed;
    	JButton[] bouttonsGrille = new JButton[rows * columns];
     
    	Solver solver;
     
    	public class BouttonGrille extends JButton implements ActionListener {
     
    		int numero;
     
    		/**
                     * @return the numero
                     */
    		public int getNumero() {
    			return numero;
    		}
     
    		public BouttonGrille(int numero) {
    			setText("");
    			this.numero = numero;
    			addActionListener(this);
    		}
     
    		public void actionPerformed(ActionEvent event) {
    			buttonPressed = this;
    			Point p = new Point(getX(), getY());
     
    			SwingUtilities.convertPointToScreen(p, window);
     
    			pave.setLocation(p);
    			pave.setVisible(true);
    		}
    	}
     
    	public class BouttonChiffrePave extends JButton implements ActionListener {
     
    		int numero;
     
    		public BouttonChiffrePave(int numero) {
    			setText(numero + "");
    			addActionListener(this);
    			this.numero = numero;
    		}
     
    		public void actionPerformed(ActionEvent event) {
    			buttonPressed.setText(numero + "");
    			System.out.println("Boutton Grille : " + buttonPressed.getNumero());
    			System.out.println("Boutton pave : " + numero);
    			solver.setGrille(buttonPressed.getNumero(), numero);
    			pave.setVisible(false);
    		}
    	}
     
    	public class BouttonEffacerPave extends JButton implements ActionListener {
     
    		public BouttonEffacerPave() {
    			setText("C");
    			addActionListener(this);
    		}
     
    		public void actionPerformed(ActionEvent event) {
    			for (int row = 0; row < rows; row++) {
    				for (int column = 0; column < columns; column++) {
    					bouttonsGrille[row * 9 + column].setText("");
    					solver.setGrille(row*9+column, 0);
    				}
    			}
    			pave.setVisible(false);
    		}
    	}
     
    	public class BouttonBlancPave extends JButton implements ActionListener {
     
    		public BouttonBlancPave() {
    			setText("0");
    			addActionListener(this);
    		}
     
    		public void actionPerformed(ActionEvent event) {
    			buttonPressed.setText("");
    			solver.setGrille(buttonPressed.getNumero(),0);
    			pave.setVisible(false);
    		}
    	}
     
    	public class BouttonGoPave extends JButton implements ActionListener {
     
    		public BouttonGoPave() {
    			setText("G");
    			addActionListener(this);
    		}
     
    		public void actionPerformed(ActionEvent event) {
    			pave.setVisible(false);
     
    			new Thread(new Runnable() {
    				public void run() {
    					solver.go();
    				}
    			}).start();
     
    		}
    	}
     
     
     
    	public Ihm(Solver solver) {
    		super();
    		this.solver = solver;
    	}
     
    	@Override
    	public void update(Observable o, Object arg) {
    		afficher(((Solver) o).getGrille());
    	}
     
    	public void afficher(int[] grille) {
    		for (int i = 0; i < grille.length; i++) {
    			bouttonsGrille[i].setText(grille[i] + "");
    		}
    	}
     
    	public void initGrille() {
    		window = new JFrame("Solver de sudoku");
    		window.getContentPane().setLayout(new GridLayout(rows, columns));
    		for (int row = 0; row < rows; row++) {
    			for (int column = 0; column < columns; column++) {
    				JButton button = new BouttonGrille(row * 9 + column);
    				if ((column / 3 == 0 && row / 3 == 0)
    						|| (column / 3 == 2 && row / 3 == 0)
    						|| (column / 3 == 1 && row / 3 == 1)
    						|| (column / 3 == 0 && row / 3 == 2)
    						|| (column / 3 == 2 && row / 3 == 2)) {
    					button.setBackground(new Color(100, 149, 237));
    				} else {
    					button.setBackground(new Color(46, 139, 87));
    				}
    				bouttonsGrille[row * 9 + column] = button;
    				window.getContentPane().add(button);
    			}
    		}
    		window.setSize(600, 600);
    		window.setVisible(true);
    	}
     
    	public void initPave() {
    		pave.setLayout(new GridLayout(4, 3));
    		for (int row = 1; row <= 3; row++) {
    			for (int column = 1; column <= 3; column++) {
    				JButton button = new BouttonChiffrePave(
    						((row - 1) * 3 + column));
    				pave.add(button);
    			}
    		}
    		JButton buttonC = new BouttonEffacerPave();
    		pave.add(buttonC);
     
    		JButton buttonBlanc = new BouttonBlancPave();
    		pave.add(buttonBlanc);
     
    		JButton buttonGo = new BouttonGoPave();
    		pave.add(buttonGo);
     
    		pave.setSize(150, 150);
    		pave.setVisible(false);
    	}
     
    	public void init() {
    		initGrille();
    		initPave();
    	}
     
    	public static void main(String[] args) {
    		Solver solver=new Solver();
    		Ihm ihm = new Ihm(solver);
    		ihm.init();
     
    		/* La vue s'enregistre au prêt du modèle*/
    		solver.addObserver(ihm);
    	}
    }
    Et le modèle :

    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
     
    package com.modele;
     
    import java.util.*;
    import javax.swing.SwingUtilities;
     
    /* Le solver est le modèle */
     
     
    public class Solver extends Observable{
     
    	private int[] grille=new int[81];
     
    	/**
             * Indicateur pour stopper la recursion 
             */
     
    	int fin = 0;
     
     
    	public void setGrille(int index,int value) {
    		grille[index]=value;
    	}
     
    	public int[] getGrille() {
    		return grille;
    	}
     
     
     
    	/**
             * 
             * Methode permettant de verifier qu'un chiffre place a l'indice case de la
             * grille est valide
             * 
             * @param laGrille
             * @param laCase
             * @return boolean
             */
     
    	private boolean estValide(int[] laGrille, int laCase) {
     
     
    	/**
             * Fonction principale appelee recursivement
             * @param laGrille
             * @param laCase
             */
     
    	void resoudre(int[] laGrille, int laCase) {
     
    		int i=0;
     
    		/* Condition de fin */
    		setChanged();
    		notifyObservers(laGrille);
     
    		if (laCase >= 81||fin==1) {
    			fin=1;
    			return;
     
    		}
     
    		/* Si un chiffre de l'ennonce est present dans la grille on traite la
    		 * case suivante
    		 */
     
    		if (laGrille[laCase] != 0) {
    			resoudre(laGrille, laCase + 1);
    		}
     
    		/*
    		 * Si aucun chiffre n'est present, on test pour une case donnee les
    		 * chiffres de 1 a 9 : Si on trouve une solution satisfaisante, on
    		 * relance la fonction sur la case suivante. Si aucun chiffre ne peut
    		 * etre insere dans la grille, on indique que la case peut etre modifiee
    		 * et on laisse la fonction se terminer
    		 */
     
    		else {
    			for (i = 1; i <= 9; i++) {
    				laGrille[laCase] = i;
    				if (estValide(laGrille, laCase)) {
    					resoudre(laGrille, laCase + 1);
    					if (fin == 1) {
    						return;
    					}
    				}
    			}
    			laGrille[laCase] = 0;
    		}
     
    	}
     
    	public void go() {
    		resoudre(grille, 0);
    	}	
     
    }
    J'ai par endroit simplifié mon code pour le rendre plus lisible.

    En gros l' IHM comporte une grille composé de bouttons dont les labels affichent les résultats :
    - Je stocke les références des bouttons de ma grille dans un tableau.
    - J'ai des classes internes qui correspondent à l'action de mes bouttons sur mon
    ihm.
    - Enfin j'ai une instance de mon modèle pour pouvoir modifier un tableau
    contenant les chiffres de la grille quand j'effectue une action sur mon IHM.
    - Ma classe IHM s'enregistre comme observer de la classe Solver (le modèle)

    Ma classe Solver quand à elle hérite de Observable et notifie un changement à l'IHM.

    Donc je souhaite savoir si je respecte un peu près la philosophie du pattern sachant que je ne comprend pas trop trop l'intérêt du (des ? ) controleurs dans l'architecture. Est-ce pertinent ? Utile ? horrible ? débile ? de stocker une instance du modèle dans la vue ?

    Quel serait l'intérêt de rajouter un controleur ?

    P.S. Si je me trompe ne soyez pas trop dur je débute dans le noble art du java et de la POO

  2. #2
    Membre averti Avatar de Tux++
    Étudiant
    Inscrit en
    Avril 2008
    Messages
    281
    Détails du profil
    Informations professionnelles :
    Activité : Étudiant

    Informations forums :
    Inscription : Avril 2008
    Messages : 281
    Points : 379
    Points
    379
    Par défaut
    Bonjour,

    tu n'as pas vraiment compris l'intêret des controller.
    Cela permets de lier ton modèle et ta vue., ici tu as mélangé vue et controlleurs

    Imagine tu as une vue réprésentant une grille avec des boutons.
    Un controlleur comprenant chaque actionListener nécéssaire dans cette vue et le modèle pour les calculs et changements logiques.

    Ton modèle étends Observable, et ta vue implémente Observer.

    Dans le constructeur du controller tu auras:

    LeNomDuModel m = new LeNomDuModel();
    LeNomDeLaVue v = new LeNomDeLaVue();
    m.addObservers(v);

    ce qui te permets de faire le lien entre les deux.

    Donc quand tu auras un changement dans modèle, tu pourras faire: setChanged(); notifyObservers(Object o)...
    Certified Oracle Advanced PL/SQL Professional
    Certified Oracle APEX Expert
    Certified Oracle SQL Expert

Discussions similaires

  1. Mon code n'est pas interprété !
    Par korriganez dans le forum Langage
    Réponses: 3
    Dernier message: 31/05/2006, 16h46
  2. [Exécutable]puis je creer un executable a partir de mon code ?
    Par youpileouf dans le forum Général Java
    Réponses: 3
    Dernier message: 17/06/2005, 10h15
  3. Optimiser mon code ASP/HTML
    Par ahage4x4 dans le forum ASP
    Réponses: 7
    Dernier message: 30/05/2005, 11h29
  4. Réponses: 1
    Dernier message: 21/02/2005, 13h40
  5. [langage] algo de bissection dans mon code
    Par killy dans le forum Langage
    Réponses: 5
    Dernier message: 19/01/2004, 19h35

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo