Remove layout on a button click programmatically in android studio

Question

I am adding layouts on a button click.

 private void addLayout() {
    layout2 = LayoutInflater.from(mContext).inflate(R.layout.product_layout, mLinearLayout, false);
    productAuto = (AutoCompleteTextView) layout2.findViewById(R.id.tv_product);
    qtyEditText = (EditText) layout2.findViewById(R.id.prod_qty);
    prodPriceEditText = (EditText)layout2.findViewById(R.id.prod_price);
    prodSpecsEditText = (EditText)layout2.findViewById(R.id.prod_specs);
    removeProduct = (Button)layout2.findViewById(R.id.btn_rmv);
    mLinearLayout.addView(layout2);
    setProd();
    this.initListenersPrd(getActivity());
}
private void initListenersPrd(final Context context) {
    productAuto.setOnItemClickListener((parent, view, position, id) -> {
        String newName = parent.getItemAtPosition(position).toString();
        ProductDetail productDetail = mProductManager.getProductByPrdName(newName);
        if (productDetail != null) {
           this.prodPriceEditText.setText(productDetail.getPrice());
           this.prodSpecsEditText.setText(productDetail.getProductSpec());
        }
    });

    removeProduct.setOnClickListener(v -> {
       if (mLinearLayout.getChildCount()>0)
       {
           ((LinearLayout)mLinearLayout.getParent()).removeView(mLinearLayout);
       }

    });}

Now I want to remove a particular product form from the application so I have done ((LinearLayout)mLinearLayout.getParent()).removeView(mLinearLayout); on remove button click. But it removes the entire layouts that are added dynamically.

GUI

As you can see in the above image I have added 4-5 layouts of the product but when I click the remove button I remove all the layouts.

Update 1

I have added the below layout in my main layout and then called the product layout inside it programatically

The product layout is created separately and I am calling it in my main layout.

The above layout is called whenever I click on Add New Product Button.

How can I remove a particular layout??

Any help would be highly appreciated.

user_6287910 · Answer

You are trimming too high in the view hierarchy. If the button is a direct child of the LinearLayout you want to remove, you can do something like the following in a fragment. (It is similar for an Activity.)

Add five sub-layouts to the top-level layout.

int addCount = 0;
LinearLayout mLinearLayout = binding.linearLayoutId;
for (int i = 0; i < 5; i++) {
    LinearLayout newLayout = new LinearLayout(getActivity());
    newLayout.setBackground(new ColorDrawable(0xFFAAAAAA));
    TextView tv = new TextView(getActivity());
    tv.setTextSize(24);
    tv.setText("Hello World! " + ++addCount);
    newLayout.addView(tv);
    Button button = new Button(getActivity());
    button.setText("Click me");
    newLayout.addView(button);
    mLinearLayout.addView(newLayout);
    ((LinearLayout.MarginLayoutParams) newLayout.getLayoutParams()).setMargins(margin, margin, margin, margin);
    button.setOnClickListener(v -> { // v is the button
        mLinearLayout.removeView((ViewGroup) v.getParent());
    });
}

Where the top layout is simply

In the following, I simply click the buttons for the odd layouts then the buttons for the even layouts.

user_9647447 · Answer

Using RecyclerView with Adapter is your best bet in these situations. My working code is as followes

Class listAdapter

package com.akshita.recycler;

import android.content.Context;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.AutoCompleteTextView;
import android.widget.Button;
import android.widget.EditText;

import androidx.annotation.NonNull;
import androidx.recyclerview.widget.RecyclerView;

import java.util.ArrayList;

public class listAdapter extends RecyclerView.Adapter {
    private ArrayList product_name;
    private ArrayList quantity;
    private ArrayList price;
    private ArrayList specs;
    private static Context scontext;

    public listAdapter(Context context)
    {
        scontext = context;
        this.product_name = new ArrayList();
        this.price = new ArrayList();
        this.quantity = new ArrayList();
        this.specs = new ArrayList();
    }

    public listAdapter(Context context, ArrayList pdname, ArrayList quantity, ArrayList price, ArrayList specs)
    {
        this.product_name = pdname;
        this.price = price;
        this.quantity = quantity;
        this.specs = specs;
        scontext = context;
    }

    @NonNull
    @Override
    public listAdapter.ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
        LayoutInflater layoutInflater =  LayoutInflater.from(parent.getContext());
        View listItem = layoutInflater.inflate(R.layout.productview,parent,false);
        ViewHolder viewHolder = new ViewHolder(listItem);
        return viewHolder;
    }

    @Override
    public void onBindViewHolder(@NonNull listAdapter.ViewHolder holder, int position) {
        holder.actv.setText(product_name.get(position));
        holder.qty.setText(quantity.get(position).toString());
        holder.price.setText(price.get(position).toString());
        holder.desc.setText(specs.get(position));

    }
    public void addView(String pdname, Double prc, Integer quant, String spec ) {

        product_name.add(pdname);
        quantity.add(quant);
        price.add(prc);
        specs.add(spec);
        notifyItemInserted(product_name.size());
        notifyItemInserted(quantity.size());
        notifyItemInserted(price.size());
        notifyItemInserted(specs.size());
    }

    public void removeAt(int position) {

        product_name.remove(position);
        quantity.remove(position);
        price.remove(position);
        specs.remove(position);
        notifyItemRemoved(position);
        notifyItemRangeChanged(position, product_name.size());
        notifyItemRangeChanged(position, quantity.size());
        notifyItemRangeChanged(position, price.size());
        notifyItemRangeChanged(position, specs.size());
    }

    @Override
    public int getItemCount() {
        return product_name.size();
    }

    public class ViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener{
        public AutoCompleteTextView actv;
        public EditText qty;
        public EditText price;
        public EditText desc;
        public Button button;
        public ViewHolder(View itemView) {
            super(itemView);
            this.actv = itemView.findViewById(R.id.tv_product);
            this.qty = itemView.findViewById(R.id.prod_qty);
            this.price = itemView.findViewById(R.id.prod_price);
            this.desc = itemView.findViewById(R.id.prod_specs);
            this.button = itemView.findViewById(R.id.btn_rmv);
            button.setOnClickListener(this);
        }


        @Override
        public void onClick(View v) {
            if(v.equals(button)){
                removeAt(getAdapterPosition());
        }

        }
    }
}

My Main Class

MainActivity.java

package com.akshita.recycler;

import android.os.Bundle;
import android.view.View;

import androidx.appcompat.app.AppCompatActivity;
import androidx.recyclerview.widget.LinearLayoutManager;
import androidx.recyclerview.widget.RecyclerView;

public class MainActivity extends AppCompatActivity implements View.OnClickListener{
    listAdapter adapter;
    RecyclerView hs;
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        hs = findViewById(R.id.rcview);

        hs.setHasFixedSize(false);
        hs.setLayoutManager(new LinearLayoutManager(this, LinearLayoutManager.VERTICAL, false));
    }

    @Override
    public void onClick(View v) {
        switch(v.getId())
        {
            case R.id.add: addView(v);
        }
    }

    public void addView(View v)
    {
        if(adapter == null) {
            adapter = new listAdapter(this);
            hs.setAdapter(adapter);
        }
        adapter.addView("Name",0.0, 0, "Specs");
    }
}

productview.xml

activity_main.xml

Whole working code is available here on GitHub

user_7872110 · Answer

It's because you are removing the entire layout instead you should remove layout been added dynamically I'm supposing that remove product is a direct child in product_layout main view group

    removeProduct.setOnClickListener(v -> {
   if (mLinearLayout.getChildCount()>0)
   {
       ((LinearLayout)mLinearLayout.getParent()).removeView(removeProduct.getParent());
   }

});}

But you should keep in mind that this is totally not recommended and instead you should use recyclerview and on adding new product add an object to products list and call notifyDataSetChanged(), also you can use System.currentTimeMillis() as a unique id for product object.

Remove layout on a button click programmatically in android studio

3 Answers