Discussion :

[tolliob]

Bonjour à tous,


Comme exercice, j'ai réalisé une JComboBox réalisant de l'autocomplétion. L'objectif était de pouvoir utilisé un dictionnaire complet (genre 300 000 mots), donc d'implémentant un arbre de complétion. J'ai donc réalisé une classe ComboBoxWithAutomaticCompletion héritant de JComboBox dont l'un des attribut est l'arbre de complétion (objet de la classe Dictionnary)

Tout semble parfait, mais...


Lorsque je fait le test suivant, tout va bien :

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
 
import java.awt.BorderLayout;
import java.awt.FlowLayout;
import java.awt.GridLayout;
 
import javax.swing.JComboBox;
import javax.swing.JFrame;
 
import Exceptions.NoDictionnaryAvailableException;
import Functionning.Dictionnary;
import Windows.ComboBoxWithCompletion.ComboBoxWithAutomaticCompletion;
 
public class Essai extends JFrame {
	private static final long serialVersionUID = 1L;
	private final static String repertoryPath = "...";
	private final static String dicoRelativPath = "Dico/";
	String name = "liste.de.mots.francais.frgut.txt";
	String completeDicoPath = repertoryPath + dicoRelativPath + name;
 
    public Essai() {
    	Dictionnary dico = null;
    	try {
    		dico = new Dictionnary(completeDicoPath, "ISO-8859-1");
    	} catch (NoDictionnaryAvailableException e) {
    		System.out.println("Dictionnaire introuvable, vérifiez votre chemin d'accès !");
    	}
        JComboBox<String> jCombo = new ComboBoxWithAutomaticCompletion(dico);
        getContentPane().setLayout(new BorderLayout());
        getContentPane().add(jCombo);
 
	this.setSize(200, 100);
        this.setVisible(true);
    }
 
    public static void main(String args[]) {
        new Essai();
    }
}

Si je modifie la ligne

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

getContentPane().add(jCombo);

en

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

getContentPane().add(jCombo, BorderLayout.SOUTH);

, la JFrame ne s'affiche plus... Le programme ne ressort pas de la ligne

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

this.setVisible(true);

Avez vous déjà rencontré un cas similaire ???

Si je modifie la ligne

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

getContentPane().setLayout(new BorderLayout());

en

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

getContentPane().setLayout(new FlowLayout());

, c'est la même chose. Par contre, si j'utilise un GridLayout, ca marche nickel : je peux avoir un tableau complet de ma nouvelle JComboBox !!!


Je n'ai aucune idée d'où cela peut bien venir... Mon implémentation de l'autocomplétion ? des Layout ? Je pencherais bien pour la première solution, car lorsque je fais charger un petit dictionnaire (genre 10 mots), je peux bien mettre ma JComboBox au sud d'un BorderLayout, ou utiliser un FlowLayout...


SI quelqu'un à une idée... je suis preneur !

Merci d'avance,

[joel.drigo]

Salut,

Je pense que le problème vient de la différence entre la détermination de la taille des zones layoutées : dans le cas de GridLayout ou de BorderLayout CENTER (ce qui est le cas par défaut lorsque tu ne précises pas de contrainte comme dans ton premier appel), le layout impose une taille, alors que dans les autres cas, c'est la taille du composant qui prévaut (la preferredSize). La méthode getDisplaySize() de la JComboBox, qui est utilisée pour déterminer cette dimension, parcourt l'ensemble des valeurs du modèle. Je suppose que dans le cas de 300000 valeurs, cela prend un certain temps qui te donne l'impression que ça bloque. Essayes d'affecter une valeur de prototype pour déterminer la taille sans parcourir l'ensemble des valeurs du modèle (méthode setPrototypeDisplayValue()) : rapidement pour commencer en mettant une valeur bidon (comme "xxxxxxxxx"), juste pour valider (et au final la plus longue des valeurs dans le dictionnaire, à priori, déterminable au chargement du dictionniaire, ou une valeur arbitraire, si tu veux).

[tolliob]

Bien joué !!! En plein dans le mille !

Bravo, c'est exactement ça !

Tu es mon dieu vivant, Joël !!!!

C'est nickel maintenant !

[joel.drigo]

En fait, pour être totalement nickel, il faudrait calculer la largeur en pixels de chaque valeur du dictionnaire pour la police courante, puisque celle-ci peut être choisie par l'utilisateur (dans le thème de son environnement) donc inconnue d'avance (et non pas la longueur en caractères). Parce si le mot Pomme est plus grand que le mot Rififi en Arial (malgré qu'il ait moins de lettres), il est plus petit en Courrier New, par exemple. Et calculer cette dimension au chargement ne fait que déplacer le temps de calcul (ce qui est toujours mieux que de le faire à chaque nouvelle combo, la valeur calculée étant cachée dans l'instance de ComboBoxUI). Ce serait quand même probablement plus rapide, parce qu'on ne déterminerait que les dimensions de la valeur et pas du composant de rendu, mais je ne sais pas évaluer cette différence sans faire de mesures.