2016-03-17 2 views
0

Я выполняю подстановку переменных в выражении jquery. Это просто выглядит уродливо. Интересно, знает ли кто-нибудь лучший способ сделать это?Попытка украсить строку jQuery

... пробегает по календарю, глядя на даты до высокого света в календарном таблице, фрагмент коды:

var = 31; 
$('#s' + var + '').css({"background-color":"blue","color":"white"}); 

Это работает, только синтаксис немного не-элегантное. Благодарю.

+0

https://jsfiddle.net/hgssreh6/2/ .. скопируйте свой код в jsfiddle и нажмите на ссылку 'Tidy' сверху .. дает лучший читаемый код. Есть и другие js beautifiers, но это то, что я использую. – Krishna

+0

Спасибо Кришне. :-) –

+0

Чтобы упростить код, вы должны предоставить нам более подробный вид вашего кода, например, всю функцию, выполняющую эту операцию, чтобы мы могли предложить лучшие способы делать вещи. Простое переписывание одной строки кода не является открытым для очень многих оптимизаций. – jfriend00

ответ

2

Вам не нужен $('#s' + var + ''). Этого должно быть достаточно:

var xyz = 31; 
$('#s' + xyz).css({"background-color":"blue","color":"white"}); 

Я уверен, что это всего лишь пример, но не называют вар var.

+0

Ничего себе, спасибо за быстрый ответ. Да, вар просто вскочил. Это сумасшедшее имя. Это действительно день. –

0

Если украшение это то, что вы хотите, моя структура кода ниже.

var xyz = 31; 
    $('#s' + xyz).css({ 
     "background-color":"blue", 
     "color":"white" 
    }); 

Я сделаю это, как в моем файле CSS, потому что для меня, читаемого кода красивого код.

1

мая Вы можете создать класс как

.highlight{ 
background:blue; 
color:white; 
} 

А может просто добавить или удалить, как вы хотите

abc = 31; 
$('#s' + abc).addClass("highlight"); 

, если вы хотите, чтобы удалить стиль, Просто используйте

$('#s' + abc).removeClass("highlight"); 
+0

Да, гораздо чище. –

+0

для подсветки, это хорошая практика, чтобы закрепить стиль в новом классе. , – Sachin