1

I am continually getting an error when trying to run an android app telling me that there is a null object reference. I am still a beginner with android and java and I would greatly appreciate any help I can get.

Here is the logcat:

01-17 17:11:49.156  27778-27778/com.example.owner.sketchy D/AndroidRuntime﹕ Shutting down VM
    --------- beginning of crash
01-17 17:11:49.157  27778-27778/com.example.owner.sketchy E/AndroidRuntime﹕ FATAL EXCEPTION: main
    Process: com.example.owner.sketchy, PID: 27778
    java.lang.RuntimeException: Unable to start activity ComponentInfo{com.example.owner.sketchy/com.example.owner.sketchy.MainActivity}: java.lang.NullPointerException: Attempt to invoke virtual method 'android.view.View android.widget.LinearLayout.getChildAt(int)' on a null object reference
            at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2298)
            at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2360)
            at android.app.ActivityThread.access$800(ActivityThread.java:144)
            at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1278)
            at android.os.Handler.dispatchMessage(Handler.java:102)
            at android.os.Looper.loop(Looper.java:135)
            at android.app.ActivityThread.main(ActivityThread.java:5221)
            at java.lang.reflect.Method.invoke(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:372)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:899)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:694)
     Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'android.view.View android.widget.LinearLayout.getChildAt(int)' on a null object reference
            at com.example.owner.sketchy.MainActivity.onCreate(MainActivity.java:33)
            at android.app.Activity.performCreate(Activity.java:5933)
            at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1105)
            at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2251)
            at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2360)
            at android.app.ActivityThread.access$800(ActivityThread.java:144)
            at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1278)
            at android.os.Handler.dispatchMessage(Handler.java:102)
            at android.os.Looper.loop(Looper.java:135)
            at android.app.ActivityThread.main(ActivityThread.java:5221)
            at java.lang.reflect.Method.invoke(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:372)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:899)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:694)

and here is my MainActivity.java:

    package com.example.owner.sketchy;

import android.support.v7.app.ActionBarActivity;
import android.os.Bundle;
import android.view.Menu;
import android.view.MenuItem;
import android.view.View;
import android.widget.ImageButton;
import android.widget.LinearLayout;

public class MainActivity extends ActionBarActivity {


    private ImageButton currPaint;
    private DrawingView drawView;

    public void paintClicked(View view) {
        if (view != currPaint) {

            ImageButton imgView = (ImageButton) view;
            String color = view.getTag().toString();
            imgView.setImageDrawable(getResources().getDrawable(R.drawable.paint_pressed));
            currPaint.setImageDrawable(getResources().getDrawable(R.drawable.paint));
            currPaint = (ImageButton) view;
            drawView.setColor(color);
        }
    }

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        drawView = (DrawingView)findViewById(R.id.drawing);
        LinearLayout paintLayout = (LinearLayout)findViewById(R.id.paint_colors);
        currPaint = (ImageButton)paintLayout.getChildAt(0);
    }
    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        // Inflate the menu; this adds items to the action bar if it is present.
        getMenuInflater().inflate(R.menu.menu_main, menu);
        return true;
    }

    @Override
    public boolean onOptionsItemSelected(MenuItem item) {
        // Handle action bar item clicks here. The action bar will
        // automatically handle clicks on the Home/Up button, so long
        // as you specify a parent activity in AndroidManifest.xml.
        int id = item.getItemId();

        //noinspection SimplifiableIfStatement
        if (id == R.id.action_settings) {
            return true;
        }

        return super.onOptionsItemSelected(item);
    }

}
Dave Schweisguth
  • 36,475
  • 10
  • 98
  • 121
Caedon Bates
  • 23
  • 1
  • 1
  • 3

2 Answers2

3

Your paintLayout is null, and your problem is onCreate(). When overriding methods from a super class, you should always (most of the times) call the method from that class using super.method(), to allow normal processing.

Also, you are missing a call to setContentView(R.layout.your_layout) . So don't forget these two lines:

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

    drawView = (DrawingView)findViewById(R.id.drawing);
    LinearLayout paintLayout = (LinearLayout)findViewById(R.id.paint_colors);
    currPaint = (ImageButton)paintLayout.getChildAt(0);
}
natario
  • 24,954
  • 17
  • 88
  • 158
  • Thank you so much! I was extremely frustrated because I was unable to figure out the problem but this worked! I am extremely thankful and happy to have learned little more about java/android! – Caedon Bates Jan 18 '15 at 00:28
1

Your onCreate() method has two flaws:

  1. It does not chain to the superclass with super.onCreate(), which is typically the very first line in the onCreate() method.

  2. It does not do anything to create a user interface, such as calling setContentView().

You are crashing because findViewById() is returning null, because you have no widgets yet.

CommonsWare
  • 986,068
  • 189
  • 2,389
  • 2,491