How to delete value on variable

Question

I am new to Android and I have a question. I have a recyclerview that can be clicked and will send a query. When clicked, it will get a number/ID of each position that is clicked,

like this:

fno = arrivals.get(position).flightno;
codeCity = arrivals.get(position).arr;

When clicked for the first time it succeeds, but when clicked for the second or third time, it gets an error. I think this because the variable fno and codeCity has been filled. So how to delete the value on the variable?

This my code when the recyclerview clicked:

mRecyclerView.addOnItemTouchListener(new RecyclerView.OnItemTouchListener() {
            GestureDetector gestureDetector = new GestureDetector(mContext, new GestureDetector.SimpleOnGestureListener() {

                public boolean onSingleTapUp(MotionEvent e){
                    return true;
                }
            });

            @Override
            public boolean onInterceptTouchEvent(RecyclerView rv, MotionEvent e) {
                View child = rv.findChildViewUnder(e.getX(), e.getY());
                if (child != null && gestureDetector.onTouchEvent(e)){
                    int position = rv.getChildAdapterPosition(child);

                    fno = arrivals.get(position).flightno;
                    codeCity = arrivals.get(position).arr;

                    final Bundle i = new Bundle();
                    i.putString("city", codeCity.toString()); // Key1
                    i.putString("flightno", fno.toString()); // Key1
                    open2(i);
                }
                return false;
            }

            @Override
            public void onTouchEvent(RecyclerView rv, MotionEvent e) {

            }

            @Override
            public void onRequestDisallowInterceptTouchEvent(boolean disallowIntercept) {

            }
        });

This the query

public void open2(Bundle args) {
        codeIata = args.getString("city");
        String flightdate = args.getString("flightdate");
        fno = args.getString("flightno");
        ArrivalActivity.detail(codeIata, flightdate, fno);
    }
    
public static void detail(String s, String date, String fno) {

        DateFormat df = new SimpleDateFormat("yyyy-MM-d");
        date = df.format(Calendar.getInstance().getTime());
        Call<GetArrivals> arrCall = mApiInterface.getArr(s, "2019-03-04", "", fno);
        arrCall.enqueue(new Callback<GetArrivals>() {
            @Override
            public void onResponse(Call<GetArrivals> call, Response<GetArrivals>
                    response) {
                System.out.println("status code "+response.code());

                arrivals = response.body().getResult();
                mAdapter = new ArrivalAdapter(mContext, arrivals);
                //mRecyclerView.setAdapter(mAdapter);
            }

            @Override
            public void onFailure(Call<GetArrivals> call, Throwable t) {
                Log.e("Retrofit Get", t.toString());
            }
        });
    }

Thank you!

you need onClick instead of onTouch so you can get correct position of element you have clicked refer this my answer https://stackoverflow.com/a/44963520/1506395 — Rajesh, Mar 06 '19 at 07:00
@BachVu pointing at line fno = arrivals.get(position).flightno; — Belsazer Razer B, Mar 06 '19 at 07:28
@Rajesh but i can't send a query if i put onClick in Adapter — Belsazer Razer B, Mar 06 '19 at 07:30

Molly · Answer 1 · 2019-03-06T07:31:24.973

There are two approaches you can capture a click event on item of a recyclerview:

First approach

override onClick method instead of onInterceptTouchEvent.

I am providing you with an example.

mRecyclerView.addOnItemTouchListener(new RecyclerTouchListener(this,
        recyclerView, new ClickListener() {
    @Override
    public void onClick(View view, final int position) {
        fno = arrivals.get(position).flightno;
        codeCity = arrivals.get(position).arr;
    }

}));

class RecyclerTouchListener implements RecyclerView.OnItemTouchListener{

    private ClickListener clicklistener;
    private GestureDetector gestureDetector;

    public RecyclerTouchListener(Context context, final RecyclerView recycleView, final ClickListener clicklistener){

        this.clicklistener=clicklistener;
        gestureDetector=new GestureDetector(context,new GestureDetector.SimpleOnGestureListener(){
            @Override
            public boolean onSingleTapUp(MotionEvent e) {
                return true;
            }

            @Override
            public void onLongPress(MotionEvent e) {
                View child=recycleView.findChildViewUnder(e.getX(),e.getY());
                if(child!=null && clicklistener!=null){
                    clicklistener.onLongClick(child,recycleView.getChildAdapterPosition(child));
                }
            }
        });
    }

    @Override
    public boolean onInterceptTouchEvent(RecyclerView rv, MotionEvent e) {
        View child=rv.findChildViewUnder(e.getX(),e.getY());
        if(child!=null && clicklistener!=null && gestureDetector.onTouchEvent(e)){
            clicklistener.onClick(child,rv.getChildAdapterPosition(child));
        }

        return false;
    }

    @Override
    public void onTouchEvent(RecyclerView rv, MotionEvent e) {

    }

    @Override
    public void onRequestDisallowInterceptTouchEvent(boolean disallowIntercept) {

    }
}

Second Approach(a more conventional one)

override the OnClickListener of the viewholder of your adapter:

public class MyAdapter extends RecyclerView.Adapter<MyAdapter.MyViewHolder> {

    LayoutInflater inflater;
    Context context;

    private MyAdapter.RecyclerViewClickListener mListener;

    public MyAdapter(Context context, RecyclerViewClickListener mListener) {
        this.context = context;
        this.mListener = mListener;
        inflater = LayoutInflater.from(this.context);

    }

    public class MyViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener {

        public MyViewHolder(View item, RecyclerViewClickListener listener) {
            super(item);
            mListener = listener;
            item.setOnClickListener(this);
        }


        @Override
        public void onClick(View view) {
            mListener.onClick(view, getAdapterPosition());
        }
    }

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

        return new MyViewHolder(itemView,mListener);
    }
    @Override
    public void onBindViewHolder(MyViewHolder holder, int position) {

    }


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

    @Override
    public int getItemViewType(int position) {
        return position;
    }


    @Override
    public long getItemId(int position) {
        return 0;
    }


    public interface RecyclerViewClickListener {
        void onClick(View view, int position);
    }
}

and in your Fragment/Activity:

myadapter = new MyAdapter(getContext(), new MyAdapter.RecyclerViewClickListener() {
            @Override
            public void onClick(View view, int position) {
                fno = arrivals.get(position).flightno;
                codeCity = arrivals.get(position).arr;
            }
        });
        recyclerView.setAdapter(MyAdapter);
        MyAdapter.notifyDataSetChanged();

Hope this helps you.

I have implemented your code but still get same error, still pointing at line fno = arrivals.get(position).flightno — Belsazer Razer B, Mar 06 '19 at 07:59
"Invalid index 1, size is 1" and pointing at line fno = arrivals.get(position).flightno — Belsazer Razer B, Mar 08 '19 at 03:40

How to delete value on variable

1 Answers1