colinzeidler / p2pchat Goto Github PK

two or more users connected

1. force close one users client

all remaining users should remove old user from lists

only one client will be able to run at once,

Launch one client on the computer
launch a second client on the same computer.
second client should display an error that when dismissed exits the app

A popup box that appears when the program is started,
it will contain a list of all local servers, which when clicked will be connected to.
It will also contain a field to enter the address of the host you want to join

every 10 or so seconds will send a heartbeat packet to all users, and handle the case of connection reset errors.

upon receival of a heartbeat message, do nothing.

should be run after a result for the dialog is received.

I have run into a problem using Bytebuffer, with 3+ users

when trying to send a message to multiple users the functions used in the getBytes method empties the ByteBuffer on the first call making following calls fail.

Send the actual file once someone requests it.
should spawn a new Thread since this can take a while

there is aparrently a setting for this, not sure what it is though

Add the ability for the user to remove on of their file advertisements, may cause issues if a file is already being transferred

For the while (true) loop in Controller

1. choose a delay amount
2. get time before handleing users
3. get time after handling users
4. subtract difference from delay amount
5. sleep for that time

This will reduce the CPU load of the program

appeared after calling the incomingCOnnect function
Solution:

• move the for loop that iterates over the newUsers array outside of the userList forloop

The code for finding meighbors will call this function for each neighbor discovered.

providing the ip as a string.

saving the incoming file to disk

multiple users in room.
one user disables their network connection
this user should remove all other clients from list
all other clients should remove the one user from their list

helpful for actually knowing there is a new message

Need to know if it finds all the IPs correctly, followed what the one link you posted did fairly closely

part of the menu bar, possible a file explorer dialog, returns the string for the file location

Should be easy to do, the dialog sets your name, and then closes.

Controler.run() should handle this at the bottom
create a new function to call from run to keep stuff tidy

loop through all Users contained in Model
read from stream, if null continue to next (the sockets SoTimeout must be set, best is probably 1)
if not null, data should be a number (0 = text, 1 = file advert, 2 = file request, 3 = connect)

• text
  • readline (this is the actual message)
• file advert
  • readline (this is the file name)
• file request
  • send file size
  • send file
• connect
  • readline (address to connect to)

Handle logic for encoding text OR file data

NOTE: Not the actual transfer of the data itself

Functions for writing and reading Packet objects with Input and Output streams

Within the user class 'sendText/File' functions need to be changed to incorporate this.

Just need to make sure each user gets it

will require a keyevent listener, but should make use of the same code for the send button

This is the "server" that allows for network discovery
check the text of the packet, and if correct respond

incoming message: p2p_chat_discovery_packet
outgoing message: p2p_chat_discovery_response

all users should remove the user who left.
User who left should not show anyone as being in their room

currently message is assembled in the User class
this means that it must be done for every user in the room. which may be quite a lot.
This can take lots of time, also the only part of the sendText function that is client specific is the actual sending

better format is to create the full message string in the getSendListener function. then loop through all the Users and sendText them.

And an action handler for the ui that will send data when the send button (or Enter) is pressed
View class needs a get message text function

• clears the message field
• returns the text that was in it

in the Controllers getSendListener

• call the views get message function
• add that message to the model
• loop through all users and call sendText(message)

should only display files from remote users, nothing localy offered

disconnect, and connect should be options in the menu bar
disconnect will d/c the user and then bring up the connection dialoge
connect will bring up the connection dialoge, and only d/c if the user chooses a new room to connect to

Client 1,2 are connected
Client 1 advertises file
Client 2 can see the file
Client 1 removes the ad
Client 2 says Failed to remove advertisement on file! Not Removed!

error appears to occur in Controller.java: line 137 (myModel.removeFile(removeFile))

This will find people who are joining.

1. accept user
2. tell user how many users are in room
3. tell all other users to connect to new user

since this is on a new thread will have to be thread safe when modifying user list