Is there anything wrong with this flappy bird code?

[Br0ken robot]

I'm not sure if this is correct but this is the script for a flappy bird pipe spawner can anyone see any missing commas.
{
spawnknife();
}

// Update is called once per frame
void Update()
{
if (timer < spawnRate)
{
timer = timer + Time.deltaTime;
}
else
{
spawnknife();
timer = 0;
}

}

void spawnknife()
{
float lowestPoint = transform.position.y - heightOffset;
float highestPoint = transform.position.y + heightOffset;

Instantiate(Knife, new Vector3(transform.position.x, Random.Range(lowestPoint, highestPoint), 1), transform.rotation);
}
}

 

[Phro0244]

The code appears to be intended to generate knives that are instantiated and thrown into the game. However, it lacks some details such as setting variables, choosing parameters, and initializing objects. Here's a full version of the code you can use:

using UnityEngine;

public class KnifeSpawner : MonoBehaviour
{
    public GameObject Knife;
    public float spawnRate = 2f;
    public float heightOffset = 1f;

    private float timer = 0f;

  
    void Start()
    {
        spawnknife();
    }

    void Update()
    {
        if (timer < spawnRate)
        {
            timer = timer + Time.deltaTime;
        }
        else
        {
            spawnknife();
            timer = 0;
        }
    }

    void spawnknife()
    {
        float lowestPoint = transform.position.y - heightOffset;
        float highestPoint = transform.position.y + heightOffset;

        Instantiate(Knife, new Vector3(transform.position.x, Random.Range(lowestPoint, highestPoint), 0), transform.rotation);
    }
}