а вопросы "правильно ли я спроектировал БД" и "ER-диаграмма" - не связаны.

За проектирование реляционных БД отвечает "теория БД" (или что-то аналогичное, что вам должны бы были читать). Смотрите например топики Нормальные формы или вот . (часто советуют довести примерно до третьей нормальной формы... но опять же все зависит от задач которые перед вами ставятся... )
Ну и ещё вопросы быстродейтсвия и выборки данных тоже влияют - но это уже зависит от конкретной СУБД вами используемой.

А ER-диаграмма - в принципе может отразить как любую фактическую структуру таблиц БД (в "физической модели"... за название не ручаюсь) так и и любую "структуру сущностей" (в "логической модели"). ER - диаграмме все равно правильно вы сделаи БД или нет.

Правильно или нет - решать вам и вашему преподавателю. Мы тут даже задачи не знаем какую вы преследуете создавая такую таблицу)))