Why are you parsing every time, instead to use a constructor?