NPE在线程中引用对象时?

时间:2018-07-10 08:20:15

标签: java android thread-safety

我想使用套接字对象。看来这是使用它的“标准做法”,或者实际上是线程内与互联网相关的任何事情。

我想通过按下按钮在此套接字上发送一些数据,所以自然地我需要在我的onClick事件处理程序中引用此套接字对象。套接字将在mainActivity的onCreate方法中启动的线程中构造。这是代码:

package com.mypack.app.demo;

import java.io.BufferedWriter;
import java.io.IOException;
import java.io.OutputStreamWriter;
import java.io.PrintWriter;
import java.net.InetAddress;
import java.net.Socket;
import java.net.UnknownHostException;
import android.app.Activity;
import android.os.Bundle;
import android.view.View;
import android.widget.EditText;

public class MainActivity extends Activity {

    private Socket socket;
    private static final int SERVERPORT = 12345;
    private static final String SERVER_IP = "111.11.111.11";

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
        new Thread(new ClientThread()).start();
    }

    public void onClick(View view) {
        try {

            EditText et = (EditText) findViewById(R.id.EditText01);
            String str = et.getText().toString();
            PrintWriter out = new PrintWriter(new BufferedWriter(
                    new OutputStreamWriter(socket.getOutputStream())),
                        true);
            out.println(str);
            out.flush();
            out.close();
        } catch (UnknownHostException e) {
            e.printStackTrace();
        } catch (IOException e) {
            e.printStackTrace();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

    class ClientThread implements Runnable {

        @Override
        public void run() {
            try {
                InetAddress serverAddr = InetAddress.getByName(SERVER_IP);
                socket = new Socket(serverAddr, SERVERPORT);
            } catch (UnknownHostException e1) {
                e1.printStackTrace();
            } catch (IOException e1) {
                e1.printStackTrace();
            }
        }
    }
}

奇怪的是,您认为在调用onClick时,套接字将被实例化。但是我在logcat中收到以下错误消息:

  

07-09 19:31:42.911 2243-2243 / com.myapp.app.mydemo D / OpenGLRenderer:启用调试模式0   07-09 19:32:31.961 2243-2243 / com.myapp.app.mydemo W / System.err:java.lang.NullPointerException           在com.myapp.app.mydemo.MainActivity.onClick(MainActivity.java:62)

错误消息所指向的特定行是:

 new OutputStreamWriter(socket.getOutputStream()))

此错误非常严重,每当我单击屏幕上的按钮时就会发生。显然由于某些原因,“插座”物体在按下按钮之前无法存活足够长的时间。

该如何解决?有人建议我使用某种同步块或类似的东西。我是Java的新手,所以我应该怎么做?

1 个答案:

答案 0 :(得分:1)

您的代码有很多问题:

1)您无法在onClick中为套接字创建OutputWriter,因为您将在主线程上写入套接字。这将导致NetworkOnMainThreadError。

2)如果在创建套接字时发生任何异常,您将得到一个NPE,因为未编写套接字变量

3)套接字在这里需要是易失的,以便在两个线程上可靠地访问它。 (尽管您根本不应该在主线程上访问它。)

4)即使具有可变性,在竞争条件下,在套接字构造函数完成之前可以单击按钮,这也将使socket变量为null。