Идиома, чтобы закрыть курсор

Question

Какое из следующих двух я должен использовать, чтобы убедиться, что все курсоры закрыты?

Cursor c = getCursor(); 

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

    OR 

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

    }finally{ 
     c.close(); 
    } 

EDIT:
Несколько вопросов:
1. Нужно ли нам называть близко() на курсор, который имеет счетчик 0?
2. Поскольку в этом случае для первой идиомы, close() никогда не будет вызван. Он предполагает, что для курсора, не имеющего элементов, курсор никогда не будет открыт. Действительно ли это предположение?

Просьба сообщить.

Answer 1

Ни то, ни другое было ближе всего.

• Вариант 1 не правильно закрыть Курсор при GetCount() == 0
• Вариант 2 оставляет наконец, блок подвергается исключения нулевого указателя

Я хотел бы использовать:

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() выдает исключение.

Кроме того, некоторые отступы не помешает :)

Answer 3

Я бы сказал, что первый из них, главным образом потому, что вторая одна будет пытаться вызвать c.close() даже если c является null. Кроме того, согласно документам, getCount() не выбрасывает никаких исключений, поэтому нет необходимости включать его в блок try.

Answer 4

Лучшая практика является один ниже:

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 5

Это даже лучше:

• не использует c.getCount() - подсчет может потребоваться дополнительная работа для базы данных и не необходимо
• инициализировать курсор перед блоком запроса, поэтому не удалось создать запрос на завершение блока

Код:

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 может быть пустым в случае ошибки или пустой курсор. См. https://stackoverflow.com/a/16108435/952135. Тем не менее, я бы сообщал значение null возвращаемого значения в случае пустого курсора.

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