Ad
  • Custom User Avatar

    I like your use of enum here, hadn't thought of that.. (:

    that said, in my humble opinion, I wouldn't assign default values like that in the constructor arguments. :/ purely in the interest of clearity and readability, I feel it's better to do that in the constructor body.