This is a bad pattern as it is duplicating code from line 66 and again from line 87. Instead this should look like

notice the use of the variable naming -  `default_$var`
https://github.com/ghoneycutt/puppet-module-ntp/blob/master/manifests/init.pp#L117

and then we check if a value has been supplied from hiera or to use the default
https://github.com/ghoneycutt/puppet-module-ntp/blob/master/manifests/init.pp#L172