Community

Question

I'm making a FPS game, but reloading won't work!I'm writing it with JavaScript Can anyone help?Code:

public var BulletPrefab : Transform;
public var BulletSpeed : float = 6000;
public var GunMuzzle : Transform;
public var ClipSize : float = 30;
public var Clip : float = 30;
public var ReloadTime : float = 2.0;

function Update () {
    if (Input.GetButtonDown("Fire1")) { 
       if (!BulletPrefab | !BulletSpeed ) {
         Debug.Log("[shoot] Undefined Variables");

       } else {
         if (Clip >0) {
         var BulletSpawn = Instantiate (BulletPrefab,GunMuzzle.transform.position,Quaternion.identity);
         BulletSpawn.rigidbody.AddForce(transform.forward * BulletSpeed );
         Clip -= 1;
        }
    }  
}


}
if (Clip < 1 ){
       Debug.Log("Reloading");
       yield WaitForSeconds (ReloadTime) ;
       Clip = ClipSize ;

}

Answer 1

0

you cant have a yield in an update function. What you can do is use a while loop in your start function.

function Start(){
     while(true)
     {
          while(!Reloading) yield;
          Debug.Log("reloading");
          yield WaitForSeconds(ReloadTime);
          Clip = ClipSize;
          Reloading = false;
     }

}

function Update(){
     if(Input.GetButtonDown("Fire1"))
     {
            Reloading = true;
     }
}

please remember that you will get a much faster answer if you ask your question in a way that does not give people a hard time ot figure out what the problem is.

answered Jun 07 '11 at 07:16 PM

Anxowtf
1.6k ● 22 ● 27 ● 37

I'm trying to wrap my head around this; does your first yield statement kick out to the Update(), run through one frame, then resume?

Jun 07 '11 at 08:42 PM Chris D

my first while statement just makes sure that the start function loops. As you know Chris, Start only runs once just before the update function. so the while(true) which is always true will cause the while to loop all game long. Without a yield or some sort of delay, a while true in start will crash your game/editor but with a delay it works well. the second while tells it not to go on unless reloading is true, when reloading becomes true it will pass that line of code and start to reload. at the end reloading is set back to false so it will only run reload once till reloading is set back to true externally.

Jun 08 '11 at 12:03 AM Anxowtf

OH MY GOD!Thank you, Thank you! It Works now :D Thanks again

Jun 08 '11 at 06:10 AM Deelis

@Anxo Ah, excellent! I know about Start() only running once, but I haven't dealt with yield before. Thanks for the explanation.

Jun 08 '11 at 04:08 PM Chris D

(comments are locked)

Community

Fps game gun reloading problem

Follow this question

Welcome to Unity Answers

Unity Answers tutorial video!

Related Questions