Corrected code but NPN connection not working correctly - always on