Simple Transistor, why is it working?

[joeman112]

Hi,

Any ideas why this isn't working?
I am new to electronics, but Ive done a lot of reading up and I think this should work:


Thanks,

Joe

 

[zia]

negative switch or give ground to base then chck

 

[davoud]

1-You should use LED D5 by a resistor
2- You should use NPN transistor for switch.
Regard.

 

[btbass]

You need series resistors for the LED's to control the current!
D5 will short the output from the micro

 

[joeman112]

Excellent, thanks guys, My real problem was that the wire going to D1 also went to ground (off the initial image) which caused a short.

 

[D.A.(Tony)Stewart]

That's a problem but a current limiter is still not shown.