¿Es esto correcto?

[Germano6]

Buenas tengo un código que funciona, pero no sé si es chiripa porque apesta un poco.
Tengo una función que al hacer scroll se ejecuta y también dice que al hacer click en un objeto se activa otra fnción, pero de hecho funciona al hacer click aunque no esté haciendo scroll en ese momento. La cosa es que tiene el comportamiento deseado, pero creo que está mal...

Código HTML:

<div id="objeto"></div>
 

Código Javascript:

onscroll = function funcion1() { 
//hace sosas
var objeto = document.getElementById('objeto');
objeto.onclick = funcion2;
};
function funcion2() {
 //hace mas cosas 
}
 

Mi pregunta es si está todo bien, y en caso negativo cómo podría escribirlo bien de tal forma que tenga el mismo comportamiento.

[PHPeros]

Es correcto pero no tiene mucho sentido asociar el mismo manejador para el evento cada vez que se hace scroll.

Si es necesario que se active el onclick cuando se hace scroll, tendrás que hacer una condición utilizando por ejemplo una bandera para asegurarte que solo se ejecuta la primera vez.