Size: a a a

JavaScript Noobs — сообщество новичков

2021 September 30

IS

Ivan Savin in JavaScript Noobs — сообщество новичков
у вас коммент от человека, у которого нет ни одного реплая/лайка (т. е. подтверждения/согласия) и который не прикрепил своё решение, которое может быть ошибочным, очевидно
vs.
мои слова, что всё работает, которые я могу подтвердить и скинув решение, и кинув скрины

так что либо поверьте мне, либо я вам всё-таки кину скрины, раз вы не хотите верить хд
источник

AB

Alex Bessmertnov in JavaScript Noobs — сообщество новичков
помогите народ плз,  над в путь сунуть каждый рандомный айтем

export const setrandomImage = selector => {
 const items = [
   '1', '2', '3', '4', '5',
   '6', '7', '8', '9', '10',
   '11', '12', '13', '14', '15'
 ]

 const shuffle = arr => arr.sort(() => Math.random() - 0.5)
 const random = shuffle(items)

 random.forEach(el => {
   return selector.forEach(item => item.src.replace(/\d/, el))
 })
}

путь "./assets/img/gallery/g7.jpg" и таких 15 и чет не работает
источник

IS

Ivan Savin in JavaScript Noobs — сообщество новичков
selector это что?
источник

AB

Alex Bessmertnov in JavaScript Noobs — сообщество новичков
псевдомассив => document.querySelectorAll('.item-img')
источник

IS

Ivan Savin in JavaScript Noobs — сообщество новичков
ну, я не могу не отметить сразу, что названия отвратительные, имхо

дальше, вы меняете строку - и?
от этого свойство не поменяется
источник

IS

Ivan Savin in JavaScript Noobs — сообщество новичков
вы просто достаёте значения, реплейсите в них циферки на ходу, всё, они в воздухе
источник

IS

Ivan Savin in JavaScript Noobs — сообщество новичков
ну и зачем цикл в цикле тоже не совсем понятно, конечно
источник

AB

Alex Bessmertnov in JavaScript Noobs — сообщество новичков
работает не так как я ожидал рендерится с рандным индеком все 15, не каждый
источник

IS

Ivan Savin in JavaScript Noobs — сообщество новичков
оно вообще работать не должно
источник

j

jk in JavaScript Noobs — сообщество новичков
Погуглите другой способ shuffle, это кошмар.
источник

IS

Ivan Savin in JavaScript Noobs — сообщество новичков
это пока хрен с ним хд
источник

{

{ Max: 'JS Engineer'... in JavaScript Noobs — сообщество новичков
есть хороший алгоритм, не помню название

const shuffle = arr => {
 const len = arr.length
 for (let i = 0; i < len - 1; i += 1) {
   let r = Math.floor(Math.random() * (len - i))
   r && ([arr[i], arr[i + r]] = [arr[i + r], arr[i]])
 }
 return arr
}
источник

{

{ Max: 'JS Engineer'... in JavaScript Noobs — сообщество новичков
в честь какого-то программиста назван
источник

IS

Ivan Savin in JavaScript Noobs — сообщество новичков
// magic numbers это тоже плохо, кстати
const items = Array.from({ length: 15 }, (_, i) => `${i + 1}`);

// тут ваш шафл
const random = shuffle(items)
 
// почему не images хотя бы???
selector.forEach(item => item.setAttribute('src', item.src.replace(/\d/, random.pop());
источник

IS

Ivan Savin in JavaScript Noobs — сообщество новичков
хотя бы так
источник

IS

Ivan Savin in JavaScript Noobs — сообщество новичков
но это тоже гадость, канеш
источник

AB

Alex Bessmertnov in JavaScript Noobs — сообщество новичков
я только что допер что такое items в вашем коде)
источник

AB

Alex Bessmertnov in JavaScript Noobs — сообщество новичков
сколько прошло 5 минут больше?)
источник

IS

Ivan Savin in JavaScript Noobs — сообщество новичков
да это просто пример. основная суть, что хардкодить - плохо, имена - плохие, magic numbers - плохие, shuffle - плохой, цикл в цикле не нужен, вот
источник

AB

Alex Bessmertnov in JavaScript Noobs — сообщество новичков
ну я не спорю, но это во мне может проснется, а может уже и нет) хоть как бы написать для начала
источник