Returns wrong row number with ENTER key in JTable...

Question

I am having a problem to get exact row number ofJTable when ENTER key is pressed.
table.getSelectedRow() always retuns one more value when ENTER key is pressed(fired). like
if i have 5 rows in a table and when my focus is on 1st row and then i press enter key focus moves to next row and it returns 1 instead of 0 for the 1st row.
I have written a code bellow, please check it out and tell me why it happens. Please click on Add Row button for creating rows in table.
import javax.swing.*;
   import javax.swing.table.*;
   import java.awt.*;
   import java.awt.event.*;
   import java.util.Vector;
   class TableTest extends JFrame {
     MyTableModel myModel;
     JTable table;
     JButton button;
     JButton buttonQuery;
     int count = 0;
     public TableTest() {
       myModel = new MyTableModel();
       table = new JTable(myModel);
       table.setPreferredScrollableViewportSize(new Dimension(500, 200));
       JScrollPane scrollPane = new JScrollPane(table);
      table.addKeyListener(new KeyAdapter(){
          public void keyTyped(KeyEvent ke)
            if(ke.getKeyChar() == ke.VK_ENTER){
              System.out.println(table.getSelectedRow());
       getContentPane().add(scrollPane, BorderLayout.CENTER);
       getContentPane().add(button = new JButton("Add Row"),
       BorderLayout.SOUTH);
       getContentPane().add(buttonQuery = new JButton("Query"), BorderLayout.NORTH);
       button.addActionListener(new ActionListener() {
    public void actionPerformed(ActionEvent e) {
      Object [] aRow = new Object [8];
     String aRow1;
     for (int i=0; i<5; i++) {}
     myModel.addRow(aRow);
   buttonQuery.addActionListener(new ActionListener() {
     public void actionPerformed(ActionEvent e) {
       System.out.println(myModel.getRowCount());
       System.out.println(myModel.getValueAt(1, 3));
   addWindowListener(new WindowAdapter() {
     public void windowClosing(WindowEvent e) {
       System.exit(0);
   class MyTableModel extends AbstractTableModel {
   protected final String[] headers =
   { "No", "Vehicle No", "Date of Loss", "Image Type", "Image Description", "Claim Type", "Scan Date", "User Id" };
   Vector data;
   MyTableModel() {
    data = new Vector();
   public int getColumnCount() {
     return headers.length;
   public int getRowCount() {
    return data.size();
   public Object getValueAt(int row, int col) {
   if (row < data.size() && col < 8) {
     Object [] aRow = (Object []) data.elementAt(row);
   return aRow[col];
   } else {
     return null;
   public void addRow(Object [] aRow) {
     data.addElement(aRow);
    fireTableRowsInserted(data.size()-1, data.size()-1);
   public static void main(String[] args) {
   TableTest frame = new TableTest();
   frame.pack();
   frame.setVisible(true);
   }Regards..
jaya

Answer 1

Dear Jaya,
I have a look at your code. And modified a bit as well to really check the getSelectedRow(). Here's the modified code: I think when key event fired, the table row is changed and then is displays the get selected row. You can check this, if you are on the last row, and you press enter, it displays 0, it means first it changes the row and then displays the selected row which is obviously rowNumber+1.
Hope it helps.
import javax.swing.*;
import javax.swing.table.*;
import java.awt.*;
import java.awt.event.*;
import java.util.Vector;
class TableTest
    extends JFrame {
  MyTableModel myModel;
  JTable table;
  JButton button;
  JButton buttonQuery;
  JButton buttonSelectedRow;
  int count = 0;
  public TableTest() {
    myModel = new MyTableModel();
    table = new JTable(myModel);
    table.setPreferredScrollableViewportSize
        (new Dimension(500, 200));
    JScrollPane scrollPane = new JScrollPane(table);
    table.addKeyListener(new KeyAdapter() {
      public void keyTyped(KeyEvent ke) {
        if (ke.getKeyChar() == ke.VK_ENTER) {
          System.out.println(table.getSelectedRow());
    getContentPane().add(scrollPane, BorderLayout.CENTER);
    getContentPane().add(button = new JButton("Add Row"), BorderLayout.SOUTH);
    getContentPane().add(buttonQuery = new JButton("Query"), BorderLayout.NORTH);
    getContentPane().add(buttonSelectedRow = new JButton("Get Selected Row Number"), BorderLayout.NORTH);
    button.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent e) {
        Object[] aRow = new Object[8];
        String
            aRow1;
        for (int i = 0; i < 5; i++) {}
        myModel.addRow(aRow);
    buttonSelectedRow.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent e) {
        System.out.println("Selected Row is: " + table.getSelectedRow());
    buttonQuery.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent e) {
        System.out.println(myModel.getRowCount());
        System.out.println(myModel.getValueAt(1, 3));
    addWindowListener(new WindowAdapter() {
      public void windowClosing(WindowEvent e) {
        System.exit(0);
  class MyTableModel
      extends AbstractTableModel {
    protected final String[] headers = {
        "No", "Vehicle No", "Date of Loss", "Image Type", "Image Description",
        "Claim Type", "Scan Date", "User Id"};
    Vector data;
    MyTableModel() {
      data = new Vector();
    public int getColumnCount() {
      return headers.length;
    public int getRowCount() {
      return data.size();
    public Object getValueAt(int row, int col) {
      if (row < data.size() && col < 8) {
        Object[] aRow = (Object[]) data.elementAt(row);
        return aRow[col];
      else {
        return null;
    public void addRow(Object[] aRow) {
      data.addElement(aRow);
      fireTableRowsInserted(data.size() - 1, data.size() - 1);
  public static void main(String[] args) {
    TableTest frame = new TableTest();
    frame.pack();
    frame.setVisible(true);