Are this variable required ?
I would also use `{}`for the `if below