0

El problema que tengo es que para el onBackPressed quiero que tenga dos funciones:

  1. Cerrar el NavigationView si ese está abierto.
  2. Que los iconos del BottomNavigationView se cambien solos según el fragmento en que esté el usuario

Ambas cosas funcionan, pero tengo que llamar dos veces a super.onBackPressed();

Ahora, el problema es que al llamarlo dos veces las imágenes en dos de los fragmentos se me devuelve duplicadas, pero si quito uno de los ````super.onBackPressed();``` la página no se cambia. se vuelve a cargar la misma página en que estés, pero no te vuelve a la página anterior.

Cómo puedo escribir yo el código como para llamar al super.onBackPressed(); solo una vez para que las imágenes no se me devuelva duplicadas?

Así lo tengo ahora, y las imágenes se me devuelve duplicadas

 @Override
    public void onBackPressed() {
        if (mDrawer.isDrawerOpen(GravityCompat.START)) {
            mDrawer.closeDrawer(GravityCompat.START);
        }
        super.onBackPressed();

        Fragment fragment = getSupportFragmentManager().findFragmentById(R.id.fragment_container);
        if (fragment instanceof HomeFragment) {
            mBottomNavigationView.setSelectedItemId(R.id.nav_home);
        } else if (fragment instanceof AttendingEventFragment) {
            mBottomNavigationView.setSelectedItemId(R.id.nav_save);
        } else if (fragment instanceof NotificationsFragment) {
            mBottomNavigationView.setSelectedItemId(R.id.nav_notifications);
        } else if (fragment instanceof ProfileFragment) {
            mBottomNavigationView.setSelectedItemId(R.id.nav_profile);
        }
        super.onBackPressed();
    }

También lo he intentado a hacer así, pero entonces la página no se cambia...

@Override
    public void onBackPressed() {
        Fragment fragment = getSupportFragmentManager().findFragmentById(R.id.fragment_container);
        if (mDrawer.isDrawerOpen(GravityCompat.START)) {
            mDrawer.closeDrawer(GravityCompat.START);
        } else {
            if (fragment instanceof HomeFragment) {
                mBottomNavigationView.setSelectedItemId(R.id.nav_home);
            } else if (fragment instanceof AttendingEventFragment) {
                mBottomNavigationView.setSelectedItemId(R.id.nav_save);
            } else if (fragment instanceof NotificationsFragment) {
                mBottomNavigationView.setSelectedItemId(R.id.nav_notifications);
            } else if (fragment instanceof ProfileFragment) {
                mBottomNavigationView.setSelectedItemId(R.id.nav_profile);
            }
        }
        super.onBackPressed();
    }

BottomNavigationView MainActivity

private BottomNavigationView.OnNavigationItemSelectedListener navigationItemSelectedListener = menuItem -> {
        switch (menuItem.getItemId()) {
            case R.id.nav_home:
                mSelectedFragment = new TabLayoutFragment();
                break;
            case R.id.nav_notifications:
                seenNotification();
                mSelectedFragment = new NotificationsFragment();
                break;
            case R.id.nav_profile:
                SharedPreferences.Editor editor = getSharedPreferences("PREFS", MODE_PRIVATE).edit();
                editor.putString("profileid", Objects.requireNonNull(FirebaseAuth.getInstance().getCurrentUser()).getUid());
                editor.apply();
                mSelectedFragment = new ProfileFragment();
                break;
            case R.id.nav_save:
                mSelectedFragment = new AttendingEventFragment();
                break;
        }

        if (mSelectedFragment != null) {
            getSupportFragmentManager().beginTransaction().replace(R.id.fragment_container, mSelectedFragment, null).addToBackStack(null).commit();
        }

        return true;
    };
2
  • A que imagenes te refieres que se cargan dos veces?
    – Jorgesys
    Commented el 10 jul. 2020 a las 13:58
  • @Jorgesys tengo imagenes en el ProfileFragment que se cargan dos veces, pero lo he comprobado y sé que es porque llamo dos veces al super.onBackPressed();, pero si no lo hago ni me vuelve a la página anterior, ni el NavigationView se cierra... Hay mejor forma de escribir ese código en onBackPressed(); para que haga ambas cosas y solo tenga que llamar a onBackPressed(); tan solo una vez?
    – johnnnn
    Commented el 10 jul. 2020 a las 14:01

1 respuesta 1

0

Si las imágenes y otros elementos aparecen duplicados es porque estas creando instancias del fragmento y asignándolas al contenedor mas de una vez. Debes manejar la pila de fragmentos usando banderas, o simplemente añadiendo un fragmento si aun no esta añadido, y en el caso de regresar al anterior, ocultar el actual y mostrar el anterior, es un juego de añadir-ocultar-mostrar según se requiera. Aquí un ejemplo guardando un historial usando el enfoque inicial que tenias (usando replace para cada transacción).

package cm.aptoide.myapplication;

import android.os.Bundle;
import android.view.MenuItem;

import androidx.annotation.NonNull;
import androidx.appcompat.app.AppCompatActivity;
import androidx.fragment.app.Fragment;

import com.google.android.material.bottomnavigation.BottomNavigationView;

import java.util.ArrayList;
import java.util.List;

public class MainActivity extends AppCompatActivity {

    BottomNavigationView navView;
    List<String> historic = new ArrayList<>();
    int MAX_HISTORIC = 5; //Cambiar de acuerdo al historial que desees implementar

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        navView = findViewById(R.id.nav_view);
        navView.setOnNavigationItemSelectedListener(new BottomNavigationView.OnNavigationItemSelectedListener() {
            @Override
            public boolean onNavigationItemSelected(@NonNull MenuItem item) {
                setSelected(item,false);
                return false;
            }
        });
        navView.setSelectedItemId(R.id.navigation_home);
    }

    private void setSelected(MenuItem item,boolean isBack) {
        Fragment mSelectedFragment = null;
        switch (item.getItemId()) {
            case R.id.navigation_home:
                mSelectedFragment = new TabLayoutFragment();
                break;
            case R.id.navigation_dashboard:
                mSelectedFragment = new AttendingEventFragment();
                break;
            case R.id.navigation_notifications:
                mSelectedFragment = new NotificationsFragment();
                break;
        }
        if (mSelectedFragment != null) {
           getSupportFragmentManager().beginTransaction().replace(R.id.nav_host_fragment, mSelectedFragment, null).addToBackStack(null).commit();
            if(!isBack)saveHistoric(String.valueOf(item.getTitle()));
            item.setChecked(true);
        }
    }
    private void saveHistoric(String currentItem) {

        int i = historic.size();
        if(i != 0) {
            if (i < MAX_HISTORIC) {
                if (!historic.get(i - 1).equals(currentItem)) {
                    historic.add(currentItem);
                }
            } else {
                if (!historic.get(i - 1).equals(currentItem)) {
                    historic.add(currentItem);
                    historic.remove(0);
                }
            }
        }else historic.add(currentItem);
    }
    @Override
    public void onBackPressed() {
        if(historic.size() > 1) {
            switch (historic.get(historic.size() - 2)) {
                case "Home":
                    setSelected(navView.getMenu().getItem(0),true);
                    break;
                case "Dashboard":
                    setSelected(navView.getMenu().getItem(1),true);
                    break;
                case "Notifications":
                    setSelected(navView.getMenu().getItem(2),true);
                    break;
            }
            historic.remove(historic.size() - 1);
        }else
            finish();
    }
}
14
  • De acuerdo, pero me puedes indicar cómo lo puedo cambiar con el código que tengo allí arriba? Po ejemplo, arriba puedes ver el que pone como lo tengo ahora, y seguro que es eso, pero que harias tu diferent con ese código?
    – johnnnn
    Commented el 10 jul. 2020 a las 14:08
  • El código que adjuntas esta bien, deberías editarlo e incluir la parte donde se asignan los fragmentos al seleccionar cada ítem, porque es allí donde se verifica que procedimiento realizar
    – Marcos F.
    Commented el 10 jul. 2020 a las 14:13
  • Acabo de subirlo. Echale un vistazo por favor. Ademas, en el metodo super.onBackPressed(); lo llamo dos veces que no esta bien, verdad? Hay otra forma de hacerlo, escribirlo para que solo lo llame una vez?
    – johnnnn
    Commented el 10 jul. 2020 a las 14:16
  • ¿Estas seguro que fragment tiene un valor no nulo?, porque puede ser que nunca se cumpla el condicional
    – Marcos F.
    Commented el 10 jul. 2020 a las 14:37
  • Por qué iba a tener un valor nulo?? Acabo de comprobarlo y me parece que sí, pero cómo lo arreglo?
    – johnnnn
    Commented el 10 jul. 2020 a las 14:52

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.