Some error handling

Impiccato/src/it/polito/tdp/impiccato/ImpiccatoController.java

@@ -72,14 +72,33 @@ private void updateView() {
 
 		txtErrori.setText(String.format("%d", model.getErrori()));
 		txtParola.setText(model.getMaschera());
-		pbErrori.setProgress( (double)model.getErrori()/model.getMAX_ERRORI()) ;
+		pbErrori.setProgress((double) model.getErrori() / model.getMAX_ERRORI());
 
 	}
 
 	@FXML
 	void doStart(ActionEvent event) {
 
-		model.startGame(txtSegreto.getText());
+		String segreto = txtSegreto.getText();
+
+		// remove spaces and converto to uppercase
+		segreto = segreto.trim().toUpperCase();
+
+		// must not be empty
+		if (segreto.length() == 0) {
+			txtSoluzione.setText("ERRORE parola non valida");
+			return;
+		}
+
+		// must only contain letters
+		for (int i = 0; i < segreto.length(); i++) {
+			if (!Character.isUpperCase(segreto.charAt(i))) {
+				txtSoluzione.setText("ERRORE parola non valida");
+				return;
+			}
+		}
+
+		model.startGame(segreto);
 
 		inMatch = true;
 		updateView();
@@ -89,11 +108,11 @@ void doStart(ActionEvent event) {
 	void doTry(ActionEvent event) {
 
 		String t = comboLettera.getValue();
-		if(t==null) {
+		if (t == null) {
 			// no selected letter
-			return ;
+			return;
 		}
-		
+
 		model.tryLetter(t);
 
 		if (model.isWinner()) {
@@ -125,7 +144,7 @@ void initialize() {
 		for (char ch = 'A'; ch <= 'Z'; ch++) {
 			comboLettera.getItems().add(String.valueOf(ch));
 		}
-		
+
 	}
 
 	public void setModel(ImpiccatoModel model) {

Impiccato/src/it/polito/tdp/impiccato/model/ImpiccatoModel.java

@@ -39,6 +39,7 @@ public ImpiccatoModel() {
 	 */
 	public void startGame(String segreto) {
 		this.segreto = segreto.toUpperCase();
+
 		errori = 0;
 		provate.clear();