Java rowAtPoint () не возвращает правильное значение - PullRequest
2 голосов
/ 12 февраля 2012

Я делаю Java-проект и использую JTable. То, что я хочу сделать, это связать popupmenu с jtable. popupmenu имеет четыре различных items. Некоторые из items должны знать, выбранный row в jtable. Проблема в selectRow -функции. я не могу получить функцию для возврата правильного row. я нажимаю jtable с right mouse button, и я получаю popupmenu в том же столбце, который щелкнул, как и должно быть. но когда я использую addSelectedRow или removeSelectedRow -функции (в menuItem с меткой "addSelected" или "deleteSelected") и любые row (s) не выбраны, я использую selectRow -функцию для поиска row я нажимаю на right mousebutton. Кажется, что rowAtPoint(Point p) не может просто найти правильный ряд. Возвращает eather 0 или -1. Я боролся с этой проблемой почти неделю, поэтому прошу прощать мой код: D Я надеюсь, что код достаточно чистый: D

<- ред -> http://painkiller.comlu.com/images/1.jpg

Вот скриншот проблемы. JTable всегда имеет хотя бы один ряд. Я нажимаю произвольно любую строку (в этой ситуации секунда row). Row не выбран по назначению. Теперь, если я нажму «Lisää valittu», что означает «Add selected», будет вызвана функция selectRow().

/ ** Местоположение

jTbl.getLocation(): java.awt.Point[x=0,y=0]
e.getPoints() java.awt.Point[x=4,y=8]
SwingUtilities.convertPoint(e.getComponent(), e.getPoint(), jTbl) java.awt.Point[x=350,y=31]

сначала я попробовал e.getPoint() * e.getPoints() -> java.awt.Point[x=4,y=8] порядковый номер возвращаемой строки из rowAtPoint(): 0

тогда я попробовал SwingUtilities.convertPoint() * SwingUtilities.convertPoint(tablePopMenu, e.getPoint(), jTbl); -> java.awt.Point[x=28,y=10] порядковый номер возвращаемой строки из rowAtPoint(): 2

так что, кажется, работает, но иногда функция выдает -1 или 0. (нужно исправить эту ошибку), но в основном работает довольно хорошо.

<- ->

private void TableMousePressed(java.awt.event.MouseEvent evt) {
        try
        {
            /*
             * The popupmenu content (items) are standart for all tables
             * The label of popupmenu items are used to select proper 
             * command in select case
             * 1. Add selected-> Add only selected row(s)
             * 2. Add all -> Add all rows
             * 3. Delete selected -> Remove only selected row(s)
             * 4. Delete all-> Remove all row(s)
            */

            javax.swing.JTable temp;    //Temporar jTable alien
            String label = ((javax.swing.JMenuItem)evt.getSource()).getText();   //Switch Case


            temp = (javax.swing.JTable)((JPopupMenu)(evt.getComponent().getParent())).getInvoker();
            switch(label)   //Select rigt method by jMenuItem.getText() method (the label of menu item)            {
                case "Add selected":
                    addSelectedRow(evt, temp);
                    break;
                case "Add all":
                    addAllRows(temp);
                    break;
                case "Delete selected":
                    removeSelectedRow(evt, temp);
                    break;
                case "Delete all":
                    removeAllRows(temp);
                    break;
            }            
        }
        catch(Exception ex){ex.printStackTrace();}
    }

    /**
     * @param e - MouseEvent
     * @param jTbl - JTable which will be used
     */
    private void addSelectedRow(java.awt.event.MouseEvent e, javax.swing.JTable jTbl){
        if(jTbl.getSelectedRow() == -1){ //If no row(s) is selected 
            selectRow(e, jTbl); //select row
        }  
        int cellCount = jTbl.getModel().getColumnCount(); //amount of columns to apply
        int rows[] = jTbl.getSelectedRows(); //amount of rows to apply
        cellCollection = new Object[jTbl.getModel().getColumnCount()]; //Object which contains row and cells. will be added to a jTable as a new row

        for(int row = 0; row < rows.length; row++){ //Creates new row by excisting data
            for(int cell = 0; cell < cellCount; cell++){
                cellCollection[cell] = jTbl.getModel().getValueAt(rows[row], cell);
            }
            ((DefaultTableModel)jTbl.getModel()).addRow(cellCollection); //Adds a new row with data
        }         
    }

    /**
     * @param jTbl - JTable which will be used
     */
    private void addAllRows(javax.swing.JTable jTbl){
        int rowCount = jTbl.getModel().getRowCount(); //Amount of adding rows
        int cellCount = jTbl.getModel().getColumnCount(); //Amount of adding colums
        cellCollection = new Object[jTbl.getModel().getColumnCount()]; //New object which contains a row with columns filled with excisting data in a for-loop

        for(int row = 0; row < rowCount; row++){ 
            for(int cell = 0; cell < cellCount; cell++){
                cellCollection[cell] = jTbl.getModel().getValueAt(row, cell);
            }
            ((DefaultTableModel)jTbl.getModel()).addRow(cellCollection);
        }     
    }

    /**
     * @param e - MouseEvent
     * @param jTbl - JTable which will be used
     */
    private void removeSelectedRow(java.awt.event.MouseEvent e, javax.swing.JTable jTbl){
        if(jTbl.getSelectedRow() == -1){ //If no row(s) is selected
            selectRow(e, jTbl); //selects a row
        }
        int[] rows = jTbl.getSelectedRows(); //Get selected row(s)
        for(int i = rows.length - 1; i >= 0; i--)
            ((DefaultTableModel)jTbl.getModel()).removeRow(rows[i]); //Removes the rows
        getTableToLife(jTbl); //Adds a new empty row if jtable has no rows after removing
    }

    /**
     * @param jTbl - JTable which will be used
     */
    private void removeAllRows(javax.swing.JTable jTbl){
        int rowCount = jTbl.getModel().getRowCount(); //Amount of removing rows
        for(int row = rowCount-1; row >=0; row--){ 
            ((DefaultTableModel)jTbl.getModel()).removeRow(row); //Remove rows
        }
        getTableToLife(jTbl);//Adds a new empty row if jtable has no rows after removing        
    }

    /**
     * @param jTbl - JTable which will be saved
     */
    private void getTableToLife(javax.swing.JTable jTbl){
        if(jTbl.getModel().getRowCount() == 0){ //If jtable has no rows
            int cellCount = jTbl.getModel().getColumnCount(); //How many cells will be added
            cellCollection = new Object[cellCount];
            for(int cell = 0; cell < cellCount; cell++){
                cellCollection[cell] = null; //No new data is required                }   
            ((DefaultTableModel)jTbl.getModel()).addRow(cellCollection); //Adding a new empty row
        }
    }

    /**
     * @param e - Mouse event
     */
    private void selectRow(java.awt.event.MouseEvent e, javax.swing.JTable jTbl)
    {

    //      This is the main problem at the moment

    //    System.out.print("\r " + jTbl.rowAtPoint(e.getPoint()));
    //    ListSelectionModel selector = jTbl.getSelectionModel();
    //    selector.removeSelectionInterval(0, jTbl.getHeight()); //I though it he could help, but no effect (row [0] seems to be selected
    //    int p = jTbl.rowAtPoint(e.getPoint()); //or this function seems to be uncapable to find right row i clicked on jtable
    //    selector.setSelectionInterval(p, p); //returns 0 or -1. reason is unknown

    //    New solution
    ListSelectionModel selector = jTbl.getSelectionModel();
    Point newP = SwingUtilities.convertPoint(tablePopMenu, e.getPoint(), jTbl);


    int p = jTbl.rowAtPoint(newP);
    System.out.print("\rRow number: " + (p));
    if(p <=0)
        selector.setSelectionInterval(p, p);
    else
        selector.setSelectionInterval(p-1, p-1);
    }

Спасибо вам за спасение моего дня!

Ответы [ 3 ]

3 голосов
/ 12 февраля 2012

Я не совсем уверен, но, возможно, метод getPoint () MouseEvent возвращает точку в координатном пространстве всплывающего меню.

Что ожидает rowAtPoint (p) в JTable, это точка в координатном пространстве таблицы.

Вы можете использовать SwingUtilities.convertPoint () для перевода координат.

1 голос
/ 12 февраля 2012

В качестве альтернативы используйте пользовательский TableCellEditor, как показано здесь . Вы сможете однозначно определить уязвимую строку в реализации модели setValueAt(), когда редактор завершит работу.

1 голос
/ 12 февраля 2012

этот код должен показывать реальную проблему, зависит от кода до / после того, как этот код был пропущен, или если он используется

1) используется тип SelectionModel

2) Сортировка и фильтрация , читайте о методах convertXxxIndexToXxx

3) отредактируйте свой вопрос с помощью SSCCE , например

...