It's all really good except for the infinite loop when I add