TabLayout与ViewPager中Fragment动态数据不同步问题求助

你遇到的这个数据错位问题，核心原因是静态变量a在OneFragment的newInstance方法中被全局共享覆盖，再加上ViewPager默认的预加载机制（默认会提前创建当前页面左右各一个Fragment），导致第一个显示的Fragment拿到的是第二个Fragment的标签值，从而加载了错误的数据。下面我会一步步拆解问题并给出修复方案：

1. 静态变量引发的全局覆盖问题 #

在OneFragment的newInstance方法里，你用了静态变量a来存储标签名称：

private static String a;
// ...
public static OneFragment newInstance(int position, Bundle item) {
    Bundle args = item;
    a = args.getString("item"); // 静态变量会被后续创建的Fragment覆盖
    OneFragment fragment = new OneFragment();
    fragment.setArguments(args);
    return fragment;
}

静态变量属于类级别，所有OneFragment实例都会共享这个变量。当ViewPager创建第一个Fragment（TOPS）后，会立刻预加载第二个Fragment（LAUNDRY），这时候静态变量a会被替换成LAUNDRY，导致第一个Fragment在加载数据时用了这个被覆盖的值。

修复方案：移除静态变量a，直接从Fragment的arguments中获取标签名称，不要用静态变量传递数据。

2. ViewPager预加载与数据加载时机问题 #

另外，你的数据加载逻辑放在onCreateView里的RecyclerViewAdapterMethod，但ViewPager预加载时这个方法会被提前调用，再加上静态变量的问题导致数据错误。同时onStart里的进度条处理逻辑也不合理，notifyDataSetChanged在这里调用没有意义，因为数据还没从Firebase加载完成。

修复方案：

• 从getArguments()中直接获取当前Fragment的标签名称，不依赖静态变量
• 调整数据加载逻辑，确保每个Fragment都用自己的标签值请求数据
• 优化进度条显示逻辑，在数据加载完成/失败后再隐藏

修改后的完整代码 #

修正后的MyTabAdapter #

MyTabAdapter本身问题不大，只是可以简化newInstance的参数（你代码里没用到传入的position）：

public class MyTabAdapter extends FragmentStatePagerAdapter { 
    private ArrayList<TabListing> tabListings; 

    public MyTabAdapter(FragmentManager supportFragmentManager, ArrayList<TabListing> tabListings) { 
        super(supportFragmentManager); 
        this.tabListings = tabListings; 
    } 

    @Override 
    public Fragment getItem(int position) { 
        TabListing l = tabListings.get(position); 
        String tabName = l.getName(); 
        Bundle bundle = new Bundle(); 
        bundle.putString("item", tabName); 
        return OneFragment.newInstance(bundle); // 简化参数，只传Bundle
    } 

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

    @Nullable 
    @Override 
    public CharSequence getPageTitle(int position) { 
        TabListing l = tabListings.get(position); 
        return l.getName(); 
    } 
}

修正后的OneFragment #

public class OneFragment extends Fragment { 
    RecyclerView recyclerview; 
    AdapterToActivity act; 
    ListingAdapter adapter; 
    FirebaseDatabase mDatabase; 
    DatabaseReference mReference; 
    ContentLoadingProgressBar pBar; 
    ArrayList<ItemList> itm = new ArrayList<>(); 
    private String tabName; // 用实例变量代替静态变量

    public OneFragment() { } 

    // 简化newInstance方法，移除静态变量，直接传递Bundle
    public static OneFragment newInstance(Bundle args) { 
        OneFragment fragment = new OneFragment(); 
        fragment.setArguments(args); 
        return fragment; 
    } 

    @Override 
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) { 
        View v = inflater.inflate(R.layout.fragment_one, container, false); 
        recyclerview = v.findViewById(R.id.all_product_list); 
        pBar = v.findViewById(R.id.progress); 
        
        // 从当前Fragment的arguments中获取标签名称
        if (getArguments() != null) {
            tabName = getArguments().getString("item");
        }

        RecyclerView.LayoutManager layoutManager = new GridLayoutManager(getActivity(), 1); 
        recyclerview.setLayoutManager(layoutManager); 
        adapter = new ListingAdapter(getActivity(), itm, tabName); 
        recyclerview.setAdapter(adapter); // 提前设置适配器，避免数据加载时重复设置
        
        loadFirebaseData(); // 单独提取数据加载方法，逻辑更清晰
        return v; 
    } 

    // 单独处理Firebase数据加载逻辑
    private void loadFirebaseData() { 
        if (tabName == null) return;
        
        pBar.show(); 
        mDatabase = FirebaseDatabase.getInstance(); 
        mReference = mDatabase.getReference(tabName); 
        mReference.addChildEventListener(new ChildEventListener() { 
            @Override 
            public void onChildAdded(DataSnapshot dataSnapshot, String s) { 
                ItemList l = dataSnapshot.getValue(ItemList.class); 
                itm.add(l); 
                adapter.notifyDataSetChanged(); 
                pBar.hide(); 
            } 

            @Override public void onChildChanged(DataSnapshot dataSnapshot, String s) { } 
            @Override public void onChildRemoved(DataSnapshot dataSnapshot) { } 
            @Override public void onChildMoved(DataSnapshot dataSnapshot, String s) { } 
            @Override public void onCancelled(DatabaseError databaseError) { 
                pBar.hide(); // 加载失败也要隐藏进度条，避免卡住
            } 
        }); 
    } 

    @Override 
    public void onStart() { 
        super.onStart(); 
        // 移除无意义的notifyDataSetChanged和进度条操作，数据加载已在onCreateView中处理
    } 
}

额外优化建议 #

• 给ListingAdapter添加清空数据的方法，在Fragment重新可见时（比如onResume）清空旧数据并重新加载，避免数据累积
• 如果你的标签数量不多，考虑使用FragmentPagerAdapter代替FragmentStatePagerAdapter，前者的缓存机制更适合固定数量的标签页
• 在onDestroyView中调用mReference.removeEventListener(this)，移除Firebase监听，避免内存泄漏

内容的提问来源于stack exchange，提问作者rituparna