Home.... | log in.... | Pub my question! | Sign-up....

problem with JTable getSelectedRows()

We are using JTables to display values in our desktop application and we are using jdk 1.5.0_06.

However when we continuously clicks on the jTable's rows and retrives values of the selected row, it does work fine up to a certain point. But afterwards it always returns the same index value or -1 causing a NullPointerException.

Please help this is really really urgent. The main problem is we cannot really decide when does this really happen, sometimes after a lots of click on the JTable and sometimes after 5 or 6 clicks?

Please help.

[565 byte] By [SachithKevina] at [2007-11-26 12:59:08]

«« using a library (jcurses) that uses jni
»» Select and then Copy Text (JTextArea in an Applet)

# 1

Test the selectedIndex to make sure it is not -1 before you try to use it to access any data/information

int selectedIndex = table.getSelectedRow();

if(selectedIndex != -1) {

// okay to proceed

}

// otherwise, never mind...

crwooda at 2007-7-7 16:57:55 >

top of Java-index,Desktop,Core GUI APIs...

# 2

Well its not always -1. However we are currently checking whether it is -1. The problem is getSelectedRow doesnt get refreshed and it returns an old index value which we have selected previously.The main problem is we cannot decide when does this actually happens.

SachithKevina at 2007-7-7 16:57:55 >

# 3

> The main problem is we cannot decide when does this actually happens.

Not sure how you expect us to decide this either, given you have written custom code and we have no idea what your custom code looks like.

If you need further help then you need to create a [url http://homepage1.nifty.com/algafield/sscce.html]Short, Self Contained, Compilable and Executable, Example Program[/url] (SSCCE) that demonstrates the incorrect behaviour, because I can't guess exactly what you are doing based on the information provided.

And don't forget to use the [url http://forum.java.sun.com/help.jspa?sec=formatting]Code Formatting Tags[/url] so the code retains its original formatting.

camickra at 2007-7-7 16:57:55 >

# 4

Attached is the code. But please note that it requires a Database Connection and it has no main method since this class is called from another class. Please try and provide us a solution since we are in a critical condition

Thanks in advance

package com.epic.wfm.gui.workOrder;

import java.awt.BorderLayout;

import javax.swing.JPanel;

import javax.swing.JInternalFrame;

import javax.swing.JScrollPane;

import java.awt.FlowLayout;

import java.sql.SQLException;

import java.text.SimpleDateFormat;

import java.util.Date;

import java.util.Hashtable;

import java.util.Vector;

import com.epic.wfm.util.CommonInfo;

import javax.swing.JOptionPane;

import javax.swing.JPanel;

import javax.swing.JInternalFrame;

import javax.swing.JLabel;

import javax.swing.JTextField;

import javax.swing.JComboBox;

import javax.swing.JTable;

import javax.swing.JScrollPane;

import javax.swing.event.TableModelEvent;

import javax.swing.event.TableModelListener;

import javax.swing.table.DefaultTableCellRenderer;

import javax.swing.table.DefaultTableModel;

import javax.swing.table.JTableHeader;

import javax.swing.table.TableCellRenderer;

import javax.swing.table.TableColumn;

import javax.swing.JButton;

import javax.swing.ImageIcon;

import javax.swing.JTextArea;

import java.awt.event.KeyEvent;

import java.awt.event.KeyListener;

import com.epic.wfm.businessLogic.SystemFault;//This is to check the assignbility

import com.epic.wfm.gui.workOrder.FaultAssign;

import com.epic.wfm.data.Fault;

import com.epic.wfm.data.WorkGroup;

import com.epic.wfm.data.master.CustomerObservation;

import com.epic.wfm.data.master.OPMC;

import com.epic.wfm.data.master.Status;

import com.epic.wfm.data.master.WGCategory;

import com.epic.wfm.gui.MainMDI;

import com.epic.wfm.gui.workOrder.CustomTableCellRenderer;

import com.epic.wfm.gui.workOrder.FaultDocket;

import com.epic.wfm.util.SystemMessage;

import com.epic.wfm.util.log.StackTraceUtil;

import com.epic.wfm.util.log.SysLogger;

import com.epic.wfm.util.ui.ETextField;

import com.epic.wfm.util.ui.JTextFieldFilter;

import com.epic.wfm.util.ui.calendar.JDateChooser;

import javax.swing.table.TableModel;

import java.awt.AWTEvent;

import java.awt.Color;

import java.awt.Component;

import java.awt.Font;

import java.awt.Dimension;

import java.awt.event.ActionEvent;

import java.awt.event.ActionListener;

import java.beans.PropertyVetoException;

public class FaultExplorer extends JInternalFrame implements KeyListener, TableModelListener {

private JPanel jContentPane = null;

private JPanel pnlSearch = null;

private JPanel pnlFaultTable = null;

private JPanel pnlBottom = null;

private JScrollPane scpSearchResult = null;

public JTable tblFault = null;

private JLabel lbl1 = null;

private JLabel lbl2 = null;

private JLabel lbl3l = null;

private JLabel lbl4 = null;

private JLabel lblTo = null;

private JLabel jLabel = null;

private JLabel jLabel1 = null;

private JLabel jLabel2 = null;

private JLabel jLabel3 = null;

private JTextField txtFaultid = null;

private JTextField txtExchangeArea= null;

private JTextField txtSubscriberNo = null;

private JTextField txtWorkOrderNo = null;

private JComboBox cmbOpmcCode= null;

private JComboBox cmbStatus = null;

private JTextField txtCabDp = null;

private JButton cmdSearch = null;

private JButton cmdExit = null;

JDateChooser calReportedFrom = null;

JDateChooser calReportedTo= null;

JDateChooser calModifiedTo= null;

JDateChooser calModifiedFrom= null;

//Declaration of user variables

OPMC opmc= null;

Vector tmpAllOpmcVec= null;

Hashtable tmpOpmcHashTable= null; // This Hash table for insert code and description

int counter;

Statusstate= null;

VectortmpAllStateVec= null;

Hashtable tmpStateHashTable= null;

String selectedOpmc = null;

Vector rows,columns= null;// used to set the rows and columns in the table tblFault

public final static long MILLLS_PER_DAY= 86400000;// milliseconds per day.

//Hashtables for the Customer Observations and Status

Hashtable hashCustomerObservation= new Hashtable();

Hashtable hashStatusCode= new Hashtable();

//Vectors for Customer Observations and Status

Vector vecCustomerObservation;

DefaultTableModel tabModel;

// Hash tables for worgroups and workgroupcategories

Hashtable hashWorkgroup= new Hashtable();

WorkGroup workGroup;

Vector vecWorkGroup= new Vector();

Hashtable hashWorkgroupCat= new Hashtable();

WGCategory workGroupCat;

Vector vecWGCategory= new Vector();

private JLabel lblReportedate = null;

private JLabel jLabel4 = null;

private JLabel jLabel5 = null;

private JLabel jLabel6 = null;

private JLabel lblStatusInfo = null;

private JTextField txtFaultType = null;

private JLabel lblFaultRefNo = null;

private ETextField txtfaultrefNo = null;

private JLabel lblcusRefNo = null;

private JTextField txtCusRefNo = null;

private JLabel lblLoop = null;

private JTextField txtLoop = null;

public static JButton cmdAssign = null;

public static JButton cmdView = null;

public static JButton cmdReAssign = null;

private JLabel lblWGCategory = null;

private JLabel lblWG = null;

private JComboBox cmbWGCategory = null;

private JComboBox cmbWG = null;

public FaultExplorer(){

super();

// TODO Auto-generated constructor stub

}

public FaultExplorer(String arg0) {

super(arg0);

// TODO Auto-generated constructor stub

initialize();

}

public FaultExplorer(String arg0, boolean arg1) {

super(arg0, arg1);

// TODO Auto-generated constructor stub

initialize();

}

public FaultExplorer(String arg0, boolean arg1, boolean arg2) {

super(arg0, arg1, arg2);

// TODO Auto-generated constructor stub

initialize();

}

public FaultExplorer(String arg0, boolean arg1, boolean arg2, boolean arg3) {

super(arg0, arg1, arg2, arg3);

// TODO Auto-generated constructor stub

initialize();

}

public FaultExplorer(String arg0, boolean arg1, boolean arg2, boolean arg3,

boolean arg4) {

super(arg0, arg1, arg2, arg3, arg4);

// TODO Auto-generated constructor stub

initialize();

opmc= new OPMC();

state= new Status();

workGroup= new WorkGroup();

workGroupCat= new WGCategory();

CustomerObservationtmpCusObser= new CustomerObservation();

try

{

tmpAllStateVec= state.getStatusByCategory("WKOD");

tmpAllOpmcVec= CommonInfo.getactOPMCVect();

vecCustomerObservation= tmpCusObser.getAllCustomerObservations();

}

catch (SQLException e)

{

SysLogger.logError(java.util.logging.Level.SEVERE, this.toString(), "FaultExplorer()", StackTraceUtil.getStackTrace(e));

e.printStackTrace();

}

catch (Exception e)

{

SysLogger.logError(java.util.logging.Level.SEVERE, this.toString(), "FaultExplorer()", StackTraceUtil.getStackTrace(e));

e.printStackTrace();

}

tmpOpmcHashTable= new Hashtable();

tmpStateHashTable= new Hashtable();

cmbOpmcCode.addItem("--Select OPMC--");

String tmpOpmc= null;

for(int i=0;i<tmpAllOpmcVec.size() ;i++)

{

OPMC currentOPMC= (OPMC) tmpAllOpmcVec.elementAt(i);

String tmpOPMCCode= currentOPMC.getOPMCCode();

String tmpDescription= currentOPMC.getDescription();

tmpOpmcHashTable.put(tmpDescription,tmpOPMCCode);

tmpOpmc = tmpOPMCCode;

cmbOpmcCode.addItem(tmpDescription);

}

cmbStatus.addItem("--Select Status--");

for(int i=0;i<tmpAllStateVec.size() ;i++)

{

Status currentState= (Status) tmpAllStateVec.elementAt(i);

String tmpStatusCode= currentState.getStatusCode();

String tmpDescription= currentState.getDescription();

tmpStateHashTable.put(tmpDescription,tmpStatusCode);

hashStatusCode.put(tmpStatusCode,tmpDescription);

cmbStatus.addItem(tmpDescription);

}

for(int i=0;i<vecCustomerObservation.size() ;i++)

{

CustomerObservation currentState= (CustomerObservation) vecCustomerObservation.elementAt(i);

String tmpStatusCode= currentState.getCustomerObservationCode();

String tmpDescription= currentState.getDescription();

hashCustomerObservation.put(tmpStatusCode,tmpDescription);

}

try

{

vecWorkGroup= workGroup.allWorkGroupByOPMCObj(tmpOpmc);

vecWGCategory= workGroupCat.getAllWGCategories();

cmbWGCategory.addItem("--Select WG Category--");

for(int i=0;i<vecWGCategory.size() ;i++)

{

WGCategory currentState= (WGCategory) vecWGCategory.elementAt(i);

String desc= currentState.getDescription();

String code= currentState.getWGCCode();

if(!desc.equals("DEFXXX"))

{

hashWorkgroupCat.put(desc,code);

cmbWGCategory.addItem(desc);

}

//If the user role is MDF get the mdf category and disable the combo box

if(CommonInfo.userLevel.equals("MD"))

{

cmbWGCategory.setSelectedItem("MDF Workgroup");