Close() ไม่เคยถูกเรียกอย่างชัดเจนบนฐานข้อมูล แม้ว่าจะใช้งาน close() แล้วก็ตาม

Eclipse แสดงข้อผิดพลาดข้างต้นให้ฉันทราบ แม้ว่าฉันได้ใช้เมธอดดังกล่าวแล้ว close()

นี่คือรหัส:

public void update_project(View view) {

    EditText cinone = (EditText) findViewById(R.id.pname);
    pname = cinone.getText().toString();

    String fDate = pday + ". " + pmonth + ". " + pyear;

    projectdatabase dbupdate = new projectdatabase(UpdateProject.this);

    dbupdate.open();
    dbupdate.updateProject(id, pname, fDate); 
    dbupdate.close();

}

ล็อกแคท:

12-31 22:05:51.114: E/SQLiteDatabase(4180): close() was never explicitly called on  
database '/data/data/com.kk.project/databases/project_db' 
12-31 22:05:51.114: E/SQLiteDatabase(4180):  
android.database.sqlite.DatabaseObjectNotClosedException: Application did not close the     
cursor or database object that was opened here
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at  
android.database.sqlite.SQLiteDatabase.<init>(SQLiteDatabase.java:1943)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at 
android.database.sqlite.SQLiteDatabase.openDatabase(SQLiteDatabase.java:1007)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at   
android.database.sqlite.SQLiteDatabase.openDatabase(SQLiteDatabase.java:986)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at 
android.database.sqlite.SQLiteDatabase.openOrCreateDatabase(SQLiteDatabase.java:1051)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at 
android.app.ContextImpl.openOrCreateDatabase(ContextImpl.java:770)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at  
android.content.ContextWrapper.openOrCreateDatabase(ContextWrapper.java:221)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at 
android.database.sqlite.SQLiteOpenHelper.getWritableDatabase(SQLiteOpenHelper.java:157)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at     
com.kk.project.projectdatabase.open(projectdatabase.java:66)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at  
com.kk.project.ProjectExplorer.getinfo(ProjectExplorer.java:30)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at    
com.kk.project.ProjectExplorer.onCreate(ProjectExplorer.java:25)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at  
android.app.Activity.performCreate(Activity.java:4465)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at  
android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1049)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at 
android.app.ActivityThread.performLaunchActivity(ActivityThread.java:1920)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at 
android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:1981)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at  
android.app.ActivityThread.access$600(ActivityThread.java:123)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at 
android.app.ActivityThread$H.handleMessage(ActivityThread.java:1147)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at  
android.os.Handler.dispatchMessage(Handler.java:99)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at 
android.os.Looper.loop(Looper.java:137)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at  
android.app.ActivityThread.main(ActivityThread.java:4424)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at   
java.lang.reflect.Method.invokeNative(Native Method)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at     
java.lang.reflect.Method.invoke(Method.java:511)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at 
com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:784)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at   
com.android.internal.os.ZygoteInit.main(ZygoteInit.java:551)
12-31 22:05:51.114: E/SQLiteDatabase(4180):     at 
dalvik.system.NativeStart.main(Native Method)
12-31 22:05:51.165: E/System(4180): Uncaught exception thrown by finalizer
12-31 22:05:51.174: D/dalvikvm(4180): GREF has increased to 201
12-31 22:05:51.204: I/dalvikvm(4180): threadid=3: reacting to signal 3
12-31 22:05:51.324: E/System(4180): java.lang.IllegalStateException: Don't have   
database lock!
12-31 22:05:51.324: E/System(4180):     at  
android.database.sqlite.SQLiteDatabase.verifyLockOwner(SQLiteDatabase.java:2090)
12-31 22:05:51.324: E/System(4180):     at  
android.database.sqlite.SQLiteDatabase$1.entryRemoved(SQLiteDatabase.java:2182)
12-31 22:05:51.324: E/System(4180):     at   
android.database.sqlite.SQLiteDatabase$1.entryRemoved(SQLiteDatabase.java:2178)
12-31 22:05:51.324: E/System(4180):     at  
android.util.LruCache.trimToSize(LruCache.java:197)
12-31 22:05:51.324: E/System(4180):     at   
android.util.LruCache.evictAll(LruCache.java:285)
12-31 22:05:51.324: E/System(4180):     at   
android.database.sqlite.SQLiteDatabase.deallocCachedSqlStatements  
(SQLiteDatabase.java:2143)
12-31 22:05:51.324: E/System(4180):     at  
android.database.sqlite.SQLiteDatabase.closeClosable(SQLiteDatabase.java:1126)
12-31 22:05:51.324: E/System(4180):     at  
android.database.sqlite.SQLiteDatabase.finalize(SQLiteDatabase.java:1914)
12-31 22:05:51.324: E/System(4180):     at   
java.lang.Daemons$FinalizerDaemon.doFinalize(Daemons.java:182)
12-31 22:05:51.324: E/System(4180):     at  
java.lang.Daemons$FinalizerDaemon.run(Daemons.java:168)
12-31 22:05:51.324: E/System(4180):     at java.lang.Thread.run(Thread.java:856)

สิ่งที่แปลกคือฉันใช้โค้ดที่คล้ายกันอย่างมีค่าและทำงานได้อย่างไม่มีที่ติ นอกจากนี้วิธีการอื่นจากคลาสฐานข้อมูลเดียวกันก็ทำงานได้อย่างสมบูรณ์

รหัสของกิจกรรมที่เรียก updateProject:

public class UpdateProject extends Activity implements OnItemSelectedListener,  DatePickerDialog.OnDateSetListener {

int mPos;
String mSelection;

String pname;
String pdifficulty;

int pday;
int pmonth;
int pyear;

String id;


@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_update_project);

    id = getIntent().getExtras().getString("idforupdate");

    userdatabase viewname = new userdatabase(this);             //Get username
    viewname.open();
    String name = viewname.getusername();
    viewname.close();

    if(name.equals("")) {

        DialogFragment newFragment = new NouserexitFragment();
        newFragment.show(getFragmentManager(), "exit");

    }

    Spinner spinner = (Spinner) findViewById(R.id.difficultyspinner); ArrayAdapter<CharSequence> 
    adapter = ArrayAdapter.createFromResource(this,R.array.difficultyarray, android.R.layout.simple_spinner_item);
    adapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
    spinner.setAdapter(adapter);
    spinner.setOnItemSelectedListener(this);

    pdifficulty = spinner.getSelectedItem().toString();
}

@Override
public boolean onCreateOptionsMenu(Menu menu) {
    // Inflate the menu; this adds items to the action bar if it is present.
    getMenuInflater().inflate(R.menu.activity_update_project, menu);
    return true;
}

@TargetApi(Build.VERSION_CODES.HONEYCOMB)
public void datepicker(View v) {                                                        //Date picker   
    DialogFragment newFragment = new DatePickerForUpdate();
    newFragment.show(getFragmentManager(), "datePickerforupdate");

}


    @Override
public void onDateSet(DatePicker view, int year, int month, int day) {
    // TODO Auto-generated method stub

    pday = day;
    pmonth = month +1;
    pyear = year;

    month = month + 1;
    Button activityButton = (Button)findViewById(R.id.datebutton);
    activityButton.setText (day + "." + month + "." + year);

}

@Override
public void onItemSelected(AdapterView<?> parent, View view, int pos, long id) {
    // TODO Auto-generated method stub

    UpdateProject.this.mPos = pos;
    UpdateProject.this.mSelection = parent.getItemAtPosition(pos).toString();

}

@Override
public void onNothingSelected(AdapterView<?> arg0) {
    // TODO Auto-generated method stub

}

public void update_project(View view) {

EditText cinone = (EditText) findViewById(R.id.pname);
pname = cinone.getText().toString();

String fdate = pday + ". " + pmonth + ". " + pyear;

projectdatabase update = new projectdatabase(UpdateProject.this);
update.open();
update.updateProject(id, pname, fdate);
update.close();


}

}

รหัสสำหรับคลาสฐานข้อมูล:

public class projectdatabase {

public static final String KEY_ROWID = "_id";
public static final String PROJECT_NAME = "project_name";
public static final String PROJECT_ID = "project_id";
public static final String PROJECT_DIFFICULTY = "project_difficulty";
public static final String PROJECT_STATUS = "project_status";
public static final String PROJECT_START_DATE = "project_start_date";
public static final String PROJECT_FINISH_DATE = "project_finsish_date";

private static final String DATABASE_NAME = "project_db";
static final String DATABASE_TABLE = "project_details";
private static final int DATABASE_VERSION = 1;

private DbHelper ourHelper;
private final Context projectdbContext;
private SQLiteDatabase projectDatabase;

private static class DbHelper extends SQLiteOpenHelper {
    public DbHelper(Context context) {

        super(context, DATABASE_NAME, null, DATABASE_VERSION);

    }

    @Override
    public void onCreate(SQLiteDatabase db) {
        // TODO Auto-generated method stub

        db.execSQL("CREATE TABLE "+ DATABASE_TABLE + " (" +
                KEY_ROWID + " INTEGER PRIMARY KEY AUTOINCREMENT, " +
                PROJECT_NAME + " TEXT NOT NULL, " +
                PROJECT_ID + " INTEGER, " +
                PROJECT_START_DATE + " TEXT, " +
                PROJECT_FINISH_DATE + " TEXT, " +
                PROJECT_DIFFICULTY + " TEXT, " +
                PROJECT_STATUS + " TEXT);"
               ); 

    }

    @Override
    public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
        // TODO Auto-generated method stub

        db.execSQL("DROP TABLE IF EXIST " + DATABASE_NAME);
        onCreate(db);

    }
}

public projectdatabase (Context c) {                            
     projectdbContext = c;
}

public projectdatabase open() {                                                 //Open database
    ourHelper = new DbHelper(projectdbContext);
    projectDatabase = ourHelper.getWritableDatabase();
    return this;
}

public void close() {                                                           //Close database
    ourHelper.close();
}

public long createEntry(String name, String pid, String startdate, String finishdate, String difficulty, String Status) {           //Enter project data into database

    ContentValues cv = new ContentValues();
    cv.put(PROJECT_NAME, name);
    cv.put(PROJECT_ID, pid);
    cv.put(PROJECT_START_DATE, startdate);
    cv.put(PROJECT_FINISH_DATE, finishdate);
    cv.put(PROJECT_DIFFICULTY, difficulty);
    cv.put(PROJECT_STATUS, Status);

    return projectDatabase.insert(DATABASE_TABLE, null, cv);
}

public String getData() {                                                       //Retrieve all project data

    String[] columns = new String[] {KEY_ROWID, PROJECT_NAME, PROJECT_ID, PROJECT_START_DATE, PROJECT_FINISH_DATE, PROJECT_DIFFICULTY, PROJECT_STATUS};
    Cursor c = projectDatabase.query(DATABASE_TABLE, columns, null, null, null, null, null);
    String result = "";

    int iRow = c.getColumnIndex(KEY_ROWID);
    int iName = c.getColumnIndex(PROJECT_NAME);
    int iID = c.getColumnIndex(PROJECT_ID);
    int iStartDate = c.getColumnIndex(PROJECT_START_DATE);
    int iFinishDate = c.getColumnIndex(PROJECT_FINISH_DATE);
    int iDifficulty = c.getColumnIndex(PROJECT_DIFFICULTY);
    int iStatus = c.getColumnIndex(PROJECT_STATUS);

    for(c.moveToFirst(); !c.isAfterLast(); c.moveToNext()) {

        result = result + c.getString(iRow) + "|" + c.getString(iName) + "|" + c.getString(iID) + "|" + c.getString(iStartDate) + "|" + c.getString(iFinishDate) + "|" + c.getString(iDifficulty) + "|" + c.getString(iStatus) + "\n";

    }

    return result;
}

public Cursor getDataforDisplay () {                                            //Project data for list view

    String[] columns = new String[] {KEY_ROWID, PROJECT_NAME, PROJECT_FINISH_DATE, PROJECT_DIFFICULTY, PROJECT_STATUS};
    Cursor c = projectDatabase.query(DATABASE_TABLE, columns, null, null, null, null, null);

    c.moveToFirst();
    return c;

}

public String getProjectName(String id) {                   

    String[] columns = new String[] {PROJECT_NAME,KEY_ROWID};
    Cursor c = projectDatabase.query(DATABASE_TABLE, columns, KEY_ROWID + "=" + id, null, null, null, null);

    if(c != null) {

        int iName = c.getColumnIndex(PROJECT_NAME);

        c.moveToFirst();
        String name = c.getString(iName);
        return name;

    }

    return null;
}

สตริงสาธารณะ getProjectStatus (รหัสสตริง) {

    String[] columns = new String[] {PROJECT_STATUS,KEY_ROWID};
    Cursor c = projectDatabase.query(DATABASE_TABLE, columns, KEY_ROWID + "=" + id, null, null, null, null);

    if(c != null) {

        int iStatus = c.getColumnIndex(PROJECT_STATUS);

        c.moveToFirst();
        String status = c.getString(iStatus);
        return status;

    }

    return null;
}

สตริงสาธารณะ getProjectDifficulty (รหัสสตริง) {

String[] columns = new String[] {PROJECT_DIFFICULTY,KEY_ROWID};
Cursor c = projectDatabase.query(DATABASE_TABLE, columns, KEY_ROWID + "=" + id, null, null, null, null);

if(c != null) {

    int iDiff = c.getColumnIndex(PROJECT_DIFFICULTY);

    c.moveToFirst();
    String diff = c.getString(iDiff);
    return diff;

}

return null;

}

สตริงสาธารณะ getProjectstartdate (รหัสสตริง) {

String[] columns = new String[] {PROJECT_START_DATE,KEY_ROWID};
Cursor c = projectDatabase.query(DATABASE_TABLE, columns, KEY_ROWID + "=" + id, null, null, null, null);

if(c != null) {

    int iSdate = c.getColumnIndex(PROJECT_START_DATE);

    c.moveToFirst();
    String sdate = c.getString(iSdate);
    return sdate;

}

return null;

}

สตริงสาธารณะ getProjectfinishdate (รหัสสตริง) {

String[] columns = new String[] {PROJECT_FINISH_DATE,KEY_ROWID};
Cursor c = projectDatabase.query(DATABASE_TABLE, columns, KEY_ROWID + "=" + id, null, null, null, null);

if(c != null) {

    int iFdate = c.getColumnIndex(PROJECT_FINISH_DATE);

    c.moveToFirst();
    String fdate = c.getString(iFdate);
    return fdate;

}

return null;

}

โมฆะสาธารณะ updateProject (รหัสสตริง, สตริง PName, สตริง FDate) { // วิธีการอัปเดตฐานข้อมูล

ContentValues cvUpdate = new ContentValues();
cvUpdate.put(PROJECT_NAME, PName);
cvUpdate.put(PROJECT_FINISH_DATE, FDate);

projectDatabase.update(DATABASE_TABLE, cvUpdate, KEY_ROWID + "=" + id, null); 

}

}

แก้ไข: ถ้าฉันแทนที่เมธอด updateProject() ด้วยวิธีอื่นจากคลาสฐานข้อมูล ข้อผิดพลาดจะหายไปและแอปจะทำงานได้โดยไม่มีปัญหา


person krtkush    schedule 31.12.2012    source แหล่งที่มา
comment
รหัสการทำงานอันมีค่าของคุณมีความแตกต่างกันอย่างไร?   -  person Robert Harvey    schedule 31.12.2012
comment
อันนั้นไม่มีส่วนของตัวเลือกวันที่ที่ใช้งานได้ แค่นั้นแหละ. รหัสที่เหลือก็เหมือนกัน ฉันลบอันนั้นแล้วเขียนใหม่เป็นกิจกรรมใหม่ และตอนนี้ฉันได้รับปัญหานี้แล้ว   -  person krtkush    schedule 31.12.2012
comment
นี่เป็นที่เดียวที่คุณเปิด db หรือไม่   -  person Geobits    schedule 31.12.2012
comment
ฉันพนันได้เลยว่าถ้าคุณวางโค้ดทั้งสองไว้เคียงข้างกันแล้วดูทีละบรรทัด คุณจะเห็นความแตกต่างอื่นๆ   -  person Robert Harvey    schedule 31.12.2012
comment
@Geobits ในกิจกรรมนี้ ใช่แล้ว! @ RobertHarvey นั่นคือปัญหา ฉันไม่มีรหัสนั้นอีกต่อไป!   -  person krtkush    schedule 31.12.2012
comment
อา. ถ้าอย่างนั้นคุณกำลังพูดถึงโค้ดล้ำค่าที่มีอยู่แล้วซึ่งไม่มีอยู่จริง   -  person Robert Harvey    schedule 31.12.2012
comment
ตรวจสอบว่า เคอร์เซอร์ของคุณทั้งหมด ปิดอยู่เช่นกัน   -  person varevarao    schedule 31.12.2012
comment
@varevarao ฉันจะทำอย่างไร?   -  person krtkush    schedule 31.12.2012
comment
ฉันสมมติว่าคุณเรียกใช้แบบสอบถามบน Db ในเมธอด updateProject() ของคุณ แบบสอบถามนี้จะส่งคืนเคอร์เซอร์ที่ต้องปิดด้วย สำหรับเคอร์เซอร์แต่ละตัว เมื่อคุณได้ข้อมูลทั้งหมดที่คุณต้องการเสร็จแล้ว ให้เรียก cursorName.close() ไว้ นี่คือข้อมูลอ้างอิงที่เว็บไซต์ dev   -  person varevarao    schedule 31.12.2012
comment
@varevarao ฉันได้เพิ่มคำจำกัดความของวิธีการในคำอธิบายแล้ว ฉันควรเพิ่ม dbupdate.close() ต่อท้ายคำจำกัดความหรือไม่?   -  person krtkush    schedule 31.12.2012
comment
คุณสามารถเพิ่มรหัสคลาสของคุณสำหรับคลาส ProjectDatabase ได้ไหม นั่นจะช่วยให้เข้าใจปัญหาที่นี่ได้จริงๆ   -  person varevarao    schedule 31.12.2012
comment
@varevarao ฉันได้เพิ่มฐานข้อมูลและกิจกรรมแล้ว โปรดดู!   -  person krtkush    schedule 31.12.2012


คำตอบ (2)


คุณต้องออกแบบคลาส SQLiteOpenHelper ของคุณอย่างถูกต้อง ผู้เชี่ยวชาญหลายคนแนะนำให้สร้างการอ้างอิงแบบคงที่เพื่อให้แน่ใจว่ามีอินสแตนซ์ DatabaseHelper เพียงอินสแตนซ์เดียวเท่านั้นได้ตลอดเวลา

โค้ดด้านล่างนี้จะให้แนวคิดเกี่ยวกับเรื่องนี้แก่คุณ (นี่ไม่ใช่รหัสที่สมบูรณ์ แต่จะให้คำแนะนำแก่คุณ)

public class DatabaseHelper extends SQLiteOpenHelper { 
    private static DatabaseHelper mDBHelper;

    public static DatabaseHelper getInstance(Context ctx) {

        if (mDBHelper == null) { //this will ensure no multiple instances out there.
            mDBHelper = new DatabaseHelper(ctx.getApplicationContext());
        }
        return mDBHelper;
    }

}

ตอนนี้ทำเคล็ดลับ

เปิดฐานข้อมูล

private SQLiteDatabase db;
    public synchronized SQLiteDatabase open() throws SQLException {

        if(db ==null){
            db = DBHelper.getWritableDatabase();
        }

        return db;
    }

ปิดฐานข้อมูล

public synchronized void close() {

        //do nothing. This is a trick.
    }

เหตุผล : สำหรับแอปพลิเคชันบนมือถือ ไม่จำเป็นต้องเปิดและปิดกลไก เหมาะที่สุดสำหรับเว็บแอปพลิเคชันที่มีผู้ใช้หลายคนพยายามเข้าถึง db แต่ในกรณีของแอปพลิเคชันมือถือ จะมีผู้ใช้เพียงคนเดียวเท่านั้นที่จะใช้งานแอปพลิเคชันของคุณ

แฮ็คนี้ยังช่วยให้แน่ใจว่าคุณจะไม่มีวันได้รับข้อผิดพลาด Close() was never explicitly called on database อีกด้วย ข้อยกเว้นนี้จะเกิดขึ้นเมื่อคุณเปิดอินสแตนซ์ SQLiteDatabase มากกว่าที่คุณปิด และตอนนี้ในกรณีของเรา จะมีอินสแตนซ์ SQLiteDatabse เพียงอินสแตนซ์เดียวเท่านั้น วิธีนี้จะจัดการกับปัญหาการเข้าถึง/เขียน db ที่เกิดขึ้นพร้อมกันด้วย

หวังว่าแฮ็คนี้จะช่วยให้คุณเข้าใจปัญหาที่แท้จริง

เพื่อความเข้าใจที่ดีขึ้น คุณสามารถอ้างอิงลิงก์นี้ .

person Moin Ahmed    schedule 01.01.2013

public projectdatabase open() {
    ourHelper = new DbHelper(projectdbContext);
    projectDatabase = ourHelper.getWritableDatabase();
    return this;
}

public void close() {
    ourHelper.close();
}

ด้านบนนี้ ให้ลองเปลี่ยน ourHelper.close() เป็น projectDatabase.close() คุณต้องปิดอินสแตนซ์ของ WriteableDatabase ของ SQLiteDatabase ที่คุณเปิด ไม่ใช่ DBHelper

ดังนั้นฟังก์ชันปิดของคุณจะกลายเป็น:

public void close() {
    projectDatabase.close();
}

[แก้ไข]: จากเว็บไซต์ sqlite ฉันพบข้อมูลนี้ เหตุผลที่ใช้งานได้เมื่อคุณแทนที่ updateProject() ด้วยวิธีอื่นใด เนื่องจากนั่นเป็นวิธีเดียวที่คุณได้รับล็อกการเขียนบนฐานข้อมูล ฐานข้อมูล SQLite ไม่มีการล็อคการอ่าน มีเพียงการล็อคการเขียนเท่านั้น

คุณได้ลองเรียกปิดบน projectDatabase ที่ส่วนท้ายของเมธอด updateProject() ของคุณ (ข้างใน) แล้วหรือยัง

คำแนะนำอีกอย่างหนึ่งที่ฉันจะทำคือทำให้คลาสฐานข้อมูลของคุณ ซิงเกิลตัน วิธีนี้จะหลีกเลี่ยงข้อขัดแย้งใดๆ เนื่องจากมีเพียงวัตถุเดียวที่เข้าถึงฐานข้อมูลในเวลาใดก็ตาม

person varevarao    schedule 31.12.2012
comment
ฉันดำเนินการนี้ ข้อผิดพลาดหายไปแล้ว! ขอบคุณ! แต่ตอนนี้ฐานข้อมูลของฉันไม่ได้รับการอัพเดต - person krtkush; 31.12.2012