1

Здравствуйте. Хочу переписать скрипт, используя конструкцию:

условие ? выражение1 : выражение2

$('#search-btn').click(function () {
    console.log('test')
    if ( $(".boolean ").hasClass("checked")) {
        return
    }
    else if( $('#search_material').val() == ''){
        $('#search_material').focus()
    }
    else $('.type-checkbox ').addClass('active-confirm');    
});

У меня, к сожалению, не выходит сделать рабочий вариант. Подскажите, в чем ошибка?

   $('#search-btn').click(function () {

    ( $(".boolean ").hasClass("checked") ) ?  return :

   ( $('#search_material').val() == '') ?  $('#search_material').focus() :

    $('.type-checkbox ').addClass('active-confirm');

 });
6
  • return : - так можно? Нет, нельзя.
    – user176262
    Commented 13 окт. 2017 в 15:08
  • В тернарных операторах нельзя использовать другие операторы, в частности, <code>return</code>. Используйте его, чтобы возвращать результат условия. Commented 13 окт. 2017 в 15:21
  • 1
    Ошибка заключается в использовании не того инструмента. В данном случае не стоит переписывать условия с помощью тернарных операторов
    – Grundy
    Commented 13 окт. 2017 в 15:30
  • из одного костыля другой собрались делать? Если у вас внутри if написан return то на кой черт вы пишите else вообще? просто продолжайте код дальше
    – teran
    Commented 13 окт. 2017 в 15:30
  • тут лучше вообще без условий обойтись: $(".boolean ").hasClass("checked") || $('#search_material').val() == '' && $('#search_material').focus() || $('.type-checkbox ').addClass('active-confirm')
    – Grundy
    Commented 13 окт. 2017 в 15:32

4 ответа 4

2

Упростим задачу:

function aaa() {
  (true)? return : 2;
}

Смотрим ошибку в консоли.

1

Вместо return поставьте пустое выражение '' всё равно у вас по логике функции после условия нет никаких действий.

Чтобы выполнялось каждое подвыражение - "условие ? выражение1 : выражение2" бери в круглые скобки, например:

((условие) ? выражение1 : выражение2)

$('#search-btn').click(function () {

    ( $(".boolean ").hasClass("checked") ) ?  '' :

   (( $('#search_material').val() == '') ?  $('#search_material').focus() :

    $('.type-checkbox ').addClass('active-confirm'));

 });
0

А если вот так?

$('#search-btn').click(function () {
    var $s = $('#search_material'),
        $c = $('.type-checkbox '),

        b = $(".boolean").hasClass("checked"),
        s = $s.val();

    b ? null : s ? $s.focus() : $c.addClass('active-confirm');      
});
-1

А так будет работать? Если второе условие полностью взять в скобки. У вас же сейчас поидее получается a ? b : c ? d : e а надо a ? b : (c ? d : e)

 $('#search-btn').click(function () {

    ( $(".boolean ").hasClass("checked") ) ?  return :

   (( $('#search_material').val() == '') ?  $('#search_material').focus() :

    $('.type-checkbox ').addClass('active-confirm'));

 });

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.