How to implement a delay between each time my gun fires (using coroutines or otherwise)?...

So I've taken some parts from the FPS tutorial script and chopped them up for my use. I'm right now only testing a sniper, and it will kill the "monster" i have, it shoots as fast as a machine gun, I dont know whats going on, everything here seems right:

var damage = 20;
var clips = 20;
var reloadTime = 2;
private var hitParticles : ParticleEmitter;

function Start () {
    hitParticles = GetComponentInChildren(ParticleEmitter);

    // We don't want to emit particles all the time, only when we hit something.
    if (hitParticles)
        hitParticles.emit = false;
}

function Fire () {
    var direction = transform.TransformDirection(Vector3.forward);
    var hit : RaycastHit;

    // Did we hit anything?
    if (Physics.Raycast (transform.position, direction, hit)) {
                if (hitParticles) {
            hitParticles.transform.position = hit.point;
            hitParticles.transform.rotation = Quaternion.FromToRotation(Vector3.up, hit.normal);
            hitParticles.Emit();
        }

        // Send a damage message to the hit object          
        hit.collider.SendMessageUpwards("ApplyDamage", damage, SendMessageOptions.DontRequireReceiver);
    }

    Reload();
}

function Reload(){
yield WaitForSeconds(reloadTime);
clips--;
}

I get no errors in the debug box and everything should work, but it still shoots like a machine gun. I cant figure it out, maybe you can

because you're calling "Reload()", and Reload() is a coroutine, your coroutine starts running separately to your main "Fire" function, and your "Fire" function is free to continue executing at full speed.

If you want your firing to repeat, with a certain delay between every shot, while the fire button is held down, you could use something like this:

function Start() {
    // start up the shooting timer coroutine:
    ShootTimer();
}

function ShootTimer() {
    // a repeating test to see whether the user is pressing fire
    while (true) {
        if (Input.GetButton ("Fire1")) {
            Fire();
            yield WaitForSeconds(reloadTime);
        } else {
            yield;
        }
    }
}

function Fire() {
   // perform one shot (raycast, particles, etc),
   // but no reload timing goes in here
}


Alternatively, you could use the more "traditional" approach instead of coroutines (which can sometimes be simpler to understand), where you measure the amount of time elapsed since the last shot yourself:

var reloadTime = 0.5;
private var nextFireTime = 0.0;

function Update () {
    if (Input.GetButton ("Fire1") && Time.time > nextFire) {
        nextFireTime = Time.time + fireRate;
        Fire();
    }
}

Just a guess:

You didn't post your Update() function or the function where you actually fire your weapon (where you call your Fire() function. By looking at the above code, could it be that you can even fire without any ammunition? Possibly add an if statement that you can only fire when a bullet is loaded. What you have up there is simply the WaitForSeconds() to reload. Your described behavior would result if you continuously fire your gun without ammunition and the reloading just happens without any consequence.