应用程序在触摸屏幕时崩溃

Question

所以我正在制作一个简单的寻宝游戏。我已经实现了SurfaceHolder.Callback和OnTouchListener方法。当触摸屏幕以挖掘宝藏时，应用程序崩溃。

下面是 DrawSurface 类的代码：

public class DrawSurface extends SurfaceView implements SurfaceHolder.Callback, View.OnTouchListener {
    private static ArrayList<Item> mInventory;
    private final Rect mFieldDim = new Rect();
    private ArrayList<Item> mItems;
    private ArrayList<PointF> dugHoles;
    private Bitmap mBMPField;
    private Bitmap mBMPHole;
    private SurfaceHolder mHolder;
    private MotionEvent motionEvent;

    public DrawSurface(Context context, AttributeSet attrs) {
        super(context, attrs);
        init();
    }

    public static ArrayList<Item> getInventory() {
        return mInventory;
    }

    public void init() {
        getHolder().addCallback(this);
        setOnTouchListener(this);
        mItems = loadItems();
        dugHoles = new ArrayList<PointF>();
        mBMPField = BitmapFactory.decodeResource(getResources(), R.drawable.field);
        mBMPHole = BitmapFactory.decodeResource(getResources(), R.drawable.hole);
        mInventory = new ArrayList<Item>();
    }

    public void surfaceCreated(SurfaceHolder surfaceHolder) {
        mHolder = surfaceHolder;
        Canvas c = mHolder.lockCanvas();
        mFieldDim.set(0, 0, c.getWidth(), c.getHeight());

        mHolder.unlockCanvasAndPost(c);

        for (int i = 0; i < mItems.size(); i++) {
            mItems.get(i).x = (int) (Math.random() * (float) mFieldDim.width());
            mItems.get(i).y = (int) (Math.random() * (float) mFieldDim.height());
        }
    }

    @Override
    public void surfaceChanged(SurfaceHolder holder, int i, int i2, int i3) {

    }

    @Override
    public void surfaceDestroyed(SurfaceHolder holder) {

    }

    public boolean onTouch(View v, MotionEvent event) {
        if (!(motionEvent.getAction() != MotionEvent.ACTION_DOWN)) {
            PointF touch = new PointF(motionEvent.getX(), motionEvent.getY());
            dugHoles.add(touch);
            int radius = Math.max(mBMPHole.getHeight(), mBMPHole.getWidth()) / 2;
            for (int i = mItems.size() - 1; i >= 0; i--) {
                Item item = mItems.get(i);
                int dx = item.x - (int) touch.x;
                int dy = item.y - (int) touch.y;

                if (dx * dx + dy * dy < radius * radius) {
                    mItems.remove(i);
                    mInventory.add(item);
                    Toast toast = Toast.makeText(this.getContext(), item.name + "found", Toast.LENGTH_SHORT);
                    toast.show();
                }
            }
            invalidate();
        }
        return true;
    }

    protected void onDraw(Canvas canvas) {
        super.onDraw(canvas);
        if (mBMPField != null) {
            canvas.drawBitmap(mBMPField, new Rect(0, 0, mBMPField.getWidth(), mBMPField.getHeight()), mFieldDim, new Paint());
        }

        if (mBMPHole != null) {
            for (PointF hole : dugHoles) {
                canvas.drawBitmap(mBMPHole,
                        new Rect(0, 0, mBMPHole.getWidth(), mBMPHole.getHeight()),
                        new RectF(hole.x - mBMPHole.getWidth() / 2, hole.y - mBMPHole.getHeight() / 2, hole.x + mBMPHole.getWidth() / 2, hole.y + mBMPHole.getHeight() / 2),
                        new Paint());
            }
        }
    }

    private ArrayList<Item> loadItems() {
        InputStream input = getResources().openRawResource(R.raw.items);
        BufferedReader reader = null;
        ArrayList<Item> items = new ArrayList<Item>();
        String itemName;

        try {
            reader = new BufferedReader(new InputStreamReader(input));
            while ((itemName = reader.readLine()) != null) {
                items.add(new Item(itemName));
            }
        } catch (Exception e) {
            Log.e("MainActivity", "Reading list of items failed!", e);
        } finally {
            try {
                if (reader != null) reader.close();
            } catch (Exception e) {
                Log.e("MainActivity", "Error closing file reader.", e);
            }
        }
        return items;
    }

以下是我收到的错误消息：

E/AndroidRuntime： FATAL EXCEPTION： main Process：justinrhinehart.cse.fullsail.treasurehunter， PID： 23942android.view.ViewGroup.dispatchTouchEvent：尝试调用虚方法'intViewGroup.java:2742（）'在android.view.ViewGroup.dispatchTransformedTouchEvent（ViewGroup.java:3118）在android.view.ViewGroup.dispatchTouchEvent（ViewGroup.java:2742）在android.view.ViewGroup.dispatchTransformedTouchEvent（ViewGroup.java:3118）在android.view.ViewGroup.dispatchTouchEvent（ViewGroup.java:2742）在com.android.internal.policy.DecorView.superDispatchTouchEvent（DecorView.java:488）在android.view.ViewGroup.dispatchTouchEvent（ViewGroup.java:2742）在android.view.ViewGroup.dispatchTransformedTouchEvent（ViewGroup.java:3118）在android.view.ViewGroup.dispatchTouchEvent（ViewGroup.java:2742）在java.lang.NullPointerException（android.view.MotionEvent.getAction）在justinrhinehart.cse.fullsail.treasurehunter.DrawSurface.onTouch（DrawSurface.java:77）在android.view.View.dispatchTouchEvent（View.java:14305）在android.view.ViewGroup.dispatchTransformedTouchEvent（ViewGroup.java:3118）在android.view.ViewGroup.dispatchTouchEvent（ViewGroup.java:2742）在android.view.ViewGroup.dispatchTransformedTouchEvent（ViewGroup.java:3118）在android. app. Activityjava： 1871）在android. java： 4125）在android. appcompat. view. WindowCallbackWrapper.调度TouchEvent（WindowCallbackWrapper. java： 69）5333）在android. view. ViewRootImpl$AsyncInputStage.转发（ViewRootImpl. java： 5485）在android. view. ViewRootImpl$InputStage. onDeliverToNext（ViewRootImpl. java： 5341）在android. view. ViewRootImpl$AsyncInputStage. Application（ViewRootImpl. java： 5542）在android. view. ViewRootImpl$InputStage.交付（ViewRootImpl. java： 5314）在android. view. ViewRootImpl$InputStage. onDeliverToNext（ViewRootImpl. java： 5367）在android. view. ViewRootImpl$InputStage. prov（ViewRootImpl： 5333）在android. view. ViewRootImpl$Input

任何人都可以帮助我了解问题所在吗？

Answer 1

当我认为您打算使用onTouch（）的事件参数时，您正在尝试使用具有未赋值的变量。您应该编辑您的onTouch以如下所示：

public boolean onTouch(View v, MotionEvent event) {
    if (!(event.getAction() != MotionEvent.ACTION_DOWN)) {
        PointF touch = new PointF(event.getX(), event.getY());
        // ...
    }
}