logo
Free, unlimited AI code reviews that run on commit
git-lrc git-lrc GitHub Install Now We'd appreciate a star git-lrc - Free, unlimited AI code reviews that run on commit | Product Hunt git-lrc - Free, unlimited AI code reviews that run on commit | Product Hunt

Perl::Critic::Policy::ValuesAndExpressions::RequireNumberSeparators - Write " 141_234_397.0145 " instead

Affiliation

       This Policy is part of the core Perl::Critic distribution.

Author

       Jeffrey Ryan Thalhammer <jeff@imaginative-software.com>

Configuration

       The minimum absolute value of numbers that must contain separators can be configured via the "min_value"
       option.  The default is 10,000; thus, all numbers >= 10,000 and <= -10,000 must have separators.  For
       example:

           [ValuesAndExpressions::RequireNumberSeparators]
           min_value = 100000    # That's one-hundred-thousand!

Description

       Long numbers can be difficult to read.  To improve legibility, Perl allows numbers to be split into
       groups of digits separated by underscores.  This policy requires number sequences of more than three
       digits to be separated.

           $long_int = 123456789;   #not ok
           $long_int = 123_456_789; #ok

           $long_float = 12345678.001;   #not ok
           $long_float = 12_345_678.001; #ok

Name

       Perl::Critic::Policy::ValuesAndExpressions::RequireNumberSeparators - Write " 141_234_397.0145 " instead
       of " 141234397.0145 ".

Notes

       As it is currently written, this policy only works properly with decimal (base 10) numbers.  And it is
       obviously biased toward Western notation.  I'll try and address those issues in the future.

See Also