Created
August 12, 2012 18:50
-
-
Save vsavkin/3333671 to your computer and use it in GitHub Desktop.
Roles
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
class TransferringMoney | |
include Context | |
... | |
def transfer amount | |
... | |
end | |
module SourceAccount | |
include ContextAccssor | |
def transfer_out amount | |
raise "Insufficient funds" if balance < amount | |
decrease_balance amount | |
context.destination_account.transfer_in amount | |
update_log "Transferred out", amount | |
end | |
end | |
module DestinationAccount | |
include ContextAccssor | |
def transfer_in amount | |
increase_balance amount | |
update_log "Transferred in", amount | |
end | |
end | |
end |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Not sure if I'm a fan of the roles accessing the context. It feels like a bit of a boundary issue to me and it's unnecessary. Why is
context.destination_account.transfer_in
better than:The other odd thing is both roles have the ContextAccssor mixin, but only one even uses it.