1. Код скрипта надо подключать в самом конце тега <boby>, перед его закрытием. Иначе он просто не увидит ваших изображений.
<html>
<head>
</head>
<body>
<img src="http://savepic.ru/11286158.png">
<img src="http://savepic.ru/11330177.png">
<script>
//Сюда пишем код скрипта
</script>
</body>
</html>
2. Ошибка не в Math.random(), a в цикле. Вы случайно вставили не знак «меньше» (<), а его HTML-код «<»; Правильный цикл выглядит так:
for (i; i < max; i +=1) {
//какой-то код
}
3. Не используйте в качестве переменной слово «top». Оно зарезервировано (тут можно почитать). Лучше напишите что-нибудь вроде «topPos» или что-нибудь другое. Для единообразия то же самое можно сделать и с переменной «left».
Drew96's post in Неправильно работает Math.random() was marked as the answer