Java Android socket客户端-can'；不要发送消息

首先，我对Android、Java和Stacks Overflow是一个完全的新手，所以如果我做错了什么，请道歉

我正在尝试为一个使用套接字的项目创建一个简单的应用程序。我想在安卓设备上设置一个客户端，向运行在PC上的服务器发送命令。我已经用Python编写了我的服务器，并对其进行了测试，所以我知道项目的这一方面是可行的。我在安卓应用程序的实现上遇到了问题

我按照以下指南编写了我的客户端应用程序：

当我运行我的应用程序时，我的服务器会确认客户端已连接到它。但是，当我尝试发送数据时，服务器没有收到任何信息。这就是我的代码的样子（与上面链接中的代码几乎相同）：

MainActivity.java

package com.example.user.sockettest;

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 = 8000;
    private static final String SERVER_IP = "192.168.0.17";

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        new Thread(new ClientThread()).start();
    }

    public void onClick(View view) {
        try {
            EditText et = (EditText) findViewById(R.id.editText);
            String str = et.getText().toString();
            PrintWriter out = new PrintWriter(new BufferedWriter(
                    new OutputStreamWriter(socket.getOutputStream())),
                    true);
            out.println(str);
        } 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();
            }

        }

    }
}

活动\u main.xml

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

        <EditText
            android:id="@+id/editText"
            android:layout_width="fill_parent"
            android:layout_height="wrap_content"
            android:text="TestData" >
        </EditText>

        <Button
            android:id="@+id/myButton"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:onClick="onClick"
            android:text="Send" >
        </Button>

    </LinearLayout>

<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package="com.example.user.sockettest">

    android:versionCode="1"
    android:versionName="1.0" >

    <uses-sdk
        android:minSdkVersion="8"
        android:targetSdkVersion="17" />

    <uses-permission android:name="android.permission.INTERNET" >
    </uses-permission>

    <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" >
    </uses-permission>

    <uses-permission android:name="android.permission.ACCESS_WIFI_STATE" />

    <application
        android:allowBackup="true"
        android:icon="@mipmap/ic_launcher"
        android:label="@string/app_name"
        android:theme="@style/AppTheme" >

        <activity android:name="com.example.user.sockettest.MainActivity">
            <intent-filter>
                <action android:name="android.intent.action.MAIN" />

                <category android:name="android.intent.category.LAUNCHER" />
            </intent-filter>
        </activity>
    </application>

</manifest>

---

我很惊讶你没有撞车。单击处理程序在主线程上运行。主线程无法执行网络IO-它应该抛出一个异常以重试。BufferedWriter可能会将您保存在那里—如果消息很小，它可能不会写入实际的套接字并缓冲所有内容，因为您没有刷新流

---

因此，您需要做的第一件事是在另一个线程（可能是您的ClientThread）上执行写操作，并使用某种形式的消息传递向它发送写命令。第二件事是简化输出编写器-您真的不需要将其封装在任何东西中，只需直接写入输出流即可使用。

BufferedWriter的默认缓冲区大小似乎是8192

char

s，因此您的猜测似乎是有可能的。感谢您的回答Gabe！我实现了我对你答案的解释，并且它是有效的（我对这一点还不熟悉，所以我不知道我的解释是否准确）！我已经编辑了我的原始帖子以包含工作代码。我不确定我是否打破了任何最佳实践（特别是无限循环检查要发送的消息）。线程上的无限循环相当常见。它还应该检查isInterrupted，这样如果线程被告知结束，它就可以自行退出。你也不应该像那样忙着等待——你应该等待消息队列或循环器之类的东西。这样，操作系统可以暂停你的应用程序，并将核心用于其他可能需要CPU时间的进程。您还应该在线程末尾的finally{}块中关闭套接字，以正确关闭套接字，并在活动的onDestroy（）中中断线程，以便正确取消所有线程并释放资源。感谢这些指针，非常感谢。我将尝试在我的项目中实现这些技术！

public class MainActivity extends Activity {


    private Socket socket;
    boolean sendData = false;   // Flag to indicate whether data should be sent
    String message;

    private static final int SERVERPORT = 8000;
    private static final String SERVER_IP = "192.168.0.17";

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        new Thread(new ClientThread()).start();
    }

    public void onClick(View view) {
        EditText et = (EditText) findViewById(R.id.editText);
        message = et.getText().toString();
        sendData = true;
    }

    class ClientThread implements Runnable {

        @Override
        public void run() {
            try {
                InetAddress serverAddr = InetAddress.getByName(SERVER_IP);

                socket = new Socket(serverAddr, SERVERPORT);
                BufferedReader in = new BufferedReader(new InputStreamReader(socket.getInputStream()));

                // Loop forever checking if a message needs to be send
                while (socket.isConnected()){
                    if (sendData){
                        // If a message is ready to be sent, call sendMessage method
                        sendMessage();
                    }
                }
            } catch (UnknownHostException e1) {
                e1.printStackTrace();
            } catch (IOException e1) {
                e1.printStackTrace();
            }
        }

        private void sendMessage(){
            // Send message over socket
            try {
                PrintWriter out = new PrintWriter(socket.getOutputStream());
                out.println(message);
                out.flush();
                // Reset flag
                sendData=false;
            } catch (UnknownHostException e) {
                e.printStackTrace();
            } catch (IOException e) {
                e.printStackTrace();
            } catch (Exception e) {
                e.printStackTrace();
            }
        }
    }
}