JPANEL CALENDRIER

Commentaire de alex1er le 26/08/2005 09:27:29

Il manque la classe Grille

@+

Commentaire de XIIIOreiller le 30/08/2005 11:25:24

Bon, je n'ai pas utilisé ton source mais par contre, je vais me permettre d'en faire une analyse rapide.

1) Un GROS défaut selon moi, l'utilisation de la méthode setBounds alors que ton calendrier héritant de JPanel, il sera positionné à l'aide du LayoutManager donc, les coordonnées seront calculées automatiquement

2) Quelques coquilles dans le code mais rien de bien grave (si tu veux que je t'en dise un peu plus, dis le moi, on ne va pas s'éterniser là dessus)

3) Une question me vient à l'esprit, pourquoi n'as tu pas utiliser certaines classes gérant la date??? Cela t'aurais éviter de faire du code pour rien (mais tu dois avoir tes raisons que je ne connais pas mai qui m'intéresse)

Voila, si tu veux que l'on discute plus longuement du comment rendre ton code plus propre (et donc plus exploitable), n'hésite pas demande moi, il n'y a pas de problème.

Commentaire de EPS32HAY le 30/08/2005 11:36:05

Salut,

Moi je connais pas grand chose en JAVA, mais je tiens à te remercier pour ce code qui va surement m'aider.

Bonne prog

Commentaire de tncoder le 30/08/2005 12:36:14

merci ESP32HAY , moi meme je suis en train d'apprendre java et ce site m'aide ennormément , amors bon courage à nous deux ;)

Commentaire de tarzent le 03/10/2005 21:20:47

Voilà, j'en ai codé un assez rapidement. Ca va très vite lorsqu'on a les bonnes classes à disposition. J'ai même eu le temps d'ajouter un changement de locale : les jours sont affichés et disposés (premier jour = dimanche ou lundi) en fonction de cette locale.
Si mon code n'est pas suffisament lisible, n'hésitez pas à me poser des questions.

J'envisage d'en poster un un peu plus évolué dans quelques temps.

###################################################################################

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.text.DateFormatSymbols;
import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.Calendar;
import java.util.Date;
import java.util.GregorianCalendar;
import java.util.Locale;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;


public class JCalendar
extends JPanel {
        
    private Locale _locale = getDefaultLocale();
    
    private GregorianCalendar _calendar = new GregorianCalendar();
    
    private DateFormatSymbols _dateSymbols = new DateFormatSymbols();
    
    private SimpleDateFormat _formatMY = new SimpleDateFormat("MMMM yyyy");
    
    private int _firstDayOfWeek = _calendar.getFirstDayOfWeek();
    
    private final JLabel _monthYear = new JLabel("", SwingUtilities.CENTER);
         
    private final JLabel[] _daysOfWeek = new JLabel[7];
    
    private final JButton[] _daysNumber = new JButton[42];
    
    private final ActionListener _changeMonth = new ActionListener() {
        public void actionPerformed(final ActionEvent ae) {
            final int nb = "next".equals(ae.getActionCommand()) ? 1: -1;
            _calendar.add(Calendar.MONTH, nb);
        updateMonthYear();
        updateDaysNumber();
        }
    };
    
    public JCalendar() {
        setLayout(new BorderLayout());
        // Month Panel
        final JPanel monthPanel = new JPanel();
        final JButton previous = new JButton("<<");
        previous.addActionListener(_changeMonth);
        final JButton next = new JButton(">>");
        next.addActionListener(_changeMonth);
        next.setActionCommand("next");
        _monthYear.setPreferredSize(new Dimension(120, 20));
        monthPanel.add(previous);
        monthPanel.add(_monthYear);
        monthPanel.add(next);
        add(monthPanel, BorderLayout.NORTH);
        // Day Panel
        final JPanel dayPanel = new JPanel();
        dayPanel.setLayout(new GridLayout(7, 7));
        for (int i = 0; i < 7; ++i)
            dayPanel.add(_daysOfWeek[i] = new JLabel("", SwingUtilities.CENTER));
        for (int i = 0; i < _daysNumber.length; ++i)
            dayPanel.add(_daysNumber[i] = new JButton());
        add(dayPanel, BorderLayout.CENTER);
        // Remplissage des composants
        updateMonthYear();
        updateDaysOfWeek();
        updateDaysNumber();
    }
    
    // Réactualise la locale et réaffiche le contenu des composants
    private void updateLocale(final Locale locale) {
        _locale = locale;
        final Date tmp = _calendar.getTime();
        _calendar = new GregorianCalendar(_locale);
        _calendar.setTime(tmp);
        _firstDayOfWeek = _calendar.getFirstDayOfWeek();
        _dateSymbols = new DateFormatSymbols(_locale);
        _formatMY = new SimpleDateFormat("MMMM yyyy", _locale);
        updateMonthYear();
        updateDaysOfWeek();
        updateDaysNumber();
    }
    
    // Affiche le mois et l'année en cours
    private void updateMonthYear() {
        _monthYear.setText(_formatMY.format(_calendar.getTime()));
    }
    
    // Affiche les jours de la semaine
    private void updateDaysOfWeek() {
        final String[] weekDays = _dateSymbols.getShortWeekdays();
        for (int i = 1; i < weekDays.length; ++i) {
            final int index = (i - 2 + _firstDayOfWeek) % 7 + 1;
            _daysOfWeek[i - 1].setText(weekDays[index]);
        }
    }
    
    // Affiche le numéro des jours
    private void updateDaysNumber() {
        final Date tmp = _calendar.getTime();
        _calendar.set(Calendar.DAY_OF_MONTH, 1);
        final int firstDay = _calendar.get(Calendar.DAY_OF_WEEK);
        final int LocalFirstDay = (firstDay - _firstDayOfWeek + 7) % 7 + 1;
        boolean full = false;
        for (int i = 0; i < _daysNumber.length; ++i) {
            //Détermine si le composant est affiché ou non
            final boolean isNotEmpty = i < LocalFirstDay - 1 || full;
            _daysNumber[i].setVisible(!isNotEmpty);
            // Affichage du jour
            if (!isNotEmpty) {
                final int dayOfMonth = _calendar.get(Calendar.DAY_OF_MONTH);
                _daysNumber[i].setText(String.valueOf(dayOfMonth));
                _calendar.add(Calendar.DAY_OF_MONTH, 1);
                full = 1 == _calendar.get(Calendar.DAY_OF_MONTH);
            }
        }
        _calendar.setTime(tmp);
    }
    
    
    public static void main(final String[] args)
    throws ParseException {
        SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                final JFrame frame = new JFrame();
                frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                frame.getContentPane().add(new JCalendar());
                frame.pack();
                frame.setVisible(true);
            }
        });
    }
}

Commentaire de XIIIOreiller le 22/10/2005 09:55:30

Je suis d'accord avec Natha pour dire qu'il ne sert pas à grand chose de refaire des choses existantes et qu'utiliser ce qui existe est bien, mais n'est il pas plus important d'apprendre (c a d refaire des choses simples pour apprendre les base - comme utiliser un layout ou en faire un soit même). Donc pour moi, le seul truc qui me gene, c'est la non utilisation de layout (non conforme à la norme swing et reutilisabilité que je qualifierait d'inexistante), pour ce qui est de refaire un calendar, là j'aurais tendance à dire pas de problème, c'est même bien car on peut vouloir certaine spécificités, donc d'un point de vue pédagogique, c'est très bien.

Commentaire de XIIIOreiller le 08/11/2005 10:39:37

Tarzent, pour ce qui est du _ pour les propriétés private d'une classe, ce n'est pas la norme car tu ne trouvera jamais une norme de programmation utilisé ceci (je suis prestataire de service en informatique et j'ai vu un paquet de norme de programmation).

Pour une programmation "propre", l'ensemble des propriétés d'une classe devrait être private ou protected (pour l'héritage) avec des méthodes public permettant de modifier ces propriétés.
ex :

private int nomDeLaVariable = 0;

et
public void setVariable(int value) {
   nomDeLaVariable = value;
}
public int getVariable() {
   return nomDeLaVariable ;
}

Après, il est nécessaire de nommer la variable de façon à bien la reconnaitre et une des méthodes le plus souvent employée est d'utiliser le préfixe m_ (pour variable membre) plutot que "_". Après cela est à l'appréciation de chacun mais la méthode que je te donne permet d'utiliser plusieurs préfixe genre s_ pour des constantes static (ceci n'est pas utiliser dans java, généralement c'est en majuscule).

Bref, tout cela pour dire qu'il n'y a pas de norme universelle mais qu'il est intéressant de voir comment cela est fait dans les classes de base de java et de faire pareil pour qu'il y ait une certaine homogénéité dans le code source (cela aide les gens qui utilise votre code).