成语关闭光标

两者都没有，但第二个是最接近的。

• 当getCount（）== 0时，选项1没有正确关闭Cursor
• 选项2使finally块暴露于空指针exception

我会用：

 Cursor c = getCursor(); try { if(c!=null && c.getCount()>0){ // do stuff with the cursor } } catch(..) { //Handle ex } finally { if(c != null) { c.close(); } } 

…或者如果您希望光标经常为空，您可以稍微转过头：

 Cursor c = getCursor(); if(c != null) { try { if(c.getCount()>0) { // do stuff with the cursor } } catch(..) { //Handle ex } finally { c.close(); } } 

这甚至更好：

• 不使用c.getCount（） – 计数可能需要额外的数据库工作，不需要
• 在查询块之前初始化游标，因此创建查询失败后不会跟随finally块

代码：

 Cursor c = query(....); if (c != null) { try { while (c.moveToNext()) { // If empty or after last record it returns false. // process row... } } finally { c.close(); } } 

请注意，如果出现错误或空光标，则c可能为null。 请参阅https://stackoverflow.com/a/16108435/952135 。 但是，如果将空光标作为错误，我会报告null返回值。

最佳做法如下：

 Cursor c = null; try { c = query(....); while (c.moveToNext()) { // If empty or next to last record it returns false. // do stuff.. } } finally { if (c != null && !c.isClosed()) { // If cursor is empty even though should close it. c.close(); c = null; // high chances of quick memory release. } 

取决于你捕获的内容，但我会说第二个，以防c.getCount()抛出exception。

另外，一些缩进不会有问题:)

我会说第一个，主要是因为第二个将尝试调用c.close()即使c为null 。 另外，根据文档， getCount()不会抛出任何exception，因此不需要将它包含在try块中。

我认为我的答案是最好的答案：

  Cursor cursor = null; try { cursor = rsd.rawQuery(querySql, null); if (cursor.moveToFirst()) { do { // select your need data from database } while (cursor.moveToNext()); } } finally { if (cursor != null && !cursor.isClosed()) { cursor.close(); cursor = null; } } 

我认为@ skylarsutton是这个问题的正确答案。 但是，我想留下问题的代码（答案中的任何代码似乎都有一些缺陷）。 请考虑使用我的代码。

 Cursor c = query(....); if (c != null) { try { //You have to use moveToFirst(). There is no quarantee that a cursor is located at the beginning. for(c.moveToFirst();!c.isAfterLast();c.moveToNext()) { // process row... } } finally { c.close(); } }