and it should also mention them in the interface rather than the implementation (which is also done in my previous comment)