2016-07-30 6 views
1
public void addFavourite(User user,Winkel winkel){ 
    ContentValues values = new ContentValues(); 
    values.put(COLUMN_NAME,winkel.getNaam()); 
    values.put(COLUMN_POSTCODE, winkel.getPostcode()); 
    values.put(COLUMN_DEELGEMEENTE, winkel.getDeelGemeente()); 
    values.put(COLUMN_GEMEENTE, winkel.getGemeente()); 
    values.put(COLUMN_ADRES, winkel.getAdres()); 
    Log.d("lol",user.getUserName()); // Log is succesfull 
    values.put(COLUMN_USERNAME, user.getUserName()); 
    SQLiteDatabase db = this.getWritableDatabase(); 
    db.insert(TABLE_FAVOURITES, null, values); 
    db.close(); 

} 

public void deleteFavourite(User user,Winkel winkel){ 

    String selectQuery = "DELETE FROM " + TABLE_FAVOURITES + " WHERE " + COLUMN_USERNAME+"=" + user.getUserName() + " AND " + COLUMN_NAME + "=" + winkel.getNaam(); 
    SQLiteDatabase db = this.getWritableDatabase(); 
    db.execSQL(selectQuery); 
    db.close(); 
} 


    String CREATE_FAVOURITES_TABLE = "CREATE TABLE IF NOT EXISTS " + 
      TABLE_FAVOURITES + "(" 
      + COLUMN_ID2 + " INTEGER PRIMARY KEY," 
      + COLUMN_NAME + " TEXT," 
      + COLUMN_POSTCODE + " INTEGER," 
      + COLUMN_DEELGEMEENTE + " TEXT," 
      + COLUMN_GEMEENTE + " TEXT," 
      + COLUMN_ADRES + " TEXT," 
      + COLUMN_USERNAME + " TEXT, " 
      + "FOREIGN KEY(" + COLUMN_USERNAME+ ") REFERENCES " 
      + TABLE_USERS + "(username) " + ")"; 
    db.execSQL(CREATE_FAVOURITES_TABLE); 

Sql Ausnahme bei dem Versuch, Favorit zu löschen:SQLlite - keine solche Spalte in einer DELETE-Befehl

android.database.sqlite.SQLiteException: no such column: Gebruikersnaam (code 1): , while compiling: DELETE FROM favorieten WHERE username=Gebruikersnaam AND naam=EUROCITY 

Hinweis: so gehe ich davon aus, entweder etwas falsch ist, wenn der Favorit hinzufügen, und der Benutzername tatsächlich wird nicht zur Tabelle hinzugefügt, oder es ist etwas falsch in meiner create table-Anweisung, oder es ist etwas falsch in meiner deletequery. Ich kann nicht herausfinden, was es ist. Warum funktioniert das nicht?

+1

Sie benötigen Strings in einfachen Anführungszeichen. Ie .: 'WHERE username = 'Gebruikersnaam' UND naam = 'EUROCITY'' –

+2

Sie haben die Anführungszeichen Ihrer Werte vergessen. Aber eigentlich sollten Sie Prepared Statements verwenden –

+0

@juergend Ja, gebundene Parameter funktionieren besser. Sie kümmern sich auch um die Verdoppelung der einzelnen Anführungszeichen im Falle einer Zeichenkette, die einen oder mehrere Apostrophe enthält (''=>'' '') –

Antwort

1

Sie müssen Ihre Strings in einfache Anführungszeichen setzen.
IE.E .: WHERE username = 'Gebruikersnaam' UND naam = 'EUROCITY'

Gebundene Parameter funktionieren besser.
Sie kümmern sich auch um die Verdoppelung der einfachen Anführungszeichen im Falle einer Zeichenkette, die einen oder mehrere Apostrophe enthält ("'" =>"''").
Sie müssen sie also nicht manuell ersetzen.

In Ihrem Fall ist es so etwas wie diese:

// Put your parameters in this array (no matter if they are numeric types 
// because the SQL statement is a string anyway), 
// in the exact order of appearance in your query or SQL command 
String[] strParams = new String[]{user.getUserName(), winkel.getNaam()}; 
// Your modified SQL command with the parameter placeholders (?) 
String selectQuery = "DELETE FROM " + TABLE_FAVOURITES + 
    " WHERE " + COLUMN_USERNAME + " = ? AND " + COLUMN_NAME + " = ?"; 
SQLiteDatabase db = this.getWritableDatabase(); 
// Use this execSQL() syntax 
db.execSQL(selectQuery, strParams); 
db.close(); 
Verwandte Themen