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

Jump to bottom

Discuss about Command pattern for Little Kai #8

Open
Cotel opened this issue Mar 12, 2017 · 2 comments
Open

Discuss about Command pattern for Little Kai #8

Cotel opened this issue Mar 12, 2017 · 2 comments
Assignees
@tonilopezmr @Cotel @srgtrujillo @pedrodaviddev @Starkie @ConradoMateu @josegarm @Martrout96 @frnlpz
Labels
discuss OOP

Comments

@Cotel
Copy link
Member

Cotel commented Mar 12, 2017

After implementing the pattern (yet to see if it is correct) I still don't see its utility.

I think it is much easier to call Cart methods directly in my example. After all, it is an interface and it is already decoupled. Superior layers in the abstraction won't know how this methods are implemented.

Please, enlighten me 🙏
@Cotel Cotel added the discuss label Mar 12, 2017
@tonilopezmr tonilopezmr mentioned this issue Mar 12, 2017
Merged
@tonilopezmr
Copy link
Member

You are alone @Cotel

@Cotel
Copy link
Member Author

Cotel commented Mar 28, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tonilopezmr tonilopezmr

@Cotel Cotel

@srgtrujillo srgtrujillo

@pedrodaviddev pedrodaviddev

@Starkie Starkie

@ConradoMateu ConradoMateu

@josegarm josegarm

@Martrout96 Martrout96

@frnlpz frnlpz

Labels
discuss OOP
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
9 participants
@tonilopezmr @Cotel @srgtrujillo @pedrodaviddev @Starkie @ConradoMateu @josegarm @Martrout96 @frnlpz