{"comment":{"url":"https://api.github.com/repos/SeriousBusiness/Lost-Cities/comments/10456959","html_url":"https://github.com/SeriousBusiness/Lost-Cities/commit/c6f1427aa34e85b53ed3ce056587dc3c8b4095d1#commitcomment-10456959","id":10456959,"user":{"login":"Pertus","id":10560961,"avatar_url":"https://avatars.githubusercontent.com/u/10560961?v=3","gravatar_id":"","url":"https://api.github.com/users/Pertus","html_url":"https://github.com/Pertus","followers_url":"https://api.github.com/users/Pertus/followers","following_url":"https://api.github.com/users/Pertus/following{/other_user}","gists_url":"https://api.github.com/users/Pertus/gists{/gist_id}","starred_url":"https://api.github.com/users/Pertus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Pertus/subscriptions","organizations_url":"https://api.github.com/users/Pertus/orgs","repos_url":"https://api.github.com/users/Pertus/repos","events_url":"https://api.github.com/users/Pertus/events{/privacy}","received_events_url":"https://api.github.com/users/Pertus/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"c6f1427aa34e85b53ed3ce056587dc3c8b4095d1","created_at":"2015-03-29T12:11:20Z","updated_at":"2015-03-29T12:11:20Z","body":"My idea for having both TakeCard() in Player and TakeTopCard() in Deck is to reduce couplings. The Game class could then be the central hub and give a card to a player by calling activePlayer.TakeCard(deck.TakeTopCard()); which gives the effect of no coupling between the Player class and the Deck class. The idea is that everything goes through the Game Class. I don't know if this is a good solution, maybe it's overly complicated?"}}