Thành ngữ để đóng một con trỏ

Question

Tôi nên sử dụng điều nào sau đây để đảm bảo rằng tất cả các con trỏ đều bị đóng?

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:
Một vài câu hỏi:
1. Đỗ chúng ta cần phải gọi close() trên một con trỏ đó có tổng số là 0?
2. Bởi vì trong trường hợp đó cho thành ngữ đầu tiên, close() sẽ không bao giờ được gọi. Giả định rằng đối với con trỏ không có phần tử, con trỏ sẽ không bao giờ được mở. Đây có phải là giả định hợp lệ không?

Vui lòng thông báo.

Answer 1

Không, nhưng điểm thứ hai là gần nhất.

• Lựa chọn 1 không đúng cách đóng Cursor khi getCount() == 0
• Lựa chọn 2 rời khỏi khối finally tiếp xúc với một con trỏ ngoại lệ vô

Tôi sẽ sử dụng:

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

... hoặc nếu bạn dự kiến ​​con trỏ sẽ không có giá trị thường xuyên, bạn có thể biến nó trên đầu một chút:

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

Answer 2

Phụ thuộc vào những gì bạn đang bắt, nhưng tôi muốn nói điều thứ hai, chỉ trong trường hợp c.getCount() ném ngoại lệ.

Ngoài ra, một số thụt đầu dòng sẽ không đi không ổn :)

Answer 3

tôi muốn nói là người đầu tiên, chủ yếu là do điều thứ hai sẽ cố gắng gọi c.close() ngay cả khi c là null. Ngoài ra, theo các tài liệu, getCount() không ném bất kỳ ngoại lệ nào, do đó, không cần phải bao gồm nó trong khối try.

Answer 4

Thực hành tốt nhất là dưới đây:

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

này thậm chí còn tốt hơn:

• không sử dụng c.getCount() - đếm có thể yêu cầu thêm công việc cho các cơ sở dữ liệu và không phải là cần
• khởi chạy con trỏ trước khối truy vấn, do đó không thể tạo truy vấn không được theo sau bởi khối cuối cùng

Mã:

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

Lưu ý rằng c có thể là null trong trường hợp lỗi hoặc con trỏ rỗng. Xem https://stackoverflow.com/a/16108435/952135. Tuy nhiên, tôi sẽ báo cáo giá trị trả về null trong trường hợp con trỏ trống làm lỗi.

Answer 6

Tôi nghĩ rằng câu trả lời của tôi là tốt nhất:

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

Tôi nghĩ @ skylarsutton là một câu trả lời đúng cho câu hỏi này.Tuy nhiên, tôi muốn để lại mã cho câu hỏi (bất kỳ mã nào trong câu trả lời dường như có một số sai sót). Vui lòng xem xét sử dụng mã của tôi.

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