2017-08-07 58 views
0

我正在尝试通过AsyncTask中的联系人簿中的号码查找联系人姓名。有时我会遇到以下错误,我认为这会导致应用程序崩溃。请任何帮助。应用程序未关闭游标或数据库对象android

logcat的

08-05 09:00:49.627 24988-27371/com.fake.fake E/CursorLeakDetecter: PossibleCursorLeak:content://com.android.contacts/phone_lookup/%2B923310433708,QueryCounter:5 
android.database.sqlite.DatabaseObjectNotClosedException: Application did not close the cursor or database object that was opened here 
    at android.content.ContentResolver.query(ContentResolver.java:323) 
    at com.fake.fake.GV.getContactName(GV.java:177) 
    at com.fake.fake.MySQLiteHelper.GetNumbers(MySQLiteHelper.java:361) 
    at com.fake.fake.MainActivity$taskLoadContacts.doInBackground(MainActivity.java:930) 
    at com.fake.fake.MainActivity$taskLoadContacts.doInBackground(MainActivity.java:921) 
    at android.os.AsyncTask$2.call(AsyncTask.java:264) 
    at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:305) 
    at java.util.concurrent.FutureTask.run(FutureTask.java:137) 
    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1076) 
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:569) 
    at java.lang.Thread.run(Thread.java:856) 

从数据库中检索联系人号码。

public ArrayList<MyMessageDetails> GetNumbers(String query) { 
     ArrayList<MyMessageDetails> numbers = new ArrayList<MyMessageDetails>(); 
     Cursor cursor = db.rawQuery(query, null); 
     if (cursor != null && cursor.moveToFirst()) { 
      do { 
       MyMessageDetails item = new MyMessageDetails(); 
       item.setNumber(cursor.getString(0)); 
       String name = GV.getContactName(item.getNumber());  
       if (name == null) { 
        name = item.getNumber(); 
       } 
       item.setName(name); 
       item.setTime(cursor.getString(3)); 
       numbers.add(item); 
      } while (cursor.moveToNext()); 

     } 
     return numbers; 
    } 

检索联系人姓名。

public static String getContactName(String phoneNumber) { 
     if (phoneNumber != null) { 
      ContentResolver cr = ApplicationContextProvider.getContext().getContentResolver(); 
      Uri uri = Uri.withAppendedPath(ContactsContract.PhoneLookup.CONTENT_FILTER_URI, Uri.encode(phoneNumber)); 
      Cursor cursor = cr.query(uri, new String[]{ContactsContract.PhoneLookup.DISPLAY_NAME}, null, null, null); 
      if (cursor == null) { 
       return ""; 
      } 
      String contactName = null; 
      if (cursor.moveToFirst()) { 
       contactName = cursor.getString(cursor.getColumnIndex(ContactsContract.PhoneLookup.DISPLAY_NAME)); 
      } 

      if (cursor != null && !cursor.isClosed()) { 
       cursor.close(); 
      } 
      return contactName; 
     } 

     return phoneNumber; 
    } 
+0

请参阅:https://stackoverflow.com/questions/ 4991179/application-did-not-close-the-cursor-or-database-object-that-opened-here-rq = 1 –

+0

感谢您的链接,但错误来自“getContactName”而不是来自GetNumbers。 – user934820

回答

0

所以我在这里看到的几个问题。

首先,请始终围绕try catch finally块中的查询执行环节,并关闭finally块中的所有游标和数据库操作。所以这两个你的方法。

我认为你在做什么错在这里首先调用GetNumbers()方法,然后getContactName()。现在你不要在代码中关闭GetNumbers()中的db,因此在那之后你尝试在db上做的任何操作都会给出这个错误,因此出现错误getContactName()

+0

谢谢你的“试试最后”的建议。我想让我的数据库保持打开状态,这就是为什么它没有关闭。我是否也应该在finally块中关闭游标? – user934820

+1

是的,你应该在finally块中做到这一点。最好关闭数据库并在需要时再次打开。或者您可以在该方法中执行与该部分数据相关的所有数据库操作并关闭它。 –

2

使用Cursor#close()完成读取数据后。例如:

Cursor cursor = ... 
try { 
    // fetch whatever you need here 
} finally { 
    cursor.close(); 
} 
+0

谢谢让我检查。 – user934820

相关问题