0

Как правильно дать ссылку на переменную temp?

Работает:

$('.selectCategory option:contains("Хлеб")').prop('selected',true);

Не работает(

 var temp = ("Хлеб");
 $('.selectCategory option:contains(temp)').prop('selected',true);
1
  • Для чего вокруг строки "Хлеб" круглые скобки?
    – Arnial
    30 сен 2016 в 10:25

2 ответа 2

4

То что находится внутри '' просто строка. Вам нужно явно записать в неё значение из переменной.

Можно сделать так:

var temp = "Хлеб";
$('.selectCategory option:contains("'+temp+'")').prop('selected',true);

либо используя es6

const temp = "Хлеб";
$(`.selectCategory option:contains("${temp}")`).prop('selected',true);
4
  • Так пробовал "'temp'", так "'+temp+'" не пробовал. Сенкс. Странный синтаксис
    – helloWorld
    30 сен 2016 в 10:29
  • 1
    @BogdanShulga, почему странный? Обычная конкатенация.
    – JimmDiGriz
    30 сен 2016 в 10:36
  • 1
    Если бы js подставлял значения переменной в середину строки, котораяя по какойто причине, сожержит в себе набор символов последовательность которых совпадает с назвванием одной из переменных, определённых гдето ранее, на нём невозможно былобы писать. Т.к. никогда не знаешь что там внезапно появится.
    – Arnial
    30 сен 2016 в 10:37
  • А теперь заменяем "Хлеб" на ")_(" и наблюдаем последствия :) 30 сен 2016 в 11:26
3

Не надо писать сложные селекторы через конкатенацию строк только для того чтобы потом jquery разбирала их обратно.

Вот так это делается правильно:

$('.selectCategory option')
  .filter(function() { return this.text.indexOf(temp) !== -1; })
  .prop('selected', true);

Или даже вот так:

$('.selectCategory option').each(function() {
  if (this.text.indexOf(temp) !== -1)
    this.selected = true;
}); 

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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