Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

findViewById inside a Static Method

I have this static method:

public static void displayLevelUp(int level, Context context) {

    LayoutInflater inflater = (LayoutInflater) context
            .getSystemService(Context.LAYOUT_INFLATER_SERVICE);

    View layout = inflater.inflate(R.layout.custom_level_coast,
            (ViewGroup) findViewById(R.id.toast_layout_root));  // this row

    TextView text = (TextView) layout.findViewById(R.id.toastText);
    text.setText("This is a custom toast");

    Toast toast = new Toast(context);
    toast.setGravity(Gravity.CENTER_VERTICAL, 0, 0);
    toast.setDuration(Toast.LENGTH_LONG);
    toast.setView(layout);
    toast.show();

    Toast.makeText(context, String.valueOf(level), Toast.LENGTH_SHORT)
            .show();

}

However, I can't figure out how to get the firstfindViewById to play nice with this as it says it is a non-static method. I understand why it says that, but there must be a workaround? I passed context into this method but couldn't work them out together.

like image 695
TheLettuceMaster Avatar asked May 02 '13 15:05

TheLettuceMaster


3 Answers

One thing you can do is make the view a class wide variable and use that. I dont actually recommend doing that but it will work if you need something quick and dirty.

Passing in the view as a parameter would be the preferred way

like image 95
tyczj Avatar answered Oct 29 '22 15:10

tyczj


it is a little weird. But you can pass the root view as parameter.

//some method...
ViewGroup root = (ViewGroup) findViewById(R.id.toast_layout_root);
displayLevelUp(level, context, root);
//some method end...


public void displayLevelUp(int level, Context context, ViewGroup root) {

LayoutInflater inflater = (LayoutInflater) context
        .getSystemService(Context.LAYOUT_INFLATER_SERVICE);

View layout = inflater.inflate(R.layout.custom_level_coast,
        root);

TextView text = (TextView) layout.findViewById(R.id.toastText);
text.setText("This is a custom toast");

Toast toast = new Toast(context);
toast.setGravity(Gravity.CENTER_VERTICAL, 0, 0);
toast.setDuration(Toast.LENGTH_LONG);
toast.setView(layout);
toast.show();

Toast.makeText(context, String.valueOf(level), Toast.LENGTH_SHORT)
        .show();

}
like image 33
buptcoder Avatar answered Oct 29 '22 15:10

buptcoder


If you want to stick with a static method use Activity instead of Context as parameter and do a activity.findViewById like so:

public static void displayLevelUp(int level, Activity activity) {
    LayoutInflater inflater = (LayoutInflater) activity.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
    View layout = inflater.inflate(R.layout.toastText, (ViewGroup) activity.findViewById(R.id.abs__action_bar_container));  // this row

Another way to do it is to pass the parent ViewGroup as parameter instead of a Context or Activity:

public static void displayLevelUp(int level, ViewGroup rootLayout) {
    View layout = rootLayout.inflate(rootLayout.getContext(), R.layout.custom_level_coast, rootLayout.findViewById(R.id.toast_layout_root));  // this row
like image 27
Emanuel Moecklin Avatar answered Oct 29 '22 16:10

Emanuel Moecklin