Problem with holding down UI button to instantiate prefab.

I have a button in my UI that has the following script attached to it.

using UnityEngine;
using System.Collections;
using UnityEngine.EventSystems;
using UnityEngine.UI;

public class LaserControl : MonoBehaviour,IEventSystemHandler {
	public GameObject character;
	bool buttonHeld = false;
	public GameObject laser;
	public float laserSpeed;
	public AudioClip laserShot;
	public float firingSpeed;
	public void pressed (BaseEventData eventData)
		buttonHeld = true;
	public void notpressed(BaseEventData eventData)
		buttonHeld = false;
	void Start () 
		character = GameObject.FindWithTag ("Player");
	void Update(){
		if (buttonHeld){
			Debug.Log ("click");
			InvokeRepeating ("shootLaser", 0.001f, firingSpeed);
			Debug.Log ("not click");
	void shootLaser() {
		GameObject shot = Instantiate (laser, character.transform.position,Quaternion.identity) as GameObject;
		shot.GetComponent<Rigidbody2D>().velocity = new Vector2 (0f, laserSpeed);
		AudioSource.PlayClipAtPoint (laserShot,transform.position);

The issue I am having is that I am trying to instantiate at a given interval when the button is held. When I push the button right now, I am invoking at a massive rate. I had the same code being initialized Input.GetKey( and the timing was perfect. But when I introduce events, it stops working correctly, and goes into overdrive.

This is my first time using the IEventSysemHandler. I am using Pointer Down and Pointer Up as my BaseEventData at Runtime only.

Any help would be great! I am at a loss where to go from here.

InvokeRepeating (“shootLaser”, 0.001f, firingSpeed);

^That means you’re repeatedly calling shootLaser() at fireSpeed. Whatever you declared in the inspector.

But if that’s set up all correctly, then it’s due to the eventData. Why not use EventTrigger?