ViewPager中的奇怪行为(bug?)

时间:2016-02-24 04:37:13

标签: java android android-viewpager

尝试从与ViewPager不同的类中使用addOnPageChangeListener时,我注意到它会导致一种奇怪的行为......指令在第一次被调用后会被执行多次。我做了一个非常自动生成的示例项目,以便您自己查看:

MainActivity.java:

package com.example.laptop.test;

import android.support.design.widget.FloatingActionButton;
import android.support.design.widget.Snackbar;
import android.support.v7.app.AppCompatActivity;
import android.support.v7.widget.Toolbar;
import android.support.v4.app.Fragment;
import android.support.v4.app.FragmentManager;
import android.support.v4.app.FragmentPagerAdapter;
import android.support.v4.view.ViewPager;
import android.os.Bundle;
import android.view.View;

public class MainActivity extends AppCompatActivity {

    private SectionsPagerAdapter mSectionsPagerAdapter;
    static ViewPager mViewPager;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
        setSupportActionBar(toolbar);

        mSectionsPagerAdapter = new SectionsPagerAdapter(getSupportFragmentManager());

        mViewPager = (ViewPager) findViewById(R.id.container);
        mViewPager.setAdapter(mSectionsPagerAdapter);

        FloatingActionButton fab = (FloatingActionButton) findViewById(R.id.fab);
        fab.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                Snackbar.make(view, "Replace with your own action", Snackbar.LENGTH_LONG)
                        .setAction("Action", null).show();
            }
        });

    }

    public class SectionsPagerAdapter extends FragmentPagerAdapter {

        public SectionsPagerAdapter(FragmentManager fm) {
            super(fm);
        }

        @Override
        public Fragment getItem(int position) {
            switch (position){
                case 0:
                    return  new TestActivity();
                case 1:
                    return new TestActivity2();
                case 2:
                    return new TestActivity2();
            }    return null;
        }

        @Override
        public int getCount() {
            // Show 3 total pages.
            return 3;
        }
    }
}

TestActivity.java:

package com.example.laptop.test;

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

public class TestActivity extends android.support.v4.app.Fragment {

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        View rootView = inflater.inflate(R.layout.fragment_test, container, false);
        return rootView;
    }
    }

TestActivity2.java:

package com.example.laptop.test;

import android.os.Bundle;
import android.support.v4.view.ViewPager;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.Toast;

public class TestActivity2 extends android.support.v4.app.Fragment {

    int i=0;

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        final View rootView = inflater.inflate(R.layout.fragment_test2, container, false);

        MainActivity.mViewPager.addOnPageChangeListener(new ViewPager.OnPageChangeListener() {
            @Override
            public void onPageScrolled(int position, float positionOffset, int positionOffsetPixels) {

            }

            @Override
            public void onPageSelected(int position) {

                if(position==1){

                    Toast.makeText(rootView.getContext(),i+"",Toast.LENGTH_SHORT).show();
                    i++;
                }

            }

            @Override
            public void onPageScrollStateChanged(int state) {

            }
        });

        return rootView;
    }
    }

activity_main.xml中:

<?xml version="1.0" encoding="utf-8"?>
<android.support.design.widget.CoordinatorLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/main_content"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:fitsSystemWindows="true"
    tools:context="com.example.laptop.test.MainActivity">

    <android.support.design.widget.AppBarLayout
        android:id="@+id/appbar"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:paddingTop="@dimen/appbar_padding_top"
        android:theme="@style/AppTheme.AppBarOverlay">

        <android.support.v7.widget.Toolbar
            android:id="@+id/toolbar"
            android:layout_width="match_parent"
            android:layout_height="?attr/actionBarSize"
            android:background="?attr/colorPrimary"
            app:layout_scrollFlags="scroll|enterAlways"
            app:popupTheme="@style/AppTheme.PopupOverlay">

        </android.support.v7.widget.Toolbar>

    </android.support.design.widget.AppBarLayout>

    <android.support.v4.view.ViewPager
        android:id="@+id/container"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        app:layout_behavior="@string/appbar_scrolling_view_behavior" />

    <android.support.design.widget.FloatingActionButton
        android:id="@+id/fab"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_gravity="end|bottom"
        android:layout_margin="@dimen/fab_margin"
        android:src="@android:drawable/ic_dialog_email" />

</android.support.design.widget.CoordinatorLayout>

fragment_test.xml:

<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity$PlaceholderFragment"
    android:background="#ffffff">

    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="Fragment 1"
        android:id="@+id/textView"
        android:layout_alignParentTop="true"
        android:layout_alignParentLeft="true"
        android:layout_alignParentStart="true" />
</RelativeLayout>

fragment_test2.xml:

<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity$PlaceholderFragment"
    android:background="#ffffff">

    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="Fragment 2"
        android:id="@+id/textView2"
        android:layout_alignParentTop="true"
        android:layout_alignParentLeft="true"
        android:layout_alignParentStart="true" />
</RelativeLayout>

如TestActivity2所示,系统应该在达到第2页时释放显示数字的Toast,每次到达页面时向变量添加1,就是这样。第一次滑动到第2页就是这样,但在随后的滑动中,输出变得更奇怪,更奇怪,显示了几个吐司而不是一个。

除此之外,我注意到当页面数量为2而不是3时,问题就会消失。

这个问题只发生在我身上吗?我怎么能解决这个问题呢?这是一个错误还是预期的错误?我错过了什么?

1 个答案:

答案 0 :(得分:1)

首先,我必须说,对视图进行静态引用是内存泄漏和头痛的根源。考虑在您的活动中注册OnPageChangeListener,然后将数据传递到您的片段,如文档和教程所示:Communicating with other fragments

关于手头的问题,由于你的静态ViewPager,也会出现问题。您在静态变量中添加了对片段的引用,因此这些片段永远不会被垃圾收集,并且只要该静态变量与OnPageChangeListener一起存在。因此,当ViewPager创建一个新片段时,最后一个片段的OnPageChangeListener仍然存在,因此会有多个祝酒词。

默认情况下,

ViewPager将2个页面加载到内存中,您可以通过setOffscreenPageLimit(int)更改此页面。当您有3个页面时,其中一个页面在您访问它(或其前面的页面)时会被销毁并更新。这是我上面描述的根本原因。

您可以通过在活动中仅注册一个OnPageChangeListener来解决此问题,使ViewPager成为非静态字段,然后通过您最初向我指出的链接中了解的方式通知您的更改片段

或者,您可以查看EventBus作为与片段进行通信的替代方式。