Wywolanie metody (View view) Android Studio

Question

Witam, problem jest tego typu mam taki kod 

 public void obliczPozyc(View view) {
        // Is the button now checked?
        int latt = 0;
        boolean checked = ((RadioButton) view).isChecked();

        // Check which radio button was clicked
        switch(view.getId()) {

            case R.id.rbMeska:
                if (checked)
                {
                    String wynik = result.getText().toString();
                    int wyn = Integer.parseInt(String.valueOf(result));

                    latt = wyn + 6 ;  //age+6
                    if (latt > 25) {
                        latt = 25;
                    }
                    if (latt < 13) {
                        latt = 13;
                    }

                }
                    break;

            case R.id.rbZenska:
                if (checked)
                {
                    String wynik = result.getText().toString();
                    int wyn = Integer.parseInt(String.valueOf(result));

                    latt = wyn - 7; //age-7

                    if (latt > 12) {
                        latt = 12;
                    }
                    if (latt < 0) {
                        latt = 0;
                    }
                }
                    break;
        }

metoda zostala utworzona wraz z utworzeniem radiobuttonow w radiogroup.

Moje pytanie jest nastepujace, chcialbym wywolac ja w innym miejscu, gdy podaje nazwe metody, wola rowniez o parametry. Co mam podac jako parametr, badz jak to ominac, tak aby View view nie bylo potrzebne?

Comments

A co potrzebujesz z tej metody, bo zawsze możesz stworzyć jakąś dodatkową inną metodę.

---

No wlasciwie to wszystko, bo jest to metoda ktora jest czesto uzywana. Myslalem o tym aby po prostu zamiast View view, podac parametr boolean zmienna  i sprawdzac true/false w if'ach. Mialoby to sens?

Answer 1

W tym wydaniu ta metoda niemal całkowicie zależy od parametru View. Tylko elementy w if-ach są niezależne od view, więc prawdopodobnie te bloki da się wyciągnąć do odrębnej metody. Mogłoby to wyglądać np. tak:

   public void obliczPozyc(View view)
    {
        // Is the button now checked?
        int latt = 0;
        boolean checked = ((RadioButton) view).isChecked();

        // Check which radio button was clicked

        if (checked)
        {
            switch (view.getId())
            {

                case R.id.rbMeska:
                {
                    operationForRbMeska();
                    break;
                }

                case R.id.rbZenska:
                {
                    operationForRbZenska();
                    break;
                }
            }
        }
    }

    public void operationForRbMeska()
    {
        int latt;
        String wynik = result.getText().toString();
        int wyn = Integer.parseInt(String.valueOf(result));

        latt = wyn + 6;  //age+6
        if (latt > 25)
        {
            latt = 25;
        }
        if (latt < 13)
        {
            latt = 13;
        }
    }

    public void operationForRbZenska()
    {
        int latt;
        String wynik = result.getText().toString();
        int wyn = Integer.parseInt(String.valueOf(result));

        latt = wyn - 7; //age-7

        if (latt > 12)
        {
            latt = 12;
        }
        if (latt < 0)
        {
            latt = 0;
        }
    }

Przeniosłem jeszcze to if(checked) ponad switcha ponieważ oba case to robią więc zdaje się że można to zrobić raz. Ponadto metody operationForXXX są właściwie identyczne więc można by je jeszcze jakoś uprościć.

A jeśli View jest Ci niezbędny, to trzeba zmienić go na zmienną o zasięgu klasy, tylko wtedy będzie można pozbyć się  parametru z metody.

Comments

A i wcześniej jakoś na to nie zwróciłem uwagi. Takie coś:

if (latt > 12)
{
    latt = 12;
}
if (latt < 0)
{
    latt = 0;
}

W tym przypadku jest mało optymalne, dlatego że musi się wykonać każdy if, a jeśli np. pierwszy jest spełniony to automatycznie drugi nie, a i tak musi się wykonać. Trzeba by zastosować konstrukcje else if przy drugim ifie:

if (latt > 12)
{
    latt = 12;
}
else if (latt < 0) //wykona się tylko wtedy gdy pierwszy if nie będzie spełniony
{
    latt = 0;
}