LINUX.ORG.RU

История изменений

Исправление crutch_master, (текущая версия) :

    font_arr['1'] =  new Array();
    font_arr['1']['fontsize'] = '14px';
    font_arr['1']['lineheight'] = '30px';
    font_arr['1']['h1'] = '22px';
    font_arr['1']['h2'] = '18px';
    font_arr['1']['h3'] = '16px';
    font_arr['2'] =  new Array();
    font_arr['2']['fontsize'] = '18px';
    font_arr['2']['lineheight'] = '35px';
    font_arr['2']['h1'] = '26px';
    font_arr['2']['h2'] = '22px';
    font_arr['2']['h3'] = '20px';
    font_arr['3'] =  new Array();
    font_arr['3']['fontsize'] = '24px';
    font_arr['3']['lineheight'] = '45px';
    font_arr['3']['h1'] = '32px';
    font_arr['3']['h2'] = '28px';
    font_arr['3']['h3'] = '26px';

Какой гнусный, отвратительный говнокод. Ты в курсе, что можно было, хотя-бы вот так:

font_arr = [
{},
{ fontsize : "14px", lineheight : "30px", h1 : "22px", h2 : "18px", h3: "16px"},
{ fontsize : "18px", lineheight : "35px", h1 : "26px", h2 : "22px", h3: "20px"},
{ fontsize : "24px", lineheight : "45px", h1 : "32px", h2 : "28px", h3: "26px"}
];
Почему вот это
document.getElementById('fz419enable').style.display = 'none';
нельзя завернуть в функцию? Почему все вот эти
var anchors = document.getElementsByTagName('a');
    		for(i = 0; i < anchors.length; i++) {anchors[i].style.background = "none";}
    		var divs = document.getElementsByTagName('div');
    		for(i = 0; i < divs.length; i++) {divs[i].style.background = 'none';}
    		var spans = document.getElementsByTagName('span');
    		for(i = 0; i < spans.length; i++) {spans[i].style.background = 'none';}
    		var bodies = document.getElementsByTagName('body');
    		for(i = 0; i < bodies.length; i++) {bodies[i].style.background = 'none';}
    		var tables = document.getElementsByTagName('table');
    		for(i = 0; i < tables.length; i++) {tables[i].style.background = 'none';}
    		var tds = document.getElementsByTagName('td');
    		for(i = 0; i < tds.length; i++) {tds[i].style.background = 'none';}
    		var trs = document.getElementsByTagName('tr');
    		for(i = 0; i < trs.length; i++) {trs[i].style.background = 'none';}
    		var lis = document.getElementsByTagName('li');
    		for(i = 0; i < lis.length; i++) {lis[i].style.background = 'none';}
    		var uls = document.getElementsByTagName('ul');
    		for(i = 0; i < uls.length; i++) {uls[i].style.background = 'none';}
anchors, divs, spans, etc нельзя было сделать строками, запихать в массив, а потом по нему пробежаться? И сделать forEach вместо цикла (да, for быстрее, но тут это не критично же)
Это достойно сайта govnokod.ru

Исправление crutch_master, :

    font_arr['1'] =  new Array();
    font_arr['1']['fontsize'] = '14px';
    font_arr['1']['lineheight'] = '30px';
    font_arr['1']['h1'] = '22px';
    font_arr['1']['h2'] = '18px';
    font_arr['1']['h3'] = '16px';
    font_arr['2'] =  new Array();
    font_arr['2']['fontsize'] = '18px';
    font_arr['2']['lineheight'] = '35px';
    font_arr['2']['h1'] = '26px';
    font_arr['2']['h2'] = '22px';
    font_arr['2']['h3'] = '20px';
    font_arr['3'] =  new Array();
    font_arr['3']['fontsize'] = '24px';
    font_arr['3']['lineheight'] = '45px';
    font_arr['3']['h1'] = '32px';
    font_arr['3']['h2'] = '28px';
    font_arr['3']['h3'] = '26px';

Какой гнусный, отвратительный изврат. Ты в курсе, что можно было, хотя-бы вот так:

font_arr = [
{},
{ fontsize : "14px", lineheight : "30px", h1 : "22px", h2 : "18px", h3: "16px"},
{ fontsize : "18px", lineheight : "35px", h1 : "26px", h2 : "22px", h3: "20px"},
{ fontsize : "24px", lineheight : "45px", h1 : "32px", h2 : "28px", h3: "26px"}
];
Почему вот это
document.getElementById('fz419enable').style.display = 'none';
нельзя завернуть в функцию? Почему все вот эти
var anchors = document.getElementsByTagName('a');
    		for(i = 0; i < anchors.length; i++) {anchors[i].style.background = "none";}
    		var divs = document.getElementsByTagName('div');
    		for(i = 0; i < divs.length; i++) {divs[i].style.background = 'none';}
    		var spans = document.getElementsByTagName('span');
    		for(i = 0; i < spans.length; i++) {spans[i].style.background = 'none';}
    		var bodies = document.getElementsByTagName('body');
    		for(i = 0; i < bodies.length; i++) {bodies[i].style.background = 'none';}
    		var tables = document.getElementsByTagName('table');
    		for(i = 0; i < tables.length; i++) {tables[i].style.background = 'none';}
    		var tds = document.getElementsByTagName('td');
    		for(i = 0; i < tds.length; i++) {tds[i].style.background = 'none';}
    		var trs = document.getElementsByTagName('tr');
    		for(i = 0; i < trs.length; i++) {trs[i].style.background = 'none';}
    		var lis = document.getElementsByTagName('li');
    		for(i = 0; i < lis.length; i++) {lis[i].style.background = 'none';}
    		var uls = document.getElementsByTagName('ul');
    		for(i = 0; i < uls.length; i++) {uls[i].style.background = 'none';}
anchors, divs, spans, etc нельзя было сделать строками, запихать в массив, а потом по нему пробежаться? И сделать forEach вместо цикла (да, for быстрее, но тут это не критично же)
Это достойно сайта govnokod.ru

Исходная версия crutch_master, :

    font_arr['1'] =  new Array();
    font_arr['1']['fontsize'] = '14px';
    font_arr['1']['lineheight'] = '30px';
    font_arr['1']['h1'] = '22px';
    font_arr['1']['h2'] = '18px';
    font_arr['1']['h3'] = '16px';
    font_arr['2'] =  new Array();
    font_arr['2']['fontsize'] = '18px';
    font_arr['2']['lineheight'] = '35px';
    font_arr['2']['h1'] = '26px';
    font_arr['2']['h2'] = '22px';
    font_arr['2']['h3'] = '20px';
    font_arr['3'] =  new Array();
    font_arr['3']['fontsize'] = '24px';
    font_arr['3']['lineheight'] = '45px';
    font_arr['3']['h1'] = '32px';
    font_arr['3']['h2'] = '28px';
    font_arr['3']['h3'] = '26px';

Какой гнусный, отвратительный изврат. Ты в курсе, что можно было, хотя-бы вот так:

font_arr = [
{},
{ fontsize : "14px", lineheight : "30px", h1 : "22px", h2 : "18px", h3: "16px"},
{ fontsize : "18px", lineheight : "35px", h1 : "26px", h2 : "22px", h3: "20px"},
{ fontsize : "24px", lineheight : "45px", h1 : "32px", h2 : "28px", h3: "26px"}
];
Почему вот это
document.getElementById('fz419enable').style.display = 'none';
нельзя завернуть в функцию? Почему все вот эти
var anchors = document.getElementsByTagName('a');
    		for(i = 0; i < anchors.length; i++) {anchors[i].style.background = "none";}
    		var divs = document.getElementsByTagName('div');
    		for(i = 0; i < divs.length; i++) {divs[i].style.background = 'none';}
    		var spans = document.getElementsByTagName('span');
    		for(i = 0; i < spans.length; i++) {spans[i].style.background = 'none';}
    		var bodies = document.getElementsByTagName('body');
    		for(i = 0; i < bodies.length; i++) {bodies[i].style.background = 'none';}
    		var tables = document.getElementsByTagName('table');
    		for(i = 0; i < tables.length; i++) {tables[i].style.background = 'none';}
    		var tds = document.getElementsByTagName('td');
    		for(i = 0; i < tds.length; i++) {tds[i].style.background = 'none';}
    		var trs = document.getElementsByTagName('tr');
    		for(i = 0; i < trs.length; i++) {trs[i].style.background = 'none';}
    		var lis = document.getElementsByTagName('li');
    		for(i = 0; i < lis.length; i++) {lis[i].style.background = 'none';}
    		var uls = document.getElementsByTagName('ul');
    		for(i = 0; i < uls.length; i++) {uls[i].style.background = 'none';}
anchors, divs, spans, etc нельзя было сделать строками, запихать в массив, а потом по нему пробежаться?
Это достойно сайта govnokod.ru