Changing model during runtime after button pressed

[Sachnmacha]

This is my first post, if I did anything wrong please point it out!

I want to change the model of game object after a button gets pressed.

I have written the code, but I can’t find the problem…

using UnityEngine;
using System.Collections;

public class Player1Controller : MonoBehaviour {

public GameObject model1;
public GameObject model2;
public GameObject model3;
private GameObject currentModel;

void Start ()
{
	currentModel = Instantiate (model1, transform.position, transform.rotation) as GameObject;
	currentModel.transform.parent = transform;
}

void Update () 
{
	if (Input.GetButtonDown ("Jump")) 
	{
		if (currentModel == model1)	
		{
			GameObject thisModel = Instantiate (model2, transform.position, transform.rotation) as GameObject;
			Destroy (currentModel);
			thisModel.transform.parent = transform;
			currentModel = thisModel;
		}
		else
		{
			if (currentModel == model2)
			{
				GameObject thisModel = Instantiate (model3, transform.position, transform.rotation) as GameObject;
				Destroy (currentModel);
				thisModel.transform.parent = transform;
				currentModel = thisModel;
			}
			else
			{
				GameObject thisModel = Instantiate (model1, transform.position, transform.rotation) as GameObject;
				Destroy (currentModel);
				thisModel.transform.parent = transform;
				currentModel = thisModel;
			}
		}
	}
}

}

thank you all for your help!

[b1gry4n]

Instantiating and destroying models over and over isnt as efficient as just turning the models on/off. For the below script to work you would need to drag/drop the models into the corresponding model slot in the inspector(the panel that shows up inside unity).

public GameObject modelA;
	public GameObject modelB;
	public GameObject modelC;
     
	private int modelNumber;
	 
	 
    void Start ()
    {
		modelNumber = 1;
		modelB.SetActive(false);
		modelC.SetActive(false);
    }
     
	 void ModelSwitch(){
		if(modelNumber == 1){
			modelA.SetActive(false);
			modelB.SetActive(true);
			modelNumber = 2;			
		}else if(modelNumber == 2){
			modelB.SetActive(false);
			modelC.SetActive(true);
			modelNumber = 3;
		}else if(modelNumber == 3){
			modelC.SetActive(false);
			modelA.SetActive(true);
			modelNumber = 1;
		}
	 } 
	 
    void Update ()
    {
		if (Input.GetButtonDown ("Jump")){
			ModelSwitch();
		}
    
    }