This is fine but take a look at using templates in the future