264dc9bbde
I expect a function called "is_numeric" or "is_integer" to check if a
variable is an integer or a number even if the variable passed by isn't
a string nor a number at all. Otherwise we should call them
is_string_a_number and is_string_an_integer and we have then to remove
the check for .is_a?(Number) and .is_a?(FixNum)
now checking also if it is a hex or octal number
improved/corrected checking for integer
* checking against Integer instead of Fixnum so that
also Bignum is matching
* now .is_a? Integer is done first so this is quiet fast
Now many types of numerics are recognized.
1. Float/Integer values (signed or unsigned, with exponent or without)
2. octal and hex check
3. except hex numbers and the "0." in a float lower than 1 can be prefixed
with a '0'.
whitespaces shouldn't be allowed as prefix/suffix
string representation of numbers should not contain any type of
whitespace.. the user is responsible to clean a string before checking
it..
fix documentation and added more checks
tried to be 99.9% backward compatible
* for now the decission is post poned if hex and octal numbers
should be allowed or not (is_numeric)
* native Bignum is now also a valid integer class
fix problem with old 1.8 ruby and Hash.to_s/Array.to_s
In ruby < 1.9 array and hashes would be recognized as numeric
if they have a special format:
1.8:
[1,2,3,4].to_s = "1234"
{1=>2}.to_s = "12"
1.9:
[1,2,3,4].to_s = "[1, 2, 3, 4]"
{1=>2}.to_s = "{1=>2}"
73 lines
1.8 KiB
Ruby
73 lines
1.8 KiB
Ruby
#
|
|
# is_numeric.rb
|
|
#
|
|
|
|
module Puppet::Parser::Functions
|
|
newfunction(:is_numeric, :type => :rvalue, :doc => <<-EOS
|
|
Returns true if the variable passed to this function is a number.
|
|
|
|
The function recognizes only integer and float but not hex or octal
|
|
numbers (for now) until a decision is made how to handle these types.
|
|
|
|
The parameter can be in the native format or given as string representation
|
|
of a number.
|
|
|
|
Valid examples:
|
|
|
|
77435
|
|
10e-12
|
|
-8475
|
|
0.2343
|
|
-23.561e3
|
|
EOS
|
|
) do |arguments|
|
|
|
|
if (arguments.size != 1) then
|
|
raise(Puppet::ParseError, "is_numeric(): Wrong number of arguments "+
|
|
"given #{arguments.size} for 1")
|
|
end
|
|
|
|
value = arguments[0]
|
|
|
|
# Regex is taken from the lexer of puppet
|
|
# puppet/pops/parser/lexer.rb but modified to match also
|
|
# negative values and disallow invalid octal numbers or
|
|
# numbers prefixed with multiple 0's (except in hex numbers)
|
|
#
|
|
# TODO these parameters should be constants but I'm not sure
|
|
# if there is no risk to declare them inside of the module
|
|
# Puppet::Parser::Functions
|
|
|
|
# TODO decide if this should be used
|
|
# HEX numbers like
|
|
# 0xaa230F
|
|
# 0X1234009C
|
|
# 0x0012
|
|
# -12FcD
|
|
#numeric_hex = %r{^-?0[xX][0-9A-Fa-f]+$}
|
|
|
|
# TODO decide if this should be used
|
|
# OCTAL numbers like
|
|
# 01234567
|
|
# -045372
|
|
#numeric_oct = %r{^-?0[1-7][0-7]*$}
|
|
|
|
# Integer/Float numbers like
|
|
# -0.1234568981273
|
|
# 47291
|
|
# 42.12345e-12
|
|
numeric = %r{^-?(?:(?:[1-9]\d*)|0)(?:\.\d+)?(?:[eE]-?\d+)?$}
|
|
|
|
if value.is_a? Numeric or (value.is_a? String and (
|
|
value.match(numeric) #or
|
|
# value.match(numeric_hex) or
|
|
# value.match(numeric_oct)
|
|
))
|
|
return true
|
|
else
|
|
return false
|
|
end
|
|
end
|
|
end
|
|
|
|
# vim: set ts=2 sw=2 et :
|