1

Есть 2 класса.

Первый:

import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;

public class FirstClass extends AppCompatActivity
{
    SecondClass mSecondClass; //объявляется второй класс
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_first_class);
        Button button = (Button) findViewById(R.id.button); //кнопка и обработчик для неё
        button.setOnClickListener(new View.OnClickListener() 
        {
            @Override
            public void onClick(View v) 
            {
                mSecondClass.methodToast(); //вызывается метод из другого класса
            }
        });
    }
}

И второй:

import android.app.Activity;
import android.widget.Toast;

public class SecondClass extends Activity
{
    public void methodToast() {
        Toast.makeText(SecondClass.this, "Click", Toast.LENGTH_SHORT).show();
    }
}

При нажатии на кнопку возникает ошибка: java.lang.NullPointerException, а в LogCat указывается проблемный участок: mSecondClass.methodToast();. То есть возникают проблемы при вызове метода.

Подскажите, где я ошибся и как правильно следует вызывать методы из другого класса?

4 ответа 4

2

Второй класс не должен расширять класс активити. Если вам нужно в нём вызвать Toast, то вам надо передать туда Context текущей активити. Например через конструктор.

public class SecondClass
{
    Context ctx;

    public SecondClass(Context ctx) { this.ctx = ctx; }

    public void methodToast() {
        Toast.makeText(ctx, "Click", Toast.LENGTH_SHORT).show();
    }
}

Теперь в основной активити вам надо создать экземпляр класса, передать через конструктор Context и только потом вызывать метод.

mSecondClass = new SecondClass(FirstClass.this);
mSecondClass.methodToast();

либо модифицировать метод, задав ему Context в качестве аргумента:

public class SecondClass
{

    public SecondClass() {}

    public void methodToast(Context ctx) {
        Toast.makeText(ctx, "Click", Toast.LENGTH_SHORT).show();
    }
}

Вызывать надо тоже только после создания экземпляра класса

mSecondClass = new SecondClass();
mSecondClass.methodToast(FirstClass.this);

А ещё можно метод сделать статическим. тогда экземпляр класса создавать не придётся:

public class SecondClass
{    
    public static void methodToast(Context ctx) {
        Toast.makeText(ctx, "Click", Toast.LENGTH_SHORT).show();
    }
}

Вызвать можно будет так:

SecondClass.methodToast(FirstClass.this);
5
  • От всей души спасибо за разъяснения. А каким образом можно сделать так, чтобы при нажатии на кнопку "назад" (метод onBackPressed, вызываемый в первом классе) выполнялся метод из класса, который наследуется от ListFragment? То есть SecondClass наследуется не от Activity, а от ListFragment. Буду очень признателен за помощь.
    – Win_D
    Commented 10 янв. 2016 в 20:36
  • 1
    @Win_D, можно, например, через FragmentManager найти фрагмент и вызвать его метод. Если сразу не выйдет - создавайте новый вопрос.
    – ЮрийСПб
    Commented 10 янв. 2016 в 20:37
  • Боюсь, что не выйдет, т. к. у меня в ListFragment не используются другие фрагменты (только ArrayAdapter, которые меняются при нажатии на пункт). В любом случае, спасибо за помощь, попробую самостоятельно по изучать.
    – Win_D
    Commented 10 янв. 2016 в 20:41
  • 1
    Ваш ListFragment точно можно найти через мэнеджер. Что в нём происходит - не важно.
    – ЮрийСПб
    Commented 10 янв. 2016 в 20:42
  • Пожалуйста, взгляните на этот вопрос: ru.stackoverflow.com/questions/481722/…
    – Win_D
    Commented 11 янв. 2016 в 16:44
2

Как ответил ЮрийСПб, У вас во втором классе не хватает контекста, поэтому необходимо создать обьект второго класса и передать ему в конструктор контекст первого класса. Тогда это не будет вызывать исключение. Надеюсь мой совет поможет Вам.

Это можно сделать примерно так

В первом классе в обработчике нажатия кнопки пишем

 @Override
 public void onClick(View v) {
      new SecondClass(FirstClass.this).methodToast(); // здесь создается сразу обьект второго класса и вызывается метод из другого класса
}

во втором классе создаете конструктор с контекстом,

0

У вас не инициализирован mSecondClass. Например:

SomeClass someClass = new SomeClass();
someClass.doSomething();
1
  • Не сработает. точнее решит текущую ошибку, но приведёт к другой из-за нужды Toast.makeText в Context, коего не будет в классе-активити, коя не отображается.
    – ЮрийСПб
    Commented 10 янв. 2016 в 19:44
0

Я так понимаю, вам нужно создать объект класса.

SecondClass mSecondClass = new SecondClass ();  

И создать конструктор во втором классе

SecondClass() {}
1
  • Не сработает. точнее решит текущую ошибку, но приведёт к другой из-за нужды Toast.makeText в Context, коего не будет в классе-активити, коя не отображается.
    – ЮрийСПб
    Commented 10 янв. 2016 в 19:43

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.