- 1
paramsObj.templateName = templateNewWindow.items.items[0].items.items[0].items.items[0].items.items[0].getValue();
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+159.1
paramsObj.templateName = templateNewWindow.items.items[0].items.items[0].items.items[0].items.items[0].getValue();
Получение значения ExtJS'ного контрола, который спрятан глубоко в дереве.
+157.3
function SetStyleForFilterOver(this2) {
this2.style.border = "1px solid #ea0000";
this2.style.color = "white";
this2.style.background = "#404040";
}
function SetStyleForFilterOut(this2) {
this2.style.border = "1px solid #404040";
this2.style.color = "#ea0000";
this2.style.color = "#ea0000";
this2.style.background = "#eeeeee";
}
Тс-с-с-с... Только не говорите автору про css и селектор ":hover" :о)
+158.1
case 'down':
if (parseInt(pageTop) + parseInt(pageHeight) - parseInt(elementTop) - parseInt(elementHeight) - parseInt(elementBorderTop) - parseInt(elementBorderBottom) >= parseInt(stepOfMove)) {
var topParam = parseInt(elementTop) - 0 + parseInt(stepOfMove) + 'px';
elementToMove.style.top = topParam;
this.updateElementsParams(id,'top',topParam);
} else {
stepOfMoveElement.value = parseInt(pageTop) - 0 + parseInt(pageHeight) - parseInt(elementTop) - parseInt(elementHeight) - parseInt(elementBorderTop) - parseInt(elementBorderBottom);
}
break;
и так миллион тыщ раз...
+163.8
//Я ТИПА ЗДЕСЬ СДЕЛАЛ НА ГАЛЕРЕЕ 4 КАРТИНКИ
if(this.id=='gallery') {
this.previewCount = 4;
} else {
this.previewCount = 4;
}
Недавно код пересматривали. Авторы долго отнекивались
+152.8
Перл #1:
...
setCityId(getCityId());
...
Перл #2:
function performOrgSearch(){
// ...
cityId = getCityId(); // функция getCityId вычищает всё из некого input type=text
+162.5
//было:
if(this.focusCount == this.blurCount + 1)
{
this.counter++;
}
//стало:
if(this.focusCount == this.blurCount + 1)
{
this.counter++;
}
else this.counter++;
Надо было из функции убрать проверку условия.
+149.8
function getOffset(elem) {
if (elem.getBoundingClientRect) {
// "правильный" вариант
return getOffsetRect(elem)
} else {
// пусть работает хоть как-то
return getOffsetSum(elem)
}
}
+159
function block_select_from(block,type){
if(type=='all'){
document.getElementById('block_'+block+'_all').style.display='';
document.getElementById('block_'+block+'_blogs').style.display='none';
document.getElementById('block_'+block+'_articles').style.display='none';
}
else if(type=='blogs'){
document.getElementById('block_'+block+'_all').style.display='none';
document.getElementById('block_'+block+'_blogs').style.display='';
document.getElementById('block_'+block+'_articles').style.display='none';
}
else if(type=='articles'){
document.getElementById('block_'+block+'_all').style.display='none';
document.getElementById('block_'+block+'_blogs').style.display='none';
document.getElementById('block_'+block+'_articles').style.display='';
}
}
Классика жанра с сайта popmech.ru
+156.2
function ChangeDisc(n, size, l_x, l_y, r_x, r_y)//а-а-а-а-а-а убейте меня
{
var disc_left = document.getElementById('sDisc1');
var disc_right = document.getElementById('sDisc2');
var disc_left_div = document.getElementById('sDisc1_div');
var disc_right_div = document.getElementById('sDisc2_div');
switch (size)
{
case 13:
disc_left_div.style.paddingLeft = (l_x+3)+'px';
disc_left_div.style.paddingTop = (l_y+2)+'px';
disc_right_div.style.paddingLeft = (r_x+6)+'px';
disc_right_div.style.paddingTop = (r_y+2)+'px';
break;
case 14:
disc_left_div.style.paddingLeft = (l_x+1)+'px';
disc_left_div.style.paddingTop = (l_y+1)+'px';
disc_right_div.style.paddingLeft = (r_x+1)+'px';
disc_right_div.style.paddingTop = (r_y+1)+'px';
break;
case 15:
disc_left_div.style.paddingLeft = l_x+'px';
disc_left_div.style.paddingTop = l_y+'px';
disc_right_div.style.paddingLeft = r_x+'px';
disc_right_div.style.paddingTop = r_y+'px';
break;
case 16:
disc_left_div.style.paddingLeft = (l_x-2)+'px';
disc_left_div.style.paddingTop = (l_y-2)+'px';
disc_right_div.style.paddingLeft = (r_x-2)+'px';
disc_right_div.style.paddingTop = (r_y-2)+'px';
break;
case 17:
disc_left_div.style.paddingLeft = (l_x-3)+'px';
disc_left_div.style.paddingTop = (l_y-4)+'px';
disc_right_div.style.paddingLeft = (r_x-5)+'px';
disc_right_div.style.paddingTop = (r_y-4)+'px';
break;
case 18:
disc_left_div.style.paddingLeft = (l_x-5)+'px';
disc_left_div.style.paddingTop = (l_y-5)+'px';
disc_right_div.style.paddingLeft = (r_x-8)+'px';
disc_right_div.style.paddingTop = (r_y-6)+'px';
break;
case 19:
disc_left_div.style.paddingLeft = (l_x-8)+'px';
disc_left_div.style.paddingTop = (l_y-8)+'px';
disc_right_div.style.paddingLeft = (r_x-8)+'px';
disc_right_div.style.paddingTop = (r_y-8)+'px';
break;
case 20:
disc_left_div.style.paddingLeft = (l_x-10)+'px';
disc_left_div.style.paddingTop = (l_y-10)+'px';
disc_right_div.style.paddingLeft = (r_x-10)+'px';
disc_right_div.style.paddingTop = (r_y-10)+'px';
break;
}
disc_left.src = Images[n].src;
disc_right.src = Images[n].src;
scroll(0,0);
}
копипаста процветает... правильно, зачем структурировать код? проще так.
+149.8
//простите меня за эти строки, но просто альтернативный метод swapNode (нативный только в IE)
//удаляет оригинальный нод и его приходится заново по id искать :(
document.getElementById(element['drag'].id).style.border = "2px solid gray";
document.getElementById(element['acce'].id).style.border = "2px solid gray";
:))))
каммент жжот
мой старый JS-быдлокод