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 +13 -8
2 changed files with 13 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 702fb399c3 - Show all commits

11
src/tools/TestPort.java
View File

@ -1,5 +1,7 @@
package tools;
import javafx.util.Pair;
/** Test ports.
* @author Louis Royer
* @author Flavien Haas
@ -8,18 +10,19 @@ package tools;
*/
louis_royer commented 2020-03-21 19:27:50 +01:00
Outdated
Review
Copy Link

missing javadoc

missing javadoc
public class TestPort {
louis_royer commented 2020-03-21 19:26:53 +01:00
Outdated
Review
Copy Link

méthode static

méthode static
louis_royer commented 2020-03-21 19:29:39 +01:00
Outdated
Review
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)
public boolean testPortServer(int port) {
public boolean testPortServer(int port, String message) {
louis_royer commented 2020-03-24 13:08:10 +01:00
Outdated
Review
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
Outdated
Review
Copy Link

ping @flavien @js_auge

ping @flavien @js_auge
if((port > 7000) && (port < 7999)){
return true;
}
message = message + "Wrong port (7000 -> 7999), using default port 7070";
return false;
louis_royer commented 2020-03-21 19:27:05 +01:00
Outdated
Review
Copy Link

méthode static

méthode static
}
louis_royer commented 2020-03-21 19:29:47 +01:00
Outdated
Review
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)
public boolean testPortTracker(int port) {
public Pair<Integer,String> testPortTracker(int port) {
if((port > 6000) && (port < 6999)){
return true;
return new Pair<>(0,"");
}
return false;
return new Pair<>(1, "Wrong port (6000 -> 6999), using default port 6969");
}

10
src/tracker/Tracker.java
View File

@ -1,6 +1,7 @@
package tracker;
import java.util.Scanner;
import javafx.util.Pair;
import tracker.TrackerManagementTCP;
import tracker.TrackerManagementUDP;
import tools.Directories;
@ -42,7 +43,7 @@ public class Tracker {
final String defaultPort = "6969";
final String defaultHostname = "localhost";
louis_royer commented 2020-03-21 19:31:16 +01:00
Outdated
Review
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)
Scanner scanner = new Scanner(System.in);
TestPort testPortTracker = new TestPort();
TestPort testPort = new TestPort();
String hostname;
String port;
Tracker t;
@ -73,14 +74,15 @@ public class Tracker {
}
System.out.println("using hostname : " + hostname);
if(testPortTracker.testPortTracker(Integer.parseInt(port))){
Pair<Integer, String> PtestPortTracker = testPort.testPortTracker(Integer.parseInt(port));
if(PtestPortTracker.getKey() == 0){
System.out.println("using port : " + port);
t = new Tracker(port);
}
else {
System.out.println("Wrong port (6000 -> 6999), using default port 6969");
System.out.println(PtestPortTracker.getValue());
t = new Tracker(defaultPort);
t.logger.write("Wrong port (6000 -> 6999), using default port 6969", LogLevel.Warning);
t.logger.write(PtestPortTracker.getValue(), LogLevel.Warning);
}
TrackerManagementUDP tmudp = new TrackerManagementUDP(hostname, t.port, t.logger);