Your signature (Jassim) points to a site seems to be commercial... It is not really accepted here and can cause you to be banned for spamming...
Please remove it (or restore it to its previous value - your personal site)!
Skipper: We'll fix it. Alex: Fix it? How you gonna fix this? Skipper: Grit, spit and a whole lotta duct tape.
The instructor table looks a bit odd to me, having both ID and InstructorID I would assume you meant InstructorID and InstructorName.
And as an addition to what Mycroft said, in a fully normalized database you always use the same name for an entity throughout the database. It's not just a good idea, it's a standard[^].
And just like Mycroft said, for a surrogate key, using <tablename> + "ID" is a good idea
if OP put it in the Lounge they'd get flamed by many.
I don't see why, as it is not a programming question, more a general request for opinions and experiences. Plenty of similar discussions appear in the Lounge. I know these forums are titled "Discussions", but in reality they are for asking Technical Questions.