关闭光标的习惯用法

Question

我应该使用以下哪两个来确保所有游标都关闭？

    Cursor c = getCursor(); 

    if(c!=null && c.getCount()>0){ 
        try{ 
            // read values from cursor 
        }catch(..){} 
        finally{ 
            c.close(); 
        } 
    }//end if
    

或

    Cursor c = getCursor(); 
    try{ 
        if(c!=null && c.getCount()>0){ 
            // read values from cursor 
        }//end if 
    }catch(..){
        
    }finally{ 
        c.close(); 
    } 

请指教。

Answer 1

两者都不是，但第二个最接近。

• 选项 1 无法正确关闭
  Cursor when getCount() == 0
• 选项 2 使 finally 块暴露于空指针异常，

我会使用：

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(); 
    }
}

Answer 2

这甚至更好：

• 不使用 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。不过，如果游标为空，我会报告空返回值作为错误。

Answer 3

最佳实践如下：

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.
}

Answer 4

取决于您要捕获的内容，但我会说第二个，以防万一 c.getCount() 抛出异常。

另外，一些缩进不会出错:)

Answer 5

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

Answer 6

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

    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;
        }
    }

Answer 7

我认为@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();
    }
}