generateView int position is always 0

[vaggelis-net]

Why the position is always 0 when generateView is called?
I want to make a listview and i hold a reference to a list with all the objects in the swipeadapter.
I have a trash imageview in the bottom view (just like in the tutorial) which when clicked it deletes that item, but i can't notify the adapter with the position since it's always 0. So the wrong item gets deleted.

[daimajia]

Could you please paste your code?

[vaggelis-net]

I did some more testing and i found out that if i pass a list that aready contains some items everything seem to work fine. The position is correct and the right item is deleted.
If the list is empty and start adding later, the position will always be 0 on the new items.

public class MyAdapter extends SwipeAdapter
{

    private Context context;
    //holding the reference, i have an add button on the main activity that adds more strings in the list
    private List<String> mylist;

    public MyAdapter(Context context, List<String> mylist)
    {
        this.context = context;
        this.mylist = mylist;
    }

    @Override
    public View generateView(final int position, ViewGroup viewGroup)
    {
        View v = LayoutInflater.from(context).inflate(R.layout.list_layout, null);
        final SwipeLayout swipeLayout = (SwipeLayout)v.findViewById(getSwipeLayoutResourceId(position));
        swipeLayout.setShowMode(SwipeLayout.ShowMode.LayDown);

        swipeLayout.findViewById(R.id.myimageview).setOnClickListener(new View.OnClickListener()
        {
            @Override
            public void onClick(View view)
            {
                //The position here is always 0 for the new items
                Toast.makeText(context, "my position : " + String.valueOf(position), Toast.LENGTH_SHORT).show();
                //mylist.remove(position);
                //notifyDataSetChanged();
            }
        });

        return v;
    }

   @Override
    public void fillValues(int position, View view)
    {
        TextView textView = (TextView) view.findViewById(R.id.mytextview);
        textView.setText(mylist.get(position));
    }

    @Override
    public int getCount()
    {
        return mylist.size();
    }

    @Override
    public Object getItem(int position)
    {
        return mylist.get(position);
    }

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

[daimajia]

Hey guy.

You should view the source of the SwipeAdapter, and try to know the principle of the SwipeAdapter.

remember: generateView should only do as it's name, just generate views.. do not bind any listeners or fill values.

So, a quick way to solve this problem:

move the following code:

        swipeLayout.findViewById(R.id.myimageview).setOnClickListener(new View.OnClickListener()
        {
            @Override
            public void onClick(View view)
            {
                //The position here is always 0 for the new items
                Toast.makeText(context, "my position : " + String.valueOf(position), Toast.LENGTH_SHORT).show();
                //mylist.remove(position);
                //notifyDataSetChanged();
            }
        });

to

   @Override
    public void fillValues(int position, View view)
    {
        TextView textView = (TextView) view.findViewById(R.id.mytextview);
        textView.setText(mylist.get(position));
    }

[daimajia]

Have a try, if any problem, feel free to contact me.

[vaggelis-net]

Yes, fillValues position will work since you pass the position from getView. My mistake then i though at first the position from geverateView is the same as getView too

[daimajia]

It works right?

I think I should add this notice in Wiki. It's also my mistake. :-D

[vaggelis-net]

Yes i just tested it. Well i fall for it because of the View in both generateView and the usual getView

[daimajia]

ask a favor:

The problem you met is really a pitfall (my mistake)... My English is not very well, I'm not good at writing, explaining. If you are available, could you please help this project to improve the wiki to prevent other developers get into this trouble again:
Only this page:
https://github.com/daimajia/AndroidSwipeLayout/wiki/SwipeAdapter

[vaggelis-net]

Yes i can do some editing if you want :) But to let you know English is not my native language too!

[daimajia]

Do your best !
I trust you ! 😄

[angelen10]

Click and swipe have some comflict.I want to enter another Activity when I click an item, but also I want to swipe left to show the BottomView and handle the BottomView.My code has something wrong?
@OverRide
public void fillValues(int position, View convertView) {

    SwipeLayout swipeLayout = (SwipeLayout) convertView
            .findViewById(getSwipeLayoutResourceId(position));
    swipeLayout.setShowMode(SwipeLayout.ShowMode.LayDown);

    LinearLayout linoutMemo = (LinearLayout) swipeLayout
            .findViewById(R.id.linoutMemo);

    linoutMemo.setOnClickListener(new OnClickListener() {...});

}

My layout code:

<com.daimajia.swipe.SwipeLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:id="@+id/swipe"
android:layout_width="match_parent"
android:layout_height="match_parent" >

<!-- BottomView -->
<LinearLayout
    android:layout_width="40dp"
    android:layout_height="120dp"
    android:background="#333"
    android:gravity="center"
    android:tag="Bottom2" >

    <ImageView
        android:id="@+id/trash"
        android:layout_width="22dp"
        android:layout_height="22dp"
        android:src="@drawable/ic_trash" />
</LinearLayout>

<!-- SurfaceView -->

<LinearLayout
    android:id="@+id/linoutMemo"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:background="#ffffff"
    android:orientation="vertical"
    android:padding="10dp" >

    <TextView
        android:id="@+id/txvTitle"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:text="This is the title" />

    <TextView
        android:id="@+id/txvContent"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:text="This is the content" />
</LinearLayout>

</com.daimajia.swipe.SwipeLayout>
How to handle this problem?
Sorry for my poor English.

[vaggelis-net]

Try this:
LinearLayout linoutMemo = (LinearLayout) convertview.findViewById(R.id.linoutMemo);

instead of
LinearLayout linoutMemo = (LinearLayout) swipeLayout.findViewById(R.id.linoutMemo);