SQLite запрос неправильный, onItemClick вызывает сбой приложения - PullRequest
Новая
       21

SQLite запрос неправильный, onItemClick вызывает сбой приложения

0 голосов
/ 07 мая 2019

У меня есть представление списка, которое я заполняю именами, введенными в базу данных через SQLite.Когда я нажимаю на одно из этих имен, приложение вылетает.

Пожалуйста, если вы можете найти проблему, я был бы благодарен.Спасибо!

Я добавил часть logcat, которая появляется, когда я нажимаю на имя из списка.COL0 = "ID" и COL1 = "Имя".

Метод OnClick: 

listView.setOnItemClickListener(new AdapterView.OnItemClickListener() {
            @Override
            public void onItemClick(AdapterView<?> adapterView, View view, int i, long l) {
                String name = adapterView.getItemAtPosition(i).toString();
                Log.e(TAG, name);
                Log.e(TAG, "" + i);
                Cursor player = dbHelper.getItemID(name);  //this is line 51 -> see logcat
                int itemID = -1;
                while(player.moveToNext())
                    itemID = player.getInt(0);
                if(itemID > -1) {
                    Intent intent = new Intent(PlayersList.this, PlayerCharacters.class);
                    intent.putExtra("id", itemID);
                    intent.putExtra("name", name);
                    startActivity(intent);
                } else Toast.makeText(PlayersList.this, "No ID associated with that name.", Toast.LENGTH_SHORT).show();
            }
        });
    }

Метод getItemID () в классе базы данных: 

public Cursor getItemID(String name) {
        SQLiteDatabase db = this.getWritableDatabase();
        String query = "SELECT " + COL0 + " FROM " + TABLE_NAME + " WHERE " + COL1 + " = " + name;
        Cursor data = db.rawQuery(query, null);  //this is line 62 -> see logcat
        return data;
    }

Ошибкажурнал, который я получаю при нажатии на элемент (имя из списка): 

D/AbsListView: onTouchUp() mTouchMode : 0
E/PlayersListActivity: Eu
    3
E/SQLiteLog: (1) no such column: Eu
D/AndroidRuntime: Shutting down VM
E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.example.characters, PID: 30728
    android.database.sqlite.SQLiteException: no such column: Eu (code 1): , while compiling: SELECT ID FROM PlayersTable WHERE Name = Eu
    #################################################################
    Error Code : 1 (SQLITE_ERROR)
    Caused By : SQL(query) error or missing database.
        (no such column: Eu (code 1): , while compiling: SELECT ID FROM PlayersTable WHERE Name = Eu)
    #################################################################
        at android.database.sqlite.SQLiteConnection.nativePrepareStatement(Native Method)
        at android.database.sqlite.SQLiteConnection.acquirePreparedStatement(SQLiteConnection.java:1096)
        at android.database.sqlite.SQLiteConnection.prepare(SQLiteConnection.java:661)
        at android.database.sqlite.SQLiteSession.prepare(SQLiteSession.java:588)
        at android.database.sqlite.SQLiteProgram.<init>(SQLiteProgram.java:59)
        at android.database.sqlite.SQLiteQuery.<init>(SQLiteQuery.java:37)
        at android.database.sqlite.SQLiteDirectCursorDriver.query(SQLiteDirectCursorDriver.java:44)
        at android.database.sqlite.SQLiteDatabase.rawQueryWithFactory(SQLiteDatabase.java:1746)
        at android.database.sqlite.SQLiteDatabase.rawQuery(SQLiteDatabase.java:1685)
        at com.example.characters.DatabaseHelper.getItemID(DatabaseHelper.java:62)  //---->here
        at com.example.characters.PlayersList$1.onItemClick(PlayersList.java:51)   //----->here
        at android.widget.AdapterView.performItemClick(AdapterView.java:350)
        at android.widget.AbsListView.performItemClick(AbsListView.java:1683)
        at android.widget.AbsListView$PerformClick.run(AbsListView.java:4094)
        at android.widget.AbsListView$10.run(AbsListView.java:6583)
        at android.os.Handler.handleCallback(Handler.java:789)
        at android.os.Handler.dispatchMessage(Handler.java:98)
        at android.os.Looper.loop(Looper.java:164)
        at android.app.ActivityThread.main(ActivityThread.java:6944)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.Zygote$MethodAndArgsCaller.run(Zygote.java:327)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1374)
Application terminated.

1 Ответ

0 голосов
/ 07 мая 2019

Строковое значение name должно быть заключено в одинарные кавычки: 

String query = "SELECT " + COL0 + " FROM " + TABLE_NAME + " WHERE " + COL1 + " = '" + name + "'";

Без этих одинарных кавычек строковое значение name считается именем столбца, и, поскольку оно не существует, такой столбец внутриВ таблице приложение вылетает с ошибкой вроде: 

no such column: Eu

Но безопасный и рекомендуемый способ передачи параметров в запрос таков: 

String query = "SELECT " + COL0 + " FROM " + TABLE_NAME + " WHERE " + COL1 + " = ?";
Cursor data = db.rawQuery(query, new String[] {name});

Таким образом, у вас нетбеспокоиться о одинарных кавычках, так как это будет обрабатываться методом rawQuery().

...