Temps de chargement long (JTable)

**docv266** · 22/08/2007, 17h37

Bonjour,

J'ai une JTable qui affiche des entiers dans chacunes de ses cases.

Elle prend ses valeur dans un vecteur qui lui se rempli à partir d'un fichier xml.

Lorsque la table contient peu de cases (environ 20), tout marche bien.

Lorsque le nombre de case approche des 2000, le temps de chargement est trés long, voire ça ne charge pas du tout...

Pourtant dans le cas de 2100 cases, j'ai 30 lignes et 70 colonnes, c'est quand même pas énorme ...

Vous pensez que je doit utiliser la classe HugeTableModel ou c'est autre chose?

Merci d'avance.

**®om** · 22/08/2007, 18h21

Montre nous quelques bouts de code

**docv266** · 23/08/2007, 11h03

Pas de souci, alors :

Voici le code de la partie JDOM, création de mes objets.
En fait ça me sert à définir deux vecteurs, qui contiennent chacun les titres des lignes et des colonnes.

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
 
public void makeTableauLigneColonne(String famillePiece, String familleEvent)
	{
		tableauLigne.removeAllElements();
		tableauColonne.removeAllElements();
 
 
		Boolean present1 = false, present2 = false;
		for (int i=0;i<VObjet.size();i++)
		{
			Objet o = VObjet.get(i);
			StringTokenizer token = new StringTokenizer(o.getDate(), "/");
			token.nextToken();
			int mois = Integer.parseInt(token.nextToken());
 
			if (mois == mois_selec || mois_selec == 0)
			{
			if (tableauLigne.isEmpty() && (o.getFamillePiece().equals(famillePiece) || famillePiece.equals("Tout")) && (o.getFamilleEvent().equals(familleEvent) || familleEvent.equals("Tout")))
			{
				tableauLigne.add(o.getPiece());
			}
			else
			{
				present1 = false;
				for (int j=0;j<tableauLigne.size();j++)
				{
					if (o.getPiece().equals(tableauLigne.get(j)))
					{
						present1=true;
						break;
					}
				}
				if (!present1 && (o.getFamillePiece().equals(famillePiece) || famillePiece.equals("Tout")) && (o.getFamilleEvent().equals(familleEvent) || familleEvent.equals("Tout")))
				{
					tableauLigne.add(o.getPiece());
				}
			}
 
			if (tableauColonne.isEmpty() && (o.getFamillePiece().equals(famillePiece) || famillePiece.equals("Tout")) && (o.getFamilleEvent().equals(familleEvent) || familleEvent.equals("Tout")))
			{
				tableauColonne.add(o.getEvent());
			}
			else
			{
				present2 = false;
				for (int j=0;j<tableauColonne.size();j++)
				{
					if (o.getEvent().equals(tableauColonne.get(j)))
					{
						present2=true;
						break;
					}	
				}
				if (!present2 && (o.getFamillePiece().equals(famillePiece) || famillePiece.equals("Tout")) && (o.getFamilleEvent().equals(familleEvent) || familleEvent.equals("Tout")))
				{
					tableauColonne.add(o.getEvent());
				}
			}
		}
		}
	}

Ensuite voici la classe de mon JPanel qui contient ma JTable :

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
 
model = new Model(arbre, (String)fenetre.panelHaut.piece.getSelectedItem(), (String)fenetre.panelHaut.event.getSelectedItem(), fenetre.panelHaut.mois.getSelectedIndex());
 
		TableColumnModel cm = new DefaultTableColumnModel()
		{
			private static final long serialVersionUID = 968242089331222569L;
 
			public void addColumn(TableColumn tc)
			{
				super.addColumn(tc);
			}
		};
 
 
		table = new JTable(model, cm);
 
 
		table.setRowSelectionAllowed(true);
		table.setColumnSelectionAllowed(true);
 
		table.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
 
		table.setBackground(new Color(255, 255, 255));
 
		// redéfinition du comportement à l'affichage des cellules
		table.setDefaultRenderer(String.class, model);
 
		table.setAutoCreateColumnsFromModel(true);
		table.createDefaultColumnsFromModel();
 
 
		spane = new JScrollPane();
		spane.setPreferredSize(new Dimension(700, 500));
		spane.getViewport().add(table);
 
		add(spane);
 
		table.addMouseListener(this);
		table.setModel(model);
		table.revalidate();
		table.repaint();
		table.setVisible(true);
		spane.setVisible(true);
		setVisible(true);

Et enfin le model de ma table :

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
 
public class Model extends AbstractTableModel implements TableModelListener, TableCellRenderer 
{
 
 
	/**
         * 
         */
	private static final long serialVersionUID = 3961249787453440907L;
 
	public Xml arbre;
 
	private static Color COULEUR_LIGNE_SELECTION_SIMPLE = new Color(10, 10, 100, 50);
 
	private static Color COULEUR_LIGNE_TITRE = new Color(10, 120, 10, 50);
 
	public Model(Xml arbre, String famillePiece, String familleEvent, int mois_sele)
	{
		this.arbre = arbre;
		this.arbre.mois_selec = mois_sele;
		this.arbre.makeTableauLigneColonne(famillePiece, familleEvent);
		addTableModelListener(this);
	}
 
	public void tableChanged(TableModelEvent e)
	{
 
	}
 
	public Component getComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column)
	{
		JLabel label = new JLabel("");
		label.setHorizontalAlignment(JLabel.CENTER);
		label.setFont(new Font("Arial", Font.BOLD, 12));
		label.setBackground(new Color(220, 220, 220, 100));
 
		Component compo = null;
		// initialisation du composant
 
		// Composant pour afficher du texte
		label.setText(value == null ? "" : value.toString());
		label.setOpaque(true);
		compo = label;
 
		if (isSelected || hasFocus)
		{
			compo.setBackground(COULEUR_LIGNE_SELECTION_SIMPLE);
		}
		else if (column == 0)
		{
			compo.setBackground(COULEUR_LIGNE_TITRE);
		}
		else 
		{
			int couleur = Integer.parseInt(arbre.getValue(row, column, true));
 
			couleur = couleur*50;
 
			if (couleur < 0)
			{
				couleur = 0;
			}
			if (couleur > 255)
			{
				couleur = 255;
			}
			compo.setForeground(new Color(couleur,0,0,255));
		}
 
		String message= arbre.getValue(row, column, false);
		((JComponent)compo).setToolTipText(message);
 
		return compo;
	}
 
	public Component getTableCellRendererComponent(JTable table, Object value,	boolean isSelected, boolean hasFocus, int row, int column)
	{
		return getComponent(table, value, isSelected, hasFocus, row, column);
	}
 
	public Class<?> getColumnClass(int col)
	{
		return String.class;
	}
 
	public boolean isCellEditable(int row, int col)
	{
		return false;
	}
 
	public int getColumnCount() 
	{
		return arbre.getTabEvent().size()+1;
	}
 
	public int getRowCount() 
	{
		return arbre.getTabPiece().size();
	}
 
	public Object getValueAt(int rowIndex, int columnIndex)
	{
		return arbre.getValue(rowIndex, columnIndex, true);
	}
 
	public String getColumnName(int col)
	{
		if (col == 0)
		{
			return "Lieu";
		}
		else
		{
			return arbre.getTabEvent().get(col-1);
		}
	}
}

Je me rend compte que ça fait beaucoup...

**eclesia** · 23/08/2007, 11h27

le probleme doit venir de ta classe Xml et de l'appelle a la methode getValue :

Code :

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

1
2
3
4
 
public Object getValueAt(int rowIndex, int columnIndex){
return arbre.getValue(rowIndex, columnIndex, true);
}

cette methode doit etre assez longue a executer je pense. sinon il n'y a pas de raison que ca soit long a afficher. (j'ai fait des tables avec une 20 de colonnes et quelques milliers de lignes sans lenteur particulière)

**docv266** · 23/08/2007, 12h22

Merci, cependant c'est pas ça.

J'ai testé ça :

Code :

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

1
2
3
4
5
 
public Object getValueAt(int rowIndex, int columnIndex)
{
return "TEST";
}

Et ça mets toujours autant de temps.

**Yannick_from_31** · 23/08/2007, 13h05

Je vais peut-être dire une bêtise mais on sait jamais !

Dans mon application assez lourde en traitements je vois une différence en rapidité d'exécution quand je m'applique à déclarer certains objects en dehors des boucles (même pour une simple référence).

Exemple :

au lieu de :

Code :

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

1
2
3
4
5
6
7
8
9
 
for (int i=0;i<VObjet.size();i++)
		{
			Objet o = VObjet.get(i);
			StringTokenizer token = new StringTokenizer(o.getDate(), "/");
			token.nextToken();
			int mois = Integer.parseInt(token.nextToken());
 [...]
}

tu pourrais essayer ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
StringTokenizer token;
Objet o;
int mois;
for (int i=0;i<VObjet.size();i++)
		{
			o = VObjet.get(i);
			token = new StringTokenizer(o.getDate(), "/");
			token.nextToken();
			mois = Integer.parseInt(token.nextToken());
 [...]
}

C'est pas grand chose, mais ça peut jouer beaucoup au final sur de gros traitements.

EDIT : fautes

Temps de chargement long (JTable)

Composants Java

Vue hybride

Discussions similaires

Partager

Partager