I find it confusing to have local vars shadowing class vars. Could you drop the `Value` prefix for locals ?