Do you need to take care of non-integer values here - eg. strings?