Android - 'cursor.close()' may produce 'java.lang.nullpointerexception'?

Question

I have bellow code which works just fine, but produces a warning:

method invocation 'cursor.close()' may produce 'java.lang.nullpointerexception'

code with warning

This is only a warning, so you can probably ignore it. Also, above that, you have `catch (nullPointerException)`, Which is a [bad idea](http://stackoverflow.com/questions/2586290/is-catching-a-null-pointer-exception-a-code-smell). — Jonas Czech, Apr 08 '15 at 06:35

score 1 · Accepted Answer · answered Apr 08 '15 at 06:29

1

cursor_id may not be initialized, as, e.g., your try block checks. You need to add the same validation in the finally block:

} finally { 
    if (cursor_id != null) {
        cursor_id.close();
    }
}

answered Apr 08 '15 at 06:29

Mureinik

1 Answers1