Sounds reasonable to me. There are no rules about how you do data access, you don't have to do it the same way I do, you can use Linq, or Entity Framework or NHibernate or whatever you like. Though using those technologies you may need to add/uncomment/update the connectionStrings section in Web.config whereas the included code uses it from appSettings.
Best,
Joe