Вы можете проверить свои синтаксические ошибки, прежде чем продолжить; Я не знаю, скопировал ли ты весь свой код, но пропускаешь точки с запятой в конце своих строк. Помимо синтаксических проблем, один из возможных способов достижения того, что вы описываете, заключается в назначении координаты x и y каждой точки случайному числу. Изучение кода показывает, что только начальное значение i
присваивается значению random()
. Увеличение i
сделает координаты будущих точек зависимыми от начального значения i
, которое вы, возможно, захотите принять во внимание. Но нигде в вашем коде я не вижу, как вы меняете положение каждого элемента на основе значений, которые вы генерируете.
Я настоятельно рекомендую вам использовать HTML5 Canvas вместо того, чтобы пытаться перемещать элементы HTML; последний будет громоздким и приведет к грязному и неэффективному коду. Если вы все еще хотите придерживаться метода, который вы пытаетесь использовать сейчас, убедитесь, что для свойства CSS display
для этих элементов установлено значение block
. Вы используете метод getElementById()
, который не очень полезен в этом случае, поскольку идентификаторы уникальны в файлах HTML. Я бы предложил использовать getElementsByTagName()
и использовать вместо этого те возвращенные элементы с определенным атрибутом class
.