JavaScript — Говнокод.ру

JavaScript / Говнокод #1873

+158.1

1
2
3
4
5
6
7
8
9

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;

и так миллион тыщ раз...

Sadie, 21 Сентября 2009

Комментарии (1)

JavaScript / Говнокод #1870

+163.8

1
2
3
4
5
6

//Я ТИПА ЗДЕСЬ СДЕЛАЛ НА ГАЛЕРЕЕ 4 КАРТИНКИ
               if(this.id=='gallery') {
                    this.previewCount = 4;
               } else {
                    this.previewCount = 4;
               }

Недавно код пересматривали. Авторы долго отнекивались

hagnar, 19 Сентября 2009

Комментарии (3)

JavaScript / Говнокод #1869

+152.8

1
2
3
4
5
6
7
8
9

Перл #1:
 ...
 setCityId(getCityId()); 
 ...

Перл #2:
function performOrgSearch(){
  // ...
  cityId = getCityId(); // функция getCityId вычищает всё из некого input type=text

yvu, 18 Сентября 2009

Комментарии (4)

JavaScript / Говнокод #1855

+162.5

01
02
03
04
05
06
07
08
09
10
11
12
13
14

//было:

  if(this.focusCount == this.blurCount + 1)
   {
     this.counter++;
   }

//стало:

   if(this.focusCount == this.blurCount + 1)
   {
     this.counter++;
   }
   else this.counter++;

Надо было из функции убрать проверку условия.

psychos, 17 Сентября 2009

Комментарии (0)

JavaScript / Говнокод #1853

+149.8

1
2
3
4
5
6
7
8
9

function getOffset(elem) {
    if (elem.getBoundingClientRect) {
        // "правильный" вариант
        return getOffsetRect(elem)
    } else {
        // пусть работает хоть как-то
        return getOffsetSum(elem)
    }
}

code_master, 16 Сентября 2009

Комментарии (9)

JavaScript / Говнокод #1839

+159

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19

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

MV28jam, 14 Сентября 2009

Комментарии (10)

JavaScript / Говнокод #1795

+156.2

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61

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);
}

копипаста процветает... правильно, зачем структурировать код? проще так.

gorky, 09 Сентября 2009

Комментарии (7)

JavaScript / Говнокод #1780

+149.8

1
2
3
4

//простите меня за эти строки, но просто альтернативный метод swapNode (нативный только в IE)
//удаляет оригинальный нод и его приходится заново по id искать :(
document.getElementById(element['drag'].id).style.border = "2px solid gray";
document.getElementById(element['acce'].id).style.border = "2px solid gray";

:))))
каммент жжот

мой старый JS-быдлокод

danilissimus, 06 Сентября 2009

Комментарии (11)

JavaScript / Говнокод #1728

+151.6

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21

var addEventListener = function() {
    // Если браузер IE:
    if (window.attachEvent)
        // Возвращаем функцию для IE:
        return function(element, event, callback) {
            // Фиксуем callback, чтоб установить правильный
            // this, и передавать объект события первым параметром:
            callback.__callbackfix = function() {
                callback.call(element, window.event);
            };
            // Устанавливаем обработчик (не забываем про "on")
            return element.attachEvent("on" + event, callback.__callbackfix);
        }
    // Если браузер W3C-совместим:
    if (window.addEventListener)
        // Возвращаем функцию без фиксов:
        return function(element, event, callback) {
            // Последний параметр всегда false, для совместимости с IE:
            return element.addEventListener(event, callback, false);
        }
}();

Найдено на просторах рунета. А теперь пробуем повесить событие на window.

kolyaj, 31 Августа 2009