working on ForEach