flavien/Projet_JAVA_P2P_STRI2A
3
2
Fork 0
Code Issues 11 Pull Requests Releases 13 Activity

WIP: Aide tracker #63

Closed
louis_royer wants to merge 12 commits from clitracker into etape4
pull from: clitracker
merge into: flavien:etape4
Conversation 1 Commits 12 Files Changed 7 +189 -40
louis_royer commented 2020-03-21 19:25:54 +01:00
Collaborator
Copy Link

Fix #60.

Fix #60.
louis_royer reviewed 2020-03-21 19:33:40 +01:00
louis_royer left a comment
Author
Collaborator
Copy Link

Des changements mineurs

Des changements mineurs
README.md Outdated
@ -8,1 +8,4 @@
# Usage
tracker : java -ea tracker.Tracker, to specify port (default localhost:6969, range 6000 -> 6999): java -ea trackerP2P.trackerP2P -- <PORT>
louis_royer commented 2020-03-21 19:28:37 +01:00
Author
Collaborator
Copy Link

-ea pas obligatoire

-ea pas obligatoire
README.md Outdated
@ -9,0 +9,4 @@
# Usage
tracker : java -ea tracker.Tracker, to specify port (default localhost:6969, range 6000 -> 6999): java -ea trackerP2P.trackerP2P -- <PORT>
serveur : java -ea serveurP2P.ServeurP2P, to specify port and tracker (default for server localhost:7070 tracker localhost:6969, range 7000 -> 7999): java -ea serveurP2P.ServeurP2P -- <serveurPORT> <trackerHOSTNAME> <trackerPORT>
louis_royer commented 2020-03-21 19:28:50 +01:00
Author
Collaborator
Copy Link

-ea pas obligatoire

-ea pas obligatoire
README.md Outdated
@ -9,0 +10,4 @@
tracker : java -ea tracker.Tracker, to specify port (default localhost:6969, range 6000 -> 6999): java -ea trackerP2P.trackerP2P -- <PORT>
serveur : java -ea serveurP2P.ServeurP2P, to specify port and tracker (default for server localhost:7070 tracker localhost:6969, range 7000 -> 7999): java -ea serveurP2P.ServeurP2P -- <serveurPORT> <trackerHOSTNAME> <trackerPORT>
client/serveur : java -ea clientP2P.ClientP2P -- <serverPORT> <trackerHOSTNAME> <trackerPORT>
louis_royer commented 2020-03-21 19:29:00 +01:00
Author
Collaborator
Copy Link

-ea pas obligatoire

-ea pas obligatoire
src/tools/TestPort.java Outdated
@ -0,0 +6,4 @@
* @author JS Auge
* @version 1.0
*/
public class TestPort {
louis_royer commented 2020-03-21 19:27:41 +01:00
Author
Collaborator
Copy Link

missing javadoc

missing javadoc
src/tools/TestPort.java Outdated
@ -0,0 +7,4 @@
* @version 1.0
*/
public class TestPort {
louis_royer commented 2020-03-21 19:27:50 +01:00
Author
Collaborator
Copy Link

missing javadoc

missing javadoc
src/tools/TestPort.java Outdated
@ -0,0 +8,4 @@
*/
public class TestPort {
public boolean testPortServer(int port) {
louis_royer commented 2020-03-21 19:26:53 +01:00
Author
Collaborator
Copy Link

méthode static

méthode static
src/tools/TestPort.java Outdated
@ -0,0 +9,4 @@
public class TestPort {
public boolean testPortServer(int port) {
if((port > 7000) && (port < 7999)){
louis_royer commented 2020-03-21 19:29:39 +01:00
Author
Collaborator
Copy Link

expliquer brièvement en commentaire le pourquoi du choix de cette plage de port (avec le lien vers wikipedia si besoin)

expliquer brièvement en commentaire le pourquoi du choix de cette plage de port (avec le lien vers wikipedia si besoin)
src/tools/TestPort.java Outdated
@ -0,0 +15,4 @@
return false;
}
public boolean testPortTracker(int port) {
louis_royer commented 2020-03-21 19:27:05 +01:00
Author
Collaborator
Copy Link

méthode static

méthode static
src/tools/TestPort.java Outdated
@ -0,0 +16,4 @@
}
public boolean testPortTracker(int port) {
if((port > 6000) && (port < 6999)){
louis_royer commented 2020-03-21 19:29:47 +01:00
Author
Collaborator
Copy Link

expliquer brièvement en commentaire le pourquoi du choix de cette plage de port (avec le lien vers wikipedia si besoin)

expliquer brièvement en commentaire le pourquoi du choix de cette plage de port (avec le lien vers wikipedia si besoin)
src/tracker/Tracker.java Outdated
@ -46,2 +43,2 @@
ttcp.setName("Tracker TCP P2P-JAVA-PROJECT");
ttcp.start();
if (args[1].equals("help") || args[1].equals("-h") || args[1].equals("h")){
System.out.println("usage : java -ea trackerP2P.trackerP2P, to specify port (default 6969, range 6000 -> 6999): java -ea trackerP2P.trackerP2P -- <PORT>");
louis_royer commented 2020-03-21 19:31:16 +01:00
Author
Collaborator
Copy Link

faire une méthode dans TestPort qui permet d’afficher la range, pour tout regrouper dans une seule classe (les extrémités de numéro de ports pourront être mis dans des variables static)

faire une méthode dans TestPort qui permet d’afficher la range, pour tout regrouper dans une seule classe (les extrémités de numéro de ports pourront être mis dans des variables static)
src/tracker/Tracker.java Outdated
@ -48,0 +50,4 @@
t = new Tracker(args[1]);
}
else {
System.out.println("Wrong port (6000 -> 6999), using default port 6969");
louis_royer commented 2020-03-21 19:31:34 +01:00
Author
Collaborator
Copy Link

faire une méthode dans TestPort qui permet d’afficher la range, pour tout regrouper dans une seule classe (les extrémités de numéro de ports pourront être mis dans des variables static)

faire une méthode dans TestPort qui permet d’afficher la range, pour tout regrouper dans une seule classe (les extrémités de numéro de ports pourront être mis dans des variables static)
src/tracker/Tracker.java Outdated
@ -48,0 +51,4 @@
}
else {
System.out.println("Wrong port (6000 -> 6999), using default port 6969");
t = new Tracker(String.valueOf(defaultPort));
louis_royer commented 2020-03-21 19:28:21 +01:00
Author
Collaborator
Copy Link

FIXME

FIXME
louis_royer changed title from Aide tracker to WIP: Aide tracker 2020-03-21 22:40:59 +01:00
louis_royer commented 2020-03-23 14:38:37 +01:00
Author
Collaborator
Copy Link

Il faudrait rebase:

git checkout etape4
git pull
git checkout clitracker
git rebase etape4

et résoudre le conflit. Fait

<s>Il faudrait rebase: ``` git checkout etape4 git pull git checkout clitracker git rebase etape4 ``` et résoudre le conflit.</s> Fait
louis_royer reviewed 2020-03-24 13:15:05 +01:00
src/tools/TestPort.java Outdated
@ -0,0 +10,4 @@
*/
public class TestPort {
public boolean testPortServer(int port, String message) {
louis_royer commented 2020-03-24 13:08:10 +01:00
Author
Collaborator
Copy Link

si tu change en
public static boolean testPortServer(…)
tu n’aura plus besoin de faire ton
TestPort testPort = new TestPort(); avant de l’utiliser (tu pourra l’utiliser avec TestPort.testPortServer(port, message) directement).
D’ailleurs, je pense que tu ne devrais pas passer la variable message, mais plutôt quelque chose du genre:

private static int portMinServer = 7000;
private static int portMaxServer = 7999;

public static String getRangeServer() {
	return portMinServer + " -> " + portMaxServer;
}

public static boolean isPortServerValid(int port) {
	if ((port >= portMinServer) && (port <= maxServer)) {
    	return true;
    } else {
    	return false;
    }
}

et idem pour le tracker.

si tu change en `public static boolean testPortServer(…)` tu n’aura plus besoin de faire ton `TestPort testPort = new TestPort();` avant de l’utiliser (tu pourra l’utiliser avec `TestPort.testPortServer(port, message)` directement). D’ailleurs, je pense que tu ne devrais pas passer la variable message, mais plutôt quelque chose du genre: ```java private static int portMinServer = 7000; private static int portMaxServer = 7999; public static String getRangeServer() { return portMinServer + " -> " + portMaxServer; } public static boolean isPortServerValid(int port) { if ((port >= portMinServer) && (port <= maxServer)) { return true; } else { return false; } } ``` et idem pour le tracker.
louis_royer commented 2020-03-24 14:23:04 +01:00
Author
Collaborator
Copy Link

ping @flavien @js_auge

ping @flavien @js_auge
flavien closed this pull request 2020-03-24 16:02:29 +01:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working

  
documentation

UML/javadoc

  
duplicate

This issue or pull request already exists

  
easy

Tasks easy to do

  
enhancement

New feature

  
help wanted

Need some help

  
important

Tasks with High Priority

  
invalid

Something is wrong

  
low priority

To do but not now

  
protocol

Change in protocol required

  
question

More information is needed

  
refactoring

Code change but no new feature

  
wontfix

This won't be fixed

No Label
bug
documentation
duplicate
easy
enhancement
help wanted
important
invalid
low priority
protocol
question
refactoring
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: flavien/Projet_JAVA_P2P_STRI2A#63
No description provided.
Delete Branch "clitracker"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?