這是不好的數據庫編碼？

Question

在我的擴展SQLiteOpenHelper的DatabaseHelper類中，我設置了各種方法將Cursors返回到其他Activities，這樣我就不會在除DatabaseHelper以外的任何其他類中執行任何查詢。在這些方法中，我不關閉遊標或數據庫之後，我回到它像：

public Cursor getCoursesLeft() 
{ 
    // Open a readable database. 
    SQLiteDatabase database = this.getReadableDatabase(); 

    // Query the database and return the cursor. 
    return database.query(DEGREE_PLAN_TABLE, null, DEGREE_COLUMN_TAKEN + " = ?", 
      new String[] { "0" }, null, null, DEGREE_COLUMN_CLASS_NAME + " COLLATE NOCASE"); 
} 

從任何活動我打電話從方法，我不保證關閉後，我用它那所返回的Cursor。

由於光標是一個對象，它應該通過引用傳遞，是否正確？因此，從其他Activity關閉它應關閉原始對象，並且如果我正確理解它，關閉Cursor也會關閉數據庫。

這是一個不好的編碼習慣？

它看起來像隨機我會得到一個LogCat錯誤，說關閉從未在數據庫上調用，我可以在我的代碼中找到唯一可能的原因是我如何返回這些方法中的遊標。

Answer 1

如果我正確地理解它，關閉遊標也會關閉 數據庫。

這聽起來不太合適。在關閉所有遊標之後，您必須明確關閉數據庫。 logcat錯誤是由於你沒有關閉數據庫，可能試圖打開它的另一個實例。

順序很重要，首先是遊標，然後是數據庫實例。

<bad joke in 3.. 2.. 1...> 

其餘的聽起來不像任何不好的做法，當你得到分貝，你只是得到分貝它。 ：d

[編輯]：你說你這樣做：

public Cursor getCoursesLeft() 
{ 
    // Open a readable database. 
    SQLiteDatabase database = this.getReadableDatabase(); 
        ^^^ here you're creating a new instance of the db 
which means the db is opened for reading, and the scope of this variable 
is lost outside this function. This means you can not close this instance explicitly 

    // Query the database and return the cursor. 
    return database.query(DEGREE_PLAN_TABLE, null, DEGREE_COLUMN_TAKEN + " = ?", 
      new String[] { "0" }, null, null, DEGREE_COLUMN_CLASS_NAME + " COLLATE NOCASE"); 
} 

而是具有可以這種方法外部訪問並關閉它，一旦你做與工作數據庫變量光標（並且您已關閉光標）

SQLiteDatabase database; 
public Cursor getCoursesLeft() 
{ 
    // Open a readable database. 
    database = this.getReadableDatabase(); 

    // Query the database and return the cursor. 
    return database.query(DEGREE_PLAN_TABLE, null, DEGREE_COLUMN_TAKEN + " = ?", 
      new String[] { "0" }, null, null, DEGREE_COLUMN_CLASS_NAME + " COLLATE NOCASE"); 
} 
public void someOtherFunction() { 
    Cursor blah = getCoursesLeft(); 
    // do something with blah 
    blah.close(); 
    database.close(); 
} 

Answer 2

不關閉遊標只會導致內存泄漏。關閉數據庫是不同的。

關閉遊標就像關閉特定連接，以創建光標時生成的某些.file文件。

因此，您應該總是關閉您的光標。

這是不好的編碼？

不，也是。不要讓你的活動弄亂那些臨時文件。雖然沒有什麼會發生，但它似乎並不好看