It loops but still problems on the constructor