Question

I have a fragment that is a “timer” that I can add anywhere. In the fragment I change a textView programatically, and it runs beautifully. My problem is when it comes to using a view from the layout inflated by the constructor(? Not sure if that’s the right terminology) in another method below it.

public class Timer_fragment extends android.support.v4.app.Fragment {
    int testpins;
    String testedpin;
    TextView text;  
@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
        Bundle savedInstanceState) {
    View v = inflater.inflate(R.layout.timer_frag, container, false);

    TextView text = (TextView) v.findViewById(R.id.pwd_status);
    text.setText("Setting text in fragment not main");
    /* set the TextView's text, click listeners, etc. */
    updateStatus();
    return v;
}

All of that code works with no errors but when I try to add this method:

private void updateStatus() {
            TextView text = (TextView) findViewById(R.id.pwd_status);
            testPin();

            text.setText(testedpin);                         
        }

I get a red line under findViewById saying The method findViewById(int) is undefined for the type Timer_fragment.

I thought about inflating the view in all of my methods and not returning them, but surely that would affect performance somehow right?

Just tried inflating the layout before using the view but I get an error on the word inflater and container saying that they can’t be resolved.

Am I going about this correctly?

Answer 1

You already have a member variable in the scope of your Fragment called text. Don’t re-declare it in your methods, just assign it.

text = (TextView) v.findViewById(R.id.pwd_status);

and

private void upateStatus() {
        testPin();
        text.setText(testedpin);                         
    }