Ewout Quax ewoutquax
ewoutquax
/ gist:6547954
Last active
December 22, 2015 23:39
Dit is mijn refactor-resultaat;
variabelen hebben een naam gekregen die zoveel mogelijk hun intentie aangeeft. If-statements zijn verplaatst naar functies die aangeven wat er gecontroleerd/uitgevoerd wordt.
Ik heb geen testen gedraaid, dus het kan zijn dat de werking nu anders is ;-)
This file contains hidden or 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
| require "minitest/autorun" | |
| require "minitest/pride" | |
| class Hash | |
| def deep_merge(other_hash, required_key = nil) | |
| dup.deep_merge!(other_hash, required_key) | |
| end | |
| def deep_merge!(other_hash, required_key = nil) |