It's good. I would have found this helpful a few years ago.

How about putting the tables into normal form? ie where you currently have the company name as a string in the Person table, instead have an ID which references the Company table. It's not necessary for understanding the example shown, but it can't hurt to start off with good practice.