Форум программистов, компьютерный форум, киберфорум
JavaScript для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 5.00/7: Рейтинг темы: голосов - 7, средняя оценка - 5.00
3 / 3 / 1
Регистрация: 20.02.2018
Сообщений: 126

Нужен код ревью на скрипт модуль

15.03.2019, 13:55. Показов 1354. Ответов 5
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Здравствуйте !!!

Идея - это модуль для инпута типа range с разными методами. Задача на данный момент, красить выбранную область, учесть что он запоминает value. Подскажите как код по стилю в 2к19? Хотел отдельно приватным(обозначение _ ) сделать обработчик, но не врубаюсь как(видимо с this траблы).

HTML5
1
<input type="range" min="0" max="100" step="5" value="50"/>
JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
    class Range {
 
        constructor(settings) {
            this.settings = settings || { 
                color: 'red',
                selector: '' 
            };
 
            this._range = document.querySelector(this.settings.selector);
            this._initValue = this._range.value;
        }
 
        rangeColorProgress() {
            this._range.style.background = `-webkit-linear-gradient(left, ${this.settings.color} 0%, ${this.settings.color} ${this._initValue}%, #fff ${this._initValue}%, #fff 100%)`;
            this._range.addEventListener('input', (e) => {
                const inputValue = e.target.value;
                this._range.style.background = `-webkit-linear-gradient(left, ${this.settings.color} 0%, ${this.settings.color} ${inputValue}%, #fff ${inputValue}%, #fff 100%)`;
            });
        }
 
    }
 
    const range = new Range( { selector: 'input[type=range]', color: 'blue' } );
    range.rangeColorProgress();
0
Лучшие ответы (1)
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
15.03.2019, 13:55
Ответы с готовыми решениями:

Код ревью
Доброго времени суток, написал простейшее приложение-калькулятор. Написал ей максимально тупо, просто набор вызывающихся функций. Хотел бы...

Нужен скрипт/модуль для общения между пользователями сайта(голосом)
Здравствуйте, нужен скрипт или модуль для любого cms, через который можно звонить другому пользователю на сайте.Возможно есть какой-нибудь...

На собеседовании попросили сделать код ревью
На собеседовании задали вопрос как такой код можно улучшить, оригинал кода удалось сохранить))) Прошу прокомментировать что можно...

5
 Аватар для arcmag
347 / 322 / 203
Регистрация: 27.06.2014
Сообщений: 762
15.03.2019, 14:08
Не знаю, я конечно не эксперт, но всё таки.

Пожалуй стоит разбить функцию rangeColorProgress на отдельные составляющие.

Все обработчики событий биндить в отдельном методе (bind), так же, процесс формирования строки градиента вынести в отдельный метод (createGradientString), и получившуюся в этом методе строку уже присваивать элементу _range (rangeUpdate).
Так же, в конструкторе данные передавать через объект имеет смысл только тогда, когда аргументов становится слишком много, у вас только 2 аргумента: selector и color так что использование объекта как то избыточно на мой взгляд.

Ну и неплохо было бы не создавать объекты напрямую через new а создать какой нибудь статичный метод класса который будет это делать, ну это наверное на любителя...
1
3 / 3 / 1
Регистрация: 20.02.2018
Сообщений: 126
15.03.2019, 14:16  [ТС]
Цитата Сообщение от arcmag Посмотреть сообщение
Все обработчики событий биндить в отдельном методе (bind),
Спасибо ! это уже интересно надо погуглить будет.

Цитата Сообщение от arcmag Посмотреть сообщение
Так же, в конструкторе данные передавать через объект имеет смысл только тогда, когда аргументов становится слишком много, у вас только 2 аргумента: selector и color так что использование объекта как то избыточно на мой взгляд
Возможно и много будет со временем, да и я в целом пытаюсь найти тру вэй)
0
Эксперт JS
6496 / 3907 / 2006
Регистрация: 14.06.2018
Сообщений: 6,781
15.03.2019, 15:21
riddlejs, Опера стили не поняла. Разбираюсь, почему....
0
3 / 3 / 1
Регистрация: 20.02.2018
Сообщений: 126
15.03.2019, 15:33  [ТС]
Байнд прикрутил работает, но не совсем понятно так ли нужно.

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
class Range {
 
    constructor(settings) {
        this.settings = settings || {
            color: 'red',
            selector: ''
        };
 
        this._range = document.querySelector(this.settings.selector);
        this._initValue = this._range.value;
        this._handlerRangeColorProgress = this._bind(this._handlerRangeColorProgress, this, this.settings);
        this._drawBackgroundRange = this._bind(this._drawBackgroundRange, this, this.settings.color, this._initValue);
    }
 
    rangeColorProgress() {
        this._drawBackgroundRange(this.settings.color, this._initValue);
        this._range.addEventListener('input', this._handlerRangeColorProgress);
    }
 
    _drawBackgroundRange(color, value) {
        this._range.style.background = `-webkit-linear-gradient(left, ${color} 0%, ${color} ${value}%, #fff ${value}%, #fff 100%)`;
    }
 
    _bind(func, context) {
        return function() {
            return func.apply(context, arguments);
        };
    }
 
    _handlerRangeColorProgress(e){
        const inputValue = e.target.value;
        this._range.style.background = `-webkit-linear-gradient(left, ${this.settings.color} 0%, ${this.settings.color} ${inputValue}%, #fff ${inputValue}%, #fff 100%)`;
    }
}
Добавлено через 2 минуты
Цитата Сообщение от amr-now Посмотреть сообщение
Опера стили не поняла. Разбираюсь, почему....
Кстати возможно, я так понял данная задача в мозиле вообще решается css c помощью селектора input[type=range]::-moz-range-progress, а вот с др. браузерами еще предстоит разобраться, но этот модуль точно с хромом работает и мозилой.
0
 Аватар для arcmag
347 / 322 / 203
Регистрация: 27.06.2014
Сообщений: 762
15.03.2019, 15:55
Лучший ответ Сообщение было отмечено riddlejs как решение

Решение

Вы видимо слегка не поняли о чем я говорил, немного поправил ваш код

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
class Range {
  constructor(rangeSelector, settings) {
    this._color = settings.color;
    this._range = document.querySelector(rangeSelector);
  }
  
  createGradientString(color, value) {
    return `-webkit-linear-gradient(left, ${color} 0%, ${color} ${value}%, #fff ${value}%, #fff 100%)`;
  }
 
  rangeUpdate() {
    this._range.style.background = this.createGradientString(this._color, this._range.value);
  }
  
  bind() {
    this._range.addEventListener('input', this.rangeUpdate.bind(this));
  }
  
  static create(rangeSelector, settings) {
    const range = new Range(rangeSelector, settings);
    range.bind();
    range.rangeUpdate();
    
    return range;
  }
}
 
const range = Range.create('input[type=range]', {color: 'blue'});
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
15.03.2019, 15:55
Помогаю со студенческими работами здесь

Нужен модуль, скрипт для DLE
Нужно сделать вывод на всех страницах сайта в определенном блоке фото из папки на сервере. Например, в папке будет лежать 10 фотографий. 1...

Код ревью
В общем препод дал open source проект и дал задание сделать ревью кода какого - то класса... Вот только с java и ООП нашей группе предстоит...

Код ревью
Добрый день. Сделал ajax в ajax и вижу дубляж кода, но не знаю как от него избавиться. Правила форума 5. Запреты и ограничения. ...

Код-ревью
Всем здрасте, можете посмотреть на мою программку и рассказать как можно сделать лучше и легче? import random leftz =...

небольшой код ревью
вот делаю интерфейс, и уже сейчас у меня чрезмерно много повторяющихся элементов. &lt;Border BorderBrush=&quot;Black&quot;...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
6
Ответ Создать тему
Новые блоги и статьи
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост.
Programma_Boinc 28.12.2025
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / **********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / **********/ gallery/ bKBkQFf Пост отсюда. . .
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США.
Programma_Boinc 26.12.2025
Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru