src/app/components/add-game/add-game.component.html

@@ -28,19 +28,19 @@
 			<table class="point-entry">
 				<tr *ngIf="this.actualPlayers[0]">
 					<td><p>{{this.actualPlayers[0].firstName}}</p></td>
-					<td><input type="number" [(ngModel)]="this.actualPlayers[0].finalCardScore" (change)="this.calculateCurrentScores()"/></td>
+					<td><input type="number" [(ngModel)]="this.actualPlayers[0].finalCardScore"/></td>
 				</tr>
 				<tr *ngIf="this.actualPlayers[1]">
 					<td><p>{{this.actualPlayers[1].firstName}}</p></td>
-					<td><input type="number" [(ngModel)]="this.actualPlayers[1].finalCardScore" (change)="this.calculateCurrentScores()"/></td>
+					<td><input type="number" [(ngModel)]="this.actualPlayers[1].finalCardScore"/></td>
 				</tr>
 				<tr *ngIf="this.actualPlayers[2]">
 					<td><p>{{this.actualPlayers[2].firstName}}</p></td>
-					<td><input type="number" [(ngModel)]="this.actualPlayers[2].finalCardScore" (change)="this.calculateCurrentScores()"/></td>
+					<td><input type="number" [(ngModel)]="this.actualPlayers[2].finalCardScore"/></td>
 				</tr>
 				<tr *ngIf="this.actualPlayers[3]">
 					<td><p>{{this.actualPlayers[3].firstName}}</p></td>
-					<td><input type="number" [(ngModel)]="this.actualPlayers[3].finalCardScore" (change)="this.calculateCurrentScores()"/></td>
+					<td><input type="number" [(ngModel)]="this.actualPlayers[3].finalCardScore"/></td>
 				</tr>
 			</table>
 			<p id="score-warn" *ngIf="calculatePointSum() !== 240">Total score doesn't add up ({{getScoreDifferenceText()}})</p>
@@ -55,7 +55,7 @@
 	</div>
 	<div id="scores">
 		<p class="team-{{player.team}}" *ngFor="let player of actualPlayers">
-			{{player.firstName}}: {{player.gamePoints}}
+			{{player.firstName}}: {{calculateCurrentScore(player)}}
 		</p>
 	</div>
 </div>

src/app/components/add-game/add-game.component.ts

@@ -31,42 +31,31 @@ export class AddGameComponent implements OnInit {
 			firebonkId: 1,
 			uuid: 'abc-def-ghi-j',
 			firstName: 'Patrick',
-			team: Team.CONTRA,
+			team: Team.CONTRA
-			gamePoints: 0,
-			finalCardScore: 0,
-			foxesCaught: 1
 		});
 		this.potentialPlayers.push({
 			firebonkId: 2,
 			uuid: 'abc-def-ghi-k',
 			firstName: 'Julian',
-			team: Team.CONTRA,
+			team: Team.CONTRA
-			gamePoints: 0,
-			finalCardScore: 0
 		});
 		this.potentialPlayers.push({
 			firebonkId: 3,
 			uuid: 'abc-def-ghi-l',
 			firstName: 'Yannick',
-			team: Team.CONTRA,
+			team: Team.CONTRA
-			gamePoints: 0,
-			finalCardScore: 0
 		});
 		this.potentialPlayers.push({
 			firebonkId: 4,
 			uuid: 'abc-def-ghi-m',
 			firstName: 'Janina',
-			team: Team.CONTRA,
+			team: Team.CONTRA
-			gamePoints: 0,
-			finalCardScore: 0
 		});
 		this.potentialPlayers.push({
 			firebonkId: 5,
 			uuid: 'abc-def-ghi-n',
 			firstName: 'Moritz',
-			team: Team.CONTRA,
+			team: Team.CONTRA
-			gamePoints: 0,
-			finalCardScore: 0
 		});
 
 		this.actualPlayers.push(...this.potentialPlayers.slice(0, 4));
@@ -76,10 +65,6 @@ export class AddGameComponent implements OnInit {
 	 * Switches the entry mask UI to the next page
 	 */
 	switchToNextPage(): void {
-		if (this.currentPage >= 4) {
-			this.calculateCurrentScores();
-		}
-
 		this.currentPage++;
 	}
 
@@ -163,7 +148,7 @@ export class AddGameComponent implements OnInit {
 	 */
 	deactivateLowerAnnouncements(announcement: announcements.Announcement) {
 		// First for RE
-		switch (announcement) {
+		switch(announcement) {
 			case announcements.Announcement.RE:
 				this.deactivateAnnouncement(announcements.Announcement.RE_NO_NINETY);
 				this.deactivateAnnouncement(announcements.Announcement.RE_NO_SIXTY);
@@ -179,7 +164,7 @@ export class AddGameComponent implements OnInit {
 		}
 
 		// Now for CONTRA
-		switch (announcement) {
+		switch(announcement) {
 			case announcements.Announcement.CONTRA:
 				this.deactivateAnnouncement(announcements.Announcement.CONTRA_NO_NINETY);
 				this.deactivateAnnouncement(announcements.Announcement.CONTRA_NO_SIXTY);
@@ -212,7 +197,7 @@ export class AddGameComponent implements OnInit {
 	 */
 	activateHigherAnnouncements(announcement: announcements.Announcement) {
 		// First for RE
-		switch (announcement) {
+		switch(announcement) {
 			case announcements.Announcement.RE_NO_THIRTY:
 				this.activateAnnouncement(announcements.Announcement.RE_NO_SIXTY);
 				this.activateAnnouncement(announcements.Announcement.RE_NO_NINETY);
@@ -228,7 +213,7 @@ export class AddGameComponent implements OnInit {
 		}
 
 		// Now for CONTRA
-		switch (announcement) {
+		switch(announcement) {
 			case announcements.Announcement.CONTRA_NO_THIRTY:
 				this.activateAnnouncement(announcements.Announcement.CONTRA_NO_SIXTY);
 				this.activateAnnouncement(announcements.Announcement.CONTRA_NO_NINETY);
@@ -277,22 +262,10 @@ export class AddGameComponent implements OnInit {
 		return announcements.checkValidity(this.selectedAnnouncements);
 	}
 
-	/**
-	 * Returns the two highest announcements for this game (one for RE, one for CONTRA, if applicable)
-	 */
 	getHighestAnnouncements(): string {
 		return announcements.returnTwoHighestAnnouncements(this.selectedAnnouncements);
 	}
 
-	/**
-	 * Checks if the given announcement has been selected
-	 * @param announcement The announcement to check
-	 */
-	checkAnnouncementActive(announcement: announcements.Announcement): boolean {
-		let index = this.selectedAnnouncements.indexOf(announcement);
-		return index !== -1;
-	}
-
 	/**
 	 *   ______
 	 *  /_  __/__  ____ _____ ___  _____
@@ -361,7 +334,7 @@ export class AddGameComponent implements OnInit {
 	getScoreDifferenceText(): string {
 		let difference = this.calculatePointSum() - 240;
 
-		if (difference > 0) {
+		if(difference > 0) {
 			return difference + ' more than expected';
 		} else {
 			difference *= -1;
@@ -378,64 +351,35 @@ export class AddGameComponent implements OnInit {
 	 */
 
 	/**
-	 * Calculates the game points for all players
+	 * Calculates the game points for the given player
+	 * @param player The player to calculate points for
 	 */
-	calculateCurrentScores(): void {
+	calculateCurrentScore(player: Player): number {
 		let gameScore: number = 0;
-		let winningTeamScore = this.getWinningTeamAndScore().score;
+		let teamScore = this.getTeamScore(player);
-		let winningTeam = this.getWinningTeamAndScore().team;
 
-		// We are going to calculate the points for the winning team and then set all players points accordingly
+		// Won? If so, by how much?
-
+		if (teamScore > 210) {
-		gameScore = 1; // 1 Point for Winning
+			gameScore += 4;
-
+		} else if (teamScore > 180) {
-		// Won by how much?
-		if (winningTeamScore > 210) {
 			gameScore += 3;
-		} else if (winningTeamScore > 180) {
+		} else if (teamScore > 150) {
 			gameScore += 2;
-		} else if (winningTeamScore > 150) {
+		} else if (teamScore > 120) {
 			gameScore += 1;
+		} else if (teamScore < 30) {
+			gameScore -= 4;
+		} else if (teamScore < 60) {
+			gameScore -= 3;
+		} else if (teamScore < 90) {
+			gameScore -= 2;
+		} else {
+			gameScore -= 1;
 		}
 
-		// Announcements
+		// TODO Announcements etc
-		if (winningTeam === Team.RE) {
-			if (this.checkAnnouncementActive(announcements.Announcement.RE_NO_NINETY) && winningTeamScore > 150) {
-				gameScore += 1;
-			}
-			if (this.checkAnnouncementActive(announcements.Announcement.RE_NO_SIXTY) && winningTeamScore > 180) {
-				gameScore += 1;
-			}
-			if (this.checkAnnouncementActive(announcements.Announcement.RE_NO_THIRTY) && winningTeamScore > 210) {
-				gameScore += 1;
-			}
-		}
-		if (winningTeam === Team.CONTRA) {
-			if (this.checkAnnouncementActive(announcements.Announcement.CONTRA_NO_NINETY) && winningTeamScore > 150) {
-				gameScore += 1;
-			}
-			if (this.checkAnnouncementActive(announcements.Announcement.CONTRA_NO_SIXTY) && winningTeamScore > 180) {
-				gameScore += 1;
-			}
-			if (this.checkAnnouncementActive(announcements.Announcement.CONTRA_NO_THIRTY) && winningTeamScore > 210) {
-				gameScore += 1;
-			}
-		}
 
-		// Double Score in case of announcement
+		return gameScore;
-		if (winningTeam === Team.RE && this.checkAnnouncementActive(announcements.Announcement.RE)) {
-			gameScore *= 2;
-		}
-		if (winningTeam === Team.CONTRA && this.checkAnnouncementActive(announcements.Announcement.CONTRA)) {
-			gameScore *= 2;
-		}
-
-		// TODO: Bonus points
-		gameScore += this.getFinalFoxPoints(winningTeam);
-
-		// TODO: Check for announcements that have not been fulfilled!
-
-		this.setGameScores(gameScore, winningTeam);
 	}
 
 	/**
@@ -453,85 +397,4 @@ export class AddGameComponent implements OnInit {
 
 		return totalTeamScore;
 	}
-
-	/**
-	 * Calculates which team won and returns the Team Name and the achieved score
-	 */
-	getWinningTeamAndScore(): { team: Team, score: number } {
-		let elderPoints: number = 0;
-		let otherPoints: number = 0;
-
-		for (let player of this.actualPlayers) {
-			if (player.team === Team.RE) {
-				elderPoints += player.finalCardScore ?? 0;
-			} else {
-				otherPoints += player.finalCardScore ?? 0;
-			}
-		}
-
-		if (elderPoints > otherPoints) {
-			return {
-				team: Team.RE,
-				score: elderPoints
-			};
-		}
-		return {
-			team: Team.CONTRA,
-			score: otherPoints
-		};
-	}
-
-	/**
-	 * Sets the game scores for all players
-	 * @param score The score to set
-	 * @param winningTeam The team that won
-	 */
-	setGameScores(score: number, winningTeam: Team) {
-		for (let player of this.actualPlayers) {
-			if (player.team === winningTeam) {
-				player.gamePoints = score;
-			} else {
-				player.gamePoints = -score;
-			}
-		}
-	}
-
-	/**
-	 * Calculates the points that each team got by catching foxes
-	 */
-	calculateFoxPoints(): { re: number, contra: number } {
-		let reFoxesCaught: number = 0;
-		let contraFoxesCaught: number = 0;
-
-		for(let player of this.actualPlayers) {
-			if(player.team === Team.RE) {
-				reFoxesCaught += player.foxesCaught ?? 0;
-			} else {
-				contraFoxesCaught += player.foxesCaught ?? 0;
-			}
-		}
-
-		return {
-			re: reFoxesCaught,
-			contra: contraFoxesCaught
-		};
-	}
-
-	/**
-	 * Calculates the final fox points for the winning team
-	 * @param winningTeam The winning team
-	 */
-	getFinalFoxPoints(winningTeam: Team): number {
-		let finalPoints = 0;
-
-		if(winningTeam === Team.RE) {
-			finalPoints += this.calculateFoxPoints().re;
-			finalPoints -= this.calculateFoxPoints().contra;
-		} else {
-			finalPoints += this.calculateFoxPoints().contra;
-			finalPoints -= this.calculateFoxPoints().re;
-		}
-
-		return finalPoints;
-	}
 }

src/app/models/doppelkopf/enums/announcement.ts

@@ -15,12 +15,12 @@ export enum Announcement {
 export function getAllAnnouncementValues(): Announcement[] {
 	return [
 		Announcement.RE,
-		Announcement.RE_NO_NINETY,
-		Announcement.RE_NO_SIXTY,
-		Announcement.RE_NO_THIRTY,
 		Announcement.CONTRA,
+		Announcement.RE_NO_NINETY,
 		Announcement.CONTRA_NO_NINETY,
+		Announcement.RE_NO_SIXTY,
 		Announcement.CONTRA_NO_SIXTY,
+		Announcement.RE_NO_THIRTY,
 		Announcement.CONTRA_NO_THIRTY
 	];
 }
@@ -28,7 +28,8 @@ export function getAllAnnouncementValues(): Announcement[] {
 /**
  * Checks if the selected announcements are a valid set of announcements.
  * E.g.: If RE_NO_NINETY is newly selected, RE also has to be selected
- * @param selectedAnnouncements The list of selected announcements
+ * @param alreadySelected
+ * @param newSelected
  */
 export function checkValidity(selectedAnnouncements: Announcement[]): boolean {
 	// First check all "RE" Announcements
@@ -69,10 +70,6 @@ export function checkValidity(selectedAnnouncements: Announcement[]): boolean {
 	return true
 }
 
-/**
- * Returns the names of the two highest announcements from the given list, one for RE and one for CONTRA if applicable
- * @param selectedAnnouncements The list of announcements to check
- */
 export function returnTwoHighestAnnouncements(selectedAnnouncements: Announcement[]): string {
 	let finalString: string = '';
 

src/app/models/doppelkopf/game.ts

@@ -5,6 +5,8 @@ import {Solo} from './enums/solo';
 export interface Game {
 	gameId: number;
 	players: Player[];
+	foxesCaught: number;
+	announcements: Announcement[];
 	solo?: Solo;
 	againstTheElders: boolean;
 }

src/app/models/doppelkopf/player.ts

@@ -4,10 +4,9 @@ export interface Player {
 	firebonkId: number;
 	uuid: string;
 	firstName: string;
-	finalCardScore: number;
+	finalCardScore?: number;
-	gamePoints: number;
+	gamePoints?: number;
 	hadMarriage?: boolean;
 	hadTrumpHandoff?: boolean;
 	team?: Team;
-	foxesCaught?: number;
 }