2 -     public void AddItem(Item item) // AddItem(Item item) и AddMoney(int money), в идеале такие вещи красиво делать перегрузкой метода, назвав Add(), и в зависимости от типа данных, будет срабатывать нужный метод. Тем более имя метода вполне читается вместе с именем параметра, к примеру AddItem(Item item) и Add(Item item), в первом варианте в имени слово Item избыточно. Это не ошибка, и лучше оставить как есть, чтобы в лишний раз не наткнуться на отказ от ментора. Просто знай что такое есть. Перегрузка бы не помогла, если бы было два похожих метода, но отличающихся именем параметра, а не типом.