working on for loop