Today I\'ve stumbled upon a tricky issue with Ruby constants. In our team someone created a module, which is included into multiple models. In our (spec) test output this result
I encountered this same problem today and found a simple solution.
Since the warning is from trying to reassign a constant with its same value, I just changed
module LifeCycle
RESET = 'reset'
end
to
module LifeCycle
RESET ||= 'reset'
end
This took care of the warning and is a lot simpler than checking if each constant is defined. Let me know if you find a better solution.
This is only a problem in applications that explicitly reload, like Rails applications.
If the verbosity offends you, you can use unless
as a statement modifier instead:
module LifeCycle
RESET = 'reset' unless const_defined?(:RESET)
end
This leaves a few weak arguments against Avdi's suggestion to only use methods:
If you like Avdi's suggestion enough to ignore these, go with it.
RESET is not a constant if it keeps changing in your code. If you rename it to lower case 'reset', the problem disappears. Ruby thinks Upper case variables are constants and thus displays an error to warn you that a constant has changed.