Укоротить скрипт с условиями
 

Как можно укоротить этот индусский скрипт:

if
            (xa>1000)
            {padModule.innerHTML = 'ей'}
            else
            {padModule.innerHTML = 'я'};
            
            if
            (xa>0)
            {checklist.setAttribute("class", "list")}
            else
            {checklist.setAttribute("class", "listoff")};
            
            if
            (xa>1000)
            {modules.style.marginLeft = '132px'}
            else
            {modules.style.marginLeft = '124px'};

Например объединить отработку одинакового условия...

if (xa>1000) {
	padModule.innerHTML = 'ей'
	modules.style.marginLeft = '132px'
} else {
	padModule.innerHTML = 'я'
	modules.style.marginLeft = '124px'
};
if (xa>0) {
	checklist.setAttribute("class", "list")
} else {
	checklist.setAttribute("class", "listoff")
};

className :write:

Тогда уже плюсом тернарный оператор...

checklist.className=(xa>0)? 'list': 'listoff';

Спасибо, не знал этого, а узнать очень хотелось.
Только недавно начал изучение JS

k_DizeL, че за бред? Верстку надо делать так, чтобы не приходилось подгонять пиксель-в-пиксель.

В строке меняется падеж, соответственно меняется отступ, пришлось подгонять окно с position: absolute;, по другому ни как или не нравится

А этот колхоз уменьшается?

(parseInt(hosting.options[hosting.selectedIndex].value) >= 1) ? padej.innerHTML = 'а' : 0;
            (parseInt(hosting.options[hosting.selectedIndex].value) >= 1) ? document.getElementById('padej').style.display = 'inline' : 0;
            (parseInt(hosting.options[hosting.selectedIndex].value) == 0) ? document.getElementById('padej').style.display = 'none' : 0;
       
            (parseInt(pages2.options[pages2.selectedIndex].value) < 0) ? padPage.innerHTML = 'ы' : 0;
            (parseInt(pages2.options[pages2.selectedIndex].value) < 0) ? document.getElementById('padPage').style.display = 'inline' : 0;
            (parseInt(pages2.options[pages2.selectedIndex].value) >= 0) ? document.getElementById('padPage').style.display = 'none' : 0;

Ну ты видишь что у тебя почти вся масса кода состоит из дублей? Значит уменьшается.

hosting.options[hosting.selectedIndex].value
В переводе с индус-кода это будет hosting.value.

Да, это логично, сам не доглядел, устал уже к тому времени