Thread: Socket closed Exception being thrown.. please help

I have been working with socket programming. I was trying to develop a two way communication from server to client. The data is being received when sent from server to client but there is a problem when I am sending from client to server.

I am getting an error as "Socket is closed" pointing to the line where I have created a BufferedReader. I have simplified some part of my code to be very specific.

Kindly find the code attached.

import java.io.*;
import java.net.*;
import java.util.*;
 
public class Server
{
 
   public static void main(String args[])
   {
     String id="policeone";
     int hashcodeid=0;
     try
        {
           ServerSocket srvr=new ServerSocket(1234);
           Socket client=srvr.accept();
           System.out.println("Connected to a client");
 
           PrintWriter out=new PrintWriter(client.getOutputStream(),true);
 
         // code to perfrom operations..
         out.print(id+" ");
         out.print(hashcodeid+" ");
         //out.print(x);
         out.print(message1+" ");
 
 
         //sending data to client working... no problem.
 
 
 
 
// reading data form client... throwing exception.. "Socket is closed"   
 
 BufferedReader fromserver=new BufferedReader(new InputStreamReader(client.getInputStream()));
            while(!fromserver.ready()){}
 
String nowstring=fromserver.readLine();
System.out.println("Received from client    "+nowstring);
 
 
         }
 
catch(Exception e)
{
  e.printStackTrace();
}
    }
}

import java.io.*;
import java.net.*;
import java.util.*;
 
public class Client
{
public static void main(String args[])
{
try{
        String id="vehicle1";
        String idstring,hash,allstring;
        int hashcodeid2=0;
         Socket skt = new Socket("localhost", 1234);
         BufferedReader in = new BufferedReader(new
         InputStreamReader(skt.getInputStream()));
 
 
 
         System.out.print("Received string: '");
 
       while (!in.ready()) {}
 
       //code to read and manipulate data from server
 
 
   BufferedReader br=new BufferedReader(new InputStreamReader(System.in));
 
 
PrintWriter toserver=new PrintWriter(skt.getOutputStream(),true);
 
 
toserver.print("helo");
 
//no problem in client.. server throwing exception at this point.   
 
   }
catch(Exception e)
{
  e.printStackTrace();
 }
}
}

error::

Receiving string from client: 'java.net.SocketException: Socket is closed
at java.net.Socket.getInputStream(Socket.java:872)
at Server.main(Server.java:51)

plz help me sir....

This is original code...

import java.io.*;
import java.net.*;
import java.util.*;
 
public class Client
{
public static void main(String args[])
{
try{
        String id="vehicle1";
        String idstring,hash,allstring;
        int hashcodeid2=0;
         Socket skt = new Socket("localhost", 1234);
         BufferedReader in = new BufferedReader(new
         InputStreamReader(skt.getInputStream()));
 
 
 
         System.out.print("Received string: '");
 
       while (!in.ready()) {}
 
          allstring=in.readLine();
      System.out.println("Received               "+allstring); 
     String delims=new String(" ");
     String tokens[]=allstring.split(delims);
 
       for(int i=0;i<tokens.length;i++)
     System.out.println("\nToken number   "+i+1+":"+tokens[i]);  
    int hashid=Integer.parseInt(tokens[1]);
    int message1=Integer.parseInt(tokens[2]);
   System.out.println("Random Number:"+ (hashid^message1));
 
    System.out.println("\n  client sending::");
 
 
 
    in.close();
   }
catch(Exception e)
{
  e.printStackTrace();
 }
}
}

server

import java.io.*;
import java.net.*;
import java.util.*;
 
public class Server
{
 
   public static void main(String args[])
   {
     String id="policeone";
     int hashcodeid=0;
     try
        {
           ServerSocket srvr=new ServerSocket(1234);
           Socket client=srvr.accept();
           System.out.println("Connected to a client");
 
           PrintWriter out=new PrintWriter(client.getOutputStream(),true);
 
           for(int i=0;i<id.length();i++) 
	{
         		hashcodeid = 31*hashcodeid + id.charAt(i);
	}
               Random random = new Random ();
         int x = 100 + (int)(Math.random()*200);
 
         int message1=hashcodeid^x;
 
         System.out.println("Sending  id string: '" +id  + "'\n");
         System.out.println("Sending  hashcode of id  : '" +hashcodeid  + "'\n");
         System.out.println("Sending  random number: '" +x  + "'\n");
         System.out.println("Sending  message1: '" +message1  + "'\n");
 
         out.print(id+" ");
         out.print(hashcodeid+" ");
         //out.print(x);
         out.print(message1+" ");
 
 
 
 
   out.close();
 
 
         }
 
catch(Exception e)
{
  e.printStackTrace();
}
    }
}

--- Update ---

my problem is sending data from client to server....

client side

C:\Users\ADITYA\Desktop\project in java\half>java Client
Received string: 'Received policeone -226259832 -226259891

Token number 01:policeone

Token number 11:-226259832

Token number 21:-226259891
Random Number:197

client sending::

C:\Users\ADITYA\Desktop\project in java\half>

server side::

server.jpg

client sending the hello message... server should accept the hello msg and display the message

Sorry for posting the wrong code..

Here is the current version with the problem. The Socket Exception is not being thrown here, but the communication is not happening.

import java.io.*;
import java.net.*;
import java.util.*;
 
public class Server
{
 
   public static void main(String args[])
   {
     String id="policeone";
     int hashcodeid=0;
     try
        {
           ServerSocket srvr=new ServerSocket(1234);
           Socket client=srvr.accept();
           System.out.println("Connected to a client");
 
           PrintWriter out=new PrintWriter(client.getOutputStream(),true);
 
           for(int i=0;i<id.length();i++) 
    {
                hashcodeid = 31*hashcodeid + id.charAt(i);
    }
               Random random = new Random ();
         int x = 100 + (int)(Math.random()*200);
 
         int message1=hashcodeid^x;
 
         System.out.println("Sending  id string: '" +id  + "'\n");
         System.out.println("Sending  hashcode of id  : '" +hashcodeid  + "'\n");
         System.out.println("Sending  random number: '" +x  + "'\n");
         System.out.println("Sending  message1: '" +message1  + "'\n");
 
         out.print(id+" ");
         out.print(hashcodeid+" ");
         //out.print(x);
         out.print(message1+" ");
 
 
 
 
 
 
 BufferedReader fromserver=new BufferedReader(new InputStreamReader(client.getInputStream()));
            while(!fromserver.ready()){}
 
String nowstring=fromserver.readLine();
System.out.println("Received from client    "+nowstring);
 
 
         }
 
catch(Exception e)
{
  e.printStackTrace();
}
    }
}

import java.io.*;
import java.net.*;
import java.util.*;
 
public class Client
{
public static void main(String args[])
{
try{
        String id="vehicle1";
        String idstring,hash,allstring;
        int hashcodeid2=0;
         Socket skt = new Socket("localhost", 1234);
         BufferedReader in = new BufferedReader(new
         InputStreamReader(skt.getInputStream()));
 
 
 
         System.out.print("Received string: '");
 
       while (!in.ready()) {}
 
          allstring=in.readLine();
      System.out.println("Received               "+allstring); 
     String delims=new String(" ");
     String tokens[]=allstring.split(delims);
 
       for(int i=0;i<tokens.length;i++)
     System.out.println("\nToken number   "+i+1+":"+tokens[i]);  
    int hashid=Integer.parseInt(tokens[1]);
    int message1=Integer.parseInt(tokens[2]);
   System.out.println("Random Number:"+ (hashid^message1));
 
    System.out.println("\n  client sending::");
   BufferedReader br=new BufferedReader(new InputStreamReader(System.in));
  String myinput;
 
 
PrintWriter toserver=new PrintWriter(skt.getOutputStream(),true);
/*do
 {
   myinput=br.readLine(); 
  System.out.println("You entered       "+myinput);
  System.out.println("Sending       "+myinput+" to server");
  toserver.print(myinput);
 
}while(!(myinput.equals("n"))); */
 
toserver.print("helo");
 
 
 
   }
catch(Exception e)
{
  e.printStackTrace();
 }
}
}

Thank you so much. i have reduced the code as you suggested..but still i am having the problem. could you rectify it sir???

import java.lang.*;
import java.io.*;
import java.net.*;
 
class Client1 {
   public static void main(String args[]) {
      String data1="hello";
      try {
         Socket skt = new Socket("localhost", 1234);
         BufferedReader in = new BufferedReader(new
            InputStreamReader(skt.getInputStream()));
         System.out.print("Received string: '");
 
         while (!in.ready()) {}
         System.out.println(in.readLine()); // Read one line and output it
 
         System.out.print("'\n");
 
         PrintWriter out = new PrintWriter(skt.getOutputStream(), true);
          System.out.print("Sending string: '" + data1 + "'\n");
         out.print(data1);
         out.close();
         in.close();
      }
      catch(Exception e) {
         System.out.print("Whoops! It didn't work!\n");
      }
   }
}

import java.lang.*;
import java.io.*;
import java.net.*;
 
class Server1 {
   public static void main(String args[]) {
      String data = "Toobie ornaught toobie";
      try {
         ServerSocket srvr = new ServerSocket(1234);
         Socket skt = srvr.accept();
         System.out.print("Server has connected!\n");
         PrintWriter out = new PrintWriter(skt.getOutputStream(), true);
         System.out.print("Sending string: '" + data + "'\n");
         out.print(data);
         out.close();
 
         BufferedReader in = new BufferedReader(new
            InputStreamReader(skt.getInputStream()));
         System.out.print("Received string: '");
 
          in.close();
         skt.close();
         srvr.close();
      }
      catch(Exception e) {
         System.out.print("Whoops! It didn't work!\n");
      }
   }
}

C:\Users\ADITYA\Desktop\example>java Server1
Server has connected!
Sending string: 'Toobie ornaught toobie'
Whoops! It didn't work!


C:\Users\ADITYA\Desktop\example>javac Client1.java

C:\Users\ADITYA\Desktop\example>java Client1
Received string: 'Toobie ornaught toobie
'
Sending string: 'hello'


The client is sending hello to server, but it is not geting received on the server side.. the exception is being thrown there.. as "Socket is closed"

please sir,
i want the server receives the hello msg...

The output clearly shows client sending hello msg to server,but server is not taking the hello msg and throws exception....
I know "whoops It didnt work " is the output of printstacktrace().......

1)the server sends
toobie or naught toobie

the client receives
toobie or naught toobie

2)the client sends
hello

the server receives
whoops it didnt work

--- Update ---

i have replace "whoops it didnt work" with printStackTrace().... it shows the below error

Server has connected!
Sending string: 'Toobie ornaught toobie'
java.net.SocketException: Socket is closed
at java.net.Socket.getInputStream(Socket.java:872)
at Server1.main(Server1.java:17)