Мой jquery это правильно?

Question

мне интересно, если моя функция JQuery правильно

<script> 
window.blinker = setInterval(function(){ 
    if(window.alerta){ 
    $('a.cadastrotopo').css('color','#346698'); 
    $('a.cadastrotopo').css('text-decoration','underline'); 
     window.alerta=false; 
    } 
    else{ 
     $('a.cadastrotopo').css('color','#000'); 
     $('a.cadastrotopo').css('text-decoration','none'); 
     window.alerta = true; 
    } 
},500); 
</script> 

работает нормально, но мне интересно, если я делаю правильный путь.

Благодарю вас.

Answer 1

Лично я хотел бы использовать CSS более, в частности, классы:

a.cadostropo { 
    color: #000; 
    text-decoration: none; 
} 
a.alert { 
    color: #346698; 
    text-decoration: underline; 
} 

и затем раствор становится тривиальным:

setInterval(toggleAlert, 500); 

function toggleAlert() { 
    $("a.cadostropo").toggleClass("alert"); 
} 

Одна сторона Примечание: вместо нескольких css() вызовов можно использовать анонимные объекты укажите несколько свойств.

$("a.cadostropo").css({color: "#346698", textDecoration: "underline"}); 

Это, как говорится, я предпочитаю не использовать hardcoded CSS-манипуляции, подобные этому. Благоприятные классы. Они намного более гибкие, и вам не нужно беспокоиться о деструктивных изменениях и откатывать их назад.

Answer 2

Да, вы, хотя вы можете объединить объявления css, чтобы они выглядели так.

.css({'color' : "#346698", 'text-decoration' : "underline"});