ЙажаСценарий / Говнокод #1674 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
function ValidatorEnable(val, enable) 
{
    val.enabled = (enable != false);
    ValidatorValidate(val);
    ValidatorUpdateIsValid();
}

Обратите внимание на третью строку!
(Сначала кажется, что она ничего не делает, но на самом деле она переводит тип в boolean), но обычно пишут все-таки
val.enabled = !!enable

Trams Trams, (Updated )

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

ЙажаСценарий / Говнокод #1670 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
//Из библиотеки prototype

....
  unescapeHTML: function() {
    return this.replace(/&/g,'&').replace(/</g,'<').replace(/>/g,'>');
  }

....

Смеяться надо из-за того, что замена & на & идёт первой, таким образом &lt замениться на <, а не на <

Oleg_quadro Oleg_quadro, (Updated )

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

ЙажаСценарий / Говнокод #1615 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
  18. 18
  19. 19
  20. 20
  21. 21
  22. 22
  23. 23
  24. 24
  25. 25
  26. 26
  27. 27
  28. 28
  29. 29
  30. 30
  31. 31
  32. 32
  33. 33
  34. 34
  35. 35
  36. 36
  37. 37
  38. 38
  39. 39
  40. 40
  41. 41
  42. 42
  43. 43
  44. 44
  45. 45
  46. 46
  47. 47
  48. 48
  49. 49
  50. 50
  51. 51
  52. 52
function verify()
{	
	 //1
  if(event.clientX >= 20 && event.clientX <= 113 && event.clientY >= 20 && event.clientY <= 90 && trigger == 1)	
     {
     name.style.top=21
     name.style.left=21
     name.border=0
     if(name.name == "one"){counter1=1; document.all.ok.style.visibility = 'visible'; document.all.no.style.visibility = 'hidden'}
	 else{counter1=0; document.all.no.style.visibility = 'visible'; document.all.ok.style.visibility = 'hidden'; errors++}
	 trigger = 0
	 movements++ 
     document.all.movements.value = movements
	
	 
     }
	 //2
	 if(event.clientX >= 113 && event.clientX <= 206 && event.clientY >= 20 && event.clientY <= 90 && trigger == 1)	
     {
     name.style.top=21
     name.style.left=114
     name.border=0
     if(name.name == "two"){counter2=1; document.all.ok.style.visibility = 'visible'; document.all.no.style.visibility = 'hidden'; errors++}else{counter2=0; document.all.no.style.visibility = 'visible'; document.all.ok.style.visibility = 'hidden'; errors++}
	 trigger = 0
	 movements++ 
     document.all.movements.value = movements
	 
     }
	 //3
	 if(event.clientX >= 206 && event.clientX <= 299 && event.clientY >= 20 && event.clientY <= 90 && trigger == 1)	
     {
     name.style.top=21
     name.style.left=207
     name.border=0
	 if(name.name == "three"){counter3=1; document.all.ok.style.visibility = 'visible'; document.all.no.style.visibility = 'hidden'}else{counter3=0; document.all.no.style.visibility = 'visible'; document.all.ok.style.visibility = 'hidden'; errors++}
     trigger = 0
	 movements++ 
     document.all.movements.value = movements
	 
     }
	 //4
	 if(event.clientX >= 299 && event.clientX <= 392 && event.clientY >= 20 && event.clientY <= 90 && trigger == 1)	
     {
     name.style.top=21
     name.style.left=300
     name.border=0
	 if(name.name == "four"){counter4=1; document.all.ok.style.visibility = 'visible'; document.all.no.style.visibility = 'hidden'}else{counter4=0; document.all.no.style.visibility = 'visible'; document.all.ok.style.visibility = 'hidden'; errors++}
     trigger = 0
	 movements++ 
     document.all.movements.value = movements
	 
 //И так - 30 раз:))

Говнокодил давно, еще в школе. Года через 4, в универе, вернулся к этому коду и ничего не смог понять:) Новое решение заняло примерно в 10 раз меньше места.

huzik huzik, (Updated )

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

ЙажаСценарий / Говнокод #1607 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
var ratingid=where+id+"rating";
			document.getElementById(ratingid).innerHTML="Представьте, что курсор мышки превратился в часики";
			xmlhttp.open("GET", "/"+where+"/"+act+"/"+id+"?reqt=update", true);
			xmlhttp.onreadystatechange = function() {
        		    if (xmlhttp.readyState == 4) {
            			if(xmlhttp.status == 200) {
                		    if(xmlhttp.responseText.substring(0,2) == "OK") {
                        		s=xmlhttp.responseText.search(/!RATING!/i)+8;
                                        e=xmlhttp.responseText.search(/!ERATING!/i);
                    			document.getElementById(ratingid).innerHTML=xmlhttp.responseText.substring(s,e);
            			    }
        			}
    			    }
			};

"Представьте, что вы - Властелин Вселенной, а курсор мышки - часики"

xaionaro xaionaro, (Updated )

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

ЙажаСценарий / Говнокод #1601 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
function handleServerResponse() 
{
	document.getElementById("guest").innerHTML = "<table width='100%'><tr><td align='center' valign='center'><p style='color:gray;text-align:center;'>" + xmlHttp.responseXML.documentElement.getElementsByTagName("ok").item(0).firstChild.data + "</p></td></tr></table>\n";	
}

Обработка данных AJAX-ом.

daemon_master daemon_master, (Updated )

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

ЙажаСценарий / Говнокод #1588 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
// Any fool can write code that a computer can understand. Good programmers write code that humans can understand. (C) Martin Fowler.
function changeLogo() {
	var pane = document.getElementById("pane");
	if(pane != null) {
		pane.height = 240;
		pane.width = document.body.clientWidth;
	} else {
		alert("it's sad, so sad...");
	}
	var logo = document.getElementById("logoJpg");
	if(logo != null) {
		logo.height = 50;
		logo.width = document.body.clientWidth * 0.23;
	} else {
		alert(" ...it's a sad, sad situation, and it's getting more and more absurd");
	}
}

Код совсем ничего, но камменты рулят

domavoy domavoy, (Updated )

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