still not getting that loop right