It's a good start, but I would recommend you investigate the "merge" command. I'd also suggest that writing joins as "cross joins" (rather than "inner joins") where there is no join criteria will be somewhat less confusing.