♻️ refactor: aggregates' methods renamed to decide and evolve

Author: MateuszNaKodach

src/main/java/com/dddheroes/heroesofddd/armies/write/Army.java

@@ -31,7 +31,7 @@ class Army {
 
     @CommandHandler
     @CreationPolicy(AggregateCreationPolicy.CREATE_IF_MISSING) // performance downside in comparison to constructor
-    void handle(AddCreatureToArmy command) {
+    void decide(AddCreatureToArmy command) {
         new CanHaveMax7CreatureStacksInArmy(command.creatureId(), creatureStacks).verify();
 
         apply(
@@ -44,13 +44,13 @@ void handle(AddCreatureToArmy command) {
     }
 
     @EventSourcingHandler
-    void on(CreatureAddedToArmy event) {
+    void evolve(CreatureAddedToArmy event) {
         this.armyId = new ArmyId(event.armyId());
         creatureStacks.merge(new CreatureId(event.creatureId()), new Amount(event.quantity()), Amount::plus);
     }
 
     @CommandHandler
-    void handle(RemoveCreatureFromArmy command) {
+    void decide(RemoveCreatureFromArmy command) {
         new CanRemoveOnlyPresentCreatures(command.creatureId(), command.quantity(), creatureStacks).verify();
 
         apply(
@@ -63,7 +63,7 @@ void handle(RemoveCreatureFromArmy command) {
     }
 
     @EventSourcingHandler
-    void on(CreatureRemovedFromArmy event) {
+    void evolve(CreatureRemovedFromArmy event) {
         var creatureId = new CreatureId(event.creatureId());
         var currentQuantity = creatureStacks.get(creatureId);
         var removedQuantity = new Amount(event.quantity());

src/main/java/com/dddheroes/heroesofddd/calendar/write/Calendar.java

@@ -27,7 +27,7 @@ class Calendar {
     @CommandHandler
     @CreationPolicy(AggregateCreationPolicy.CREATE_IF_MISSING)
         // performance downside in comparison to constructor
-    void handle(StartDay command) {
+    void decide(StartDay command) {
         new CannotSkipDays(command, currentMonth, currentWeek, currentDay).verify();
 
         apply(
@@ -41,15 +41,15 @@ void handle(StartDay command) {
     }
 
     @EventSourcingHandler
-    void on(DayStarted event) {
+    void evolve(DayStarted event) {
         calendarId = new CalendarId(event.calendarId());
         currentMonth = new Month(event.month());
         currentWeek = new Week(event.week());
         currentDay = new Day(event.day());
     }
 
     @CommandHandler
-    void handle(FinishDay command) {
+    void decide(FinishDay command) {
         new CanOnlyFinishCurrentDay(command, currentMonth, currentWeek, currentDay).verify();
 
         apply(

src/main/java/com/dddheroes/heroesofddd/creaturerecruitment/write/Dwelling.java

@@ -33,7 +33,7 @@ class Dwelling {
 
     @CommandHandler
     @CreationPolicy(AggregateCreationPolicy.CREATE_IF_MISSING) // performance downside in comparison to constructor
-    void handle(BuildDwelling command) {
+    void decide(BuildDwelling command) {
         new OnlyNotBuiltBuildingCanBeBuild(dwellingId).verify();
 
         apply(
@@ -46,15 +46,15 @@ void handle(BuildDwelling command) {
     }
 
     @EventSourcingHandler
-    void on(DwellingBuilt event) {
+    void evolve(DwellingBuilt event) {
         this.dwellingId = new DwellingId(event.dwellingId());
         this.creatureId = new CreatureId(event.creatureId());
         this.costPerTroop = Cost.fromRaw(event.costPerTroop());
         this.availableCreatures = Amount.zero();
     }
 
     @CommandHandler
-    void handle(IncreaseAvailableCreatures command) {
+    void decide(IncreaseAvailableCreatures command) {
         new OnlyBuiltDwellingCanHaveAvailableCreatures(dwellingId).verify();
         // todo: check creatureId for the dwelling!
 
@@ -68,13 +68,13 @@ void handle(IncreaseAvailableCreatures command) {
     }
 
     @EventSourcingHandler
-    void on(AvailableCreaturesChanged event) {
+    void evolve(AvailableCreaturesChanged event) {
         this.availableCreatures = new Amount(event.changedTo());
     }
 
     @CommandHandler
 //    @CreationPolicy(AggregateCreationPolicy.CREATE_IF_MISSING)
-    void handle(RecruitCreature command) {
+    void decide(RecruitCreature command) {
 //        if(dwellingId == null){
 //            throw new DomainRule.ViolatedException("Only not built building can be build");
 //        }
@@ -97,7 +97,7 @@ void handle(RecruitCreature command) {
     }
 
     @EventSourcingHandler
-    void on(CreatureRecruited event) {
+    void evolve(CreatureRecruited event) {
         // todo: consider if it's OK or RecruitCreature should cause also AvailableCreaturesChanged event
         this.availableCreatures = this.availableCreatures.minus(new Amount(event.quantity()));
     }