Using multiple constructors in the aggregate

[mehdihadeli]

Hi @kgrzybek ,
I have a question about this private constructor in the Metting.

        private Meeting()
        {
            _attendees = new List<MeetingAttendee>();
            _notAttendees = new List<MeetingNotAttendee>();
            _waitlistMembers = new List<MeetingWaitlistMember>();
        }

        private Meeting(
            MeetingGroupId meetingGroupId,
            string title,
            MeetingTerm term,
            string description,
            MeetingLocation location,
            MeetingLimits meetingLimits,
            Term rsvpTerm,
            MoneyValue eventFee,
            List<MemberId> hostsMembersIds,
            MemberId creatorId)
        {
            Id = new MeetingId(Guid.NewGuid());
            _meetingGroupId = meetingGroupId;
            _title = title;
            _term = term;
            _description = description;
            _location = location;
            _meetingLimits = meetingLimits;

            this.SetRsvpTerm(rsvpTerm, _term);
            _eventFee = eventFee;
            _creatorId = creatorId;
            _createDate = SystemClock.Now;

            _attendees = new List<MeetingAttendee>();
            _notAttendees = new List<MeetingNotAttendee>();
            _waitlistMembers = new List<MeetingWaitlistMember>();

            this.AddDomainEvent(new MeetingCreatedDomainEvent(this.Id));
            var rsvpDate = SystemClock.Now;
            if (hostsMembersIds.Any())
            {
                foreach (var hostMemberId in hostsMembersIds)
                {
                    _attendees.Add(MeetingAttendee.CreateNew(this.Id, hostMemberId, rsvpDate, MeetingAttendeeRole.Host, 0, MoneyValue.Undefined));
                }
            }
            else
            {
                _attendees.Add(MeetingAttendee.CreateNew(this.Id, creatorId, rsvpDate, MeetingAttendeeRole.Host, 0, MoneyValue.Undefined));
            }
        }

Why do we need this constructor? We can just call this functionality inner CreateNew static method, like below code and remove this parameters constructor:

  internal static Meeting CreateNew(
            MeetingGroupId meetingGroupId,
            string title,
            MeetingTerm term,
            string description,
            MeetingLocation location,
            MeetingLimits meetingLimits,
            Term rsvpTerm,
            MoneyValue eventFee,
            List<MemberId> hostsMembersIds,
            MemberId creatorId)
        {
            var metting = new Meeting();

            metting.Id = new MeetingId(Guid.NewGuid());
            metting._meetingGroupId = meetingGroupId;
            metting._title = title;
            metting._term = term;
            metting. _description = description;
            metting._location = location;
            metting._meetingLimits = meetingLimits;

            metting.SetRsvpTerm(rsvpTerm, _term);
            metting._eventFee = eventFee;
            metting._creatorId = creatorId;
            metting._createDate = SystemClock.Now;

            metting._attendees = new List<MeetingAttendee>();
            metting._notAttendees = new List<MeetingNotAttendee>();
            metting._waitlistMembers = new List<MeetingWaitlistMember>();

            metting.AddDomainEvent(new MeetingCreatedDomainEvent(metting.Id));
            var rsvpDate = SystemClock.Now;
            if (hostsMembersIds.Any())
            {
                foreach (var hostMemberId in hostsMembersIds)
                {
                    metting._attendees.Add(MeetingAttendee.CreateNew(this.Id, hostMemberId, rsvpDate, MeetingAttendeeRole.Host, 0, MoneyValue.Undefined));
                }
            }
            else
            {
                metting._attendees.Add(MeetingAttendee.CreateNew(this.Id, creatorId, rsvpDate, MeetingAttendeeRole.Host, 0, MoneyValue.Undefined));
            }
        }

It has less code :D