what is wrong with my code? help me debug please.

moonraker928 · by **moonraker928** » Sat Jan 19, 2008 12:53 am

Pasted code below.

Syntax: [ Download ] [ Hide ]

Using java Syntax Highlighting

private static final String[] CONTENT_PROJECTION = new String[] {
 
            PhotoContent.Photos._ID, PhotoContent.Photos.CREATED_DATE };
 
// get current image IDs
 
Cursor current_images = managedQuery(getIntent().getData(), CONTENT_PROJECTION, null, null);
 
// store current image ids in another array
 
String[] cur_image_id = new String[current_images.count()];
 
int stored_image_id = current_images.getColumnIndex(PhotoContent.Photos._ID);
 
for (int i = 0; i < current_images.count(); i++, current_images.next()) {
 
        cur_image_id[i] = current_images.getString(stored_image_id);
 
}
Parsed in 0.031 seconds,  using GeSHi 1.0.8.4

I am getting a RuntimeException invoked by the Looper.class...and have done some debugging to localize the problem to the snippet of code posted above.

What am I doing wrong?

plusminus · by **plusminus** » Sat Jan 19, 2008 2:38 pm

Hello moonraker,

on which line exactly is it thrown :?:

Regards,
plusminus

moonraker928 · by **moonraker928** » Sat Jan 19, 2008 11:21 pm

the error is thrown inside the for loop.

the exact error goes like this:

android.database.CursorIndexOutOfBoundsException: Index -1 requested, with a size of 8

im thinking it has something to do with current_images.next() as that goes through the rows of the cursor? but still, i have no idea why that would go out of bounds...

any ideas?

wataru · by **wataru** » Sun Jan 20, 2008 11:50 am

Hi,

I could be wrong, but maybe try using

Syntax: [ Download ] [ Hide ]

Using java Syntax Highlighting

if(current_images.first())  // I believe this will move cursor to first row
Parsed in 0.031 seconds,  using GeSHi 1.0.8.4

before getting into your cursor iteration (=for) loop.
As a bonus, this should also take care
the case where there is no record returned as well

Hope it helps

plusminus · by **plusminus** » Sun Jan 20, 2008 1:18 pm

Hey moonraker928,

have a look at the :src: Working with the SQLite-Database - Cursors-Tutorial.

This is how I did it there and it worked well:

Syntax: [ Download ] [ Hide ]

Using java Syntax Highlighting

               if (c != null) {
                    /* Check if at least one Result was returned. */
                    if (c.first()) {
                         int i = 0;
                         /* Loop through all Results */
                         do {
                              i++;
                              // retrieve columns, etc...
 
                         } while (c.next());
                    }
               }
Parsed in 0.035 seconds,  using GeSHi 1.0.8.4

Regards,
plusminus

what is wrong with my code? help me debug please.

what is wrong with my code? help me debug please.

Maybe you need to call cursor.first()?

Who is online