无法向所有客户端 Java 发送消息

Question

我正在使用一个服务器，它可以接收来自所有客户端的消息并将它们发送给其他人。我可以接收消息，但我无法发送给所有客户端，尽管我从所有客户端获得了正确的输出流。

这是我的代码 如果要测试，请 cmd -> telnet localhost 端口号

import java.io.*;
import java.util.*;
import java.net.*;

public class Server 
{

    private ServerSocket s;
    private Hashtable saveOutputStreams =  new Hashtable(500,80);

    //Constructor
    public Server(int port) throws IOException
    {
        listen(port);
    }
    //Listen method
    public void listen(int port) throws IOException
    {
        s = new ServerSocket(port);

        while(true)
        {
            Socket incoming = s.accept();

            OutputStream sout = incoming.getOutputStream();

            saveOutputStreams.put(incoming, sout);
            Runnable r = new ThreadHandler(this,incoming);
            Thread t = new Thread(r);
            t.start();
        }
    }
    // Create a table of streams to process in for loop of senToAll method
    Enumeration getOutputStreams()
    {
        return saveOutputStreams.elements();
    }
    // Send message to all clients
    public void sendToAll(String message)
    {
        synchronized(saveOutputStreams)
        {
            for(Enumeration e = getOutputStreams();e.hasMoreElements();)
            {

                OutputStream getOut = (OutputStream)e.nextElement();
                PrintWriter outp  = new PrintWriter(getOut);

                try
                {
                    outp.println("sent"+ message);
                    System.out.println("Stream: "+getOut);
                    System.out.println("PrinWriter "+outp);

                }
                catch(Exception ie)
                {
                    ie.printStackTrace();
                    System.out.println("Error");
                }

                finally
                {
                    System.out.println("done sen To All");
                }
            }
        }
    }
    // Main
    public static void main(String []args) throws IOException
    {
        new Server(8015);
    }
}

class ThreadHandler implements Runnable
{
    private Socket incoming;
    private Server serverP;

    public ThreadHandler(Server server, Socket socket)
    {
        serverP = server;
        incoming = socket;
    }

    public synchronized void run()
    {
        try
        {
            try
            {

                InputStream inStream = incoming.getInputStream();
                OutputStream outStream = incoming.getOutputStream();

                Scanner in = new Scanner(inStream);
                PrintWriter out = new PrintWriter(outStream,true);

                out.println("TungAnh'Server");
                String message = in.nextLine();
                serverP.sendToAll(message);

                out.println("receieve: "+message);
                out.println("done");
                System.out.println("current Stream: "+ outStream);
                System.out.println("PrinWriter "+ out);

            }
            finally
            {
                incoming.close();
            }
        }
        catch(IOException e)
        {
            e.printStackTrace();
        }
    }
}

Answer 1

我稍微改进了你的代码：

1. 奇怪的日志被移除了
2. 添加了实例管理
3. 添加了一些关闭的流
4. ...

如果你打算使用我的代码，你肯定应该看看同步（我的方法是同步的，但它们应该与相同的锁定对象/类同步）和异常处理。我无法在这个问题上花费更多时间，对此感到抱歉。

结果（对我有用）：

package test;

import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;
import java.io.PrintWriter;
import java.net.ServerSocket;
import java.net.Socket;
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

public class Server 
{

    private ServerSocket s;

    //Constructor
    public Server(int port) throws IOException
    {
        listen(port);
    }
    //Listen method
    public void listen(int port) throws IOException
    {
        s = new ServerSocket(port);

        while(true)
        {
            Socket incoming = s.accept();
            Runnable r = new ThreadHandler(this,incoming);
            Thread t = new Thread(r);
            t.start();
        }
    }

    // Main
    public static void main(String []args) throws IOException
    {
        new Server(8015);
    }
}

class ThreadHandler implements Runnable
{
    private Socket incoming;

    private static List<ThreadHandler> instances = new ArrayList<ThreadHandler>();

    public synchronized void addInstance() {
        instances.add(this);
    }

    public synchronized void removeInstance() {
        instances.remove(this);
    }

    public ThreadHandler(Server server, Socket socket)
    {
        incoming = socket;
        addInstance();
    }

    public synchronized void sendToAll(String msg, ThreadHandler me) {
        for (ThreadHandler h : instances) {
            if (h == me) continue;

            OutputStream outStream = null;
            PrintWriter print = null;
            try {
                outStream = h.incoming.getOutputStream();
                print = new PrintWriter(outStream);
                print.println(msg);
                print.flush();
            } catch (IOException e) {
                // TODO Auto-generated catch block
                e.printStackTrace();
            }
        }
    }

    public synchronized void run()
    {
        try
        {
            try
            {

                InputStream inStream = incoming.getInputStream();
                OutputStream outStream = incoming.getOutputStream();

                Scanner in = null;
                try {
                    in = new Scanner(inStream);

                    PrintWriter out = new PrintWriter(outStream,true);

                    out.println("Vojta's Server");

                    while (true) {
                        try {
                            String message = in.nextLine();
                            sendToAll(message, this);
                        } catch (Exception e) {

                        }

                    }


                } catch (Exception e) {

                }
                finally {
                    if (in != null) {
                        in.close();
                    }
                }
            }
            finally
            {
                incoming.close();
            }
        }
        catch(IOException e)
        {
            e.printStackTrace();
        } finally {
            removeInstance();
        }
    }
}