1

Ошибка:

FATAL EXCEPTION: main Process: com.example.sapeg.chess, PID: 21296 java.lang.NullPointerException: Attempt to invoke virtual method 'void android.widget.Button.layout(int, int, int, int)' on a null object reference

Куски кода:

public class chess extends Activity {
     Button mB, bt;
LinearLayout layout;
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(new DrawView(this));
....
        class DrawView extends View {
        Paint p, paint;
        Rect rect;
    }

        public DrawView(Context context) {
            super(context);
            p = new Paint();
            rect = new Rect();
        }
.....
      @Override
            protected void onDraw(Canvas canvas) {
                 Button mButton,b;
                b = (Button) findViewById(R.id.button1);
                canvas.drawColor(Color.GREEN);
               // invalidate();

                b.layout(11, 11, 11, 11);
                b.draw(canvas);
    //далее идёт прорисовка доски

main.xml:


    <?xml version="1.0" encoding="utf-8"?>
    <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:orientation="vertical" android:layout_width="match_parent"
    android:layout_height="match_parent">

    <Button
        android:text="Button"
        android:layout_width="70dp"
        android:layout_height="wrap_content"
        android:id="@+id/button1" />
    </LinearLayout>
2
  • ....обновил....Компилирует нормально, но при запуске выдаёт ошибку.
    – sapeg
    20 фев 2017 в 3:50
  • Вы не можете получить объект из разметки методом findViewById() в методе onDraw(). В Java имена классов принято писать с Большой буквы
    – pavlofff
    20 фев 2017 в 4:21

2 ответа 2

2

Все View в Android ищутся в методе onCreate.

Попробуйте так:

public class chess extends Activity {
Button mB, bt, mButton,b;
LinearLayout layout;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.main);

    b = (Button) findViewById(R.id.button1);
....

@Override
    protected void onDraw(Canvas canvas) {

        canvas.drawColor(Color.GREEN);
       // invalidate();

        b.layout(11, 11, 11, 11);
        b.draw(canvas);

Право говоря у меня нет перегрузки метода onDraw в классе-потомке Activity...

10
  • 1
    На самом деле View через findViewById() можно получить в любом методе активити, а не только onCreate(). Метод onDraw() не принадлежит активити, поэтому там и нельзя получить объект кнопки из разметки через findViewById()
    – pavlofff
    20 фев 2017 в 4:47
  • Согласен, просто onCreate() вызывается первым.
    – GHosT
    20 фев 2017 в 4:55
  • Если я собираюсь отобразить просто кнопку в Activity то, таким образом она отображается конечно, но самое главное, как на этом же экране и canvas отобразить? Либо можно выделить область, например прямоугольником и при "тап(е)" по его области перейти к другой активити?
    – sapeg
    20 фев 2017 в 18:09
  • @sapeg попробуйте после изменения View методом View.draw() вызвать метод View.requestLayout(). Учтите также, что метод View.draw() должен вызываться только после полной загрузки View и всех ее дочерних элементов.
    – GHosT
    21 фев 2017 в 18:56
  • @sapeg Если Вам необходимо лишь сменить background кнопки, то Вы можете использовать для этого drawable ресурсы. Там Вы можете описать любое оформление кнопки.
    – GHosT
    21 фев 2017 в 19:05
2

А вообще, в данной ситуации, правильнее будет создать свое View расширяющее класс Button и в onDraw() описать все необходимые преобразования, например так:

public class MyButtonClass extends Button {
    public MyButtonClass(Context context) {
        super(context);

        // Тут можно задать Вашей View параметры по умолчанию, например:
        this.setText("My Green Button...");
    }

    @Override
    protected void onDraw(Canvas canvas) {
        canvas.drawColor(Color.GREEN);
        // Тут описываем все, что хотим сделать с View

        super.onDraw(canvas);
    }
}

В методе onCreate(), в свою очередь, пишем следующее:

protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.main);

    LinearLayout grouplayout = (LinearLayout) findViewById(R.id.grouplayout);
    MyButtonClass myBtn = new MyButtonClass(this);
    grouplayout.addView(myBtn);
}
15
  • Всё это очень интересно, но кнопка в моём случае не самое главное, а "камень преткновения". Идея в том, что я нарисовал на канве шахматную доску и мне нужна ещё и кнопка к которой я собираюсь прикрутить обработчик событий, но кнопку на канве пока не получается вывести.
    – sapeg
    22 фев 2017 в 5:53
  • То, что я написал, реализовать совсем не сложно, и это работает. Попробуйте — надо лишь скопировать код.
    – GHosT
    22 фев 2017 в 7:44
  • Ok. Буду у себя попробую.
    – sapeg
    22 фев 2017 в 16:17
  • Если можно поподробнее: что есть R.id.grouplayout. Ваше предложение я реализовал, но кроме кнопки, шахматную доску на канве не выводит.
    – sapeg
    24 фев 2017 в 14:47
  • @sapeg R.id.grouplayout — это ID родительского (parent) слоя, наследуемого от android.view.ViewGroup (например, LinearLayout, RelativeLayout), в котором необходимо расположить ваше View (в данном случае Button). Опишите, пожалуйста, как у Вас реализована шахматная доска.
    – GHosT
    24 фев 2017 в 15:18

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

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