1

このメソッドには、階層を下るプロシージャ呼び出しのいずれかが例外をスローした場合、前のプロシージャ呼び出しで行われた変更をロールバックしないなど、私が理解できなかったいくつかの問題があります....助けてください

public synchronized boolean save(DTO dto) throws DAOException,IllegalArgumentException
{           boolean retVal=false;
            boolean retVal1=false;
            boolean retVal2=false;
            boolean retVal5=true;
    try{
                connection=dataSource.getConnection();
                connection.setAutoCommit(false);
                cstmt=connection.prepareCall("{call PKG_ALL.PROC_MAIN(?,?,?)}");

                cstmt.setString(1, "A");     cstmt.setString(2, "B");
                cstmt.registerOutParameter(3,Types.VARCHAR);

                ResultSet rs=cstmt.executeQuery();                  
                String ErrMsg=cstmt.getString(3);
                if(ErrMsg.equalsIgnoreCase("Record Inserted"))  retVal=true;
                else retVal=false;

        cstmt.close();

        cstmt1=connection.prepareCall("{call PKG_ALL.PROC_CHILD1(?,?,?)}");

                cstmt1.setString(1, "A");     cstmt1.setString(2, "B");
                cstmt1.registerOutParameter(3,Types.VARCHAR);

                ResultSet rs1=cstmt.executeQuery();                 
                String ErrMsg1=cstmt1.getString(3);
                if(ErrMsg1.equalsIgnoreCase("Record Inserted")) retVal1=true;
                else retVal1=false;

        cstmt1.close();

        if(strSerialNo!=null && strSerialNo.length > 0) // for a non-mandatory multirow in the form
        {  
            cstmt2=connection.prepareCall("{call PKG_ALL.PROC_CHILD2(?,?,?)}");
                for(int k=0;k<strSerialNo.length;k++)
                    {
                        cstmt2.setString(1,"M");
                        cstmt2.setString(2,"I");
                        cstmt2.registerOutParameter(3,Types.VARCHAR);

                        ResultSet rs2=cstmt2.executeQuery();
                        String ErrMsg2=cstmt2.getString(3);
                        if(ErrMsg2.equalsIgnoreCase("Record Inserted")) retVal2=true;
                        else
                            {
                                retVal5=false;
                                retVal2=false;
                            }           
                    } 
            cstmt2.close();
        }   

        **if(retVal&&retVal1&&retVal5)**
                {
                    retVal=true;
                    connection.commit();
                }
                else
                {
                    //connection.rollback();
                    retVal=false;
                }
    }
    catch(SQLException e)
            {
                throw new DAOException(":"+e.getMessage());
            }
            catch(Exception e)
            {
                throw new DAOException(":"+e.getMessage());

            }
            finally
            {
                closeConnection(connection);
            }
            return retVal;
}
4

1 に答える 1

1

階層を下るプロシージャ呼び出しのいずれかが例外をスローした場合、前のプロシージャ呼び出しで行われた変更はロールバックされません。

多くの場合、catchSQLExceptionブロックにロールバックの呼び出しを入れません。例外をスローせずにリクエストの1つが機能しない場合にのみ、ロールバックメソッドを呼び出します。

また、リクエストの実行後に変更をコミットすることはありません。そのため、を呼び出すelseステートメントにconnection.rollback();到達すると、何もコミットされていないため、実際にはロールバックするものはありません。

このページを読んで、コミット/ロールバックを処理する方法の基本的な例を確認してください。

于 2012-10-08T14:36:36.340 に答える