public static bool Validate(Person person)
{
// are names valid
if (string.IsNullOrWhiteSpace(person.FirstName))
{
StandardMessages.DisplayErrorValidationError("first name");
return(false);
}
if (string.IsNullOrWhiteSpace(person.LastName))
{
StandardMessages.DisplayErrorValidationError("last name");
return(false);
}
return(true);
}
// A class or method should have 1 and only 1 reason to change. Bad code was amended to follow SRP!
static void Main(string[] args)
{
StandardMessages.WelcomeMessage();
Person user = PersonDataCapture.Capture(); // this is tightly coupled will be corrected in another principle
bool isUserValid = PersonValidator.Validate(user);
if (!isUserValid)
{
StandardMessages.EndApplication();
return;
}
AccountGenerator.CreatAccount(user);
StandardMessages.EndApplication();
}