From a438ab75f1236799fcd9183fcce6d6db4d794472 Mon Sep 17 00:00:00 2001
From: Isahann Hanacleto <isahanacleto@gmail.com>
Date: Fri, 23 Aug 2024 17:01:00 -0300
Subject: [PATCH] #67 refactoring methods

---
 src/app/about-window/about-window.component.html       |  2 +-
 src/app/about-window/about-window.component.ts         |  7 +++----
 .../correct-answer-window.component.html               |  2 +-
 .../correct-answer-window.component.ts                 | 10 ++++------
 src/app/main-window/main-window.component.html         |  6 +++---
 src/app/main-window/main-window.component.ts           |  6 +-----
 src/app/score-window/score-window.component.html       |  2 +-
 src/app/score-window/score-window.component.ts         |  8 +++-----
 .../wrong-answer-window.component.html                 |  2 +-
 .../wrong-answer-window.component.ts                   | 10 ++++------
 10 files changed, 22 insertions(+), 33 deletions(-)

diff --git a/src/app/about-window/about-window.component.html b/src/app/about-window/about-window.component.html
index a162ad4..793e125 100644
--- a/src/app/about-window/about-window.component.html
+++ b/src/app/about-window/about-window.component.html
@@ -15,6 +15,6 @@
       at DeviantArt</a></b></label>
 
     <app-icon-button iconPath="home.png" title="Return to home"
-                     (onButtonClick)="this.returnHome()"></app-icon-button>
+                     (onButtonClick)="this.router.navigateByUrl(PathsEnum.HOME)"></app-icon-button>
   </div>
 </div>
diff --git a/src/app/about-window/about-window.component.ts b/src/app/about-window/about-window.component.ts
index ec3d417..6d30565 100644
--- a/src/app/about-window/about-window.component.ts
+++ b/src/app/about-window/about-window.component.ts
@@ -9,12 +9,11 @@ import {PathsEnum} from "../../model/enums/PathsEnum";
 })
 export class AboutWindowComponent {
 
+  protected readonly PathsEnum = PathsEnum;
+
   constructor(
-    private readonly router: Router
+    public readonly router: Router
   ) {
   }
 
-  public async returnHome(): Promise<void> {
-    await this.router.navigateByUrl(PathsEnum.HOME);
-  }
 }
diff --git a/src/app/correct-answer-window/correct-answer-window.component.html b/src/app/correct-answer-window/correct-answer-window.component.html
index 9ae8083..68afc07 100644
--- a/src/app/correct-answer-window/correct-answer-window.component.html
+++ b/src/app/correct-answer-window/correct-answer-window.component.html
@@ -10,7 +10,7 @@
       <app-icon-button iconPath="copy.png" title="Share result" [copy-clipboard]="this.clipboardText"
                        (onButtonClick)="this.showClipboardMessage()"></app-icon-button>
       <app-icon-button iconPath="home.png" title="Return to home"
-                       (onButtonClick)="this.returnHome()"></app-icon-button>
+                       (onButtonClick)="this.router.navigateByUrl(PathsEnum.HOME);"></app-icon-button>
     </div>
     <div *ngIf="this.displayClipboardMessage">
       <label class="window-body_clipboard-text">Question result copied to clipboard!</label>
diff --git a/src/app/correct-answer-window/correct-answer-window.component.ts b/src/app/correct-answer-window/correct-answer-window.component.ts
index 6693fb3..d38d57e 100644
--- a/src/app/correct-answer-window/correct-answer-window.component.ts
+++ b/src/app/correct-answer-window/correct-answer-window.component.ts
@@ -14,6 +14,8 @@ import {GameMode} from "../../model/enums/GameModesEnum";
 })
 export class CorrectAnswerWindowComponent implements OnInit {
 
+  protected readonly PathsEnum = PathsEnum;
+
   public questionScore: number = 0;
   public clipboardText: string = '';
   public displayClipboardMessage: boolean = false;
@@ -21,7 +23,7 @@ export class CorrectAnswerWindowComponent implements OnInit {
   private correctAnswerSound: HTMLAudioElement = new Audio('assets/sounds/tada.wav');
 
   constructor(
-    private readonly router: Router,
+    protected readonly router: Router,
     private readonly route: ActivatedRoute,
     private readonly encryptionService: EncryptionService,
     private readonly templateService: TemplateService,
@@ -33,7 +35,7 @@ export class CorrectAnswerWindowComponent implements OnInit {
     await this.retrieveRouteParams();
 
     if (!this.appStorageService.canQuizBeAnswered()) {
-      await this.returnHome();
+      await this.router.navigateByUrl(PathsEnum.HOME);
       return;
     }
 
@@ -41,10 +43,6 @@ export class CorrectAnswerWindowComponent implements OnInit {
     this.saveCurrentScore();
   }
 
-  public async returnHome(): Promise<void> {
-    await this.router.navigateByUrl(PathsEnum.HOME);
-  }
-
   public async showClipboardMessage(): Promise<void> {
     this.displayClipboardMessage = true;
 
diff --git a/src/app/main-window/main-window.component.html b/src/app/main-window/main-window.component.html
index 8932e2a..016681c 100644
--- a/src/app/main-window/main-window.component.html
+++ b/src/app/main-window/main-window.component.html
@@ -12,12 +12,12 @@
 
     <div class="window-body_buttons">
       <app-icon-button iconPath="quiz.png" title="New quiz"
-                       (onButtonClick)="this.redirectTo(PathsEnum.GAME_MODE)"
+                       (onButtonClick)="this.router.navigateByUrl(PathsEnum.GAME_MODE)"
                        [disabled]="!this.quizCanBeAnswered"></app-icon-button>
       <app-icon-button iconPath="score.png" title="My score"
-                       (onButtonClick)="this.redirectTo(PathsEnum.SCORES)"></app-icon-button>
+                       (onButtonClick)="this.router.navigateByUrl(PathsEnum.SCORES)"></app-icon-button>
       <app-icon-button iconPath="about.png" title="About"
-                       (onButtonClick)="this.redirectTo(PathsEnum.ABOUT)"></app-icon-button>
+                       (onButtonClick)="this.router.navigateByUrl(PathsEnum.ABOUT)"></app-icon-button>
     </div>
   </div>
 </div>
diff --git a/src/app/main-window/main-window.component.ts b/src/app/main-window/main-window.component.ts
index e8f2fe2..62cf31a 100644
--- a/src/app/main-window/main-window.component.ts
+++ b/src/app/main-window/main-window.component.ts
@@ -18,7 +18,7 @@ export class MainWindowComponent implements OnInit {
   protected readonly PathsEnum = PathsEnum;
 
   constructor(
-    private readonly router: Router,
+    protected readonly router: Router,
     private readonly appStorageService: AppStorageService
   ) {
   }
@@ -30,10 +30,6 @@ export class MainWindowComponent implements OnInit {
       this.startCountdown();
   }
 
-  public async redirectTo(route: PathsEnum): Promise<void> {
-    await this.router.navigateByUrl(route);
-  }
-
   private async startCountdown(): Promise<void> {
     const appStorage: AppStorage = this.appStorageService.retrieveAppStorage();
 
diff --git a/src/app/score-window/score-window.component.html b/src/app/score-window/score-window.component.html
index d6d6e07..97de466 100644
--- a/src/app/score-window/score-window.component.html
+++ b/src/app/score-window/score-window.component.html
@@ -67,7 +67,7 @@
       <app-icon-button iconPath="copy.png" title="Share score" [copy-clipboard]="this.clipboardText"
                        (click)="this.showClipboardMessage()"></app-icon-button>
       <app-icon-button iconPath="home.png" title="Return to home"
-                       (onButtonClick)="this.returnHome()"></app-icon-button>
+                       (onButtonClick)="this.router.navigateByUrl(PathsEnum.HOME)"></app-icon-button>
     </div>
     <div *ngIf="this.displayClipboardMessage">
       <label class="window-body_clipboard-text">Score copied to clipboard!</label>
diff --git a/src/app/score-window/score-window.component.ts b/src/app/score-window/score-window.component.ts
index 3841ce4..fb8754d 100644
--- a/src/app/score-window/score-window.component.ts
+++ b/src/app/score-window/score-window.component.ts
@@ -22,7 +22,7 @@ export class ScoreWindowComponent implements OnInit {
   public displayClipboardMessage: boolean = false;
 
   constructor(
-    private readonly router: Router,
+    protected readonly router: Router,
     private readonly templateService: TemplateService,
     private readonly appStorageService: AppStorageService
   ) {
@@ -33,10 +33,6 @@ export class ScoreWindowComponent implements OnInit {
     await this.assembleClipboardText();
   }
 
-  public async returnHome(): Promise<void> {
-    await this.router.navigateByUrl(PathsEnum.HOME);
-  }
-
   public async showClipboardMessage(): Promise<void> {
     this.displayClipboardMessage = true;
 
@@ -75,4 +71,6 @@ export class ScoreWindowComponent implements OnInit {
     this.currentYear = currentYear;
     this.currentWeek = currentWeek;
   }
+
+  protected readonly PathsEnum = PathsEnum;
 }
diff --git a/src/app/wrong-answer-window/wrong-answer-window.component.html b/src/app/wrong-answer-window/wrong-answer-window.component.html
index ae1fa6d..32c68fd 100644
--- a/src/app/wrong-answer-window/wrong-answer-window.component.html
+++ b/src/app/wrong-answer-window/wrong-answer-window.component.html
@@ -11,7 +11,7 @@
       <app-icon-button iconPath="copy.png" title="Share result" [copy-clipboard]="this.clipboardText"
                        (onButtonClick)="this.showClipboardMessage()"></app-icon-button>
       <app-icon-button iconPath="home.png" title="Return to home"
-                       (onButtonClick)="this.returnHome()"></app-icon-button>
+                       (onButtonClick)="this.router.navigateByUrl(PathsEnum.HOME)"></app-icon-button>
     </div>
     <div *ngIf="this.displayClipboardMessage">
       <label class="window-body_clipboard-text">Question result copied to clipboard!</label>
diff --git a/src/app/wrong-answer-window/wrong-answer-window.component.ts b/src/app/wrong-answer-window/wrong-answer-window.component.ts
index 4de4d60..e29e542 100644
--- a/src/app/wrong-answer-window/wrong-answer-window.component.ts
+++ b/src/app/wrong-answer-window/wrong-answer-window.component.ts
@@ -22,7 +22,7 @@ export class WrongAnswerWindowComponent implements OnInit {
   private wrongAnswerSound: HTMLAudioElement = new Audio('assets/sounds/critical_stop.wav');
 
   constructor(
-    private readonly router: Router,
+    protected readonly router: Router,
     private readonly route: ActivatedRoute,
     private readonly encryptionService: EncryptionService,
     private readonly templateService: TemplateService,
@@ -34,7 +34,7 @@ export class WrongAnswerWindowComponent implements OnInit {
     await this.retrieveRouteParams();
 
     if (!this.appStorageService.canQuizBeAnswered()) {
-      await this.returnHome();
+      await this.router.navigateByUrl(PathsEnum.HOME);
       return;
     }
 
@@ -42,10 +42,6 @@ export class WrongAnswerWindowComponent implements OnInit {
     this.saveCurrentScore();
   }
 
-  public async returnHome(): Promise<void> {
-    await this.router.navigateByUrl(PathsEnum.HOME);
-  }
-
   public async showClipboardMessage(): Promise<void> {
     this.displayClipboardMessage = true;
 
@@ -71,4 +67,6 @@ export class WrongAnswerWindowComponent implements OnInit {
     this.correctAnswers = 'questions' in questionResult ? questionResult.correctAnswers : [questionResult.rightAnswer];
     this.clipboardText = await this.templateService.render(gameMode.templateEnum, questionResult);
   }
+
+  protected readonly PathsEnum = PathsEnum;
 }