AndroidのSQLiteで行を削除する


102

これは馬鹿げた質問かもしれませんが、私はSQLiteに不慣れで、これを理解することができないようです。私は列を持つ1台を持ってKEY_ROWIDKEY_NAMEKAY_LATITUDE、とKEY_LONGITUDE。ユーザーが選択して削除できるようにしたい。誰かが私に始める方向を教えてもらえますか?私の質問は、名前のみが指定された行を実際に削除することです。

関連コード:

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);
        }
    }
}

@iDroidの回答を使用してください...私にとってはうまくいくでしょう。iDroidに感謝します。

回答:


183

あなたはこのように試すことができます:

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

または

public boolean deleteTitle(String name) 
{
    return db.delete(DATABASE_TABLE, KEY_NAME + "=?", new String[]{name}) > 0;
}

61
このソリューションでは、セキュリティリークであるSQLインジェクションを実行できるため、Vijayの答えは正しいです。たとえば、name引数の値を使用します。name = "TRUE; <any SQL command>;"=> 'any SQL command'が実行されます。もちろん、その機能のGUIがなくても問題ありません。
bdevay 2014年

@bdevay私の答えは、クエリで実行したバックグラウンドタスクについてです。そのため、UIとは関係ありません。セキュリティを必要としないので、動的に情報を提供する必要があります。ビジェイの答えに従い、リバースエンジニアリングを行う人がいる場合は、データベースと比較しているフィールドにテーブルに関する情報が表示される可能性があります。これはクエリで直接実行しているので、ネギになる可能性はありません。
Shreyash Mahajan 14

@iDroid Explorer:もちろん、ユーザー入力やその他の種類の外部クエリ操作の可能性がない場合、セキュリティリスクは他のソリューションよりも高くありません。次のコメントに
進む

1
...しかし、私はあなたのコメントのリバースエンジニアリングの部分に同意しません。どこか、どういうわけかクエリを定義する必要があります。つまり、リバースエンジニアリングは常に(ソリューションの場合でも)セキュリティリークの可能性があります。これは、特にソースが難読化されている場合でもJavaで発生します。それだけハッキング時間を増やすことができます。一方、Googleの推奨では選択引数を使用しています。この記事を確認してください:link
bdevay

つまり、メソッドや変数に静的な値を与えないという概念です。それは最大の動的である必要があります。静的な値を与えるよりも安全です。とにかく、ユーザーが自分のアプリをどれだけ安全にしたいかはユーザー次第です。
Shreyash Mahajan 14

157

あなたがあなたの解決策を得るかもしれないようにしてみてください

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

58

whereargsも使用することをお勧めします。

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

これは、次のコマンドの使用に似ています。

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

このような方法で削除関数を使用すると、SQLインジェクションが削除されるため、優れています。


2
提供するソリューションについてもう少し説明を追加して、回答を詳しく説明していただけませんか?
abarisone

1
を使用する価値があると思いますwhereargs
msysmilu

@EnkahiこれはSQLiteで準備されたステートメントのようなものですか?私は以前にこのようなid=?構文をPHPで使用しましたが、それは非常によく似ています。
GeekWithGlasses

17

私があなたの質問を理解するまで、削除する行を選択するための2つの条件を設定します。そのためには、次のことを行う必要があります。

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);
      */  

}

13

このコードを試してください

public void deleteRow(String value)
{
SQLiteDatabase db = this.getWritableDatabase();       
db.execSQL("DELETE FROM " + TABLE_NAME+ " WHERE "+COlUMN_NAME+"='"+value+"'");
db.close();
}

削除したい場合はどうすればいいですか?db.deleteRow();
フェレス2017

削除したいパラメータとして値を渡すことで呼び出すことができます。db.deleteRow( "name");として呼び出します。
Harman Khera 2017

8

このコードを試してください...

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


3

SQLiteDatabaseを使用している場合 は、deleteメソッドがあります

削除の定義

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

実装例

これで、名前を引数として、deleteというメソッドを作成できます。

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

すべてのレコード削除する場合は、上記のメソッドにnullを渡します。

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

情報源


文字列値が外部キーに添付されている場合、??
Harsh Bhavsar 2017年

2

みんなこれはあなたがすべてのテーブルに使用できる一般的な方法です、私の場合は完全に働きました。

public void deleteRowFromTable(String tableName, String columnName, String keyValue) {
    String whereClause = columnName + "=?";
    String[] whereArgs = new String[]{String.valueOf(keyValue)};
    yourDatabase.delete(tableName, whereClause, whereArgs);
}

String.ValueOf(keyValue)=>この行を説明できますか?
Anis

1
keyValueはすでに文字列であるため、String.ValueOf(keyValue)は不要です。他の場合では、このwhereArgs配列を使用して列名の値を識別します。
Naveed Ahmad

2

テーブルから行を削除するには、delete()メソッドに行を識別する選択基準を提供する必要があります。このメカニズムは、query()メソッドの選択引数と同じように機能します。選択仕様を選択句(where句)と選択引数に分割します。

    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.

delete()メソッドの戻り値は、データベースから削除された行数を示します。


このコードは質問に答えることがありますが、このコードが質問に答える理由や方法に関する追加のコンテキストを提供すると、長期的な価値が向上します。
トーマスFlinkow

1

上記の解決策がうまくいかない場合は、私にとってもうまくいったので、これも試してください。

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

1

よく働く!

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


0
public boolean deleteRow(long l) {
    String where = "ID" + "=" + l;
    return db.delete(TABLE_COUNTRY, where, null) != 0;
}

0

あなたは私の作業コードスニペットを共有して、このようなことをすることができます

クエリがこのようなものであることを確認してください

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();
        }

0

以下のコードを試してください

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.
}

0

私のために働いた唯一の方法はこれでした

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"
         )
 }
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.