Фрагменты не отображаются правильно на android - PullRequest
0 голосов
/ 24 марта 2020

Я создаю приложение для бюджетирования в Android. В настоящее время я пытаюсь использовать fragments для отображения всех текущих кошельков пользователей (один фрагмент на кошелек). Каждый fragment будет занимать весь экран и использовать TabLayout в верхней части экрана для навигации. Поскольку мои fragments отображают данные, которые будут часто меняться, они должны быть динамическими c. Проблема сейчас в том, что они создаются (я вижу их в FragmentManager, используя точки останова и c), однако они не отображаются в ViewPager или TabLayout.

Мой XML код для основного действия:

<androidx.constraintlayout.widget.ConstraintLayout 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:layout_width="match_parent"
android:layout_height="match_parent"
tools:context=".MainActivity">

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


<com.google.android.material.tabs.TabLayout
    android:id="@+id/tabLayout"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:elevation="4dp"
    android:minHeight="?attr/actionBarSize"
    android:background="@color/colorTest"
    app:layout_constraintStart_toStartOf="parent"
    app:layout_constraintTop_toTopOf="parent"
    app:tabGravity="fill"
    app:tabMode="fixed" />

<Button
    android:id="@+id/createWallet"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:text="Create wallet"
    app:layout_constraintBottom_toBottomOf="parent"
    app:layout_constraintEnd_toEndOf="parent"
    app:layout_constraintStart_toStartOf="parent"
    app:layout_constraintTop_toTopOf="parent" />

<Button
    android:id="@+id/currentFragment"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:onClick="onClick"
    android:text="Current Fragment"
    app:layout_constraintBottom_toBottomOf="@+id/createWallet"
    app:layout_constraintEnd_toEndOf="parent"
    app:layout_constraintStart_toStartOf="parent"
    app:layout_constraintTop_toTopOf="parent" />
</androidx.constraintlayout.widget.ConstraintLayout>

Мой код для создания fragments и связывания ViewPager и TabLayout вместе в MainActivity:

ViewPager viewPager = findViewById(R.id.pager);
ViewPagerAdapter viewPagerAdapter = new ViewPagerAdapter(getSupportFragmentManager(), MainActivity.this);
viewPager.setAdapter(viewPagerAdapter);

TabLayout tabLayout = findViewById(R.id.tabLayout);
tabLayout.setupWithViewPager(viewPager);

public void loadFragments()
{
    ViewPagerAdapter viewPagerAdapter = new ViewPagerAdapter(getSupportFragmentManager(), MainActivity.this);

    for (int i = 0; i < walletList.size(); i++)
    {
        WalletClass object = walletList.get(i);

        FragmentTransaction fragmentTransaction = getSupportFragmentManager().beginTransaction();
        fragment frag = new fragment().newInstance(object);
        fragmentTransaction.add(R.id.pager, frag, "fragment" + i);
        viewPagerAdapter.addFragment(frag);
        fragmentTransaction.commit();
    }
}

Мой код класса фрагмента:

public fragment newInstance(WalletClass wallet)
{
    fragment frag = new fragment();
    Bundle bundle = new Bundle();

    bundle.putString("walletName", wallet.getWalletName());
    bundle.putInt("walletBalance", wallet.getBalance());

    frag.setArguments(bundle);
    return frag;
}


@Override
public void onCreate(@Nullable Bundle savedInstanceState)
{
    super.onCreate(savedInstanceState);

    Bundle bundle = getArguments();
    Toast.makeText(getContext(), "" + bundle.getString("walletName"), Toast.LENGTH_SHORT).show();
}

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState)
{
    // Inflate the layout for this fragment
    View view =  inflater.inflate(R.layout.fragment_layout, container, false);

    Bundle bundle = getArguments();

    TextView walletName = view.findViewById(R.id.walletName);
    walletName.setText(bundle.getString("walletName"));

    return view;
}

И мой код для адаптера ViewPager. Этот код, в основном, взят из других интерактивных учебных пособий, которые, по сути, должны быть переработаны, я думаю:

List<fragment> fragmentList = new ArrayList<>();
List<String> fragmentNameList = new ArrayList<>();

private Context context;

public ViewPagerAdapter(FragmentManager fm, Context context)
{
    super(fm);
    this.context = context;
}

@Override
public fragment getItem(int position)
{
    return fragmentList.get(position);
}

public void addFragment (Fragment fragment, WalletClass wallet)
{
    Bundle bundle = new Bundle();
    bundle.putString("walletName", wallet.getWalletName());
    bundle.putInt("walletBalance", wallet.getBalance());
    fragment.setArguments(bundle);

    fragmentList.add(fragment);
    fragmentNameList.add(wallet.getWalletName());
}

public void removeFragment (fragment fragment)
{
    fragmentList.remove(fragment);
}

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

@Nullable
public CharSequence getPageTitle(int position)
{
    return fragmentNameList.get(position);
}

Так что у кого-нибудь есть идеи о том, как динамически создавать fragments с помощью * 1028? * адаптер? Или я делаю это совершенно неправильно?

Ответы [ 2 ]

0 голосов
/ 25 марта 2020

Я думаю, что ваш fragment неправильно прикреплен к вашему ViewPager.

Сначала , в вашем loadFragment() ваш новый добавленный фрагмент не был установлен на вашем viewPager. Вы должны добавить setAdapter() снова после добавления нового фрагмента в ваш адаптер. Как упомянуто в официальной документации (это из класса ListAdapter, но в принципе то же самое):

Чтобы отобразить элементы в списке, вызовите setAdapter (android .widget. ListAdapter), чтобы связать адаптер со списком.

Второй , также в вашем loadFragment(), вам не нужно создавать новый экземпляр ViewPagerAdapter внутри метода , Используйте ваш viewPagerAdapter, который вы создали раньше. И пока вы объявляете ViewPager в начале вне вашего onCreate() метода.

Третий , вы можете удалить FragmentTransaction в loadFragment(). Я не знаю, значит это быть или нет. Но, если у вас есть адаптер, ваш новый фрагмент может присоединяться автоматически без вызова FragmentTransaction

Вот как выглядит MainActivity:

// declare ViewPager viewPager in the beginning of class.
viewPager = findViewById(R.id.pager);
viewPagerAdapter = new ViewPagerAdapter(getSupportFragmentManager(), MainActivity.this);
viewPager.setAdapter(viewPagerAdapter);

TabLayout tabLayout = findViewById(R.id.tabLayout);
tabLayout.setupWithViewPager(viewPager);

public void loadFragments()
{

    for (int i = 0; i < walletList.size(); i++)
    {
        WalletClass object = walletList.get(i);

        fragment frag = new fragment().newInstance(object);
        viewPagerAdapter.addFragment(frag, object);
        viewPager.setAdapter(viewPagerAdapter)
    }
}

Примечание : Когда я запускал ваш код, я думаю, что проблема не в вашем адаптере, а в том, как вы называете свой адаптер

Надеюсь, это поможет.

0 голосов
/ 24 марта 2020

FragmentPagerAdapter уже управляет жизненным циклом фрагментов, вам не нужно добавлять их вручную. Однако FragmentPagerAdapter кодирует положение фрагмента в свой тег фрагмента, поэтому его нельзя безопасно использовать для обработки динамических c адаптеров пейджера.

Вам необходимо использовать следующий код или эквивалентный (который, вероятно, закончится следующий код), основанный на коде FragmentPagerAdapter.

public class DynamicFragmentPagerAdapter extends PagerAdapter {
    private static final String TAG = "DynamicFragmentPagerAdapter";

    private final FragmentManager fragmentManager;

    public static abstract class FragmentIdentifier implements Parcelable {
        private final String fragmentTag;
        private final Bundle args;

        public FragmentIdentifier(@NonNull String fragmentTag, @Nullable Bundle args) {
            this.fragmentTag = fragmentTag;
            this.args = args;
        }

        protected FragmentIdentifier(Parcel in) {
            fragmentTag = in.readString();
            args = in.readBundle(getClass().getClassLoader());
        }

        @Override
        public void writeToParcel(Parcel dest, int flags) {
            dest.writeString(fragmentTag);
            dest.writeBundle(args);
        }

        protected final Fragment newFragment() {
            Fragment fragment = createFragment();
            Bundle oldArgs = fragment.getArguments();
            Bundle newArgs = new Bundle();
            if(oldArgs != null) {
                newArgs.putAll(oldArgs);
            }
            if(args != null) {
                newArgs.putAll(args);
            }
            fragment.setArguments(newArgs);
            return fragment;
        }

        protected abstract Fragment createFragment();
    }

    private ArrayList<FragmentIdentifier> fragmentIdentifiers = new ArrayList<>();

    private FragmentTransaction currentTransaction = null;

    private Fragment currentPrimaryItem = null;

    public DynamicFragmentPagerAdapter(FragmentManager fragmentManager) {
        this.fragmentManager = fragmentManager;
    }

    private int findIndexIfAdded(FragmentIdentifier fragmentIdentifier) {
        for (int i = 0, size = fragmentIdentifiers.size(); i < size; i++) {
            FragmentIdentifier identifier = fragmentIdentifiers.get(i);
            if (identifier.fragmentTag.equals(fragmentIdentifier.fragmentTag)) {
                return i;
            }
        }
        return -1;
    }

    public void addFragment(FragmentIdentifier fragmentIdentifier) {
        if (findIndexIfAdded(fragmentIdentifier) < 0) {
            fragmentIdentifiers.add(fragmentIdentifier);
            notifyDataSetChanged();
        }
    }

    public void removeFragment(FragmentIdentifier fragmentIdentifier) {
        int index = findIndexIfAdded(fragmentIdentifier);
        if (index >= 0) {
            fragmentIdentifiers.remove(index);
            notifyDataSetChanged();
        }
    }

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

    @Override
    public void startUpdate(@NonNull ViewGroup container) {
        if (container.getId() == View.NO_ID) {
            throw new IllegalStateException("ViewPager with adapter " + this
                    + " requires a view id");
        }
    }

    @SuppressWarnings("ReferenceEquality")
    @NonNull
    @Override
    public Object instantiateItem(@NonNull ViewGroup container, int position) {
        if (currentTransaction == null) {
            currentTransaction = fragmentManager.beginTransaction();
        }
        final FragmentIdentifier fragmentIdentifier = fragmentIdentifiers.get(position);
        // Do we already have this fragment?
        final String name = fragmentIdentifier.fragmentTag;
        Fragment fragment = fragmentManager.findFragmentByTag(name);
        if (fragment != null) {
            currentTransaction.attach(fragment);
        } else {
            fragment = fragmentIdentifier.newFragment();
            currentTransaction.add(container.getId(), fragment, fragmentIdentifier.fragmentTag);
        }
        if (fragment != currentPrimaryItem) {
            fragment.setMenuVisibility(false);
            fragment.setUserVisibleHint(false);
        }
        return fragment;
    }

    @Override
    public void destroyItem(@NonNull ViewGroup container, int position, @NonNull Object object) {
        if (currentTransaction == null) {
            currentTransaction = fragmentManager.beginTransaction();
        }
        currentTransaction.detach((Fragment) object);
    }

    @SuppressWarnings("ReferenceEquality")
    @Override
    public void setPrimaryItem(@NonNull ViewGroup container, int position, @NonNull Object object) {
        Fragment fragment = (Fragment) object;
        if (fragment != currentPrimaryItem) {
            if (currentPrimaryItem != null) {
                currentPrimaryItem.setMenuVisibility(false);
                currentPrimaryItem.setUserVisibleHint(false);
            }
            fragment.setMenuVisibility(true);
            fragment.setUserVisibleHint(true);
            currentPrimaryItem = fragment;
        }
    }

    @Override
    public void finishUpdate(@NonNull ViewGroup container) {
        if (currentTransaction != null) {
            currentTransaction.commitNowAllowingStateLoss();
            currentTransaction = null;
        }
    }

    @Override
    public boolean isViewFromObject(@NonNull View view, @NonNull Object object) {
        return ((Fragment) object).getView() == view;
    }

    @Override
    public Parcelable saveState() {
        Bundle bundle = new Bundle();
        bundle.putParcelableArrayList("fragmentIdentifiers", fragmentIdentifiers);
        return bundle;
    }

    @Override
    public void restoreState(Parcelable state, ClassLoader loader) {
        Bundle bundle = ((Bundle)state);
        bundle.setClassLoader(loader);
        fragmentIdentifiers = bundle.getParcelableArrayList("fragmentIdentifiers");
    }
}

Затем вы можете передать FragmentIdentifier s в этот пейджер, который будет правильно управлять вашими фрагментами на основе ваших идентификаторов фрагментов.

 List<fragment> fragmentList = new ArrayList<>();

Эти учебники пытаются сделать ваше приложение cra sh, отметьте № 3 в https://proandroiddev.com/the-seven-actually-10-cardinal-sins-of-android-development-491d2f64c8e0

...