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.

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

Batch : ACADEMY OCT-NOV 20 SUPERX , LLD1 , email : bhardwaj.kusho2@gmail.com #106

Open
wants to merge 4 commits into
base: Apri20
Choose a base branch
from

Conversation

kushb2
Copy link

@kushb2 kushb2 commented Apr 8, 2021

HW:

  1. Refactor this code such that it does not violate the SRP principle.
  2. Refactor this code such that it does not violate the LSP principle.
  3. You are given the implementation for this business requirement here. Change it for the new requirements.

…ail.com

Batch :  ACADEMY OCT-NOV 20 SUPERX , LLD1 , email : bhardwaj.kusho2@gmail.com
HW:

1. Refactor this code such that it does not violate the SRP principle.
2. Refactor this code such that it does not violate the LSP principle.
3. You are given the implementation for this business requirement here. Change it for the new requirements.
…ail.com

Batch :  ACADEMY OCT-NOV 20 SUPERX , LLD1 , email : bhardwaj.kusho2@gmail.com
HW:

1. Refactor this code such that it does not violate the SRP principle.
2. Refactor this code such that it does not violate the LSP principle.
3. You are given the implementation for this business requirement here. Change it for the new requirements.
EmployeeRepository  open for  extension , other process of saving record.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant