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

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

15.03.2019, 13:55. Показов 1359. Ответов 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
Ответ Создать тему
Новые блоги и статьи
Ритм жизни
kumehtar 27.02.2026
Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка SDL3 и Box2D из исходников с помощью CMake и Emscripten
8Observer8 27.02.2026
Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru