public Cursor getImages(long rowId) throws SQLException
{
Cursor mCursor =
db.rawQuery(\"select * from Pictures WHERE id=\" + rowId + \";
You should do it like this:
c1.getString(cursor.getColumnIndex("pic"));
and
c1.getString(cursor.getColumnIndex("comment"));
This is the way I do it
I prefer
getColumnIndex()
instead of number.
if(cursor.moveToFirst()){
do{
String varaible1 = cursor.getString(cursor.getColumnIndex("column_name1"));
String varaible2 = cursor.getString(cursor.getColumnIndex("column_name2"));
}while (cursor.moveToNext());
}
cursor.close();
Always use column name instead of position, because column position can change.
of course column name can change as well but let say if you add a new column and it position between column 1 and column 2. You need to change your code if you use number. But if you use name, you will be fine.
and it is more readable and what happened if you have 40 columns?(<-some say, it is bad)
simply use moveToNext
in your loop to iterate.
while(ct.moveToNext()){
pictures[i]=c1.getString(1);
comments[i]=c1.getString(2);
i++;
}
Cursor c = db.rawQuery("select username from user_information where username ='" + username_txt.getText() + "'", null);
c.moveToFirst();
if (c.moveToFirst()) {
username = c.getString(c.getColumnIndex("username"));
}
Use this. I hope it helps you.
And what you should do is to replace
c1.moveToFirst();
while(c1.isLast()){
//your code
}
By
//c1.moveToFirst();
while(c1.moveToNext()){
//your code
}