RogerPodacter/Code-Samples — comment on commit 297ca275086ffd9bab7e38513cbe8e5ae3f1f375
File: whittle-inverse-r.rb line 23
By RogerPodacter:

you shouldn't use `ENV` to store configuration for this script since it's accessible everywhere and who knows what other code is going to do with it. this is particularly true when you use a name like `MAXIMUM_N_ENV_VAR` lol. if you MUST put it in `ENV` at least namespace it or something to avoid collisions (and you can leave the `ENV_VAR` part off ;)

i'd make the search upper-bound a named / hash parameter of the `find_n_for` method, tho the user should probably be able to set the default via a class `attr_accessor` or something