Question

I’ve created a menu with a intents to access different activities, but I have a strange behavior, it always goes through all the cases of the switch statement after the statement selected , I’ve reviewed the value of the variable item and is correct, any ideas what could be wrong?

the snippet of code that represents the menu is:

    public static final int wiifidi = 0;
    public static final int cuentaint = 1;
    public static final int cajerosint = 2;
    public static final int indicadoresint = 3;
    public static final int promoint = 5;
    public static final int contactoint = 4;

….

….

….

@Override
    //add the items to the menu on the class
    public boolean onCreateOptionsMenu(Menu menu) {
        boolean result = super.onCreateOptionsMenu(menu);
        menu.add(0,wiifidi, 0, R.string.menu_wifi);
        menu.add(0,cuentaint, 0, R.string.menu_cuenta);
        menu.add(0,cajerosint,0,R.string.menu_cajeros);
        menu.add(0,indicadoresint,0,R.string.menu_indicadores);
        menu.add(0,contactoint,0,R.string.menu_contacto);
        menu.add(0,promoint,0,R.string.menu_promo);


        return result;
    }

    @Override
    //handle everything that happens when an item of menu is selected 
    public boolean onOptionsItemSelected(MenuItem item) {
        Toast.makeText(getApplicationContext(), "el item es " +item.getItemId(), Toast.LENGTH_LONG).show();
        switch (item.getItemId()) {     

        case wiifidi:
            wifistatus();
        case cuentaint:{
            consulta();
        }
        case cajerosint:{
            cajero();
        }
        case indicadoresint:{   
            indicador();
        }
        case contactoint:{
            contacto();
            }
        case promoint:{
            promocion();
        }

        }

        return super.onOptionsItemSelected(item);
    }

Answer 1

Remember to break out of your switches.

switch (item.getItemId()) 
    {     
        case wiifidi:
            wifistatus();
            break;
        case cuentaint:
            consulta();
            break;
        case cajerosint:
            cajero();
            break;
        case indicadoresint:   
            indicador();
            break;
        case contactoint:
            contacto();
            break;
        case promoint:
            promocion();
            break;
    }