Ошибка в этой строке
int OldP = Integer.parseInt(OldPoints);
Ошибка говорит о том, что значение OldPoints не является строкой, которая может быть преобразована в целое число, например, если это A или null;
Таким образом, либо значение, извлеченное из столбца POINTS, не является числовым, либо значение имени пользователя не соответствует столбцу NICKNAME в строке.В этом случае OldPoints будут иметь любое значение, которое было установлено до цикла.
Поскольку сами данные недоступны, вам необходимо выяснить, какая из двух ситуаций вызывает проблему.
Я бы предложил добавить несколько Входов в систему, чтобы определить, какие именно.
, например, используя что-то вроде: -
OldPoints = "my debugging value";
Log.d("MYDEBUGGING","OldPoints, before doing anything is " + OldPoints);
Cursor c_oldPoints= sqdb.query(DBHelper.TABLE_NAME2,null,DBHelper.NICKNAME+"=?",new String[]{Username},null,null,null);
int col_Points=c_oldPoints.getColumnIndex(DBHelper.POINTS);
c_oldPoints.moveToFirst();
while (!c_oldPoints.isAfterLast())
{
OldPoints=c_oldPoints.getString(col_Points);
c_oldPoints.moveToNext();
Log.d("MYDEBUGGING","Extracted the value " + OldPoints + " from position + String.valueOf(c_oldPoints.getPosition());
}
sqdb.close();
Log.d("MYDEBUGGING","Trying to convert the value " + OldPoints + " to an integer");
int OldP = Integer.parseInt(OldPoints);
Вы также не могли бы внести вышеуказанные изменения и добавить точку останова (на строкепервоначально указано), а затем используйте Run / Debug App и проверьте переменные (или используйте несколько точек останова в подходящих местах).Это может оказаться полезным при отладке Отладка вашего приложения .
Следующий код защищает от исключений, а также от попыток обновления несуществующего пользователя: -
if (count == 4) {
SQLiteDatabase sqdb = my_db.getWritableDatabase();
Cursor c_oldPoints= sqdb.query(
DBHelper.TABLE_NAME2,null,
DBHelper.NICKNAME+"=?",
new String[]{Username},
null,null,null
);
int col_Points=c_oldPoints.getColumnIndex(DBHelper.POINTS);
if (c_oldPoints.moveToFirst()) {
Oldpoints = c_oldPoints.getString(col_Points);
//Oldpoints = "oops";
int OldP = 0;
boolean can_convert_to_int = true;
try {
OldP = Integer.parseInt(Oldpoints) + countPoints;
can_convert_to_int = true;
} catch (NumberFormatException e) {
e.printStackTrace(); //TODO not necessary probably remove. just for checking the log
}
if (can_convert_to_int) {
ContentValues cv = new ContentValues();
cv.put(DBHelper.POINTS,OldP);
sqdb.update(DBHelper.TABLE_NAME2,cv, DBHelper.NICKNAME + "=?", new String[]{Username});
}
} else {
Log.d("NICKNAMENOTFOUND","No row was found when attemtping to get the old score for User " + Username);
}
}
Однако
Я бы предложил добавить пару методов в ваш класс DBHelper: *
public int increasePoints(String user, int points_to_add) {
SQLiteDatabase db = this.getWritableDatabase();
SQLiteStatement sql = db.compileStatement(
"UPDATE " + TABLE_NAME2 +
" SET " + POINTS + "=" + POINTS + " +? " +
"WHERE "+ NICKNAME + "=?"
);
sql.bindLong(1,points_to_add);
sql.bindString(2,user);
return sql.executeUpdateDelete();
}
public int getPoints(String user) {
SQLiteDatabase db = this.getWritableDatabase();
int rv = -1;
String whereclause = NICKNAME + "=?";
String[] whereargs = new String[]{user};
Cursor csr = db.query(TABLE_NAME2,new String[]{POINTS},whereclause,whereargs,null,null,null);
if (csr.moveToFirst()) {
rv = csr.getInt(csr.getColumnIndex(POINTS));
}
csr.close();
return rv;
}
Первый метод увеличенияPoints выполняет изменение точек с помощью оператора UPDATE sql и устраняет необходимость преобразования точек, извлеченных в виде строки, в целое число.Он возвращает количество строк, которые были обновлены (1, если столбец NICKNAME всегда является уникальным значением, 0, если ничего не было обновлено).
Второй метод getPoints делает так, как он говорит,он получает баллы для данного пользователя, если пользователь не существует, он вернет -1.
Ваш код может быть таким: -
if (count == 4) {
boolean updated = false; //TODO remove when happy
int old_points = my_db.getPoints(Username); //TODO remove when happy
if (my_db.increasePoints(Username,countPoints) > 0) {
updated = true;
}
int new_points = my_db.getPoints(Username); //TODO remove when happy
//TODO remove following code when happy
String result = "The result of the attempt to update the points for user " + Username;
if (updated) {
result = result + " was successful. ";
} else {
result = result + " was unsuccessful.";
}
Log.d("POINTSINCREASE",result +
" Points were " + String.valueOf(old_points) + " points are now " + String.valueOf(new_points));
}
Примечание где //TODO remove when happy закодировано, строки предназначены только для тестирования, поэтому выше может быть: -
if (count == 4) {
my_db.increasePoints(Username,countPoints);
}