Android ViewPager指示器不显示且滑动无切换问题求助

Hey, let's work through your ViewPager indicator issue together! I've spotted a few key problems in your code that are making the indicator not show up or sync with swipes. Here's how to fix them step by step:

Your LinearLayout (the container for the dots) uses android:layout_alignParentBottom="true"—but this property only works if the parent layout is a RelativeLayout or ConstraintLayout. If your root layout is a LinearLayout, this property gets ignored, which probably means your indicator is either hidden under the ViewPager or stuck in the wrong position.

How to fix it: #

Wrap your ViewPager and indicator container in a RelativeLayout (or switch to ConstraintLayout if you prefer):

<RelativeLayout
    xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="wrap_content">

    <androidx.viewpager.widget.ViewPager 
        android:id="@+id/view_pager" 
        android:layout_width="match_parent" 
        android:layout_height="170dp" />

    <LinearLayout 
        android:id="@+id/pager_dots" 
        android:layout_width="match_parent" 
        android:layout_height="wrap_content" 
        android:layout_alignParentBottom="true" 
        android:layout_marginBottom="10dp" 
        android:background="@android:color/transparent" 
        android:gravity="center_horizontal" 
        android:orientation="horizontal" />
</RelativeLayout>

Right now, you're creating the indicator dots using retroPhotoList.size() directly. But if your API returns an empty list (or response.body() is null), this will cause a crash or prevent the indicator from being created at all.

How to fix it: #

Add a quick check before initializing the dots:

private void setupPagerIndidcatorDots() {
    // Hide the indicator container if there's no data
    if (retroPhotoList == null || retroPhotoList.isEmpty()) {
        llPagerDots.setVisibility(View.GONE);
        return;
    }

    img = new ImageView[retroPhotoList.size()];
    // Rest of your dot initialization code stays the same...
}

Right now, tapping a dot only changes its alpha—it doesn't actually switch the ViewPager to that page. Let's fix that:

img[i].setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View view) {
        // Jump to the corresponding page when a dot is tapped
        viewPager.setCurrentItem(i);
    }
});

Your current page sync logic works, but we can make it a bit more efficient by only updating the previous and current dots instead of looping through all of them:

viewPager.addOnPageChangeListener(new ViewPager.OnPageChangeListener() {
    private int lastSelectedPosition = 0;

    @Override
    public void onPageScrolled(int position, float positionOffset, int positionOffsetPixels) {}

    @Override
    public void onPageSelected(int position) {
        // Reset the last selected dot
        img[lastSelectedPosition].setImageResource(R.drawable.page_indicator_unselected);
        // Highlight the current dot
        img[position].setImageResource(R.drawable.page_indicator_selected);
        lastSelectedPosition = position;
    }

    @Override
    public void onPageScrollStateChanged(int state) {}
});

Make sure your PhotoViewPagerAdapter's getCount() method returns the correct number of items. If this is wrong, the ViewPager won't swipe properly, and the indicator won't sync:

// Inside PhotoViewPagerAdapter
@Override
public int getCount() {
    // Return 0 if the list is null, otherwise return the list size
    return retroPhotoList == null ? 0 : retroPhotoList.size();
}

After making these changes, your indicator should show up correctly, sync with ViewPager swipes, and respond to taps.

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