My object just automatically "launch" or do stuff without me clicking on my mouse when it respawn?

Question

I've already set the ball (my object) initially at the start where I have to click for the ball to launch from the paddle (it's a ping pong game).

private void LockBallToPaddle() {
    Vector2 paddlePos = new Vector2(Paddle.transform.position.x,
        Paddle.transform.position.y);
    transform.position = paddlePos + paddleToBall; //For the ball to stick to the paddle
}

Launch the Ball:

private void LaunchBall() {
    if (Input.GetMouseButtonDown(0)) {
        hasStarted = true;
        rbBall.velocity = new Vector2(Random.Range(-2f, 2f),speedVelocity);
    }
}

Both of which I put under Update()

private void LaunchBall() {
    if (Input.GetMouseButtonDown(0)) {
        hasStarted = true;
        rbBall.velocity = new Vector2(Random.Range(-2f, 2f),speedVelocity);
    }
}

Now the problem is when my ball hits a certain collider, I want it to respawn the position back to the paddle. So instead of Destroy the game object, I just copy and paste those same methods under collision

if (collision.gameObject.tag == "bottom") {
    LockBallToPaddle();
    LaunchBalls();
}

But now it just launches straight away from the paddle when hit the collision without me clicking anything like what "Launchballs()" intended to. What should I do?

Answer 1

If one piece of code was working and the other is not, you can try to figure out the difference between it. In this case it is destroy();

The old code was creating a new ball at your launchpad. Your new code just takes the ball and teleports it at the launchpad BUT by doing so, it keeps the velocity it has before and your ball flies gladly in the direction it did before.

You could set the velocity inside your LockBallToPaddle since it handles already the teleport. You probably want to reset your hasStarted to false as well, whatever it does.