Java 在线程中引用对象时的NPE?

Java 在线程中引用对象时的NPE?,java,android,thread-safety,Java,Android,Thread Safety,我想使用套接字对象。这似乎是一个使用它的“标准实践”,或者是一个线程中任何与互联网相关的东西 我想通过按下按钮在这个套接字上发送一些数据,所以我自然需要在onClick事件处理程序中引用这个套接字对象。套接字将在mainActivity的onCreate方法中启动的线程中构造。代码如下: package com.mypack.app.demo; import java.io.BufferedWriter; import java.io.IOException; import java.io.O

我想使用套接字对象。这似乎是一个使用它的“标准实践”,或者是一个线程中任何与互联网相关的东西

我想通过按下按钮在这个套接字上发送一些数据,所以我自然需要在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中得到了以下错误msgs:

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)

error msg引用的特定行是:

 new OutputStreamWriter(socket.getOutputStream()))
这个错误非常持久,每当我点击屏幕上的按钮时就会发生。显然,由于某种原因,“套接字”对象在我按下按钮之前不会存在足够长的时间


我该如何解决这个问题?有人建议我使用某种同步块之类的东西。我对java有点陌生,那么我到底应该怎么做呢?

您的代码有很多问题:

1) 您不能在onClick中向套接字创建OutputWriter,因为您将在主线程上向套接字写入。这将导致NetworkOnMainThread错误

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

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


4) 即使是易变的,也存在一种竞争条件,即在套接字构造函数完成之前可以单击按钮,这也会允许套接字变量为null。

如果在套接字初始化之前触发
onClick
(这在多线程环境中肯定会发生)你现在所感觉到的一切都会发生。您可能需要添加一个
if(socket==null)
检查以防止出现这种情况。您好,谢谢Ben。我理解空检查的优点,但我只是觉得它难以置信,它应该花这么长的时间为一个套接字初始化?如果是这样的话,我认为可以安全地假设有什么东西阻止了它,在这种情况下,空检查实际上没有什么作用?