Attempt to access dead view row of persistent id 35

Question

Hi!!
I am using jdeveloper 11.1.1.5
In the given code i am Deleting all the rows and then inserting in the same row
    public void Generate(Integer yr, Integer pd, String un) {
        getDBTransaction().setLockingMode(DBTransaction.LOCK_NONE);
        // System.out.println ("Execution started");
        java.util.List<String> schclasslist = new ArrayList<String>();
        java.util.List<String> classlist = new ArrayList<String>();
        java.util.List<String> glacctlist = new ArrayList<String>();
        Row row1, row;
        ViewObjectImpl vo = this.getTpTrialBal1();
        ViewObjectImpl vo1 = this.getFinPeriods1();
        ViewObjectImpl vo2 = this.getSumGab1();
        Row vo1r = vo1.getCurrentRow();
        RowSet glacclrs = findViewObject("GlAccountClasses1").getRowSet();
        RowSet acctrs = findViewObject("GlAccts1").getRowSet();
        RowSet gapbrs = findViewObject("SumGab1").getRowSet();
        for (Row row3 = glacclrs.first(); row3 != null;
             row3 = glacclrs.next()) {
            if (row3.getAttribute("GaclPrimId") == null)
                schclasslist.add((String)row3.getAttribute("GaclId"));
        for (row = glacclrs.first(); row != null; row = glacclrs.next()) {
            for (String o : schclasslist) {
                if (o.toString().equals(row.getAttribute("GaclPrimId") ==
                                        null ? "" :
                                        row.getAttribute("GaclPrimId"))) {
                    classlist.add((String)row.getAttribute("GaclId"));
                    break;
        for (Row rowa = acctrs.first(); rowa != null; rowa = acctrs.next()) {
            for (Object o : schclasslist) {
                if (o.toString().equals(rowa.getAttribute("GlacClId"))) {
                    glacctlist.add((String)rowa.getAttribute("GlacAcct"));
                    break;
            for (Object o : classlist) {
                if (o.toString().equals(rowa.getAttribute("GlacClId"))) {
                    glacctlist.add((String)rowa.getAttribute("GlacAcct"));
                    break;
        vo2.setNamedWhereClauseParam("year", yr);
        vo2.setNamedWhereClauseParam("period", pd);
        vo2.setNamedWhereClauseParam("plant",un);
        vo2.executeQuery();
// In this i am deleting all the rows
        for (int i = 0; i < vo.getRowCount(); i++) {
                 Row vor = vo.getCurrentRow();
            vor.remove();
            System.out.println (vor);
                 this.getTransaction().commit();
//In this i am inserting the rows
        for (row1 = gapbrs.first(); row1 != null; row1 = gapbrs.next()) {
            for (Object o : glacctlist) {
                if (o.toString().equals(row1.getAttribute("GapbAcct"))) {
                    this.getTransaction().commit();
                    Row vor = vo.createRow();
                    vor.setAttribute("TtrbDesc1",
                                     row1.getAttribute("GaclDesc1"));
                    Number n = (Number)row1.getAttribute("SumGapbDebitAmt");
                    vor.setAttribute("TtrbDebitAmt", n);
                    Number n1 = (Number)row1.getAttribute("SumGapbCreditAmt");
                    vor.setAttribute("TtrbCreditAmt", n1);
                    vor.setAttribute("TtrbYear", yr);
                    vor.setAttribute("TtrbPeriod", pd);
                    Number n2 = n.add(n1);
                    vor.setAttribute("TtrbAmount", n2);
                     vor.setAttribute("TtrbSchNo",
                                     row1.getAttribute("GaclSchNo"));
                    break;
        this.getTransaction().commit();
    }

Answer 1

Wilhelm,
Not exactly sure why this happens.
The code you gave is hard to understand because of the more cryptic variables names, which sure mean something to you. For me they make the code hard to understand.
Some stuff you should think about in the code:
// In this i am deleting all the rows
        for (int i = 0; i < vo.getRowCount(); i++) {
                 Row vor = vo.getCurrentRow();
            vor.remove();
            System.out.println (vor);
                 this.getTransaction().commit();
        }This code commits the removal of each row which might be time consuming. You may want to put the commit outside the for loop like
// In this i am deleting all the rows
        for (int i = 0; i < vo.getRowCount(); i++) {
                 Row vor = vo.getCurrentRow();
            vor.remove();
            System.out.println (vor);
        this.getTransaction().commit();Why do you commit before something has happened here?
//In this i am inserting the rows
        for (row1 = gapbrs.first(); row1 != null; row1 = gapbrs.next()) {
            for (Object o : glacctlist) {
                if (o.toString().equals(row1.getAttribute("GapbAcct"))) {
                    this.getTransaction().commit(); //---------------WHY commit here?
                    Row vor = vo.createRow();
...Then you create a row but never add the row to the VO (I'm missing a vo.insertRow(vor)). Resulting in
//In this i am inserting the rows
        for (row1 = gapbrs.first(); row1 != null; row1 = gapbrs.next()) {
            for (Object o : glacctlist) {
                if (o.toString().equals(row1.getAttribute("GapbAcct"))) {
                    Row vor = vo.createRow();
                    vor.setAttribute("TtrbDesc1",
                                     row1.getAttribute("GaclDesc1"));
                    Number n = (Number)row1.getAttribute("SumGapbDebitAmt");
                    vor.setAttribute("TtrbDebitAmt", n);
                    Number n1 = (Number)row1.getAttribute("SumGapbCreditAmt");
                    vor.setAttribute("TtrbCreditAmt", n1);
                    vor.setAttribute("TtrbYear", yr);
                    vor.setAttribute("TtrbPeriod", pd);
                    Number n2 = n.add(n1);
                    vor.setAttribute("TtrbAmount", n2);
                     vor.setAttribute("TtrbSchNo",
                                     row1.getAttribute("GaclSchNo"));
                    vo.insertRow(vor);
                    this.getTransaction().commit();
                    break;
        }The commit after the for loop is obsolete.
Timo