Showing toasts from a non-activity class. Ploblem setting the context to the toast

Alex picture Alex · Jun 12, 2015 · Viewed 8.6k times · Source

Spoiler: this post may contain something stupid due to being used to C and new to Java programming

There is an activity MainActivity and a public non-activity class containing a number of methods. I need to show toast alerts for some of them

Current attempts are something like this, which fails with "Non-static method can not be referenced from a static context" for getApplicationContext():

void errorWarn (String warning) {
    Context context = android.content.ContextWrapper.getApplicationContext();
    Toast.makeText(context, "Something's wrong in " + warning, Toast.LENGTH_SHORT).show();
}

So, how to call toasts from a non-activity class?

UPD: The errorWarn is to be called from the methods in the class. So, if an error in a method of the class occurs, there should be an alert

We are in the MainActivity having an editText field. The class should get and parse the string from it. If on some step processing fails, it shows a toast in the MainActivity

UPD2: The full structure.

MainActivity:

public class MainActivity extends ActionBarActivity {
    <...>
    public void ButtonClick (View view) {
        Class.testfunc("");
    }
}

Class:

public class Class {
    void errorWarn (Context context, String warning) {
        Toast.makeText(context, "Something must be wrong. " + warning, Toast.LENGTH_SHORT).show();
    }
    void testfunc (String string) {
        errorWarn(string);
    }
}

Answer

2Dee picture 2Dee · Jun 12, 2015

Define your method so that it takes a Context in argument and pass your Activity to it.

in YourOtherClass

public class YourOtherClass {

    public void showToast(Context context, String message){
        Toast.makeText(context, message, Toast.LENGTH_SHORT).show();
    }
}

or if you want to take a Context in your constructor and use showToast only in YourOtherClass :

public class YourOtherClass {

    private Context context;

    public YourOtherClass(Context context){
        this.context = context;
    }

    private void showToast(String message){
        Toast.makeText(context, message, Toast.LENGTH_SHORT).show();
    }
}

in MainActivity

new YourOtherClass().showToast(this, message);

or if you have a Context member variable of YourOtherClass and you want to pass the Context in the constructor of YourOtherClass, you would do

new YourOtherClass(this).showToast(message);
// showToast doesn't have to take a Context as argument, it could just take one as constructor parameter and hold that. 
// But then you have to make sure YourOtherClass is not used anymore if the Activity is closed.

And for the errors you got in the code your provided :

Context context = com.example.ex3.MainActivity;

This fails because you are trying to assign a type to an instance.

MainActivity.errorWarn("here");

This fails because you are calling a non-static method (the method has no static modifier in its signature) as if it was a static one. Have a look at this question for more details on static vs non-static methods.

Without knowing what YourOtherClass does or how its lifecycle is linked to the Activity's, it's hard to say, but having to touch UI from a class that is not UI related and doesn't have any reference to Context your can use, that feels odd. Taking a Context as argument of the constructor of YourOtherClass is probably what you need, but be wary of leaking Context and Activity lifecycle.