Please help with my jump script

I want the jump in my game to be proportional to the size of the player




using UnityEngine;
using System.Collections;

public class Jump : MonoBehaviour {

public float jumpSpeed;
public bool isGround;

private Renderer rend;
private Rigidbody rb;

void Start(){
rend = GetComponent<Renderer> ().bounds.size;
rb = GetComponent<Rigidbody>();
}

void jump(){
rb.AddForce(Vector3.up * jumpSpeed + rend);
}

void OnCollisionStay (Collision collisionInfo)
{
if(collisionInfo.gameObject.CompareTag("Ground")){
isGround = true;
}
}

void OnCollisionExit (Collision collisionInfo)
{

isGround = false;

}

void Update(){

if(Input.GetButton("Jump") && isGround == true){
jump();

}


}
}



I am getting errors like :
Assets/_scripts/Jump.cs(18,20): error CS1503: Argument `#1' cannot convert `object' expression to type `UnityEngine.Vector3'

Assets/_scripts/Jump.cs(18,20): error CS1502: The best overloaded method match for `UnityEngine.Rigidbody.AddForce(UnityEngine.Vector3)' has some invalid arguments

Assets/_scripts/Jump.cs(18,37): error CS0019: Operator `+' cannot be applied to operands of type `UnityEngine.Vector3' and `UnityEngine.Renderer'

Assets/_scripts/Jump.cs(13,17): error CS0029: Cannot implicitly convert type `UnityEngine.Vector3' to `UnityEngine.Renderer'

 

GetComponent<Renderer>().bounds.size() returns a Vector3. You're trying to assign it to "rend," which you've said is type Renderer. However, if you make "rend" a Vector3 (and you might want to rename it, for clarity), that should fix most of your compilation problems.

Couple logical error, too, if I might chime in:
-When you jump, you are calling AddForce. This adds force just for one frame...and that doesn't do much. What you probably want is to add an impulse, which is done most easily by setting the velocity, something like:
rb.velocity = Vector3.up*jumpSpeed;
or
rb.velocity += Vector3.up*jumpSpeed;

Depending on whether you want to consider your initial velocity (if you want to, say, do a running jump, consider the second.

I'm not sure what you're trying to do adding the render's bounds to your jump velocity.

Additionally, I'd usually use GetButtonDown instead of GetButton. GetButtonDown will return true just on the frame you press it, whereas GetButton will return true as long as the button is held down. That means, if, for some reason, isGround isn't immediately cleared, jump will be called twice. And, if you keep jump held down, you'll jump again when you land. Maybe these are OK, but I wanted to clarify.