lib/Moose/Manual/BestPractices.pod

   1 =pod
   2
   3 =head1 NAME
   4
   5 Moose::Manual::BestPractices - Get the most out of Moose
   6
   7 =head1 RECOMMENDATIONS
   8
   9 Moose has a lot of features, and there's definitely more than one way
  10 to do it. However, we think that picking a subset of these features
  11 and using them consistently makes everyone's life easier.
  12
  13 Of course, as with any list of "best practices", these are really just
  14 opinions. Feel free to ignore us.
  15
  16 =head2 C<no Moose> and immutabilize
  17
  18 We recommend that you end your Moose class definitions by removing the
  19 Moose sugar and making your class immutable.
  20
  21   package Person;
  22
  23   use Moose;
  24
  25   # extends, roles, attributes, etc.
  26
  27   # methods
  28
  29   no Moose;
  30
  31   __PACKAGE__->meta->make_immutable;
  32
  33   1;
  34
  35 The C<no Moose> bit simply good code hygiene, as it removes all the
  36 Moose keywords that are no longer needed once your class has been
  37 built.  C<make_immutable> relinquishes your right to make further
  38 changes to your class, and allows Moose to speed up a lot of things,
  39 most notably object construction.
  40
  41 A more generic way to unimport not only L<Moose>'s exports but also those from
  42 type libraries and other modules is L<namespace::clean> and
  43 L<namespace::autoclean>.
  44
  45 =head2 Never override C<new>
  46
  47 Overriding C<new> is a very bad practice. Instead, you should use a
  48 C<BUILD> or C<BUILDARGS> methods to do the same thing. When you
  49 override C<new>, Moose can no longer inline a constructor when your
  50 class is immutabilized.
  51
  52 There are two good reasons to override C<new>. One, you are writing a
  53 MooseX extension that provides its own L<Moose::Object> subclass
  54 I<and> a subclass of L<Moose::Meta::Method::Constructor> to inline the
  55 constructor. Two, you are subclassing a non-Moose parent.
  56
  57 If you know how to do that, you know when to ignore this best practice
  58 ;)
  59
  60 =head2 Always call C<SUPER::BUILDARGS>
  61
  62 If you override the C<BUILDARGS> method in your class, make sure to
  63 play nice and call C<SUPER::BUILDARGS> to handle cases you're not
  64 checking for explicitly.
  65
  66 The default C<BUILDARGS> method in L<Moose::Object> handles both a
  67 list and hashref of named parameters correctly, and also checks for a
  68 I<non-hashref> single argument.
  69
  70 =head2 Provide defaults whenever possible, otherwise use C<required>
  71
  72 When your class provides defaults, this makes constructing new objects
  73 simpler. If you cannot provide a default, consider making the
  74 attribute C<required>.
  75
  76 If you don't do either, an attribute can simply be left unset,
  77 increasing the complexity of your object, because it has more possible
  78 states that you or the user of your class must account for.
  79
  80 =head2 Use C<builder> instead of C<default> most of the time
  81
  82 Builders can be inherited, they have explicit names, and they're just
  83 plain cleaner.
  84
  85 However, I<do> use a default when the default is a non-reference,
  86 I<or> when the default is simply an empty reference of some sort.
  87
  88 Also, keep your builder methods private.
  89
  90 =head2 Use C<lazy_build>
  91
  92 Lazy is good, and often solves initialization ordering problems. It's
  93 also good for deferring work that may never have to be done. If you're
  94 going to be lazy, use I<lazy_build> to save yourself some typing and
  95 standardize names.
  96
  97 =head2 Consider keeping clearers and predicates private
  98
  99 Does everyone I<really> need to be able to clear an attribute?
 100 Probably not. Don't expose this functionality outside your class
 101 by default.
 102
 103 Predicates are less problematic, but there's no reason to make your
 104 public API bigger than it has to be.
 105
 106 =head2 Default to read-only, and consider keeping writers private
 107
 108 Making attributes mutable just means more complexity to account for in
 109 your program. The alternative to mutable state is to encourage users
 110 of your class to simply make new objects as needed.
 111
 112 If you I<must> make an attribute read-write, consider making the
 113 writer a separate private method. Narrower APIs are easy to maintain,
 114 and mutable state is trouble.
 115
 116 In order to declare such attributes, provide a private C<writer>
 117 parameter:
 118
 119     has pizza => (
 120         is     => 'ro',
 121         isa    => 'Pizza',
 122         writer => '_pizza',
 123     );
 124
 125 =head2 Think twice before changing an attribute's type in a subclass
 126
 127 Down this path lies great confusion. If the attribute is an object
 128 itself, at least make sure that it has the same interface as the type
 129 of object in the parent class.
 130
 131 =head2 Don't use the C<initializer> feature
 132
 133 Don't know what we're talking about? That's fine.
 134
 135 =head2 Use L<MooseX::AttributeHelpers> instead of C<auto_deref>
 136
 137 The C<auto_deref> feature is a bit troublesome. Directly exposing a
 138 complex attribute is ugly. Instead, consider using
 139 L<MooseX::AttributeHelpers> to define an API that exposes those pieces
 140 of functionality that need exposing. Then you can expose just the
 141 functionality that you want.
 142
 143 =head2 Always call C<inner> in the most specific subclass
 144
 145 When using C<augment> and C<inner>, we recommend that you call
 146 C<inner> in the most specific subclass of your hierarchy. This makes
 147 it possible to subclass further and extend the hierarchy without
 148 changing the parents.
 149
 150 =head2 Namespace your types
 151
 152 Use some sort of namespacing convention for type names. We recommend
 153 something like "MyApp::Type::Foo".
 154
 155 If you're intending to package your types up for re-use using
 156 L<MooseX::Types> later, avoid using characters that are invalid in
 157 perl identifiers such as a space or period.
 158
 159 =head2 Do not coerce Moose built-ins directly
 160
 161 If you define a coercion for a Moose built-in like C<ArrayRef>, this
 162 will affect every application in the Perl interpreter that uses this
 163 type.
 164
 165     # very naughty!
 166     coerce 'ArrayRef'
 167         => from Str
 168         => via { [ split /,/ ] };
 169
 170 Instead, create a subtype and coerce that:
 171
 172     subtype 'My::ArrayRef' => as 'ArrayRef';
 173
 174     coerce 'My::ArrayRef'
 175         => from 'Str'
 176         => via { [ split /,/ ] };
 177
 178 =head2 Do not coerce class names directly
 179
 180 Just as with Moose built-in types, a class type is global for the
 181 entire interpreter. If you add a coercion for that class name, it can
 182 have magical side effects elsewhere:
 183
 184     # also very naughty!
 185     coerce 'HTTP::Headers'
 186         => from 'HashRef'
 187         => via { HTTP::Headers->new( %{$_} ) };
 188
 189 Instead, we can create an "empty" subtype for the coercion:
 190
 191     subtype 'My::HTTP::Headers' => as class_type('HTTP::Headers');
 192
 193     coerce 'My::HTTP::Headers'
 194         => from 'HashRef'
 195         => via { HTTP::Headers->new( %{$_} ) };
 196
 197 =head2 Use coercion instead of unions
 198
 199 Consider using a type coercion instead of a type union. This was
 200 covered at length in L<Moose::Manual::Types>.
 201
 202 =head2 Define all your types in one module
 203
 204 Define all your types and coercions in one module. This was also
 205 covered in L<Moose::Manual::Types>.
 206
 207 =head1 BENEFITS OF BEST PRACTICES
 208
 209 Following these practices has a number of benefits.
 210
 211 It helps ensure that your code will play nice with others, making it
 212 more reusable and easier to extend.
 213
 214 Following an accepted set of idioms will make maintenance easier,
 215 especially when someone else has to maintain your code. It will also
 216 make it easier to get support from other Moose users, since your code
 217 will be easier to digest quickly.
 218
 219 Some of these practices are designed to help Moose do the right thing,
 220 especially when it comes to immutabilization. This means your code
 221 will be faster when immutabilized.
 222
 223 Many of these practices also help get the most out of meta
 224 programming. If you used an overridden C<new> to do type coercion by
 225 hand, rather than defining a real coercion, there is no introspectable
 226 metadata. This sort of thing is particularly problematic for MooseX
 227 extensions which rely on introspection to do the right thing.
 228
 229 =head1 AUTHOR
 230
 231 Yuval (nothingmuch) Kogman
 232
 233 Dave Rolsky E<lt>autarch@urth.orgE<gt>
 234
 235 =head1 COPYRIGHT AND LICENSE
 236
 237 Copyright 2009 by Infinity Interactive, Inc.
 238
 239 L<http://www.iinteractive.com>
 240
 241 This library is free software; you can redistribute it and/or modify
 242 it under the same terms as Perl itself.
 243
 244 =cut