for cycle fixed