Thread: Homework help with echoserver and echoclient

This is a homework assignment so i don't want any code, just maybe a little hint as to what the heck i'm doing wrong. The assignment is simply to write a EchoClient and EchoServer where the server echos back everything the client sends (txt) until the client socket is closed. My code follows (please don't laugh, i'm beginner).

import java.io.*;
import java.net.*;
import java.lang.*;
 
public class EchoServer {
    public static void main (String[] args){
        try{
            ServerSocket sock;
            sock = new ServerSocket(8011);
            while(true) {
                Socket client=sock.accept();
                InputStream in=client.getInputStream();
                String line;
                BufferedReader bin=new BufferedReader(new InputStreamReader(in));
                do{
                    line=bin.readLine();
                    PrintWriter pout=new PrintWriter(client.getOutputStream(),true);
                    pout.println(line);
                } while(client.isClosed()!=true);
            sock.close();
            client.close();
            }
        }
        catch(IOException ioe) {
            System.err.println(ioe);
        }
    }
}

and the client code is:

import java.io.*;
import java.net.*;
import java.lang.*;
 
public class EchoClient {
    public static void main (String[] args){
        try{
            Socket s=new Socket("127.0.0.1",8011);
            InputStream in=s.getInputStream();
            BufferedReader buff=new BufferedReader(new InputStreamReader(in));
            String message="Hello there! Please work!";
            PrintWriter out=new PrintWriter(s.getOutputStream(),true);
            int i=0;
            while(i<4) {
                out.println(message);
                message=buff.readLine();
                i++;
            }
            s.close();
        }
        catch(IOException ioe) {
            System.err.println(ioe);
        }
 
   }
}

The while loop in client is just so I could make sure it would echo the multiple messages (and not just one)

I keep getting the following error: java.net.BindException: Address already in use

Thank you in advance. Any help is much appreciated.

Is that the entire error message?

this is all it says in the output window (using NetBeans IDE 7.4 on Ubuntu):
java.net.BindException: Address already in use
BUILD SUCCESSFUL (total time: 0 seconds)

Ok, sorry for the delay in response. There are a few things to look at.
1) You may need to manually close port 8011.

Server Code:
1) You only need to create the variable pout once.
2) Checking client.isClosed() will not give you what you expect it to. Instead, check to see if what is recieved from the client has a value.
3) One of the while loops is not needed.

Client Code:
1) Nothing technically wrong here, fix the server code and I think you will see what I mean.

Thank you very, very, very much. Your help is appreciated KucerakjM

I revised my code and am getting very frustrated because now I getting different error and can't seem to fix it. My revised code is below, any help is greatly appreciated.
Server Code:

public class EchoServer {
    public static void main (String[] args)throws IOException{
        try{
            ServerSocket server= new ServerSocket(3777);
            server.getLocalSocketAddress();
            Socket client=server.accept();
            InputStream in=client.getInputStream();
            PrintWriter out=new PrintWriter(client.getOutputStream(),true);
            BufferedReader bin=new BufferedReader(new InputStreamReader(in));
            String line;
            do{
                line=bin.readLine();
                out.println(line.toString());
            } while(line.isEmpty()==false);
            server.close();
            client.close();
        }
        catch(IOException ioe) {
            System.err.println(ioe);
        }
    }
}

and my client code:

public class EchoClient {
    public static void main (String[] args){
        try{
            Socket sock=new Socket("127.0.0.1",37777);
            PrintWriter out=new PrintWriter(sock.getOutputStream(),true);
            String message="Hello there!";
            out.println(message.toString());
            sock.close();
        }
        catch(IOException ioe) {
            System.err.println(ioe);
        }
 
   }
}

but now i keep getting this error message:
java.net.BindException: Address already in use
BUILD SUCCESSFUL (total time: 0 seconds)
and when i debug the debugger window shows this message:
Listening on 57950
User program running
User program finished

Thinking maybe it's something to do with the linux, I get even more frustrated because when i go on to my netbeans on windows machine and type in the EXACT SAME code on line 16 where my catch begins, it says <identifier> expected so it wont' run however my teacher will be running it on linux so thats what i would like it working on. PLEASE HELP

Thanks so much for all the help. Its finally working rite (at least its echoing back the clients first message), i'll post the server code below. My question is: at the end of my do-while loop where I say do ...until (line=inWhatever)!=null,does the readLine() return a null when it reads the first string? If so, how the heck can I keep the loop reading in the messages until the client stops sending? I tried testing using line.isEmpty (which didn't work). Am I going to have to make a exit String to which a match is the condition for breaking the loop?

public class EchoServer {
    public static void main (String[] args)throws IOException{
        try{
            ServerSocket server= new ServerSocket(65000);
            server.getLocalSocketAddress();
            Socket client=server.accept();
            InputStream in=client.getInputStream();
            PrintWriter out=new PrintWriter(client.getOutputStream(),true);
            BufferedReader bin=new BufferedReader(new InputStreamReader(in));
            String line;
            do{
                line=bin.readLine();
                out.println(line);
            } while((line=bin.readLine())!=null);
            server.close();
            client.close();
        }
        catch(IOException ioe) {
            System.err.println(ioe);
        }
    }
}

As far as I know, yes you will need to make an exit string if you want to indefinitely recieve data. So, unless you have a timeout on the connection the client has to "tell" the server that it is done sending or perhaps disconnecting.