would be better to receive the id in the constructor to make it testeable