Help with this "ready-bake code", implementing Runnable

[Tarek Khojah]

import java.io.*;
import java.net.*;
import java.util.*;
 
public class Setup {
 ArrayList clientOutputStreams;
 
 public class ClientHandler implements Runnable {
     BufferedReader reader;
     Socket sock;
     
     public void clientHandler(Socket clientSocket) {
         try {
             sock = clientSocket;
             InputStreamReader isReader = new InputStreamReader(sock.getInputStream());
             reader = new BufferedReader(isReader);
             
        } catch (Exception e) {e.printStackTrace();}
    }
    
    public void run() {
        String message;
        try {
            while ((message = reader.readLine()) != null) {
                System.out.println("read " + message);
                tellEveryone(message);
            }
        } catch (Exception e) {e.printStackTrace();}
    }
 }
 
 public static void main(String[] args) {
     new Setup().go();
 }
 
 public void go() {
     clientOutputStreams = new ArrayList();
     try {
         ServerSocket serverSocket = new ServerSocket(5000);
         
         while(true) {
             Socket clientSocket = serverSocket.accept();
             PrintWriter writer = new PrintWriter(clientSocket.getOutputStream());
             clientOutputStreams.add(writer);
             
             Thread t = new Thread(new ClientHandler(clientSocket));
             t.start();
             system.out.println("Got a connection!");
         }
     } catch (Exception e) {e.printStackTrace();}
 }
 
 public void tellEveryone(String message) {
     Iterator it = clientOutputStreams.iterator();
     
     while (it.hasNext()) {
         try {
             Printwriter writer = (PrintWriter) it.next();
             writer.println(message);
             writer.flush();
        } catch (Exception e) {e.printStackTrace(); }
    }
 }
}    

The "ClientHandler" class implements Runnable, but when I create the thread (Thread t = new Thread(new ClientHandler(clientSocket)) I get an error saying there is no such constructor for ClientHandler, I looked into the API and it had nothing on Runnable's constructor(s).

Help please.

[Ulf Dittmer]

public class ClientHandler

public void clientHandler (Socket clientSocket)

Capitalization matters in Java. The second line of code is currently not a constructor for the class.

[Joanne Neal]

And if you do want it to be a constructor, you will need to get rid of the 'void', otherwise it will just be a method with the same name as the class.