are you sure you've declared the 'hold' array in the main program somewhere? Also, you don't need the % after integer vars, it only makes your code slower to type and harder to read. Onece you've declared a variable you don't need to type the data type char again.