Another One of Those "Symbol Not Found" Questions

[Mitchell Huo]

Hey guys, this is my first post, so if I break any secret programmer's forum codes, please let me know. Anyways, I'm making a server that waits for connections, gets responses from that client, and writes information through the sock to the client. I made four classes/threads total: Global, Troy, KeyListener, and PortListener. The first is just for global variables, the second listens for incoming connections and hands them off to the PortListener, which listens for responses, and the KeyListener listens for commands from the console.

However, I keep getting this pesky compiler error:

Troy.java:44: cannot find symbol
symbol  : constructor PortListener(java.net.Socket,int)
location: class PortListener
        new PortListener(sock, c).start();
        ^
Troy.java:69: cannot find symbol
symbol  : constructor KeyListener(Global)
location: class KeyListener
                new KeyListener(g).start();
                ^
Troy.java:71: cannot find symbol
symbol  : constructor Troy(java.net.Socket,int,Global)
location: class Troy
                        new Troy(server.accept(), counter, g).start();
                        ^

Any help would be sincerely appreciated. Oh, and here's my source:

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

class Global {

private String command = null;

public void Global() {


}

public void setCommand (String s) {

	command = s;

}

public String getCommand () {

	return command;

}

}


public class Troy extends Thread{

private static final int PORT = 666;
Socket sock = null;
private int c;
PrintWriter out = null;
Global g = null;
String currentCommand = null;

public void Troy(Socket s, int counter, Global global) {

	sock = s;
	c = counter;	
	out = new PrintWriter(sock.getOutputStream());
	g = global;
	new PortListener(sock, c).start();
}

 public void run() {

	while (true) {
		if ((currentCommand != g.getCommand()) && (currentCommand.charAt(0) == (char) c)) {
			currentCommand = g.getCommand();
			out.println(currentCommand);

		}

	}


}

static void main(String args[]) {

Global g = null;
int counter = 1;

	try{
		g = new Global();
		ServerSocket server = new ServerSocket(PORT);
		new KeyListener(g).start();
		while (true) {
			new Troy(server.accept(), counter, g).start();
			System.out.println("Got client " + counter + ".");
			counter++;
		}
	}
	catch(IOException e) {
		System.err.println(e);
	}



}

}

class PortListener extends Thread{

Socket sock = null;
private int c;
BufferedReader in = null;

public void PortListener(Socket s, int counter) {

	sock = s;
	c = counter;


}

public void run() {

	try {
		
	in = new BufferedReader(new InputStreamReader(sock.getInputStream()));
		while (true) {

			System.out.println(c+": " + in.readLine());

		}
	}
	catch (IOException e) {

	System.err.println(e);
	}
	

}

}

class KeyListener extends Thread{

private Global g = null;
BufferedReader in = null;

public void KeyListener(Global global) {


g = global;
in = new BufferedReader(new InputStreamReader(System.in));

}

public void run() {

	try{

	while(true) {

		g.setCommand(in.readLine());
	}

	}
	catch (IOException e) {
		System.err.println(e);
	}


}

}

[Wouter Oet]

public void Troy(Socket s, int counter, Global global) { 

This is not a constructor just a regular method. So you only have the default constructor.

[Greg Brannon]

I'm going to go out on a limb and say that ALL of the code you wrote intending to be constructors are methods instead, because they specify a return type. Constructors don't specify a return type, not even 'void.'

[Mitchell Huo]

Oh, thanks guys. Sorry, like I said before, I'm new to programming. And not to make excuses, but I started at 10 and finished and midnight. I think my brain was a little fried.

[Rob Spoor]

As a result, you get only the default constructors which have no parameters.