Evidenziare riga corrispondente a ricerca elemento in una JTable

[andbin]

Il Problema è che non sempre funziona ; Non funziona se ordino la JTable cioè se clicco sul titolo di una colonna per ordinarla. Se non faccio l'ordinamento funziona.

Gli indici di riga/colonna nel "model" generalmente sono "stabili", nel senso che non cambiano (a meno che sia il table model stesso ad essere "mutabile" o comunque a poter cambiare struttura radicalmente, es. nel numero/ordine delle colonne). Ma il cambiamento è comunque programmatico, cioè è la applicazione che può essere fatta in modo da cambiare radicalmente il model .... non è detto che centri direttamente l'utente.

Gli indici di riga/colonna nella "view", ovvero quello che si vede in JTable, possono differire da quelli nel model. Gli indici di colonna possono differire se l'utente trascina e riordina le colonne (e questa feature non viene disabilitata). E se c'è implementato un qualche ordinamento, pure gli indici di riga possono differire.

In sostanza, usare gli indici del model:

for(int row = 0 ; row <= InterfacciaPrincipale.modelloJTable.getRowCount() - 1; row++){
if(elementoDaCercare.equals(InterfacciaPrincipale.modelloJTable.getValueAt(row, col)))

e poi usarli nella view:

InterfacciaPrincipale.Jtable.setRowSelectionInterval(row, row);

è "potenzialmente" causa di errori/discrepanze.

Per questo, JTable ha metodi appositi di "traslazione" degli indici quali convertColumnIndexToModel, convertColumnIndexToView, ecc...


P.S. fare accessi con es.:

InterfacciaPrincipale.campoRicerca.getText()
InterfacciaPrincipale.Jtable.setRowSelectionInterval

ovvero campi statici e pubblici di dati o addirittura componenti grafici, denota un pessimo design delle classi. Mi spiace ma è così ...

[Markus85]

Potresti propormi qualche soluzione ?

[Markus85]

Per questo, JTable ha metodi appositi di "traslazione" degli indici quali convertColumnIndexToModel, convertColumnIndexToView, ecc...

Dove vanno applicati questi metodi?
Sugli indici?

P.S. fare accessi con es.:

InterfacciaPrincipale.campoRicerca.getText()
InterfacciaPrincipale.Jtable.setRowSelectionInterval

ovvero campi statici e pubblici di dati o addirittura componenti grafici, denota un pessimo design delle classi. Mi spiace ma è così ...

Mi butti a terra... :-) :-)

Quale soluzione avrei dovuto adottare?

[andbin]

Dove vanno applicati questi metodi?
Sugli indici?

Sì, sugli indici ovviamente. I metodi di traslazione offerti da JTable servono in alcune precise circostanze (non sto a fare esempi adesso). Però nel tuo caso non è detto che servono.

Nota che nel tuo codice hai usato getValueAt sul table model (quindi riga/colonna sono nel "model") e poi hai usato quella riga per setRowSelectionInterval di JTable, che ovviamente è nella "view".

È questo che può portare a discrepanze. Se un valore lo trovi alla riga 10 nel model ..... se c'è un ordinamento sulla tabella, non è detto che si trovi fisicamente sulla riga 10 nella vista in JTable!

La soluzione è semplice e non implica necessariamente l'uso esplicito dei metodi di traslazione: semplicemente usa getRowCount / getColumnCount / getValueAt di JTable.

Mi butti a terra... :-) :-)

Beh no dai ... non era quello l'obiettivo!

Quale soluzione avrei dovuto adottare?

Il punto è che serve avere ben chiari i principi della OOP e magari anche alcuni dei più noti e comuni design pattern. Tanto per dirne una, se la funzionalità di ricerca nella tabella è da fare su più tabelle distinte, magari presenti in pannelli/finestre differenti, a me già verrebbe in mente di estendere JTable e incapsulare lì dentro la logica di ricerca.

[Markus85]

La soluzione è semplice e non implica necessariamente l'uso esplicito dei metodi di traslazione: semplicemente usa getRowCount / getColumnCount / getValueAt di JTable.

Non ho capito ; getRowCount / getColumnCount li ho utilizzati nei for , getValueAt l'ho usato pure.

public void actionPerformed(ActionEvent e){
		
String elementoDaCercare = new String(InterfacciaPrincipale.campoRicerca.getText());
int risultato = InterfacciaPrincipale.Magazzino.cercaArticolo(elementoDaCercare);
 if(risultato == -1)
  new NotificaNessunArticoloPresente();
    else{
     for(int row = 0 ; row <= InterfacciaPrincipale.modelloJTable.getRowCount() - 1; row++){
	for(int col = 0 ; col <= InterfacciaPrincipale.modelloJTable.getColumnCount() -1 ; col++){
	   if(elementoDaCercare.equals(InterfacciaPrincipale.modelloJTable.getValueAt(row, col)))
		InterfacciaPrincipale.Jtable.setRowSelectionInterval(row, row);
						
				}//FOR 2
			}//FOR 1
	}// ELSE				
}// ACTION PERFORMED

Cioè vuoi dire che invece di setRowSelectionInterval devo utilizzare getValueAt per selezionare la riga?

[Markus85]

Aspetta Aspetta....Ho capito.
Io invocavo metodi su modelloJTable e su Jtable anzichè invocarli su uno solo dei due

Cosi adesso mi funziona , anche se ordino la JTable

publicvoid actionPerformed(ActionEvent e){

String elementoDaCercare =newString(InterfacciaPrincipale.campoRicerca.getText());
int risultato =InterfacciaPrincipale.Magazzino.cercaArticolo(elementoDaCercare);
if(risultato ==-1)
newNotificaNessunArticoloPresente();
else{
for(int row =0; row <=InterfacciaPrincipale.Jtable.getRowCount()-1; row++){
for(int col =0; col <=InterfacciaPrincipale.Jtable.getColumnCount()-1; col++){
if(elementoDaCercare.equals(InterfacciaPrincipale.Jtable.getValueAt(row, col)))
InterfacciaPrincipale.Jtable.setRowSelectionInterval(row, row);

}//FOR 2
}//FOR 1
}// ELSE
}// ACTION PERFORMED

[Markus85]

Pardon

publicvoid actionPerformed(ActionEvent e){

String elementoDaCercare =newString(InterfacciaPrincipale.campoRicerca.getText());
 int risultato =InterfacciaPrincipale.Magazzino.cercaArticolo(elementoDaCercare);
  if(risultato ==-1)
   newNotificaNessunArticoloPresente();
   else{
     for(int row =0; row <=InterfacciaPrincipale.Jtable.getRowCount()-1; row++){
      for(int col =0; col <=InterfacciaPrincipale.Jtable.getColumnCount()-1; col++){
       if(elementoDaCercare.equals(InterfacciaPrincipale.Jtable.getValueAt(row, col)))
         InterfacciaPrincipale.Jtable.setRowSelectionInterval(row, row);

       }//FOR 2
     }//FOR 1
   }// ELSE 
}// ACTION PERFORMED