Well, as someone else noticed, you're comparing some fixed strings that have nothing to do with the $checkvalue, $value1 and $value2 variables. I hadn't even noticed for it's particularly byzantine. Using double quotes would have given you a wrong result. Using single quotes gives you a doubly (or better: multiply!) wrong result!!
You have:
if(('$checkvalue' gt '$value1') &&
('$checkvalue' le '$value2'))