Как правильно применить тут this?

Question

[Павел]

function setVal() {
        if ($('input#check1').is(':checked')) {
            $('input#check1').val('отмечено');
        }
        else {
            $('input#check1').val('не отмечено');
        }
    }

если поставлю $(this) вместо $('input#check1') и выведу в консоль то будет object , а не true или false что по моему пониманию логично , почему так и как правильно?

Comments

[Nik Gubin]

Прочтите про область видимости в JS. У Вас сразу же отпадёт масса вопросов.

В этом случае запишите $('input#check1') в начале функции в переменную, а потом используйте её (оборачивать переменную в $() перед использованием в условиях не нужно, т.к. в переменной уже будет jQuery объект)

[Павел]

Nik Gubin: спасибо

Answer 1

[iBird Rose]

ты должен в функцию передавать элемент

function setVal(el) {
        if (el.is(':checked')) {
            el.val('отмечено');
        }
        else {
            el.val('не отмечено');
        }
    }

setVal($('input#check1')); //передавать так
либо
setVal($(this)); //если вызываешь изнутри другой функции

Answer 2

[kulaeff]

Здесь не нужен this.

function setVal() {
  var $input = $('input#check1');

  if ($input).is(':checked')) {
    $input.val('отмечено');
  }
  else {
    $input.val('не отмечено');
  }
}

А еще лучше так:

function setVal() {
  var $input = $('input#check1');

  $input.val(($input).is(':checked') ? 'отмечено': 'не отмечено');
}

Comments

[Павел]

но это работает только с #check1 , а если id много , #check1 #check2 #check3 ... #check100500

[kulaeff]

Тогда вам подойдет решение iBird Rose