You are passing an argument to your subroutines, but using $string essentially as a global variable. That is defeatring the purpose of funnctions receiving arguments and returning values, which can thus be abstracted out as black boxes.