Обновление таблицы

[RedScreenOfLife]

Проблема следующая: не могу отправить SQL запрос для обновления таблицы, а точнее: ловит исключение SQLException.

...
rs.afterLast(); 
                while(j1 != 0)
                {
                   try{rs.previous();} catch(SQLException e){ System.out.println("упс!"); j1--; continue; }

                   System.out.println("jgf");
                  try
                    {
                   s = "UPDATE transcorp.cargo SET CargoID = \'"
                                    +((DefaultTableModel)jTable1.getModel()).getValueAt(j1-1, 0).toString()+"\', "
                                    +"Name = \'" + ((DefaultTableModel)jTable1.getModel()).getValueAt(j1-1, 1).toString()+"\', "
                                    +"CargoType = \'" + ((DefaultTableModel)jTable1.getModel()).getValueAt(j1-1, 2).toString()+"\', "
                                    +"Unit = \'" + ((DefaultTableModel)jTable1.getModel()).getValueAt(j1-1, 3).toString()+"\', "
                                    +"`Weight(kg)` = \'" + ((DefaultTableModel)jTable1.getModel()).getValueAt(j1-1, 4).toString()
                                    +"\' WHERE CargoID = \'"+(j1-1)+"\'";
                      j1--;
                      System.out.println(s);
                      st.executeUpdate(s);
                      System.out.println("execute complete");
                      j1++;
                      //System.out.println(rs.getRow());
                    }
                  catch(SQLException e){ System.out.println("Невозможно обновить некоторые поля"); return; }
                  catch(java.lang.NullPointerException e ){ System.out.println(",kz"); continue; }
                }
             jLabel2.setText("Обновлено");
             rs.close();
             st.close();
...

Выяснил, что на первой итерации цикла все проходит как надо, а вот когда начинается вторая итерация, то ловит исключение в

try{rs.previous();} catch(SQLException e){ System.out.println("упс!"); j1--; continue; }

И, как оказалось, проблема возникает не только при вызове метода rs.previous(), а еще и при вызове rs.getRow(), например


Подскажите, пожалуйста, что не так

[Fox]

StackTrace и ошибку в студию.

[RedScreenOfLife]

Ошибки никакой нет...есть исключение
а StackTrace: java.sql.SQLException: Operation not allowed after ResultSet closed

в коде произошла небольшая опечатка, но проблема осталась та же

  ...
      +"\' WHERE CargoID = \'"+rs.getString(1)+"\'";

[Fox]

В любом случае даже если запрос рабочий без текста ошибки и трейса довольно сложно понять в чем у вас ошибка.

[RedScreenOfLife]

StackTrace выложил выше)

[Fox]

Сообщение от RedScreenOfLife

Ошибки никакой нет...есть исключение
а StackTrace: java.sql.SQLException: Operation not allowed after ResultSet closed

в коде произошла небольшая опечатка, но проблема осталась та же

  ...
      +"\' WHERE CargoID = \'"+rs.getString(1)+"\'";

В данном сообщение четко сказано что Вы пытаетесь произвести операцию над ResultSet'ом после его закрытия. Соответственно могу предложить или объявление с инициализацией result set'a внести внутрь цикла или проверить как у Вас работает цикл. очень похоже что ваш while обернут каким то другим циклом. и Вы попадаете на блок

rs.close();
st.close();

а дальше снова пытаетесь что то с ними сделать.

[RedScreenOfLife]

Вот вся моя функция. Даже когда я закоментил эти две строчки, все равно выдает это исключение

private void refresh() throws SQLException{
             int j1=0, j2;
             String s;
             Connect connect = new Connect();
             Statement st = connect.getConnection().createStatement();
             ResultSet rs = st.executeQuery("SELECT * FROM transcorp.cargo");
             int rows = ((DefaultTableModel)jTable1.getModel()).getRowCount();
               while (rs.next())
               j1++;
               j2=j1;
             if(rows>j2)
             {
                 while(j2 != rows)
                 {
                   j2++;
                   try {
                  st.executeUpdate("INSERT INTO transcorp.cargo VALUES (\'"
                                   +((DefaultTableModel)jTable1.getModel()).getValueAt(j2-1, 0).toString()+"\',\'"
                                   +((DefaultTableModel)jTable1.getModel()).getValueAt(j2-1, 1).toString()+"\',\'"
                                   +((DefaultTableModel)jTable1.getModel()).getValueAt(j2-1, 2).toString()+"\',\'"
                                   +((DefaultTableModel)jTable1.getModel()).getValueAt(j2-1, 3).toString()+"\',\'"
                                   +((DefaultTableModel)jTable1.getModel()).getValueAt(j2-1, 4).toString()+"\')");
                       }
                  catch(SQLException e){ jLabel2.setText("Невозможно добавить"); continue;}
                 }
             }
                rs.afterLast();
                
                while(j1 != 0)
                {
                   try{rs.previous();} catch(SQLException e){ e.printStackTrace(); j1--; continue; }
                   System.out.println(rs.getRow());
                  try
                    {
                      s = "UPDATE transcorp.cargo SET CargoID = \'"
                                    +((DefaultTableModel)jTable1.getModel()).getValueAt(j1-1, 0).toString()+"\', "
                                    +"Name = \'" + ((DefaultTableModel)jTable1.getModel()).getValueAt(j1-1, 1).toString()+"\', "
                                    +"CargoType = \'" + ((DefaultTableModel)jTable1.getModel()).getValueAt(j1-1, 2).toString()+"\', "
                                    +"Unit = \'" + ((DefaultTableModel)jTable1.getModel()).getValueAt(j1-1, 3).toString()+"\', "
                                    +"`Weight(kg)` = \'" + ((DefaultTableModel)jTable1.getModel()).getValueAt(j1-1, 4).toString()
                                    +"\' WHERE CargoID = \'"+rs.getString(1)+"\'";
                      j1--;
                      System.out.println(s);
                      st.executeUpdate(s);
                      System.out.println("execute complete");
                      //System.out.println(rs.getRow());
                    }
                  catch(SQLException e){ System.out.println("Невозмhgjhfjожно обновить некоторые поля"+(j1));  e.printStackTrace(); return; }
                  catch(java.lang.NullPointerException e ){ System.out.println(",kz"); continue; }
                }
             jLabel2.setText("Обновлено");
             //rs.close();
             //st.close();
    }

[RedScreenOfLife]

О! А executeUpdate случаем не закрывает ResultSet и Statemant автоматически?

[alexinspir]

Тут вы создали постановку и получили с ее помощью результирующий набор:

Statement st = connect.getConnection().createStatement();
ResultSet rs = st.executeQuery("SELECT * FROM transcorp.cargo");

а тут вы выполняете обновление, с помощью той же самой постановки:

st.executeUpdate("INSERT INTO transcorp.cargo VALUES (\'"
           +((DefaultTableModel)jTable1.getModel()).getValueAt(j2-1, 0).toString()+"\',\'"
           +((DefaultTableModel)jTable1.getModel()).getValueAt(j2-1, 1).toString()+"\',\'"
           +((DefaultTableModel)jTable1.getModel()).getValueAt(j2-1, 2).toString()+"\',\'"
           +((DefaultTableModel)jTable1.getModel()).getValueAt(j2-1, 3).toString()+"\',\'"
           +((DefaultTableModel)jTable1.getModel()).getValueAt(j2-1, 4).toString()+"\')");

а теперь посмотрим в документацию, а именно в контракт интерфейса java.sql.Statement:

By default, only one ResultSet object per Statement object can be open at the same time. Therefore, if the reading of one ResultSet object is interleaved with the reading of another, each must have been generated by different Statement objects. All execution methods in the Statement interface implicitly close a statment's current ResultSet object if an open one exists.

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

rs.afterLast();