Kommentieren

Reviewed-on: #15

src/main/java/Logik/Account.java

@@ -3,6 +3,8 @@
 
 package Logik;
 
+import RestAPISchnittstelle.RestApiClient;
+
 public class Account {
 
 	// region Felder
@@ -14,7 +16,6 @@ public class Account {
 
 	private String email;
 
-
 	private int rid;
 	// endregion
 
@@ -57,6 +58,7 @@ public class Account {
 	public Account(String passwort, String name, String email) {
 		this.passwort = passwort;
 		this.name = name;
+		this.email = email;
 
 		if(this instanceof MitarbeiterAccount)
 			rid = 0;
@@ -64,5 +66,4 @@ public class Account {
 			rid = 1;
 	}
 	// endregion
-
 }

src/main/java/RestAPISchnittstelle/RestApiClient.java

@@ -42,7 +42,7 @@ public class RestApiClient implements IRestAPI{
 	 * @param controllerName Name des aufzurufenden Controllers
 	 */
 	@Override
-	public void get(String controllerName) {
+	public String get(String controllerName) {
 		URI apiUri = URI.create(String.format("%s/%s", urlBase, controllerName));
 
 		HttpRequest httpRequest = HttpRequest.newBuilder()
@@ -57,8 +57,10 @@ public class RestApiClient implements IRestAPI{
 			// Print the response status code and body
 			System.out.println("Status Code: " + httpResponse.statusCode());
 			System.out.println("Response Body: " + httpResponse.body());
+			return httpResponse.body();
 		} catch (Exception e) {
 			e.printStackTrace();
+			return null;
 		}
 	}
 
@@ -69,7 +71,7 @@ public class RestApiClient implements IRestAPI{
 	 * @param id Id der Aufzurufenden Zeile
 	 */
 	@Override
-	public void get(String controllerName, int id) {
+	public String get(String controllerName, int id) {
 		URI apiUri = URI.create(String.format("%s/%s/%s", urlBase, controllerName, id));
 
 		HttpRequest httpRequest = HttpRequest.newBuilder()
@@ -84,9 +86,12 @@ public class RestApiClient implements IRestAPI{
 			// Print the response status code and body
 			System.out.println("Status Code: " + httpResponse.statusCode() + httpResponse.body());
 
+			return httpResponse.body();
+
 			//System.out.println("Response Body: " + test);
 		} catch (Exception e) {
 			e.printStackTrace();
+			return null;
 		}
 	}
 
@@ -98,7 +103,7 @@ public class RestApiClient implements IRestAPI{
 	 * @param bezahlt TODO Warum ist das hier?
 	 */
 	@Override
-	public void get(String controllerName, int id, boolean bezahlt) {
+	public String get(String controllerName, int id, boolean bezahlt) {
 		URI apiUri = URI.create(String.format("%s/%s?%s&%s", urlBase, controllerName, id, bezahlt));
 
 		HttpRequest httpRequest = HttpRequest.newBuilder()
@@ -113,8 +118,10 @@ public class RestApiClient implements IRestAPI{
 			// Print the response status code and body
 			System.out.println("Status Code: " + httpResponse.statusCode());
 			System.out.println("Response Body: " + httpResponse.body());
+			return httpResponse.body();
 		} catch (Exception e) {
 			e.printStackTrace();
+			return null;
 		}
 	}
 

src/main/java/de/subway_surfers/vpr_app/AccounterstellungMitarbeiter.java

@@ -33,9 +33,11 @@ public class AccounterstellungMitarbeiter {
 	private ArrayList<Kind> kinder;
 
 	public void initialize() {
+		//Die Radiobuttons zum auswählen eines Accountypes werden einer ToggleGroup hinzugefügt
 		accountTyp = new ToggleGroup();
 		accountTyp.getToggles().add(typEltern);
 		accountTyp.getToggles().add(typMitarbeiter);
+
 		einmalpw.setText(einmalPwGenerieren());
 
 		kinder = new ArrayList<>();
@@ -53,7 +55,12 @@ public class AccounterstellungMitarbeiter {
 		kindDaten.getChildren().clear();
 	}
 
+	/**
+	 * Einfügen der Eingabefelder für Kinder, wenn der Eltern-Radiobutton ausgewählt wird
+	 * @param actionEvent
+	 */
 	public void onTypEltern(ActionEvent actionEvent) {
+		//Einfügen der Eingabefelder für Kinder
 		Label ueberschrift = new Label();
 		ueberschrift.setText("Daten des Kindes:");
 		kindDaten.addRow(0);
@@ -85,6 +92,7 @@ public class AccounterstellungMitarbeiter {
 		kindDaten.add(hinzufuegen, 1, 4);
 
 		kindDaten.addRow(5);
+		//wird der Button zum hinzufügen eines Kindes geclickt wird dieses Event ausgelöst
 		hinzufuegen.setOnAction(e -> {
 			Button neues = new Button(vnameEingabe.getText());
 
@@ -92,6 +100,7 @@ public class AccounterstellungMitarbeiter {
 			boolean gebGueltig = false;
 			boolean nnameGueltig = false;
 
+			//Wenn alle Eingabefelder für Kinder ausgefüllt sind ist die Eingabe gültig
 			if (!vnameEingabe.getText().equals("")) {
 				vnameGueltig = true;
 			}
@@ -108,10 +117,12 @@ public class AccounterstellungMitarbeiter {
 				Kind kind = new Kind(nnameEingabe.getText(), vnameEingabe.getText(), 0);
 				kinder.add(kind);
 				kindanzeige.getChildren().add(neues);
+				//Wird auf ein Kind in der Anzeige geclickt, wird es aus der Liste entfernt.
 				neues.setOnAction(a -> {
 					((HBox) neues.getParent()).getChildren().remove(neues);
 					kinder.remove(kind);
 				});
+				//Die Eingabefelder werden zurückgesetzt
 				vnameEingabe.setText("");
 				nnameEingabe.setText("");
 				geburtstagEingabe.getEditor().setText("");
@@ -129,22 +140,36 @@ public class AccounterstellungMitarbeiter {
 		return pw;
 	}
 
+	/**
+	 * Prüft, ob in allen Textfeldern für die Eltern Text steht.
+	 * @return true, wenn kein Feld leer ist
+	 */
 	private boolean eingabenGueltig() {
 		return !accountname.getText().equals("") && !email.getText().equals("") && (typEltern.isSelected() || typMitarbeiter.isSelected());
 	}
 
+	/**
+	 *
+	 * @param actionEvent
+	 */
 	public void onSpeichern(ActionEvent actionEvent) {
 		if (eingabenGueltig()) {
-			status.setText("Daten Erfolgreich gespeichert");
+			//Zeigt generiert ein Einmalpasswort und zeigt es an
 			einmalpw.setText(einmalPwGenerieren());
 
+			//Schnittstelle zur RestAPI
+			RestApiClient api = new RestApiClient();
 
-			//String json = String.format("{\"name\":\"%s\", \"email\":\"%s\", \"passwort\":\"%d\", \"rid\":\"%d\"}",
-			//							accountname.getText(), email.getText(), einmalpw.getText().hashCode(), (typMitarbeiter.isSelected() ? 0:1));
+			//id, die das Elternobjekt bekommt, dass im laufe der Methode erstellt wird
+			int id = api.nextId("Benutzer");
 
+			//Erstellen des neuen Accounts
 			Account neuer;
+			//Handelt es sich um einen Elternaccount, werden alle Eingegebenen Kinder erstellt.
 			if (typEltern.isSelected()) {
 				neuer = new ElternAccount(String.format("%d",einmalpw.getText().hashCode()), accountname.getText(), email.getText());
+				System.out.println(neuer.getEmail());
+				//Kinder dem neuen Account hinzufügen
 				for (Kind k : kinder) {
 					((ElternAccount)neuer).getKinder().add(k);
 				}
@@ -152,27 +177,31 @@ public class AccounterstellungMitarbeiter {
 			else if (typMitarbeiter.isSelected()) {
 				neuer = new MitarbeiterAccount(String.format("%d",einmalpw.getText().hashCode()), accountname.getText(), email.getText());
 			}
+			//Ist kein Radio Button ausgewählt, wird hier abgebrochen
 			else {
 				return;
 			}
 
+			//Die Liste, zum Speichern der gespeicherten Liste wird zurückgesetzt, da die Kinder jetzt im Elternobjekt gespeichert werden
 			kinder.clear();
 
 			Gson gson = new Gson();
 
-			System.out.println(gson.toJson(neuer));
-			RestApiClient api = new RestApiClient();
+			//Das Elternelement wird in der DB gespeichert.
 			api.post("Benutzer", gson.toJson(neuer));
+
+			//handelt es sich um einen Elternaccount, werden alle Kind-Objekte in der DB gespeichert
 			if (neuer instanceof ElternAccount) {
 				for (Kind k : ((ElternAccount)neuer).getKinder()) {
+					//setzen des Fremdschlüssels bid
+					k.setBid(id);
 					api.post("Kind", gson.toJson(k));
 				}
 			}
+			status.setText("Daten Erfolgreich gespeichert");
 		}
 		else {
 			status.setText("Accounterstellung Fehlgeschlagen");
 		}
-
 	}
-
 }

src/main/java/de/subway_surfers/vpr_app/VerwaltungController.java

@@ -1,5 +1,7 @@
 package de.subway_surfers.vpr_app;
 
+import Logik.Account;
+import RestAPISchnittstelle.RestApiClient;
 import javafx.event.ActionEvent;
 import javafx.fxml.FXML;
 import javafx.fxml.FXMLLoader;