sockets, streams, and confusion

[Adam Confino]

Hey Java Gurus,

I wrote a simple chat program. Now I am trying to modify the server so that will chat on one port and transfer a file on another. When I run this class, the writeFile() method does not diplay any of its System.out.println() messages which makes me believe it is not being called. Thanks for any help you can give. I really do appreciate it.

Adam

    

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

public class Server {
	private ServerSocket ss;
	private ServerSocket ss2;
	private Hashtable outputStreams = new Hashtable();
	
	static public void main(String args[]) throws Exception {
		int port = Integer.parseInt(args[0]);
		int port2 = Integer.parseInt(args[1]);
		new Server(port, port2);
	}

	public Server(int port, int port2) throws IOException {
		listen(port);
		writeFile(port2);
	}

	private void listen(int port) throws IOException {
		ss = new ServerSocket(port);
		System.out.println("Listening on " + ss + " and ready to chat");
		while (true) {
			Socket s = ss.accept();
			System.out.println("Connection from " + s);

			DataOutputStream dout = new DataOutputStream(s.getOutputStream());
			outputStreams.put(s, dout);

			new ServerThread(this, s);
		}
	}

	private void writeFile(int port2) throws IOException {
		System.out.println("Waiting...");
		ss2 = new ServerSocket(port2);
		System.out.println("Listening on " + ss2 + " and ready to write files");
		while (true) {
			Socket s2 = ss2.accept();
			System.out.println("Connection from " + s2);

			// reading from a socket to a file
			InputStream in = s2.getInputStream();
			FileOutputStream out = new FileOutputStream("Test.dat");

			int c;
			while ((c = in.read()) != -1) {
				out.write(c);

			}
		}
	}

	Enumeration getOutputStreams() {
		return outputStreams.elements();
	}

	void sendToAll(String message) {
		synchronized (outputStreams) {
			for (Enumeration e = getOutputStreams(); e.hasMoreElements();) {

				DataOutputStream dout = (DataOutputStream) e.nextElement();
				try {
					dout.writeUTF(message);
				} catch (IOException ie) {
					System.out.println(ie);
				}
			}
		}
	}

	void removeConnection(Socket s) {

		synchronized (outputStreams) {
			System.out.println("Removing connection to " + s);
			outputStreams.remove(s);
			try {
				s.close();
			} catch (IOException ie) {
				System.out.println("Error closing " + s);
				ie.printStackTrace();
			}
		}
	}

	
}

[David Kilcy]

In the constructor you call both listen() and writeFile(). First listen() is invoked from the constructor, but it never leaves because you have a while(true) loop in there.
So it never breaks out of the constructor.
Hope that helps.