yes yes, I am.
Good, otherwise unlimited current would harm both the output pin and the transistor.
Before I show you the code could I use MOSFETS in place of the transistors?
Yes, but there would be no special advantage as your motors seem to be small to be able to be switched with small 2N3906 transistors. Which is a PNP transistor and most arduino switched applications use a NPN transistor as a low side switch. I would have to see a drawing of your wiring to see if you are driving your motors correctly or not.
If so, what type would I need N-type?
Most switching applications used with arduino outputs using mosfets do use N type as a low side switch.
Lastly, what is the difference between using a MOSFET for this and a normal transistor
From a software point of view no difference, from a hardware point of view mosfets can have less losses in higher current applications. With your small motors there is no big advantage.