Excluindo linha no SQLite no Android

102

Esta pode ser uma pergunta estúpida, mas sou novo no SQLite e não consigo descobrir isso. Eu tenho uma tabela que tem colunas KEY_ROWID, KEY_NAME, KAY_LATITUDE, e KEY_LONGITUDE. Quero que o usuário possa selecionar um e excluí-lo; Alguém pode me dar uma direção para começar? Minha pergunta é sobre a exclusão real da linha dado apenas seu nome.

Código relevante:

public class BeaconDatabase {

    public static final String KEY_ROWID = "_id";
    public static final String KEY_NAME = "beacon_name";
    public static final String KEY_LATITUDE = "beacon_lat";
    public static final String KEY_LONGITUDE = "beacon_lon";

    private static final String DATABASE_NAME ="BeaconDatabase";
    private static final String DATABASE_TABLE ="beaconTable";
    private static final int DATABASE_VERSION = 1;

    private DbHelper helper;
    private final Context context;
    private SQLiteDatabase db;

    public BeaconDatabase(Context context) {
        this.context = context;
    }

    public BeaconDatabase open() {
        helper = new DbHelper(this.context);
        db = helper.getWritableDatabase();
        return this;
    }

    public void close() {
        helper.close();
    }

    public long createEntry(String name, Double lat, Double lon) {
        ContentValues cv = new ContentValues();
        cv.put(KEY_NAME, name);
        cv.put(KEY_LATITUDE, lat);
        cv.put(KEY_LONGITUDE, lon);
        return db.insert(DATABASE_TABLE, null, cv);
    }

    public void deleteEntry(long row) {

              // Deletes a row given its rowId, but I want to be able to pass
              // in the name of the KEY_NAME and have it delete that row.
              //db.delete(DATABASE_TABLE, KEY_ROWID + "=" + row, null);
    }

    public String getData() {
        String[] columns = { KEY_ROWID, KEY_NAME, KEY_LATITUDE, KEY_LONGITUDE };
        Cursor cursor = db.query(DATABASE_TABLE, columns, null, null, null, null, null);
        String result = "";

        int iRow = cursor.getColumnIndex(KEY_ROWID);
        int iName = cursor.getColumnIndex(KEY_NAME);
        int iLat = cursor.getColumnIndex(KEY_LATITUDE);
        int iLon = cursor.getColumnIndex(KEY_LONGITUDE);

        for (cursor.moveToFirst(); !cursor.isAfterLast(); cursor.moveToNext()) {
            result += cursor.getString(iRow) + ": " + cursor.getString(iName) + " - " + cursor.getDouble(iLat) + " latitude " + cursor.getDouble(iLon) + " longitude\n";
        }

        return result;

    }

    private static class DbHelper extends SQLiteOpenHelper {

        public DbHelper(Context context) {
            super(context, DATABASE_NAME, null, DATABASE_VERSION);
        }

        @Override
        public void onCreate(SQLiteDatabase db) {
            db.execSQL("CREATE TABLE " +  DATABASE_TABLE + " (" + 
                    KEY_ROWID + " INTEGER PRIMARY KEY AUTOINCREMENT, " +
                    KEY_NAME + " TEXT NOT NULL, " +
                    KEY_LATITUDE + " DOUBLE, " +
                    KEY_LONGITUDE + " DOUBLE);"
            );
        }

        @Override
        public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
            db.execSQL("DROP TABLE IF EXISTS " + DATABASE_TABLE);
            onCreate(db);
        }
    }
}
roboguy12
fonte
vá com a resposta @iDroid ... porque vai funcionar para mim. Obrigado iDroid.

Respostas:

183

Você pode tentar assim:

 //---deletes a particular title---
public boolean deleteTitle(String name) 
{
    return db.delete(DATABASE_TABLE, KEY_NAME + "=" + name, null) > 0;
}

ou

public boolean deleteTitle(String name) 
{
    return db.delete(DATABASE_TABLE, KEY_NAME + "=?", new String[]{name}) > 0;
}
Shreyash Mahajan
fonte
61
A resposta de Vijay é a correta porque esta solução permite fazer uma injeção de SQL que é um vazamento de segurança. Por exemplo: use o valor do argumento name: name = "TRUE; <any SQL command>;"=> 'qualquer comando SQL' será executado. Claro que não é um problema se não houver GUI para esse recurso.
dia
@bdevay Minha resposta é sobre a tarefa em segundo plano que fizemos com a consulta. Portanto, não está relacionado à IU. Você apenas tem que fornecer informações tão dinamicamente para que não haja necessidade de segurança. Se você seguir a resposta do vijay e alguém fizer engenharia reversa, você poderá obter informações sobre a tabela no banco de dados e no campo que está analisando. que estou fazendo diretamente na consulta, portanto, não há chance de obtê-lo.
Shreyash Mahajan
@iDroid Explorer: claro, se não houver entrada do usuário ou outro tipo de possibilidade de manipulação de consulta externa, o risco de segurança não é maior do que com a outra solução. Continue no próximo comentário ...
bdevay
1
... Mas não concordo com a parte de engenharia reversa do seu comentário. Você tem que definir em algum lugar e de alguma forma sua consulta, o que significa que a engenharia reversa é sempre uma possível falha de segurança (mesmo no caso de sua solução), especialmente em Java, mesmo se a fonte estiver ofuscada. Isso só pode aumentar o tempo de hackeamento. Por outro lado, a recomendação do Google é usar argumentos de seleção, consulte este artigo: link
bdevay 01 de
Quero dizer com o conceito de não dar valor estático a qualquer método ou variável. Isso deve ser máximo ou dinâmico. Para que seja mais seguro do que dar valor estático. De qualquer forma, fica a critério do usuário o quanto ele deseja proteger seu aplicativo.
Shreyash Mahajan
157

Tente assim para obter a sua solução

String table = "beaconTable";
String whereClause = "_id=?";
String[] whereArgs = new String[] { String.valueOf(row) };
db.delete(table, whereClause, whereArgs);
Vijay
fonte
58

é melhor usar whereargs também;

db.delete("tablename","id=? and name=?",new String[]{"1","jack"});

é como usar este comando:

delete from tablename where id='1' and name ='jack'

e usar a função delete dessa forma é bom porque remove injeções de sql.

Enakhi
fonte
2
Você poderia elaborar mais sua resposta adicionando um pouco mais de descrição sobre a solução que você fornece?
abarisone
1
Acho que vale a pena usar o whereargs.
msysmilu
@Enkahi Isso é como instruções preparadas no SQLite? Eu usei esse id=?tipo de sintaxe com PHP antes e parece muito semelhante a isso.
GeekWithGlasses
17

Até eu entender sua pergunta, você deseja colocar duas condições para selecionar uma linha a ser excluída. Para isso, você precisa fazer:

public void deleteEntry(long row,String key_name) {

      db.delete(DATABASE_TABLE, KEY_ROWID + "=" + row + " and " + KEY_NAME + "=" + key_name, null);

      /*if you just have key_name to select a row,you can ignore passing rowid(here-row) and use:

      db.delete(DATABASE_TABLE, KEY_NAME + "=" + key_name, null);
      */  

}
Hiral Vadodaria
fonte
13

Tente este código

public void deleteRow(String value)
{
SQLiteDatabase db = this.getWritableDatabase();       
db.execSQL("DELETE FROM " + TABLE_NAME+ " WHERE "+COlUMN_NAME+"='"+value+"'");
db.close();
}
Harman Khera
fonte
Como faço para chamar isso quando quero excluir? db.deleteRow ();
Phares
podemos chamá-lo passando o valor como parâmetro que você deseja excluir. Chame como db.deleteRow ("nome");
Harman Khera
8

Experimente este código ...

private static final String mname = "'USERNAME'";
public void deleteContact()
{
    db.delete(TABLE_CONTACTS, KEY_NAME + "=" + mname, null);
}
Giridharan
fonte
3

se você estiver usando SQLiteDatabase, então há um método delete

Definição de Excluir

int delete (String table, String whereClause, String[] whereArgs)

Exemplo de implementação

Agora podemos escrever um método chamado delete com o argumento como nome

public void delete(String value) {
    db.delete(DATABASE_TABLE, KEY_NAME + "=?", new String[]{String.valueOf(value)});
}

se você quiser deletar todos os registros , basta passar null para o método acima,

public void delete() {
    db.delete(DATABASE_TABLE, null, null);
}

Fonte de informação

Jayakrishnan
fonte
Se o valor String for anexado com uma chave estrangeira, então ??
Harsh Bhavsar
2

Pessoal esse é um método genérico que você pode usar para todas as suas tabelas, Funcionou perfeitamente no meu caso.

public void deleteRowFromTable(String tableName, String columnName, String keyValue) {
    String whereClause = columnName + "=?";
    String[] whereArgs = new String[]{String.valueOf(keyValue)};
    yourDatabase.delete(tableName, whereClause, whereArgs);
}
Naveed Ahmad
fonte
String.ValueOf (keyValue) => você pode explicar esta linha?
Anis
1
Não há necessidade de String.ValueOf (keyValue), porque o keyValue já é uma string. em outros casos, estamos usando este whereArgs Array para identificar o valor do nome da coluna.
Naveed Ahmad
2

Para excluir linhas de uma tabela, você precisa fornecer critérios de seleção que identifiquem as linhas para o delete()método. O mecanismo funciona da mesma forma que os argumentos de seleção para o query()método. Ele divide a especificação de seleção em uma cláusula de seleção (cláusula where) e argumentos de seleção.

    SQLiteDatabase db  = this.getWritableDatabase();
     // Define 'where' part of query.
    String selection = Contract.COLUMN_COMPANY_ID + " =?  and "
                       + Contract.CLOUMN_TYPE +" =? ";
   // Specify arguments in placeholder order.
    String[] selectionArgs = { cid,mode };
    // Issue SQL statement.
    int deletedRows = db.delete(Contract.TABLE_NAME, 
                       selection, selectionArgs);
    return deletedRows;// no.of rows deleted.

O valor de retorno do delete()método indica o número de linhas que foram excluídas do banco de dados.

RAM
fonte
Embora este código possa responder à pergunta, fornecer contexto adicional sobre por que e / ou como este código responde à pergunta melhora seu valor a longo prazo.
Thomas Flinkow
1

Pessoal, se as soluções acima não funcionarem para vocês, tentem essa também porque funcionou para mim.

public boolean deleteRow(String name) 
{
    return db.delete(DATABASE_TABLE, KEY_NAME + "='" + name +"' ;", null) > 0;
}
aman003
fonte
1

Funciona bem!

public void deleteNewMelk(String melkCode) {
    getWritableDatabase().delete(your_table, your_column +"=?", new String[]{melkCode});
}
Hadi Note
fonte
0

Tente este:

public void deleteEntry(long rowId) {
    database.delete(DATABASE_TABLE , KEY_ROWID 
        + " = " + rowId, null);}
Nadhir Titaouine
fonte
0
public boolean deleteRow(long l) {
    String where = "ID" + "=" + l;
    return db.delete(TABLE_COUNTRY, where, null) != 0;
}
babiro
fonte
0

Você pode fazer algo assim, compartilhando meu snippet de código de trabalho

Certifique-se de que a consulta seja assim

DELETE FROM tableName WHERE KEY__NAME = 'parameterToMatch'

public void removeSingleFeedback(InputFeedback itemToDelete) {
            //Open the database
            SQLiteDatabase database = this.getWritableDatabase();

            //Execute sql query to remove from database
            //NOTE: When removing by String in SQL, value must be enclosed with ''
            database.execSQL("DELETE FROM " + TABLE_FEEDBACKS + " WHERE "
                    + KEY_CUSTMER_NAME + "= '" + itemToDelete.getStrCustName() + "'" +
                    " AND " + KEY_DESIGNATION + "= '" + itemToDelete.getStrCustDesignation() + "'" +
                    " AND " + KEY_EMAIL + "= '" + itemToDelete.getStrCustEmail() + "'" +
                    " AND " + KEY_CONTACT_NO + "= '" + itemToDelete.getStrCustContactNo() + "'" +
                    " AND " + KEY_MOBILE_NO + "= '" + itemToDelete.getStrCustMobile() + "'" +
                    " AND " + KEY_CLUSTER_NAME + "= '" + itemToDelete.getStrClusterName() + "'" +
                    " AND " + KEY_PRODUCT_NAME + "= '" + itemToDelete.getStrProductName() + "'" +
                    " AND " + KEY_INSTALL_VERSION + "= '" + itemToDelete.getStrInstalledVersion() + "'" +
                    " AND " + KEY_REQUIREMENTS + "= '" + itemToDelete.getStrRequirements() + "'" +
                    " AND " + KEY_CHALLENGES + "= '" + itemToDelete.getStrChallenges() + "'" +
                    " AND " + KEY_EXPANSION + "= '" + itemToDelete.getStrFutureExpansion() + "'" +
                    " AND " + KEY_COMMENTS + "= '" + itemToDelete.getStrComments() + "'"
            );

            //Close the database
            database.close();
        }
Hitesh Sahu
fonte
0

Experimente o código abaixo

mSQLiteDatabase = getWritableDatabase();//To delete , database should be writable.
int rowDeleted = mSQLiteDatabase.delete(TABLE_NAME,id + " =?",
                    new String[] {String.valueOf(id)});
mSQLiteDatabase.close();//This is very important once database operation is done.
if(rowDeleted != 0){
    //delete success.
} else {
    //delete failed.
}
Durgesh
fonte
0

A única maneira que funcionou para mim foi esta

fun removeCart(mCart: Cart) {
    val db = dbHelper.writableDatabase
    val deleteLineWithThisValue = mCart.f
    db.delete(cons.tableNames[3], Cart.KEY_f + "  LIKE  '%" + deleteLineWithThisValue + "%' ", null)
}


class Cart {
    var a: String? = null
    var b: String? = null
    var c: String? = null
    var d: String? = null
    var e: Int? = null
    var f: String? = null

companion object {
    // Labels Table Columns names
    const val rowIdKey = "_id"
    const val idKey = "id"
    const val KEY_a = "a"
    const val KEY_b = "b"
    const val KEY_c = "c"
    const val KEY_d = "d"
    const val KEY_e = "e"
    const val KEY_f = "f"
   }
}

object cons {
    val tableNames = arrayOf(
            /*0*/ "shoes",
            /*1*/ "hats",
            /*2*/ "shirt",
            /*3*/ "car"
         )
 }
AllanRibas
fonte