Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

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

Long Parameter List → Introduced a DTO (BookingDetails). #66

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Long Parameter List → Introduced a DTO (BookingDetails). #66

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
84 changes: 45 additions & 39 deletions examples/csharp/csharp-booking-01_base/src/Booking/Booking.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,46 +2,52 @@

namespace CodelyTv.Booking
{
public sealed class Booking
public class BookingDetails
{
public BookingId Id { get; }
public DateTime StartDate { get; }
public DateTime EndDate { get; }
public CustomerId CustomerId { get; }
public CustomerName CustomerName { get; }
public EmailAddress CustomerEmail { get; }
public BookingType BookingType { get; }
public DiscountType DiscountType { get; }
public DiscountValue DiscountValue { get; }
public TaxType TaxType { get; }
public TaxValue TaxValue { get; }

public BookingDetails(
BookingId id, DateTime startDate, DateTime endDate, CustomerId customerId,
CustomerName customerName, EmailAddress customerEmail, BookingType bookingType,
DiscountType discountType, DiscountValue discountValue, TaxType taxType, TaxValue taxValue)
{
private readonly BookingId id;
private readonly DateTime startDate;
private readonly DateTime endDate;
private readonly CustomerId customerId;
private readonly CustomerName customerName;
private readonly EmailAddress customerEmail;
private readonly BookingType bookingType;
private readonly DiscountType discountType;
private readonly DiscountValue discountValue;
private readonly TaxType taxType;
private readonly TaxValue taxValue;
Id = id;
StartDate = startDate;
EndDate = endDate;
CustomerId = customerId;
CustomerName = customerName;
CustomerEmail = customerEmail;
BookingType = bookingType;
DiscountType = discountType;
DiscountValue = discountValue;
TaxType = taxType;
TaxValue = taxValue;
}

public Booking(
BookingId id,
DateTime startDate,
DateTime endDate,
CustomerId customerId,
CustomerName customerName,
EmailAddress customerEmail,
BookingType bookingType,
DiscountType discountType,
DiscountValue discountValue,
TaxType taxType,
TaxValue taxValue
)
{
this.id = id;
this.startDate = startDate;
this.endDate = endDate;
this.customerId = customerId;
this.customerName = customerName;
this.customerEmail = customerEmail;
this.bookingType = bookingType;
this.discountType = discountType;
this.discountValue = discountValue;
this.taxType = taxType;
this.taxValue = taxValue;
}
public Booking(BookingDetails details)
{
this.id = details.Id;
this.startDate = details.StartDate;
this.endDate = details.EndDate;
this.customerId = details.CustomerId;
this.customerName = details.CustomerName;
this.customerEmail = details.CustomerEmail;
this.bookingType = details.BookingType;
this.discountType = details.DiscountType;
this.discountValue = details.DiscountValue;
this.taxType = details.TaxType;
this.taxValue = details.TaxValue;
}

public BookingStatus StatusFor(DateTime date)
{
Expand All @@ -58,4 +64,4 @@ public BookingStatus StatusFor(DateTime date)
return BookingStatus.FINISHED;
}
}
}
}