15

私はこの質問がSOで何度も尋ねられたことを知っていますが、私は私の正確な問題を理解することができませんでした。

次のコードを使用して、データベース（Table1）からデータを取得し、取得値に基づいて別のTable2を更新しています。一部のAndroidバージョンでは正常に動作しますが、Android4.0.3でテストしたところです。私はこれjava.lang.IllegalStateException:?.attempt to re-open an already-closed objectをで取得していsum_cursor.moveToNext();ます。

このコードをAsyncTaskで使用しています。

 /** Sum of total matched values*/
            Cursor sum_cursor = db.gettotalMatchvalue(this);
             if(sum_cursor!=null)
             {
                 sum_cursor.moveToFirst();
                 for(int j=0; j<sum_cursor.getCount();j++)
                 {    
                     float totalmatchedscore = sum_cursor.getInt(0);
                     float totalingredients = Float.parseFloat(sum_cursor.getString(sum_cursor.getColumnIndex(APPDatabase.CK_TOTALINCREDIENTS)));
                     /**average = totalscore/totalingredients*/
                     double average = totalmatchedscore/totalingredients;
                     int id = Integer.parseInt(sum_cursor.getString(sum_cursor.getColumnIndex(APPDatabase.CK_ID))); 

                 db.updateAverage(id, average); 
                 sum_cursor.moveToNext(); //Here is the problem
                 }  
             }   
             db.close();

私の更新メソッドのコーディング

/** Update average */
public void updateAverage(int id,double average)
{
    SQLiteDatabase db = getWritableDatabase();
    ContentValues values = new ContentValues();
    values.put(CK_FINALVALUE,average);
    db.update(TABLE, values,CK_ID+" = "+id , null);   
}

私はここで何を間違っていますか？

私はあなたの多くがこの状況に遭遇することを知っています。みんな助けてくれませんか。

ご協力いただきありがとうございます。

4

3 に答える 3

23

クエリの結果を反復処理している間は、テーブルを更新できません。これには正当な理由があります。追加するデータによって、反復するデータが変更される場合はどうなりますか？カーソルは有効なデータを返しません。

のテーブルにデータを格納しようとするとupdateAverage()、問題が発生します。ループ中の平均値を覚えておいて、カーソルのループが終了した後、最後に1回更新する必要があります。

発生している正確なエラーをさらに説明すると、安全対策として、新しいデータを挿入するという行為により、データベースは現在開いているすべてのカーソルを閉じます。したがってsum_cursor.moveToNext()、平均を更新した後に呼び出すと、カーソルがすでに閉じられていることに少し驚いています。

于 2012-07-25T08:45:04.123 に答える
2

コメントアウトするとどうなりますdb.updateAverage(id, average)か？

于 2012-07-24T15:04:44.583 に答える
0

純粋なSQLを使用して目標を達成できます。これは、すべてのロジックが1つのSQLトランザクションに含まれるため、アーキテクチャの観点からはより高速で優れています。

REPLACE INTO table_where_to_put  SELECT *, (totalmatchedscore/totalingredients) as average  FROM table_with_input_data

REPLACEまたはUPDATEを使用する

于 2014-09-18T11:04:40.160 に答える