que estoy haciendo mal?

[IsaBelM]

cuando invocas a la función el elemento aún no está cargado. lleva esto al final del archivo antes de la etiqueta </body>

<script language="javascript">
i = 1;
cambiar_foto();
</script>

o hacer lo que te han indicado en el post anterior