refactor: fluent builder pattern (ardalis#845)

* Fluent builder pattern

* Update sample/src/NimblePros.SampleToDo.Core/ProjectAggregate/Project.cs

* Update src/Clean.Architecture.Core/ContributorAggregate/Contributor.cs

---------

Co-authored-by: Steve Smith <[email protected]>

Author: aminsharifi

sample/src/NimblePros.SampleToDo.Core/ContributorAggregate/Contributor.cs

@@ -8,14 +8,15 @@ public class Contributor : EntityBase, IAggregateRoot
 
   public Contributor(ContributorName name)
   {
-    Name = name;
+      Name = name;
   }
 
-  public void UpdateName(ContributorName newName)
+  public Contributor UpdateName(ContributorName newName)
   {
     if (Name.Equals(newName)) return;
     Name = newName;
     this.RegisterDomainEvent(new ContributorNameUpdatedEvent(this));
+    return this;
   }
 }
 

sample/src/NimblePros.SampleToDo.Core/ProjectAggregate/Project.cs

@@ -6,7 +6,7 @@ public class Project : EntityBase<Project, ProjectId>, IAggregateRoot
 {
   public ProjectName Name { get; private set; }
 
-  private readonly List<ToDoItem> _items = new();
+  private readonly List<ToDoItem> _items = [];
   public IEnumerable<ToDoItem> Items => _items.AsReadOnly();
   public ProjectStatus Status => _items.All(i => i.IsDone) ? ProjectStatus.Complete : ProjectStatus.InProgress;
 
@@ -15,17 +15,19 @@ public Project(ProjectName name)
     Name = name;
   }
 
-  public void AddItem(ToDoItem newItem)
+  public Project AddItem(ToDoItem newItem)
   {
     Guard.Against.Null(newItem);
     _items.Add(newItem);
 
     var newItemAddedEvent = new NewItemAddedEvent(this, newItem);
     base.RegisterDomainEvent(newItemAddedEvent);
+    return this;
   }
 
-  public void UpdateName(ProjectName newName)
+  public Project UpdateName(ProjectName newName)
   {
     Name = newName;
+    return this;
   }
 }

sample/src/NimblePros.SampleToDo.Core/ProjectAggregate/ToDoItem.cs

@@ -20,29 +20,31 @@ public ToDoItem(Priority priority)
 
   public Priority Priority { get; private set; }
 
-
-  public void MarkComplete()
+  public ToDoItem MarkComplete()
   {
     if (!IsDone)
     {
       IsDone = true;
 
       RegisterDomainEvent(new ToDoItemCompletedEvent(this));
     }
+    return this;
   }
 
-  public void AddContributor(int contributorId)
+  public ToDoItem AddContributor(int contributorId)
   {
     Guard.Against.Null(contributorId);
     ContributorId = contributorId;
 
     var contributorAddedToItem = new ContributorAddedToItemEvent(this, contributorId);
     base.RegisterDomainEvent(contributorAddedToItem);
+    return this;
   }
 
-  public void RemoveContributor()
+  public ToDoItem RemoveContributor()
   {
     ContributorId = null;
+    return this;
   }
 
   public override string ToString()

src/Clean.Architecture.Core/ContributorAggregate/Contributor.cs

@@ -1,21 +1,28 @@
 namespace Clean.Architecture.Core.ContributorAggregate;
 
-public class Contributor(string name) : EntityBase, IAggregateRoot
+public class Contributor : EntityBase, IAggregateRoot
 {
-  // Example of validating primary constructor inputs
-  // See: https://learn.microsoft.com/en-us/dotnet/csharp/whats-new/tutorials/primary-constructors#initialize-base-class
-  public string Name { get; private set; } = Guard.Against.NullOrEmpty(name, nameof(name));
+  public Contributor(string name)
+  {
+    UpdateName(name); // TODO: Replace with value object and use primary constructor to populate field.
+  }
+  public string Name { get; private set; } = default!;
   public ContributorStatus Status { get; private set; } = ContributorStatus.NotSet;
   public PhoneNumber? PhoneNumber { get; private set; }
+  public Contributor SetPhoneNumber(string phoneNumber)
+  {
+    PhoneNumber = new PhoneNumber(string.Empty, phoneNumber, string.Empty);
+    return this;
+  }
 
-  public void SetPhoneNumber(string phoneNumber) => PhoneNumber = new PhoneNumber(string.Empty, phoneNumber, string.Empty);
-
-  public void UpdateName(string newName) => Name = Guard.Against.NullOrEmpty(newName, nameof(newName));
+  public Contributor UpdateName(string newName)
+  {
+    Name = Guard.Against.NullOrEmpty(newName, nameof(newName));
+    return this;
+  }
 }
 
-public class PhoneNumber(string countryCode,
-  string number,
-  string? extension) : ValueObject
+public class PhoneNumber(string countryCode, string number, string? extension) : ValueObject
 {
   public string CountryCode { get; private set; } = countryCode;
   public string Number { get; private set; } = number;