This is an intersting post:
Using Transistor as a Switch.
Stop using random resistors values. Calculate them!
And this is a nice and simple circuit for driving 2 Leds (or a bi-color one) with one logical level : Hi level will illuminate one Led, low level will illuminate the other Led: