Yes this looks good. Google will then index these pages and get the doctor names from the actual page. You just need to make sure the links work and it appears they do.
The only other thing you will need to do is map the sitemap.xml link to the new page in your .htaccess file like:
RewriteRule ^sitemap\.xml$ /mdb/smgen.php [L,NC]