"Как стереть правильный элемент списка в базе данных sqlite

Question

Я хочу стереть элементы списка просмотра из базы данных, как правильно связать каждый элемент списка, который является его элементом базы данных?

Я уже пытался удалить, где _id - это позиция списка, но каждыйвремя.

это класс виджета и метод getView, где я реализую слушатель на своем классе адаптера

  private class ViewHolder{
        ImageView imageView;
        TextView nom;
        Button actualizar,borrar;
    }

    @Override
    public View getView(final int position, View view, ViewGroup parent) {
        View fila=view;
        ViewHolder holder=new ViewHolder();


        if(fila==null)
        {
            LayoutInflater inflater=(LayoutInflater) context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
            fila=inflater.inflate(layout,null);
            holder.imageView=fila.findViewById(R.id.imagencoche);
            holder.nom=fila.findViewById(R.id.nombreCoche);
            holder.actualizar=fila.findViewById(R.id.btnAct);
            holder.borrar=fila.findViewById(R.id.btnBorrar);

            holder.borrar.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View v) {
                    MyOpenHelper db=new MyOpenHelper(context);
                    db.borrarCoche(position+1+"");

                    lista=db.selectCoches();
                    notifyDataSetChanged();


                }
            });

            fila.setTag(holder);
        }
        else{
            holder=(ViewHolder)fila.getTag();

        }
        Coche coche=lista.get(position);


        holder.nom.setText(coche.getNombre());
        holder.imageView.setImageBitmap(coche.getFoto());

                return fila;
    }

метод 2, вызываемый из помощника SQLite

 public  void borrarCoche(String id)
    {

        String where="_id=?";
       SQLiteDatabase db=this.getWritableDatabase();

         db.delete("coches",where,new String[]{id});
    }

 public ArrayList<Coche> selectCoches(){
        ArrayList<Coche> lista =new ArrayList<Coche>();

        SQLiteDatabase db=this.getReadableDatabase();
        Cursor c =db.rawQuery("SELECT nombre, foto, longitud, latitud,_id from coches", null);

        if (c.moveToFirst()){
            do {
                //recogemos valores
                String nom = c.getString(0);
                byte[] foto = c.getBlob(1);
                float longitud = c.getFloat(2);
                float latitud = c.getFloat(3);
                int id = c.getInt(4);
                Coche coche = new Coche(id, longitud, latitud, DbBitMapUtility.getImage(foto), nom);
                lista.add(coche);


            }while(c.moveToNext());


        }

        return lista;

    }

он не всегда работает правильно, иногда он стирает другую запись, или когда осталось только 2 или 1, он ничего не стирает.

Answer 1

проблема в том, что у вас в массиве нет идентификатора для выбора элемента. Я использую это, и вы можете попробовать, если хотите:

Sqlite код для удаления данных:

public void borrarCoche(int id){
        this.openWriteableDB();
        String where = ConstantsBD.CAR_ID + " = ?";
        db.delete(ConstantsBD.TABLA_COCHES, where, new String[]{String.valueOf(id)});
        this.closeDB();
    }

Sqlite код для массива:

public ArrayList selectCoches() {
        ArrayList list = new ArrayList<>();
        this.openReadableDB();
        String[] campos = new String[]{ConstantsBD.COCHE_ID, //ConstantsBD is a Class where i create the sqlite tables
                ConstantsBD.NOM, ConstantsBD.FOTO,
                ConstantsBD.LONGITUD,ConstantsBD.LATITUD};
        Cursor c = db.query(ConstantsBD.TABLA_COCHES, campos, null, null, null, null, null);
        try {
            while (c.moveToNext()) {
                Coches coches=new Coches();
                ic.setIdCoche(c.getInt(0));//Your id always be in zero
                ic.setNombre(c.getString(1));
                ic.setFoto(c.getBlob(2));
                ic.setLongitud(c.getFloat(3));
                ic.setLatitud(c.getFloat(4));
                list.add(coches);
            }
        } finally { c.close(); }
        this.closeDB();
        return list;
    }

И код для ListView в вашей деятельности:

ListView listView;
ArrayList<Coches> arrayCoches;
SQLITEDB conn;

public void onResume(){
super.onResume(); 
listView = (ListView)findViewById(R.id.lvCoches);
arrayCoches = new ArrayList<Coches>();
conn = new SQLITEDB(getApplicationContext());
arrayCoches=conn.selectCoches();
adt =new adapterCoches(YourActivityName.this, arrayCoches);
lvCam.setAdapter(adt);
lvCam.setOnItemClickListener(new AdapterView.OnItemClickListener() {
            @Override
            public void onItemClick(AdapterView<?> parent, View view,final  int position, long id) {
                AlertDialog.Builder alertbox = new AlertDialog.Builder(YourActivityName.this)
                        .setTitle("¡Alert!")
                        .setPositiveButton("Erase", new DialogInterface.OnClickListener() {
                            public void onClick(DialogInterface arg0, int arg1) {
                                SQLITEDB bd = new SQLITEDB(getApplicationContext());
                                bd.borrarCoche(arrayCoches.get(position).getIdCoches() );
                                onResume();
                            }
                        })
                        .setNegativeButton("Update", new DialogInterface.OnClickListener() {
                            public void onClick(DialogInterface arg0, int arg1) {
                            }
                        });
                alertbox.show();
            }
}

Answer 2

Во-первых, я не думаю, что нужно вызывать элементы из базы данных каждый раз, когда вы удаляете элемент. Скорее вы можете удалить элемент из базы данных, удалить его из списка и вызвать notifyDatasetChanged ().

Кроме того, не вызывайте слушатель в методе getView (), а вызывайте его внутри класса ViewHoder или в onBindViewHolder (). Так что логика будет примерно такой:

public class MyViewHolder extends RecyclerView.ViewHolder {
    ImageView imageView;
    TextView nom;
    Button actualizar,borrar;

    public MyViewHolder(View view) {
        super(view);
        holder.imageView=fila.findViewById(R.id.imagencoche);
        holder.nom=fila.findViewById(R.id.nombreCoche);
        holder.actualizar=fila.findViewById(R.id.btnAct);
        holder.borrar=fila.findViewById(R.id.btnBorrar);
    }
}

@Override
public MyViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
    View itemView = LayoutInflater.from(parent.getContext())
            .inflate(R.layout.movie_list_row, parent, false);

    return new MyViewHolder(itemView);
}

@Override
public void onBindViewHolder(MyViewHolder holder, int position) {
    Coche coche = cocheList.get(position); //Where ArrayList<Coche> cocheList;
    holder.nom.setText("demo text");

    holder.borrar.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                MyOpenHelper db=new MyOpenHelper(context);
                db.borrarCoche(cocheList.getId()); //Don't use position as this is not id rather this is item index value.

                cocheList.remove(position);
                notifyDataSetChanged();
            }
        });
}

Надеюсь, вы получите решение.

Для лучшего понимания вы можете следовать https://www.androidhive.info/2016/01/android-working-with-recycler-view/, чтобы получить базовое представление о RecyclerView в Android.