Do I overcomplicate something in this code?

I would like to attain it by the cube being an object only,let him jump then when I ovecome it W button. When I keep it pushed though W button,let the cube not be hopping though continuously then.

 using UnityEngine;
 using System.Collections;
 
 public class CubePlayerJumpScript : MonoBehaviour {
     
     private float jumpSpeed = 1100f; //moveSpeed = 5f;
     private bool heClashed = false;
     
     // Use this for initialization
     void Start()
     {
         Physics.gravity = new Vector3(0, -70, 0);
     }
     
     // Update is called once per frame
     void Update()
     {
     }
     
     void OnCollisionEnter(Collision theCollision)
     {
         if(theCollision.gameObject.name == "Cube Soil")
         {
             heClashed = true;
             rigidbody.constraints = RigidbodyConstraints.FreezePositionZ | RigidbodyConstraints.FreezeRotation;
         }
     }
     
     void OnGUI()
     {
         Event e = Event.current;
         if(e.isKey)
             switch(e.keyCode)
             {
                 case KeyCode.W:
                 {
                     if(heClashed && Input.GetKeyDown(e.keyCode))
                     {
                         rigidbody.constraints = RigidbodyConstraints.FreezePositionX | RigidbodyConstraints.FreezePositionZ | RigidbodyConstraints.FreezeRotation;
                         rigidbody.AddForce(Vector3.up * jumpSpeed);
                         heClashed = false;
                     }
                     break;
                 }
                 default:
                     break;
             }
     }
     
 
 }