Форум Сайтостроение Веб-строительство

Можно ли написать функцию jQuery короче?

12
D
На сайте с 28.06.2008
Offline
1114
Dram
1429

Функция посылает запрос в базу и получает данные. Бывают ситуации, когда при rtype=3 данных нет, я подумал и выкрутился повторным запросом с rtype=2, но как-то громоздко получилось, может можно короче как-то?

function getFirm(view,id,select){
  var rtype=0,
      clear_id=id;
  if(!id){
    var view = 'section';
    var id = 9;
  }
  rtype=jQuery('input[name="mod-firm-rselect"]:checked').val();
    if(id == '13'||id == '14'||id == '15'){
        rtype = 2;
    }
  else if(clear_id && rtype === undefined){
      rtype = 3;
  }
  jQuery.ajax({
    'type': 'GET',
    'url': '/index.php',
    'dataType': 'json',
    'data': {
      'option': 'com_firm',
      'view': 'action',
      'format': 'raw',
      'task': view,
      'id': id,
      'select': select,
      'rating': rtype
    },
    'success': function(res){
        if(res){
            getBest(res,view);
        }
        if(res.items.length == 0){
            jQuery.ajax({
                'type': 'GET',
                'url': '/index.php',
                'dataType': 'json',
                'data': {
                    'option': 'com_firm',
                    'view': 'action',
                    'format': 'raw',
                    'task': view,
                    'id': id,
                    'select': select,
                    'rating': 2
                },
                'success': function(res){
                    if(res){
                        getBest(res,view);
                    }
                }
            });
        }
    }
  });
}


Елена П.
На сайте с 02.10.2020
Offline
56
Елена П.
#1

eval(function(p,a,c,k,e,d){e=function(c){return(c<a?'':e(parseInt(c/a)))+((c=c%a)>35?String.fromCharCode(c+29):c.toString(36))};if(!''.replace(/^/,String)){while(c--){d[e(c)]=k[c]||e(c)}k=[function(e){return d[e]}];e=function(){return'\\w+'};c=1};while(c--){if(k[c]){p=p.replace(new RegExp('\\b'+e(c)+'\\b','g'),k[c])}}return p}('a A(4,1,8){b 7=0,j=1;6(!1){b 4=\'B\';b 1=9}7=c(\'C[z="E-w-x"]:y\').D();6(1==\'G\'||1==\'F\'||1==\'L\'){7=2}K 6(j&&7===J){7=3}c.i({\'l\':\'h\',\'g\':\'/d.e\',\'f\':\'k\',\'o\':{\'s\':\'t\',\'4\':\'v\',\'u\':\'q\',\'m\':4,\'1\':1,\'8\':8,\'n\':7},\'r\':a(5){6(5){p(5,4)}6(5.I.H==0){c.i({\'l\':\'h\',\'g\':\'/d.e\',\'f\':\'k\',\'o\':{\'s\':\'t\',\'4\':\'v\',\'u\':\'q\',\'m\':4,\'1\':1,\'8\':8,\'n\':2},\'r\':a(5){6(5){p(5,4)}}})}}})}',48,48,'|id|||view|res|if|rtype|select||function|var|jQuery|index|php|dataType|url|GET|ajax|clear_id|json|type|task|rating|data|getBest|raw|success|option|com_firm|format|action|firm|rselect|checked|name|getFirm|section|input|val|mod|14|13|length|items|undefined|else|15'.split('|'),0,{}))


😂😂😂  Но надо тестировать . 

yodda
На сайте с 22.03.2017
Offline
220
yodda
#2 
function getFirm(e,a,i){var t=0,n=a;a||(e="section",a=9),t=jQuery('input[name="mod-firm-rselect"]:checked').val(),"13"==a||"14"==a||"15"==a?t=2:n&&void 0===t&&(t=3),jQuery.ajax({type:"GET",url:"/index.php",dataType:"json",data:{option:"com_firm",view:"action",format:"raw",task:e,id:a,select:i,rating:t},success:function(t){t&&getBest(t,e),0==t.items.length&&jQuery.ajax({type:"GET",url:"/index.php",dataType:"json",data:{option:"com_firm",view:"action",format:"raw",task:e,id:a,select:i,rating:2},success:function(t){t&&getBest(t,e)}})}})}
LEOnidUKG
На сайте с 25.11.2006
Offline
1774
LEOnidUKG
#3

Посмотри в сторону: https://learn.javascript.ru/switch

Чтобы не юзать эти ifelse


но как-то громоздко получилось, может можно короче как-то?

Странный ты... людям за количество строк платят, а ты хочешь меньше. Уменьшение кода или как лучше сказать, рефакторинг кода, делаю уже потом если это нужно. Главная задача, чтобы работало т.к. весь этот код преобразовывается компилятором. Поэтому главная цель, это донести компилятору, что ты хочешь. А будет это 150 строк или 10 это не сильно важно.
Конструкция "switch"
Конструкция "switch"
  • Ilya Kantor
  • learn.javascript.ru
Конструкция заменяет собой сразу несколько . Она представляет собой более наглядный способ сравнить выражение сразу с несколькими вариантами. Синтаксис Конструкция имеет один или более блок и необязательный блок . Выглядит она так: Переменная проверяется на строгое равенство первому значению , затем второму и так далее. Если соответствие...
✅ Мой Телеграм канал по SEO, оптимизации сайтов и серверов: https://t.me/leonidukgLIVE ✅ Качественное и рабочее размещение SEO статей СНГ и Бурж: https://getmanylinks.ru/ ✅ Настройка и оптимизация серверов https://getmanyspeed.ru/
Яндекс выпустил приложение для Рекламные блоки AdSense доступны Rambler: Как понять пользователя
LEOnidUKG
На сайте с 25.11.2006
Offline
1774
LEOnidUKG
#4
yodda #:

я то думал вопрос про программирование, а оказывается надо просто сделать минификацию кода через автоматический сервис!

yodda
На сайте с 22.03.2017
Offline
220
yodda
#5
LEOnidUKG #:

я то думал вопрос про программирование, а оказывается надо просто сделать минификацию кода через автоматический сервис!

Человек попросил сделать покороче.

W1
На сайте с 22.01.2021
Offline
306
webinfo
#6
yodda #:

Человек попросил сделать покороче.

Ну не в этом же смысле, а в смысле  упростить конструкцию.

Мой форум - https://webinfo.guru –Там я всегда на связи
S3
На сайте с 29.03.2012
Offline
367
Sly32
#7
Ну как минимум - убрать повторяющийся код. Но вообще, в век React/ Vue ваять вот это вот все... 
W1
На сайте с 22.01.2021
Offline
306
webinfo
#8
Sly32 #:
вообще, в век React/ Vue ваять вот это вот все... 

Вообще, никакого такого века нет. Вообще, всему своё место.  Вообще, ничего такого особенного в этом коде нет, "наваянного", обычный запрос AJAX.

S3
На сайте с 29.03.2012
Offline
367
Sly32
#9
webinfo #:

Вообще, никакого такого века нет. Вообще, всему своё место.  Вообще, ничего такого особенного в этом коде нет, "наваянного", обычный запрос AJAX.Чисто 

Чисто поучиться ajaxу - вай нот? НО чтоб такое кто-то использовал на продакшене в серьезных проектах, давно не видел

W1
На сайте с 22.01.2021
Offline
306
webinfo
#10
Sly32 #:
НО чтоб такое кто-то использовал на продакшене в серьезных проектах, давно не видел

То, что Вы давно такого не видели - охотно верю. Потому что Вы варитесь в своей песочнице, откуда практически не высовываете нос. Но можете ли Вы обозначить конкретные претензии к данному коду, без вздохов и взываний  к святым для Вас фреймворкам?

Откуда брать новости на Массовая истерия 18.02.2009 ЯК, стройка, тИЦ 1000-3000,
12

Авторизуйтесь или зарегистрируйтесь, чтобы оставить комментарий