Bataille navale avec interface graphique Fermé

Question

Bonjour tout le monde, Je suis étudiant en informatique et j'ai un projet de fin d'année à remettre pour le mois de mai. J'ai choisis de faire une bataille navale avec une interface graphique.. (SWING obligatoire) (si vous avez d'autres idées de projet vous pouvez aussi me les faire partager) J'ai juste une question concernant le plateau de jeu donc.. J'ai réfléchi à une manière de faire mon plateau de jeu et je n'arrive pas au résultat que je souhaite sauf en utilisant un tableau de boutons.. Je m'explique : le plateau de jeu sera constitué de cases : 10 X 10 (en abscisse les chiffres 0-9, en ordonnée les lettres A-J) chaque case aura donc une coordonnée de type A1, B0, C5, J9 ,... Y aurait il un autre moyen 'normal' de faire ce plateau de jeu (sans boutons) ? Car là, j'ai ma grille avec mes cases, où l'on peut cliquer dessus donc mais vous imaginez, faire 100 classes différentes pour 100 actions différentes en fonction du bouton sur lequel on va cliquer ? Car chaque bouton devra me renvoyer sa propre coordonnée de façon à ce que je puisse contrôler si sous cette case, il y a un bateau ou s'il y a de l'eau.. Donc voilà, je suis bien embêter.. Merci à ceux qui liront et répondront à mon message ! C'est dommage que l'on ne sait pas joindre d'images pour vous montrez le résultat mais je vous met le code de ma classe plateau : package bataillemoi; import java.awt.BorderLayout; import java.awt.Color; import java.awt.GridLayout; import javax.swing.BorderFactory; import javax.swing.JButton; import javax.swing.JPanel; public class Plateau { private JPanel total; private JButton mesBoutons[] = new JButton[121]; public Plateau() { JPanel grille = new JPanel(); total = new JPanel(); // On crée le contenair qui aura le JLabel total.setLayout(new BorderLayout()); // On le positione ! Ici, mode par défaut grille = new JPanel(); grille.setLayout(new GridLayout(11,11)); int taille = 0, lettre = 65, numero = 0; for(int i=0;i<11;i++) { for(int j=0;j<11;j++) { mesBoutons[taille] = new JButton(""); if(i==0&&j==0) mesBoutons[taille].setBackground(Color.BLACK); if(i==0&&j!=0) { mesBoutons[taille].setText(""+numero); mesBoutons[taille].setBackground(Color.ORANGE); mesBoutons[taille].setForeground(Color.red); mesBoutons[taille].setBorder(BorderFactory.createEtchedBorder(Color.blue, Color.red)); numero++; } if(j==0&&i!=0) { mesBoutons[taille].setText(""+(char)lettre); mesBoutons[taille].setBackground(Color.ORANGE); mesBoutons[taille].setBorder(BorderFactory.createEtchedBorder(Color.blue, Color.red)); lettre++; } if(i>0&&j>0) { mesBoutons[taille].setText("."); } grille.add(mesBoutons[taille]); taille++; } } total.add(grille); } public JPanel getTotal() { return total; } public JButton[] getMesBoutons() { return mesBoutons; } } Configuration: Windows 7 / Chrome 33.0.1750.149

KX · Answer

"C'est dommage que l'on ne sait pas joindre d'images pour vous montrez le résultat"
C'est pour éviter les abus. Mais je te la mets ici. Je répondrais à ta question également, le temps de l'écrire.

public static void main(String[] args)
{
    Plateau plateau = new Plateau();
    
    JFrame frame = new JFrame();
    frame.setContentPane(plateau.getTotal());
    frame.pack();
    frame.setVisible(true);
}

KX · Answer

Comme tu l'as fait, tes boutons sont enregistrés dans un tableau à une dimension, il peut déjà être bien de les mettre dans un tableau à deux dimensions pour conserver les coordonnées.

Mais dans ton cas cela ne suffira pas, car vu les interactions que tu attends il te faudra créer une classe (pas 100 hein !) pour définir chacune de tes cases. Dans cette classe, que l'on appellera Case par exemple, tu devras enregistrer le bouton (ou autre) qui te servira à représenter ta case côté swing, les coordonées de cette case au sein de la matrice (que je mettrais static dans Case plutôt que dans Plateau), ainsi que divers champs utiles pour ta case.

public class Case
{
    private static final Case[][] cases = new Case[10][10];
    
    public static Case getCase(int i, int j)
    {
        Case c = cases[i][j];
        
        if (c == null)
        {
            c = new Case(i, j);
            cases[i][j] = c;
        }
        
        return c;
    }
    
    private final JButton button;
    
    private final int x;
    private final int y;
    
    private Case(int x, int y)
    {
        if (cases[x][y] != null)
            throw new IllegalStateException("Case existe déjà");
        
        this.x = x;
        this.y = y;
        
        button = new JButton(toString());
        
        button.addActionListener(new ActionListener()
        {
            @Override
            public void actionPerformed(ActionEvent e)
            {
                System.out.println("Je suis " + toString());
            }
        });
        
        cases[x][y] = this;
    }
    
    public JButton getButton()
    {
        return button;
    }
    
    @Override
    public String toString()
    {
        return String.format("[%c%d]", 'A'+x, y);
    }
}
Pour que ça marche tu n'as qu'à remplacer ce morceau de code :

if(i>0&&j>0) {
    mesBoutons[taille].setText(".");
}
Par celui-ci :

if (i>0 && j>0) {
    mesBoutons[taille] = Case.getCase(i-1, j-1).getButton();
}
Même si du coup, il n'est certainement plus utile de conserver le tableau mesBoutons, il faut repenser le programme autour de la grille des cases.

Bataille navale avec interface graphique

2 réponses

Discussions similaires