...will nearly always overflow outputvalue. And, is useless because you overwrite the value before using it.
outputvalue