Validation fixed on server list screen.

Server List Screen done.
tags : #story[1245]
This commit is contained in:
Alistair McIntyre
2017-09-11 13:51:29 +12:00
parent 800ae2864f
commit 2b53e0d5b4
3 changed files with 30 additions and 8 deletions
@@ -72,8 +72,8 @@ public class ServerListController implements Initializable, ServerListenerDelega
RequiredFieldValidator validator = new RequiredFieldValidator(); RequiredFieldValidator validator = new RequiredFieldValidator();
textField.getValidators().add(validator); textField.getValidators().add(validator);
} }
serverHostName.getValidators().get(0).setMessage("Correct HostName Required"); serverHostName.getValidators().get(0).setMessage("Incorrect Host Name");
serverPortNumber.getValidators().get(0).setMessage("Correct Port Number Required"); serverPortNumber.getValidators().get(0).setMessage("Incorrect Port Number");
// Start listening for servers on network // Start listening for servers on network
try { try {
@@ -112,11 +112,18 @@ public class ServerListController implements Initializable, ServerListenerDelega
* *
*/ */
private void attemptToDirectConnect() { private void attemptToDirectConnect() {
if (validateHostName(serverHostName.getText()) && validatePortNumber(serverPortNumber.getText())) { if (validateDirectConnection(serverHostName.getText(), serverPortNumber.getText())) {
DirectConnect(); DirectConnect();
} }
} }
private Boolean validateDirectConnection(String hostName, String portNumber) {
Boolean hostNameValid = validateHostName(hostName);
Boolean portNumberValid = validatePortNumber(portNumber);
return hostNameValid && portNumberValid;
}
/** /**
* *
* @return * @return
@@ -150,12 +157,11 @@ public class ServerListController implements Initializable, ServerListenerDelega
return true; return true;
} else { } else {
System.out.println(portNum.toString() + " is not a valid port number"); System.out.println(portNum.toString() + " is not a valid port number");
serverPortNumber.validate();
} }
} catch (NumberFormatException e) { } catch (NumberFormatException e) {
serverPortNumber.validate();
System.out.println("Not a valid number."); System.out.println("Not a valid number.");
} }
serverPortNumber.validate();
return false; return false;
} }
@@ -2,7 +2,9 @@ package seng302.visualiser.controllers.dialogs;
import com.jfoenix.controls.JFXButton; import com.jfoenix.controls.JFXButton;
import com.jfoenix.controls.JFXSlider; import com.jfoenix.controls.JFXSlider;
import com.jfoenix.validation.RequiredFieldValidator;
import java.net.URL; import java.net.URL;
import java.util.Arrays;
import java.util.ResourceBundle; import java.util.ResourceBundle;
import com.jfoenix.controls.JFXTextField; import com.jfoenix.controls.JFXTextField;
@@ -27,14 +29,28 @@ public class ServerCreationController implements Initializable {
private JFXButton submitBtn; private JFXButton submitBtn;
//---------FXML END---------// //---------FXML END---------//
RequiredFieldValidator validator;
public void initialize(URL location, ResourceBundle resources) { public void initialize(URL location, ResourceBundle resources) {
updateMaxPlayerLabel(); updateMaxPlayerLabel();
maxPlayersSlider.valueProperty().addListener((observable, oldValue, newValue) -> { maxPlayersSlider.valueProperty().addListener((observable, oldValue, newValue) -> {
updateMaxPlayerLabel(); updateMaxPlayerLabel();
}); });
validator = new RequiredFieldValidator();
serverName.getValidators().add(validator);
submitBtn.setOnMouseClicked(event -> submitBtn.setText("CREATING...")); submitBtn.setOnMouseClicked(event -> submitBtn.setText("CREATING..."));
submitBtn.setOnMouseReleased(event -> createServer()); submitBtn.setOnMouseReleased(event -> validateServerSettings());
}
private void validateServerSettings() {
if (serverName.getText().length() >= 40) {
validator.setMessage("HI");
} else if (serverName.getText().length() == 0) {
validator.setMessage("Server Name Required.");
}
serverName.validate();
} }
public void createServer() { public void createServer() {
+1 -1
View File
@@ -52,7 +52,7 @@
#serverHostName .error-label, #serverPortNumber .error-label { #serverHostName .error-label, #serverPortNumber .error-label {
-fx-font-size: 12px; -fx-font-size: 12px;
-fx-text-fill: palevioletred; -fx-text-fill: lightblue;
} }
#connectGridPane { #connectGridPane {