Ton code peut être amélioré... Je ne discute que les premières lignes, sinon ça prendrait trop de temps.
- Indentation, ligne blanches, espaces entourant les arguments ==> code difficile à lire. Utilise un truc de formatage (si tu es sous Eclipse, fais un CTRL+SHIFT+F)
-
public class test ==> le nom doit commencer par une majuscule (convention Java)
-
JFrame f = new Fenetre();
f.setVisible(true);
==> oui, ça marche... Mais Sun recommande de faire ceci pour lancer une classe Swing:
EventQueue.invokeLater(new Runnable() {
public void run() {
try {
Fenetre frame = new Fenetre();
frame.setVisible(true);
} catch (Exception e) {
e.printStackTrace();
}
}
});
-
private JButton bouton2;
private Map<String, JCheckBox>boxes = new HashMap<String, JCheckBox>();
private Iterator it;
private static String tabl[] = new String[1000];
==> manque de commentaires, nom des variables peu significatif, déclarations qui auraient pu être faites au moment de l'utilisation des variables...
Et ainsi de suite... Encore un tit bout de chemin avant que ce code ressemble à un code de pro. Mais ne désespère pas pour autant. Lis des bouquins de programmation, c'est important...
Concernant le MVC: l'appliquer pour cette simple application (Swing + accès direct à la base) = utiliser un fusil pour tuer une mouche. On applique le MVC dans des modèles plus complexes, genre applications 3 tiers...
++