Неправильно, надо чтобы создавая новый инстанс ты уже получал значение, а не сам его забивал