On 2018-03-07 01:23 PM, Henrik Lindberg wrote:
> On 07/03/18 17:17, Gabriel Filion wrote:
>> I'm wondering if there's a current trend around style with parameter
>> data types. I couldn't find a clear mention of how this is generally
>> done in the community in the coding style guide on puppet.com.
>>
>> Is it better to align param names to the right of all data types in
>> class/define definitions like this?
>>
>> class blah (
>>    String[1]              $input,
>>    Boolean                $manage_x = true,
>>    Enum['hello', 'there'] $text     = 'hello',
>> }
>>
>> it seems to make things a lot clearer, however it can push arguments
>> quite far to the right side of the screen (and column count).

> Use type aliases if data type is long (having a name for the data type
> also makes it a lot easier to understand by a reader).

both Christopher and you are right about this one, aliases would help to
keep the width more manageable. I'll start playing around with those

> If you mix long and short names, an alternative to pushing all of them
> out is to cluster them at around 20 chars (if > 20 chars longer) let
> those be a group of their own.
> 
> class blah (
>    String[1] $input,
>    Boolean   $manage_x = true,
> 
>    Enum['hello', 'there', 'this', 'enum', 'long'] $text     = 'hello',
>    Enum['also', 'fish', 'sea', 'boat', ]          $text2    = 'boat',
> }

ah that's an interesting suggestion. thanks!

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Users" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to puppet-users+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/puppet-users/d5e05625-e1d3-634c-ee48-6883ec4bb66e%40lelutin.ca.
For more options, visit https://groups.google.com/d/optout.

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to