refactor: extract calculation into a function, recall on onSafeWithdrawalRateChange

apps/client/src/app/pages/portfolio/fire/fire-page.component.ts

@@ -178,6 +178,7 @@ export class GfFirePageComponent implements OnDestroy, OnInit {
             this.user = user;
 
             this.calculateWithdrawalRates();
+            this.calculateProjectedWithdrawalRates();
 
             this.changeDetectorRef.markForCheck();
           });
@@ -221,19 +222,7 @@ export class GfFirePageComponent implements OnDestroy, OnInit {
 
   public onCalculationComplete(calculation: FireCalculation) {
     this.fireCalculation = calculation;
-
+    this.calculateProjectedWithdrawalRates();
-    if (
-      this.fireWealth &&
-      this.user?.settings?.safeWithdrawalRate &&
-      calculation.projectedTotalAmount
-    ) {
-      this.projectedWithdrawalRatePerYear = new Big(
-        calculation.projectedTotalAmount
-      ).mul(this.user.settings.safeWithdrawalRate);
-
-      this.projectedWithdrawalRatePerMonth =
-        this.projectedWithdrawalRatePerYear.div(12);
-    }
   }
 
   public ngOnDestroy() {
@@ -250,4 +239,19 @@ export class GfFirePageComponent implements OnDestroy, OnInit {
       this.withdrawalRatePerMonth = this.withdrawalRatePerYear.div(12);
     }
   }
+
+  private calculateProjectedWithdrawalRates() {
+    if (
+      this.fireWealth &&
+      this.user?.settings?.safeWithdrawalRate &&
+      this.fireCalculation.projectedTotalAmount
+    ) {
+      this.projectedWithdrawalRatePerYear = new Big(
+        this.fireCalculation.projectedTotalAmount
+      ).mul(this.user.settings.safeWithdrawalRate);
+
+      this.projectedWithdrawalRatePerMonth =
+        this.projectedWithdrawalRatePerYear.div(12);
+    }
+  }
 }