La variable est accessible dans la classe interne. Doit être déclaré définitif

Donc, le titre dit tout. J'obtiens une erreur de compilation dans mon fichier onClick.

Voici le code.

public class fieldsActivity extends Activity {

Button addSiteButton;
Button cancelButton;
Button signInButton;


/**
 * Called when the activity is first created.
 */
@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    // to create a custom title bar for activity window
    requestWindowFeature(Window.FEATURE_CUSTOM_TITLE);

    setContentView(R.layout.fields);
    // use custom layout title bar
    getWindow().setFeatureInt(Window.FEATURE_CUSTOM_TITLE, R.layout.topbar);

    Pager adapter = new Pager();
    ViewPager mPager = (ViewPager) findViewById(R.id.fieldspager);
    mPager.setAdapter(adapter);
    mPager.setCurrentItem(1);



    addSiteButton = (Button) findViewById(R.id.addSiteButton);
    addSiteButton.setOnClickListener(new View.OnClickListener() {

        @Override
        public void onClick(View v) {
           mPager.setCurrentItem(2, true); //Compilation error happens here.
        }


    });


    cancelButton = (Button) findViewById(R.id.cancel_button);
    signInButton = (Button) findViewById(R.id.sign_in_button);

}

Si vous ne voulez pas le rendre définitif, vous pouvez toujours en faire une variable globale.

Vous pouvez déclarer la variable finale ou en faire une variable d'instance (ou globale). Si vous le déclarez définitif, vous ne pourrez pas le modifier plus tard.

Toute variable définie dans une méthode et accédée par une classe interne anonyme doit être définitive. Sinon, vous pouvez utiliser cette variable dans la classe interne, sans savoir que si la variable change dans la classe interne, puis qu'elle est utilisée plus tard dans la portée englobante, les modifications apportées à la classe interne ne sont pas conservées dans la portée englobante. Fondamentalement, ce qui se passe dans la classe interne reste dans la classe interne.

J'ai écrit une explication plus approfondie ici . Il explique également pourquoi les variables d'instance et globales n'ont pas besoin d'être déclarées finales.

L'erreur dit tout, changez:

ViewPager mPager = (ViewPager) findViewById(R.id.fieldspager);

à

final ViewPager mPager = (ViewPager) findViewById(R.id.fieldspager);

Voici une réponse amusante.

Vous pouvez déclarer un tableau final à un élément et changer les éléments du tableau autant que vous voulez apparemment. Je suis sûr que cela brise la raison même pour laquelle cette règle de compilation a été implémentée en premier lieu, mais c'est pratique lorsque vous êtes dans une contrainte temporelle comme je l'étais aujourd'hui.

Je ne peux en fait pas réclamer de crédit pour celui-ci. C'était la recommandation d'IntelliJ! Se sent un peu hacky. Mais cela ne semble pas aussi mauvais qu'une variable globale, alors j'ai pensé que cela valait la peine d'être mentionné ici. Ce n'est qu'une solution au problème. Pas nécessairement le meilleur.

final int[] tapCount = {0};

addSiteButton.setOnClickListener(new View.OnClickListener() {

    @Override
    public void onClick(View v) {
       tapCount[0]++;
    }

});

Comme @Veger l'a dit, vous pouvez faire en finalsorte que la variable puisse être utilisée dans la classe interne.

final ViewPager pager = (ViewPager) findViewById(R.id.fieldspager);

Je l'ai appelé pagerplutôt que mPagerparce que vous l'utilisez comme variable locale dans la onCreateméthode. Le mpréfixe est généralement réservé aux variables membres de la classe (c'est-à-dire les variables qui sont déclarées au début de la classe et sont disponibles pour toutes les méthodes de classe).

Si vous avez réellement besoin d'une variable de membre de classe, cela ne fonctionne pas pour la rendre finale car vous ne pouvez pas utiliser findViewByIdpour définir sa valeur tant queonCreate . La solution est de ne pas utiliser de classe interne anonyme. De cette façon, la mPagervariable n'a pas besoin d'être déclarée finale et peut être utilisée dans toute la classe.

public class MainActivity extends AppCompatActivity {

    private ViewPager mPager;
    private Button mButton;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        // ...

        mPager = (ViewPager) findViewById(R.id.fieldspager);

        // ...

        mButton.setOnClickListener(myButtonClickHandler);
    }


    View.OnClickListener myButtonClickHandler = new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            mPager.setCurrentItem(2, true);
        }
    };
}

    public class ConfigureActivity extends Activity {

        EditText etOne;
        EditText etTwo;
        @Override
        protected void onCreate(Bundle savedInstanceState) {
            super.onCreate(savedInstanceState);
            setContentView(R.layout.activity_configure);

            Button btnConfigure = findViewById(R.id.btnConfigure1);   
            btnConfigure.setOnClickListener(new View.OnClickListener() {
                        @Override
                        public void onClick(View v) {
                            configure();
                        }
                    });
    }

    public  void configure(){
            String one = etOne.getText().toString();
            String two = etTwo.getText().toString();
    }
}