1

我的 Java 套接字代码有一点问题。我正在编写一个 android 客户端应用程序，它通过直接（！）无线连接将数据发送到我的 pc 上的 java 多线程套接字服务器。它工作正常，但我想为移动应用程序改进它，因为它现在非常耗电。当我在我的代码中删除两个特殊行时，我的移动设备（htc one x）的 cpu 使用率完全没问题，但是我的连接似乎有很高的 ping 率或类似的东西......

这是我接收客户端数据的服务器代码片段：

while(true)
  {
    try {
                   ....
            Object obj = in.readObject();
            if(obj != null) {
                Class clazz = obj.getClass();
                String className = clazz.getName();
                if(className.equals("java.lang.String")) {
                    String cmd = (String)obj;
                    if(cmd.equals("dc")) {
                        System.out.println("Client "+id+" disconnected!");
                        Server.connectedClients[id-1] = false;
                        break;
                    }
                    if(cmd.substring(0,1).equals("!")) {
                        robot.keyRelease(PlayerEnum.getKey(cmd,id));
                    }
                    else {
                        robot.keyPress(PlayerEnum.getKey(cmd,id));
                    }

                }
            }

    } catch ....

这是客户端部分，我在其中循环发送我的数据：

private void networking() {
    try {
        if(client != null) {
            ....
                out.writeObject(sendQueue.poll());

            ....
        }
    } catch ....

当我写这个为什么时，每次执行while循环时我都会发送数据..当sendQueue为空时，将发送一个空的“对象”。这会导致“高”网络流量和“高”cpu 使用率。但是：几乎立即收到所有发送评论。

当我将代码更改为以下内容时：

while(true)
...
if(sendQueue.peek() != null) {
    out.writeObject(sendQueue.poll());
}
...

cpu 使用率完全没问题，但我有一些滞后 .. 命令到达的速度不够快.. 正如我所说，如果我每隔一段时间发送数据（使用空对象），它工作正常（除了 cpu 使用率）执行。但我确信这是一种非常粗糙的编码风格，因为我有点淹没网络。任何提示？

我究竟做错了什么？？

感谢您的帮助！

真诚的，maaft

4

3 回答 3

1

代码的 CPU 密集型版本正在用null值淹没输出流。它们算作要传输的数据。尽管您的服务器明确忽略了它们，但它们也有助于最终强制任何有用的数据通过。

您修改后的代码peek更合理。调用flushafter 是一种很好的形式writeObject。否则，写入的对象可能会卡在输出缓冲区中并等待更多项目的到来。缓冲是针对多个对象一起发送的情况的性能优化。不进行缓冲的流类不需要刷新。

更好的是：

Object item = sendQueue.poll();
if (item != null) {
    out.writeObject(item);
    out.flush();  // maybe not needed, depending on the class of your stream 
}

这稍微快一点；peek无论如何，如果您打算poll立即进行评估，则毫无意义。

此外，socket.setTcpNoDelay(true)在将套接字传递给之前调用套接字SocketOutputStream（假设这是您创建输出流的方式）。这会禁用Nagle 算法，这可能不一定是节省网络带宽的最佳决策，但它是一种快速检查方法，除了调整 TCP 发送/接收缓冲区之外，您的客户端和服务器是否正常工作。如果您与服务器有直接连接，我根本不用担心禁用 Nagle 算法。

于 2012-06-16T21:57:42.747 回答
0

您应该使用阻塞队列以便 poll() 阻塞，而不是返回 null。发送空值根本没有意义，这只是浪费每个人的时间、带宽和金钱。

于 2012-06-17T21:40:54.490 回答
0

进一步说明，您可能需要查看适用于 Android 的 ARO 工具，该工具可帮助您优化应用程序，包括网络使用情况。http://developer.att.com/developer/legalAgreementPage.jsp?passedItemId=9700312

于 2012-06-26T14:22:24.513 回答