Уменьшить Javascript длину функции

Question

У меня есть яваскрипта функцию, которая выглядит следующим образом:

function myfun(){ 
//product.1 
var t1=document.getElementById('1').innerHTML; 
var link = document.getElementsByClassName(t1); 

if(document.getElementsByClassName(t1).length==1){ 
document.getElementById(t1).innerHTML=link[0].outerHTML; 
document.getElementById(t1).getElementsByTagName('a')[0].className='dsad'; 

} 
if(document.getElementsByClassName(t1).length==2){ 
document.getElementById(t1).innerHTML=link[0].outerHTML+'; '+link[1].outerHTML; 
var element = document.getElementById(t1).getElementsByTagName('a')[0].className='dsad'; 
var element2 = document.getElementById(t1).getElementsByTagName('a')[1].className='dsad'; 
}... 
//product.2 
var t2=document.getElementById('2').innerHTML; 

Это идет как это до if(document.getElementsByClassName(t1).length==10), а затем продолжает элемент - document.getElementById('2') и так далее, пока он не достигнет числа элементов 10. Весь сценарий составляет около 700 строк, и я хочу как-то уменьшить его. Я думал о цикле for, но я не понимаю, как это реализовать. Какие-либо предложения?

Answer 1

Try:

for(i = 1; i <= 10; i++){ 
    if(document.getElementsByClassName(t1).length == i){ 
    document.getElementById(t1).innerHTML=link[i-1].outerHTML; 
    document.getElementById(t1).getElementsByTagName('a')[ i-1 ].className = 'dsad'; 
    } 
} 

EDIT:

Если вы хотите также элемент идентификатор, чтобы увеличить Вы должны попробовать:

for(i = 1; i <= 10; i++){ 

var t1=document.getElementById(''+i).innerHTML; 
var link = document.getElementsByClassName(t1); 

     if(document.getElementsByClassName(t1).length == i){ 
     document.getElementById(t1).innerHTML=link[i-1].outerHTML; 
     document.getElementById(t1).getElementsByTagName('a')[ i-1 ].className = 'dsad'; 
     } 
    } 

Answer 2

Вы можете попробовать это:

for(var id=1;id<=10;id++){ 
    var t1id=""+id; 
    var t1=document.getElementById(t1id).innerHTML; 
    var link = document.getElementsByClassName(t1); 
    for(var num=1;num<=10;num++){ 
     if(document.getElementsByClassName(t1).length==num){ 
      document.getElementById(t1).innerHTML=link[0].outerHTML; 
      document.getElementById(t1).getElementsByTagName('a')[0].className='dsad'; 
     } 
    } 
} 

Answer 3

function myfun(){ 
    for(i=1;i<=10;i++){ 
     var ti=document.getElementById(i).innerHTML; 
     var link = document.getElementsByClassName("t" + i); 

     if(link.length==1){ 
      link.innerHTML=link[0].outerHTML; 
      ti.getElementsByTagName('a')[i - 1].className='dsad'; 

     } 
     else if(link.length==2){ 
      ti.innerHTML=link[i - 1].outerHTML+'; '+link[i].outerHTML; 
      var element = ti.getElementsByTagName('a')[i - 1].className='dsad'; 
      var element2 = ti.getElementsByTagName('a')[i].className='dsad'; 
     } 
    } 
} 

Answer 4

Я не уверен в части кода. Поэтому я собираюсь предположить, что это зависит от идентификатора.

Первый я бы сделать, это создать объекты JavaScript с контентом, который вы хотите включить для Classname

var classNameDictionary = { 0 : 'dsad', 1 :'dsad, ...}; 

После того как вы это, то я буду реализовать цикл следующим образом:

function myFun(numElements){ 
    for(var i = 0; i < numElements; i++) { 
     var t = document.getElementById(''+i).innerHTML; 
     var link = document.getElementsByClassName(t); 
     document.getElementById(t).innerHTML = ""; 
     for(var j = 0; j < link.length; j++) { 
      document.getElementById(t).innerHTML +=link[j].outerHTML; 
      var element = document.getElementById(t).getElementsByTagName('a')[j].className = classNameDictionary[j]; 
     } 

    } 
} 

Я надеюсь, что это помогает