Как упростить повторяющийся код

@MORD93 · Регистрация: 28.12.2014

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

Всем добрый вечер. Есть два куска кода, которые отвечают за автокоплит допустим городов и улиц. У них почти всё одинаковое. И каждый раз повторять этот код не охото. Как их можно объединить, чтобы смотрелось более эстетично и проще

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
/*1 КОД*/
if ($("input.search-product-detail" + hash)) {
            var bestPictures = new Bloodhound({
                datumTokenizer: Bloodhound.tokenizers.obj.whitespace('value'),
                queryTokenizer: Bloodhound.tokenizers.whitespace,
                prefetch: '/admin/ajax/getProductsDetail',
                remote: {
                    url: '/admin/ajax/getProductsDetail/?query=%QUERY',
                    wildcard: '%QUERY'
                },
                sufficient: 10
            });
 
            $('input.search-product-detail' + hash).typeahead({
                autoselect: true,
                hint: true,
                highlight: true,
                minLength: 3
            }, {
                limit: 20,
                name: 'city',
                display: 'name',
                source: bestPictures,
 
                templates: {
                    suggestion: function (data) {
                        return '<p align="left"><a href="javascript:;">' + data.name + '</a></p>';
                    }
                }
 
            }).bind("typeahead:selected", function (obj, datum, name) {
                hash = $(this).data('hash');
                $(this).addClass('background-number');
                $(this).on( "keydown", function( e ) {
                    var code = e.keyCode;
                    if( code == 8) {
                        $('#input-number' + hash).removeClass('background-number');
                    }
                });
 
                $('#product-detail-id' + hash).val(datum.id);
            });             
        }   
 
 
 
        /*2 КОД*/
        if ($("input.search-provider" + hash)) {
            var bestPictures = new Bloodhound({
                datumTokenizer: Bloodhound.tokenizers.obj.whitespace('value'),
                queryTokenizer: Bloodhound.tokenizers.whitespace,
                prefetch: '/admin/ajax/Providers/getList',
 
                remote: {
                    url: '/admin/ajax/Providers/getList/?query=%QUERY',
                    wildcard: '%QUERY'
                },
                sufficient: 10
            });
 
            $('input.search-provider' + hash).typeahead({
                autoselect: true,
                hint: true,
                highlight: true,
                minLength: 3
            }, {
                limit: 20,
                name: 'city',
                display: 'name',
                source: bestPictures,
 
                templates: {
                    suggestion: function (data) {
                        return '<p align="left"><a href="javascript:;">' + data.name + '</a></p>';
                    }
                }
 
            }).bind("typeahead:selected", function (obj, datum, name) {
                hash = $(this).data('hash');
                $(this).addClass('background-number');
                $(this).on( "keydown", function( e ) {
                    var code = e.keyCode;
                    if( code == 8) {
                        $('#input-number_provider' + hash).removeClass('background-number');
                    }
                });
 
                $('#provider-id' + hash).val(datum.id);
            });
        }

@atanov · 08.02.2018, 16:00

Сделать функцию и вызывать её.

@MORD93 · 08.02.2018, 16:17 **[ТС]**

Я это понял. Но как? Можете привести пример на данном коде. Спасибо за ответ.

@atanov · 08.02.2018, 17:06

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

Сообщение от MORD93

У них почти всё одинаковое

, а сами Вы представляете, что именно не совпадает? На первый взгляд строка 41 со строкой 88 не бъётся. Вот отсюда и пляшите.

@MORD93 · 09.02.2018, 07:19 **[ТС]**

так ладно...
вот идет объявление переменной в каждом коде. Внутри параметры. Различия их - параметры prefetch и remote.

JavaScript
1
2
3
4
5
6
7
8
9
10
var bestPictures = new Bloodhound({
      datumTokenizer: Bloodhound.tokenizers.obj.whitespace('value'),
      queryTokenizer: Bloodhound.tokenizers.whitespace,
      prefetch: '/admin/ajax/getProductsDetail',
      remote: {
         url: '/admin/ajax/getProductsDetail/?query=%QUERY',
         wildcard: '%QUERY'
      },
      sufficient: 10
      });

остальные параметры одинаковые. Как сделать так чтобы каждый раз их не объявлять?

@j2FunOnly · 09.02.2018, 10:39

MORD93, у вас в обработчиках события typeahead:selected переменная hash специально сорит в глобальную область? Не понятно потому что в условии тоже присутствует переменная hash. И в какой момент эти if-ы с пометками код 1/2 срабатывают - при загрузке единожды или как-то по-другому?

@MORD93 · 09.02.2018, 13:31 **[ТС]**

Да, увидел ошибку с переменной. Тем более она объявлена уже выше.
Объясню как работает код. При нажатии на кнопку создается 2 поля (в tr) - область и город. hash это переменная в которой хранится рандомное 6-ти значное число, чтобы на серваке понимать и заменять на нужный мне id. При вводе в первое поле срабатывает первый код. При вводе во второе поле срабатывает второй. Создаются эти два кода каждый раз при нажатии на кнопку создать, иначе если я создам 10 полей сразу, то поиск не будет корректно работать.

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. 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 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

@atanov 640 / 481 / 172 Регистрация: 26.05.2016 Сообщений: 2,674
	08.02.2018, 16:00
	Сделать функцию и вызывать её. 0

@MORD93 0 / 0 / 0 Регистрация: 28.12.2014 Сообщений: 12
	08.02.2018, 16:17 [ТС]
	Я это понял. Но как? Можете привести пример на данном коде. Спасибо за ответ. 0

@j2FunOnly the hardway first 2475 / 1847 / 910 Регистрация: 05.06.2015 Сообщений: 3,610
	09.02.2018, 10:39
	MORD93, у вас в обработчиках события `typeahead:selected` переменная `hash` специально сорит в глобальную область? Не понятно потому что в условии тоже присутствует переменная `hash`. И в какой момент эти if-ы с пометками код 1/2 срабатывают - при загрузке единожды или как-то по-другому? 0

@MORD93 0 / 0 / 0 Регистрация: 28.12.2014 Сообщений: 12
	09.02.2018, 13:31 [ТС]
	Да, увидел ошибку с переменной. Тем более она объявлена уже выше. Объясню как работает код. При нажатии на кнопку создается 2 поля (в tr) - область и город. hash это переменная в которой хранится рандомное 6-ти значное число, чтобы на серваке понимать и заменять на нужный мне id. При вводе в первое поле срабатывает первый код. При вводе во второе поле срабатывает второй. Создаются эти два кода каждый раз при нажатии на кнопку создать, иначе если я создам 10 полей сразу, то поиск не будет корректно работать. 0