Multiuse laser beam

I'm troubled by your question. If you wrote this script, then the changes you would need to make would be clear. If you borrowed this script, aren't we doing your school work for you? This is script is not that hard to understand. All the classes and functions used here are common and well documents in the reference. And there are a multitude of answers on UA concerning the functions and concepts in this script. Don't you think you should make an effort to go through and understand this script and make the changes on your own? Shouldn't you be turning in your own work for your classes?

I learned how to do the basic script via a tutorial. I figured out how to add the sound on my own. I just want a good place to learn how to put multiple if and else statements into this script. I have searched the web but have not found any non confusing sections on it. I just need a good direction to go.

Alright I now have multiple if statements in my script. I found my answer but a new problem has come up. when I fire my laser it goes toward one of the targets but now to where I was ai$$anonymous$$g it. I have a target tagged Drone and the laser beam always heads toward it. Please tell me how to fix it. I now have multiple if statements in my script. using System.Collections;

 public class Beamlaser : $$anonymous$$onoBehaviour 
 {
     LineRenderer line;
     public GameObject other;
     public GameObject explosion;
 
     void Start () 
     {
         line = gameObject.GetComponent<LineRenderer>();
         line.enabled = false;
     }
     
     
     void Update () 
     {
         if(Input.GetButtonDown("Fire1"))
             
         {
             StopCoroutine("FireLaser");
             StartCoroutine("FireLaser", .25F);
             audio.Play();
             
         }
     }
     IEnumerator FireLaser(float lengthOfTime)
     {
         line.enabled = true;
         var t = Time.time;
         
         while(Input.GetButton("Fire1") && Time.time - t < lengthOfTime)
             
         {
             Ray ray = new Ray(transform.position, transform.forward);
             RaycastHit hit;
             
             line.SetPosition(0, ray.origin);
             
             if(Physics.Raycast(ray, out hit, 1000))
             {
                 line.SetPosition(1, hit.point);
                 if(hit.collider.gameObject.tag == "Hand")
                 {
                 Instantiate(other, hit.point, transform.rotation);
                 Destroy(hit.transform.gameObject); 
                                 }
             }
             else if(hit.collider.gameObject.tag == "Drone")    
             {
                 Instantiate(explosion, hit.point, transform.rotation);
                 Destroy(hit.transform.gameObject);
                     line.SetPosition(1, ray.GetPoint(1000));
             
             yield return null;
         }
         line.enabled = false;
         }
     }
 }

WARNING! RUNNING THE ABOVE CODE WILL CRASH UNITY! I have run and rerun this code multiple times and every time it is crashing unity. So run this code at your own risk.

Yea, it will crash Unity. The issue is that line 53:

  yield return null;

...needs to be executed every time through the while() loop. The way you have it now, it will only be executed when this line is executed and true:

   else if(hit.collider.gameObject.tag == "Drone") 

You have bracketing issue with your logic. Get all your brackets to line up correctly and look at your logic. In $$anonymous$$onodevelop, if you place your cursor at one bracket, the matching bracket will be highlighted.

Note you've also migrated this line inside your while() loop:

    line.enabled = false;