Cambiar imagenes dinamicamente

[goteen_mx]

Pues tienes un array de 4 imagenes con sus rutas, cuando le das clic en cambiar asignas la ruta a la imagen e incrementes un contador, como tienes un modulo(%) el tamaño de tu arreglo pues siembre va a estar variando entre 0 y 4, por lo tanto siempre vas a elegir cualquera de tus 4 imagenes

Puedes poner tu código completo, la función esta correcta, si acaso la precedencia no se si primero sume y luego haga el módulo o viseversa pero no debe de dar problemas,
Revisa si las imagenes estan en la ruta correcta y que tu elemento img tenga el id="imagen"

Saludos.