Algorithme fifo en interface graphique

**Ulrich237** · 05/12/2019, 11h28

Bonjour,

Je débute les interfaces graphiques en java et j'ai essayé de développer les algorithme d'ordonnancement. Actuellement je suis sur le fifo mais j'ai quelque difficultés au niveau du remplissage de mon jtable.
J'aimerais bien savoir comment récupérer les valeurs de la colonne durée et les afficher dans la colone de tempsFIn.
Le nombre de lignes du tableau est entré par l'utilisateur et lorsqu'on clique sur un bouton le nombre de lignes du tableau s'incrémente ou décrémente en fonction du nombre entré.
Bref j'ai remarqué que le temps de fin d'execution de chaque processus est égal au temps de fin du précédent + la durée du processus mais je n'arrive pas à implémenter.
Je compte sur vous merci d'avance.

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
Public void actionPerformed(ActionEvent e){
//code pour générer les lignes
Object o =e.getSource() ; 
if(o==butonV ){
  int nbre=spinner.getValue() ; 
   DefaultTableModel table = new DefaultTableModel() ; 
    table.addColumn ("processus") ;
     table.addColumn ("temp arrive") ;
      table.addColumn ("duree") ;
      table.addColumn ("finExecution") ;
    for(int i=1; i<nbre+1 ; i++){
       table. addRow (new Object[] {"p"+i})
}
  table.setModel(table ) ; 
}

**joel.drigo** · 05/12/2019, 12h45

Salut,

Envoyé par Ulrich237

J'aimerais bien savoir comment récupérer les valeurs de la colonne durée et les afficher dans la colone de tempsFIn.

Tu dois bien remplir la colonne durée à un moment donné (tu nous donne pas ce code, mais c'est forcément fait quelque part). Si tu peux affecter une valeur dans une colonne, tu peux aussi bien affecter cette valeur dans une autre. Pas besoin donc de récupérer la valeur dans la première colonne.

Code :

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

1
2
3
 
table.setValueAt( row /* index de la ligne */, 2/* index de la colonne durée*/, duree);
table.setValueAt( row /* index de la ligne */, 2/* index de la colonne finExecution*/, duree);

Seulement, à priori tu dois faire la somme de deux valeurs : ce qui n'est pas très clair dans ta question, si c'est finExecution = tempsArrive + duree, ou si finExecution du process d'avant + duree.

Dans le premier cas, il suffit de faire

Code :

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

1
2
3
4
 
int tempArrive = table.getValueAt( row /* index de la ligne */, 1/* index de la colonne arrive*/);
table.setValueAt( row /* index de la ligne */, 2/* index de la colonne durée*/, duree);
table.setValueAt( row /* index de la ligne */, 2/* index de la colonne finExecution*/, arrive + duree);

En revanche, si tu dois mettre dans la colonne finExecution la somme de la valeur dans la ligne précédente, et l'ajouter à la durée. :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
// table est l'instance de JTable
table.setValueAt( row /* index de la ligne */, 2/* index de la colonne durée*/, duree);
if ( row>0 ) {
    long ancienneFinExecution = table.getValueAt(row-1 /* index de la ligne précédente */, 3/* index de la colonne finExecution*/);
    table.setValueAt( row /* index de la dernière ligne */, 3/* index de la colonne finExecution*/, ancienneFinExecution + duree);
}
else {
    table.setValueAt( trow /* index de la ligne */, 3/* index de la colonne finExecution*/, depart + duree);
}

Attention, si tu dois pouvoir changer le tri, ou faire des filtres, il faut soit convertir les indexes de la vue vers le modèle (avec les méthodes de JTable,convertColumnIndexToModel ou convertRowIndexToModel).
Ou alors il faut récupérer le modèle pour travailler directement dessus

Code :

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

DefaultTableModel tableModel = (DefaultTableModel)table.getModel();

Tu peux aussi implémenter ton propre modèle qui fait ça automatiquement, en redéfinissant AbstractTableModel.

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
public class OrdonnanceurTableModel extends AbstractTableModel {
 
	public static final int COLUMN_PROCESSUS = 0;
	public static final int COLUMN_TEMP_ARRIVE = 1;
	public static final int COLUMN_DUREE = 2;
	public static final int COLUMN_FIN_EXECUTION = 3;
 
	private int nbProcess;
	private Map<Integer, Data> map;
 
	public OrdonnanceurTableModel(int nbProcess) {
		this.nbProcess=nbProcess;
		this.map = new HashMap<Integer,Data>();
	}
 
	@Override
	public int getRowCount() {
		return nbProcess;
	}
 
	@Override
	public int getColumnCount() {
		return 4;
	}
 
	@Override
	public String getColumnName(int column) {
		switch(column) {
		case COLUMN_PROCESSUS:
			return "processus";
		case COLUMN_TEMP_ARRIVE:
			return "temp arrive";
		case COLUMN_DUREE:
			return "duree";
		case COLUMN_FIN_EXECUTION:
			return "finExecution";
		}
		return null;
	}
 
	public void setData(int process, long tempArrive, long duree) {
		map.compute(process, (key, data)-> {
			if ( data==null ) {
				data = new Data();
			}
			data.tempArrive = tempArrive;
			data.duree = duree;
			return data;
		});
		fireTableRowsUpdated(process,process);
	}
 
	public void setTempArrive(int process, long tempArrive) {
		map.compute(process, (key, data)-> {
			if ( data==null ) {
				data = new Data();
			}
			data.tempArrive = tempArrive;
			return data;
		});
		fireTableRowsUpdated(process,process);
	}
	public void setDuree(int process, long duree) {
		map.compute(process, (key, data)-> {
			if ( data==null ) {
				data = new Data();
			}
			data.duree = duree;
			return data;
		});
		fireTableRowsUpdated(process,process);
	}
 
 
	@Override
	public Object getValueAt(int rowIndex, int columnIndex) {
		switch(columnIndex) {
		case COLUMN_PROCESSUS: // processus
			return "p"+rowIndex;
		case COLUMN_TEMP_ARRIVE: // tempArrive
			return map.getOrDefault(rowIndex, DEFAULT_DATA).tempArrive;
		case COLUMN_DUREE: // duree
			return map.getOrDefault(rowIndex, DEFAULT_DATA).duree;
		case COLUMN_FIN_EXECUTION: // fin exécution
			return map.getOrDefault(rowIndex, DEFAULT_DATA).getFinExecution();
		}
		return null;
	}
 
	private final Data DEFAULT_DATA = new Data();
 
	private class Data {
 
		public Long duree;
		public Long tempArrive;
 
		public Long getFinExecution() {
			if ( duree==null || tempArrive==null ) {
				return null;
			}
			return tempArrive + duree;
		}
 
	}
 
}

L'action listener devient :

Code :

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

1
2
3
4
5
6
7
Object o =e.getSource() ; 
if(o==butonV ){
  int nbre=spinner.getValue() ; 
   OrdonnanceurTableModel table = new OrdonnanceurTableModel(nbre) ; 
 
  table.setModel(table ) ; 
}

Il te faudra au moment d'affecter tempArrive et/ou duree récupérer le modèle.

Code :

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

1
2
OrdonnanceurTableModel tableModel = (OrdonnanceurTableModel)table.getModel();
tableModel.setData( numeroProcess, tempsArrive, duree);

**Ulrich237** · 05/12/2019, 15h08

Merci ton code fonctionne mais j"aimerai pouvoir entrer les données directement dans le tableau (le temps d'arrivé et la durée) et le programme affichera le temps de fin d'execution dans sa colonne pour chaque processus. Les cases de ton tableau ne sont pas editables donc c'est pas possibles

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
public class control implements ActionListener{
 
    Fifo fifo;
    int nbre =0;
 
    public control(Fifo fifo) {
        this.fifo = fifo;
    }
 
    @Override
    public void actionPerformed(ActionEvent e) {
        Object o = e.getSource();
 
        if (o==fifo.getJb1()) {
            nbre = (int) fifo.getSpinner().getValue();
            OrdonnanceurTableModel table = new OrdonnanceurTableModel(nbre) ;
 
            fifo.getTable().setModel(table);
 
        }
        else if(o==fifo.getJb2()){
            fifo.getTable().setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);
            int ligne =fifo.getTable().getModel().getRowCount();
            int colonne = fifo.getTable().getSelectedColumn();
            int lastRow =fifo.getTable().getModel().getRowCount()-1;
             int Tempsfinlast=0;
 
                for (int i = 1; i < ligne+1; i++) {
                for (int j = 0; j < colonne; j++) {
                    int [][] cellule=  (int[][]) fifo.getTable().getModel().getValueAt(i, j);
 
                     fifo.getTable().setValueAt( i /* index de la ligne */, 2/* index de la colonne durée*/, cellule[i][2]);
                     if (i>1) {
                         long ancienneFinExecution = (long) fifo.getTable().getValueAt(i-1 /* index de la ligne précédente */, 3/* index de la colonne finExecution*/);
                          fifo.getTable().setValueAt(i /* index de la dernière ligne */, 3/* index de la colonne finExecution*/, (int) (ancienneFinExecution +  cellule[1][2]));
                    }else {
                        fifo.getTable().setValueAt( i /* index de la ligne */, 3/* index de la colonne finExecution*/, cellule[i][1] + cellule[1][2]);
                }
              }
 
            }
        }
    }

Jb1 represente le premier bouton et Jb2 le second
Je veux que lorsqu'on click sur Jb2 les données calculer soient afficher dans la colonne temps de fin

**joel.drigo** · 05/12/2019, 15h25

Je pensais que c'était des mesures qui venaient de ton système d'ordonnancement... pas que c'était des valeurs à saisir...

dans la classe OrdonnanceurTableModel, il suffit d'ajouter une méthode isCellEditable dans cette classe pour rendre éditable les cellules. Elle doit retourner true pour une cellule éditable et false sinon.

Par exemple, pour rendre éditable "temp arrive" et "durée" :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
@Override
public boolean isCellEditable(int rowIndex, int columnIndex) {
		switch(columnIndex) {
		case COLUMN_PROCESSUS:
		case COLUMN_FIN_EXECUTION:
			return false;
		case COLUMN_TEMP_ARRIVE:
		case COLUMN_DUREE:
			return true;
		}
		return false;
}

il faut également implémenter setValueAt pour que la valeur soit stocker dans le modèle.

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
@Override
	public void setValueAt(Object value, int rowIndex, int columnIndex) {
		switch(columnIndex) {
		case COLUMN_PROCESSUS: // processus
		case COLUMN_FIN_EXECUTION: // fin exécution
			break;
		case COLUMN_TEMP_ARRIVE: // tempArrive
			if ( !Objects.equals(map.get(rowIndex).tempArrive, value) ) {
				map.get(rowIndex).tempArrive = (Long) value;
				fireTableCellUpdated(rowIndex, columnIndex);
			}
			break;
		case COLUMN_DUREE: // duree
			if ( !Objects.equals(map.get(rowIndex).duree, value) ) {
				map.get(rowIndex).duree = (Long) value;
				fireTableCellUpdated(rowIndex, columnIndex);
			}
		}
	}

Mais ça veut dire qu'il faut aussi que la JTable connaisse les types de données, ce qui se fait par la méthode getColumnClass :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
@Override
	public Class<?> getColumnClass(int columnIndex) {
		switch(columnIndex) {
		case COLUMN_PROCESSUS: // processus
			return String.class;
		case COLUMN_TEMP_ARRIVE: // tempArrive
		case COLUMN_DUREE: // duree
		case COLUMN_FIN_EXECUTION: // fin exécution
			return Long.class;
		}
		return null;
	}

Et il faut pouvoir peut-être récupérer les valeurs à l'extérieur de la JTable ?

**Ulrich237** · 05/12/2019, 15h40

la methode pour éditer les colonnes marche uniquement sur une cellule après avoir inserer les données dans une cellule, on ne peut plus inserer dans une autre et une erreur est signaler

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at controller.OrdonnanceurTableModel.setValueAt(OrdonnanceurTableModel.java:142)

**joel.drigo** · 05/12/2019, 16h24

Envoyé par Ulrich237

la methode pour éditer les colonnes marche uniquement sur une cellule après avoir inserer les données dans une cellule, on ne peut plus inserer dans une autre et une erreur est signaler

on peut éditer des lignes seulement si elles existent oui. Mais dès qu'une ligne existe, toutes ses colonnes existent, dans mon implémentation. Si on cherche à accèder à une ligne non existante, ça fait effectivement une NPE. Le modèle n'est pas prévu pour ajouter des lignes (le code que tu avais montré ne suggérait pas qu'on puisse ajouter des nouveaux processus, puisque tu créais un nouveau modèle à chaque fois. Si on doit pouvoir le faire, il faut modifier le modèle et gérer un modèle interne différement (pas une Map, mais une List).

**joel.drigo** · 05/12/2019, 15h49

Mes deux solutions étaient alternatives, dans le sens que c'est soit l'une soit l'autre.

Ce code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
                for (int i = 1; i < ligne+1; i++) {
                for (int j = 0; j < colonne; j++) {
                    int [][] cellule=  (int[][]) fifo.getTable().getModel().getValueAt(i, j);
 
                     fifo.getTable().setValueAt( i /* index de la ligne */, 2/* index de la colonne durée*/, cellule[i][2]);
                     if (i>1) {
                         long ancienneFinExecution = (long) fifo.getTable().getValueAt(i-1 /* index de la ligne précédente */, 3/* index de la colonne finExecution*/);
                          fifo.getTable().setValueAt(i /* index de la dernière ligne */, 3/* index de la colonne finExecution*/, (int) (ancienneFinExecution +  cellule[1][2]));
                    }else {
                        fifo.getTable().setValueAt( i /* index de la ligne */, 3/* index de la colonne finExecution*/, cellule[i][1] + cellule[1][2]);
                }
              }

et non seulement inadapté avec OrdonnanceurTableModel, mais faux.

déjà int [][] cellule= (int[][]) fifo.getTable().getModel().getValueAt(i, j); sous entend que ce qu'il y a dans la cellule ligne=i, column=j est un tableau à deux dimensions.
Dans le modèle on a des String (le nom du processus par exemple) et des nombres (j'ai supposé que c'était du long (ou Long) parce que c'est de base ce qu'on utilise couramment pour représenter des durées en ms, du moins ce qu'on utilisait avant java.time, mais c'est une autre histoire.
[codefor (int i = 1; i < ligne+1; i++) {
for (int j = 0; j < colonne; j++) {[/code]
ça sert à rien de parcourir toutes les données du modèle, en plus la boucle de 1 à n+1 est erronée : si ligne est le nombre de lignes dans le modèle, < ligne +1 fait qu'on va chercher à accèder à une ligne inexistante, donc plantage.
fifo.getTable().setValueAt( i /* index de la ligne */, 2/* index de la colonne durée*/, cellule[i][2]); ne sert strictement à rien : si une valeur est dans la colonne 2, alors elle déjà dans la colonne 2, ça ne sert à rien de la remettre...
Mon code était dans l'optique qu'une durée était produite par quelque chose (un process géré par un ordonnanceur). S'il ne s'agit que de saisir des valeurs, tu supprimes toute cette boucle et tu utilises ce que j'ai mis dans mon précédent message et tu vas voir que ça marche tout seul, dès que tu auras saisi une durée et un temp_arrive, le finExecution va se calculer et s'afficher.

Code :

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

 fifo.getTable().setValueAt( i /* index de la ligne */, 3/* index de la colonne finExecution*/, cellule[i][1] + cellule[1][2]);

ce code n'est à utiliser que dans l'optique d'utiliser un DefaultTableModel et non pas un OrdonnanceurTableModel.
en faisant effectivement une boucle :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
ligne = file.getTable().getModel().getRowCount();
 for (int i = 1; i < ligne; i++) {
         Long tempArrive = (Long)fifo.getTable().getValueAt(i, OrdonnanceurTableModel.COLUMN_TEMP_ARRIVE);
         Long duree = (Long)fifo.getTable().getValueAt(i, OrdonnanceurTableModel.COLUMN_DUREE);
         Long finExecution;
         if ( tempArrive==null || duree==null ) {
           finExecution=null;
         }
         else {
             finExecution=tempArrive+duree;
         }
fifo.getTable().setValueAt( i /* index de la ligne */, OrdonnanceurTableModel.COLUMN_FIN_EXECUTION/* index de la colonne finExecution*/, finExecution); 
}

Mais attention, avec un DefaultTableModel, le modèle ne connait pas les types de données, donc la saisie va se faire comme s'il n'y avait que des String. Du coup, le cast en Long ne va pas fonctionner et il faut faire un code un peu plus compliqué (testes si la cellule contient null, sinon convertir le String en Long, et si ça plante pas alors on a la valeur qu'on veut.

ça aussi ça ne va pas :

Code :

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

1
2
else if(o==fifo.getJb2()){
            fifo.getTable().setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);

Ce n'est pas au moment où on clique sur le bouton de calcul qu'on change le type de sélection, c'est au début, quand on créé la JTable.

Envoyé par Ulrich237

Jb1 represente le premier bouton et Jb2 le second
Je veux que lorsqu'on click sur Jb2 les données calculer soient afficher dans la colonne temps de fin

Quelles données calculées ? Les deux implémentations que je t'ai données calcule automatiquement la valeur dès qu'on alimente le modèle de la JTable, soit explicitement (la première méthode), soit implicitement (la seconde).

**Ulrich237** · 05/12/2019, 16h26

mdr
merci mais il reste deux problème

la premiere est qu'il ya une erreur sur cette ligne
--> long ancienneFinExecution = table.getValueAt(row-1 /* index de la ligne précédente */, 3/* index de la colonne finExecution*/);
malgré le fait que j'ai importé import javax.swing.table.DefaultTableModel; et aussi je ne sais pas si ta variable row represente un table.getRowCount() ou un getSelectedRow(); de même pour la durée

**joel.drigo** · 05/12/2019, 17h07

Envoyé par Ulrich237

mdr
merci mais il reste deux problème

la premiere est qu'il ya une erreur sur cette ligne
--> long ancienneFinExecution = table.getValueAt(row-1 /* index de la ligne précédente */, 3/* index de la colonne finExecution*/);
malgré le fait que j'ai importé import javax.swing.table.DefaultTableModel; et aussi je ne sais pas si ta variable row represente un table.getRowCount() ou un getSelectedRow(); de même pour la durée

row c'est l'index de la ligne dont tu veux écrire la fin d'exécution, dans l'hypothèse ou elle est calculée à partir de la précédente (d'où le -1, pour avoir la précédente). Maintenant, est ce que c'est la dernière, la ligne sélectionnée, ou autre, c'est toi qui le sait. Mais ce code est aussi une alternative au OrdonnanceurModel. Dans cette classe, la fin d'exécution est calculée par

Code :

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

1
2
3
4
5
6
public Long getFinExecution() {
			if ( duree==null || tempArrive==null ) {
				return null;
			}
			return tempArrive + duree;
		}

(avec l'hypothèse que finExecution est égale à tempArrive + duree de la même ligne)

Donc pas besoin de chercher à le calculer ailleurs.

**Ulrich237** · 05/12/2019, 17h10

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
public class control implements ActionListener{
 
    Fifo fifo;
    int nbre =0;
     ArrayList  list ;

    public control(Fifo fifo) {
        this.fifo = fifo;
    }
    
    @Override
    public void actionPerformed(ActionEvent e) {
        Object o = e.getSource();
             fifo.getTable().setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);
        if (o==fifo.getJb1()) {
            nbre = (int) fifo.getSpinner().getValue();
            //OrdonnanceurTableModel table = new OrdonnanceurTableModel(nbre) ;
            DefaultTableModel Table = new DefaultTableModel();
            Table.addColumn("Processus");
            Table.addColumn("date arrivée");
            Table.addColumn("durée exécution");
             Table.addColumn("Fin exécution");
            Table.addColumn("temps attente");
            Table.addColumn("temps rotation");
            for (int i = 1; i <= nbre; i++) {
                Table.addRow(new Object[] {"P"+i});
            }
            fifo.getTable().setModel(Table);
            
        }
        else if(o==fifo.getJb2()){
          int ligne =fifo.getTable().getModel().getRowCount();
            int colonne = fifo.getTable().getSelectedColumn();
            int lastRow =fifo.getTable().getModel().getRowCount()-1;
             int Tempsfinlast=0;
                         
                 for (int i = 1; i < ligne; i++) {
                         Long tempArrive = (Long)fifo.getTable().getValueAt(i, OrdonnanceurTableModel.COLUMN_TEMP_ARRIVE); Exception in thread "AWT-EventQueue-0" java.lang.ClassCastException: java.lang.String cannot be cast to java.lang.Long on signale une erreur 
                         Long duree = (Long)fifo.getTable().getValueAt(i, OrdonnanceurTableModel.COLUMN_DUREE);
                         int finExecution;
                         if ( tempArrive==null || duree==null ) {
                           finExecution=0;
                         }
                         else {
                             finExecution=(int) (tempArrive+duree);
                         }
                fifo.getTable().setValueAt( i , OrdonnanceurTableModel.COLUMN_FIN_EXECUTION, finExecution); 
                }
                 
        }
    }
}

non c'est pas cet hypothèse qu'on utilise
on est plustot sur la deuxième hypothèse

j'ai tester le code précedent uniquement pour pouvoir vérifier si laddition entre les deux colonnes fonctionnait
mais l'appli est plustot basé sur la seconde hypothèse

Algorithme fifo en interface graphique

Interfaces Graphiques en Java

Vue hybride

Discussions similaires

Partager

Partager