Will this circuit work, is it overkill or wll it self destruct?

Hi,

I'm fairly new to electronics so this might seem like a really basic question but be nice please :).

I've designed a program to go through a number of light patterns with an RGB led, it seems to be working nicely in my prototype (see pic 1) (although it's hard to tell without using an RGB LED!) but I want to use it with a 3W RGB LED I have laying around (for specs see http://dx.com/p/3w-led-emitter-on-star-multicolored-rgb-4530). As I understand it arduino can only supply 40mA/pin, not the 350mA per colour required by the LED so I need to use transistors, I've never used them before so want to make sure the circuit I've designed will do the job.

please see attachments for pics of my existing prototype and my circuit.

Just in case it's not clear what is being used in the circuit:

Any questions just shout.

Thanks in advance.

Howard