DM
Речь шла о том, чтобы иметь возможность переиспользовать код в GitHub Actions
Size: a a a
2021 April 21
DM
Я бы тоже запилил, но я в GitHub не работаю )
Кm
Плохо код написан
Зачем внутри
Зачем лишняя переменная
Сделай функцию, получающую на вход каунтер и выдающую на выход урл
Сделай функцию, которая меняет src у картинки
Это повысит читабельность кода
Зачем внутри
next() ещё одна обёртка?Зачем лишняя переменная
nextImage?Сделай функцию, получающую на вход каунтер и выдающую на выход урл
Сделай функцию, которая меняет src у картинки
Это повысит читабельность кода
NK
это так замыкание работает по примеру учебника а остальное кроме последнего так и работает только ссылка не меняется никак если знаешь как лучше перепиши я еще учусб
Кm
Здесь замыкание не к месту совсем
NK
у меня тогда счетчик после двух вообще не работает
Кm
Оно никаких проблем не решает, а только лишний код создаёт
NK
без замыкания
Кm
Потому что код плохо написан
NK
у меня щас проблема что ссылка не меняется на ту которую нужно а не то что код плохой с постоянной ссылкой оно работает отлично если знаешь как лучше напиши я еще учусь писать на js
DM
Напиши лучше, что ты хочешь сделать
Кm
Описываешь функцию с замыканием
Создаешь экземпляр
Вызываешь
function nextGenerator() {
let count = 0;
return function() {
return count++;
};
}Создаешь экземпляр
let next = nextGenerator()Вызываешь
next(); next(); next(); //0 1 2Кm
Дальше уже пишешь логику по формированию урл
DM
Вот спрашивается, чем людям классы не угодили?
DM
Типичный кейс для класса )
Кm
Человек же только изучает
DM
Не, я не про него
Кm
Но вообще да
DM
Сейчас просто модно стало классов избегать