[Solved] Causing Engine Failure

[Ontresanas]

I'm trying to make engines have a random chance of failure in which an engine simply doesn't produce thrust.  However, I can't seem to get this to work in a way that I like.

So far, I have tried setting the maxThrust of all ModuleEngines on the part to zero - which didn't work, I have tried removing all ModuleEngines - which didn't work, and I have tried shutting down all ModuleEngines whenever they become activated - which works, but produces an ungodly death rattle that I don't want.

 

This is the meat of the code I have currently.  Does anyone have any insight into what I'm doing wrong?

public override void OnStart(PartModule.StartState state)
{
    if (state == StartState.Editor) return;

    if (IsNewPart)
    {
        IsActive = true;
        IsNewPart = false;
        if (UnityEngine.Random.value < FailChance)
            HasFailed = true;
    }

    if (HasFailed)
        FailStatus = "Engine Failed";
    else FailStatus = "Engine Working";
}

public override void OnFixedUpdate()
{
    if (this.vessel == null)
        return;

    if (HasFailed)
    {
        foreach (PartModule M in part.Modules)
        {
            ModuleEngines E = M as ModuleEngines;
            if (E == null) continue;
            if(!E.engineShutdown)
                E.Shutdown();
        }
    }
}

Edited January 6, 2016 by Ontresanas
Thread is solved

[Crzyrndm]

The fields you probably want are "EngineIgnited" and "allowRestart". The method "Shutdown()" (normally triggered from the right click menu) might be better to use than the ignited property

[Ontresanas]

Thanks, that works!  However, now I've discovered that UnityEngine.Random.value doesn't seem to be giving legitimately random values.  The same engines always fail or work regardless of what I do or how many times I launch in sequence (not reverting, actually launching a new rocket).  Is there a "correct" source of randomness that I should be using?

UPDATE: I've dropped using UnityEngine.Random, and am now using a static System.Random object.  This solves the randomness problem.  Now, the hopefully-final question: is there any way to get a list of all the defined parts?  Not on the current vessel, but rather, just a list of every part, like the VAB part selection screen.

Edited January 6, 2016 by Ontresanas