¿Es esto correcto?

[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.