We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hi Oskar, I have a question about this private constructor in th Order.
Order
private Order(Guid id, Guid clientId, IReadOnlyList<PricedProductItem> productItems, decimal totalPrice) { var @event = OrderInitialized.Create( id, clientId, productItems, totalPrice, DateTime.UtcNow ); Enqueue(@event); Apply(@event); }
Why do you need this constructor? We can just call this functionality inner Initialize static method, like below code and remove this parameters constructor:
public static Order Initialize( Guid orderId, Guid clientId, IReadOnlyList<PricedProductItem> productItems, decimal totalPrice) { var order = new Order(); var @event = OrderInitialized.Create( id, clientId, productItems, totalPrice, DateTime.UtcNow ); order.Enqueue(@event); order.Apply(@event); return order; }
It is less code :D
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Hi Oskar,
I have a question about this private constructor in th
Order
.Why do you need this constructor? We can just call this functionality inner Initialize static method, like below code and remove this parameters constructor:
It is less code :D
The text was updated successfully, but these errors were encountered: