working on ex2 - details