2 replies to this topic

[Joshhua5]

Currently I'm trying to get the basics of the chat server done before adding features and optimizing it to run faster, but right now I can't get the little thing to work and I don't know what's wrong. I use two classes, one of which controls the function of the program while the other class(Users) deals with the connection, sending and receiving the messages from users. I have three ArrayLists and the id on each user should be the same throughout the three lists.

import java.io.*;
import java.net.*;
import java.util.ArrayList;
import java.util.Stack;
public class Room {

public static void Start(){
  ServerSocket serversocket;
  Socket socket;
  ArrayList<Socket> users = new ArrayList<Socket>();
  Users userfunction = new Users();
  Stack<String> outputMessage = new Stack<String>();
 
  try{
   System.out.println();
   // Created a socket in which clients can connect to.
   serversocket = new ServerSocket(7776);
   // Waits for a client to connect to the server //
   while(true){
    socket = serversocket.accept();
    // Adds User onto ArrayList
    users.add(socket);
    userfunction.addStream(socket);
    // Time to Receive messages from clients
    for(int x = 0; x < users.size(); x++){
	 outputMessage.push(userfunction.getMessage(x));
    }
    // Time to send all new messages to clients
    if(!outputMessage.empty()){
	 String out = outputMessage.pop();
	 userfunction.sendMessageALL(out);
    }
   }
  }catch(IOException e){
   System.out.println(e);
  }
}
public static void main(String[] args){
  Start();
}
}

import java.net.*;
import java.io.*;
import java.util.ArrayList;
public class Users {
ArrayList<BufferedReader> recieveAL = new ArrayList<BufferedReader>();
ArrayList<PrintWriter> sendAL = new ArrayList<PrintWriter>();
public void addStream(Socket socket){
  BufferedReader br;
  PrintWriter pw;
  try{
   br = new BufferedReader(new InputStreamReader(socket.getInputStream()));
   pw = new PrintWriter(socket.getOutputStream(), true);
   recieveAL.add(br);
   sendAL.add(pw);
  }catch(IOException e){
   e.printStackTrace();
  }
}
// Recieve a message from someone.
public String getMessage(int x){
  BufferedReader bf = recieveAL.get(x);
  try{
   return bf.readLine();
  }catch(IOException e){
   System.out.println(e);
  }
  return null;
}
// send a message to everyone
public void sendMessageALL(String send){
  for(int x = 0; x <= sendAL.size(); x++){
   PrintWriter pw = sendAL.get(x);
   pw.println(send);
  }
}
// Send a message to one person
public void sendMessage(String send,int x){
  PrintWriter pw = sendAL.get(x);
  pw.println(send);
}
}

<img class="UMSRatingIcon" id="ums_img_tooltip" />

[Joshhua5]

I've found the problem (After printing the the console at every step of the code) and it stops at the serversocket.accept(); which is waiting for a client to connect before doing anything. but the problem is if i use serversocket.setTimeout then the while brace of the try which the code is positioned in will break and cause the code to stop. how can I get the .accept to timeout without breaking a exception, if i place it into another try catch block then the variable calls out of scope once it's finished making it useless.

<img class="UMSRatingIcon" id="ums_img_tooltip" />

[hplus0603]

Look at the asynchronous functions in the "nio" package, or use select().