Taeller og løkke

Skal du have skjult alle elementer, som har et id, som starter med adresse ?-)

for så er det nemmest at:

elms=document.getElementsByTagName('DIV');for(i=0;elms.length>i;i++){if(elms[i].id.indexOf('adresse')==0){e=elms[i].style;e.display=(e.display!='block')?'block':'none';}}

-- du kan også lige gå de konkrete elementer igennem med en kode à la:

for(i=1;6>i;i++){e = document.getElementById("adresse"+i).style;e.display=(e.display!='block')?'block':'none';}

-- men som du måske kan lugte, ville jeg selv vælge den første løsning, da den er mere robust, og kan tåle et vilkårligt antal, hvor den nederste skal ændres efter antallet !-)

Hej Roenving,
tak for dit hurtige svar!
Yes, antallet vil varierer. Du får lige et link så du kan se hvad jeg vil.
http://www.verdensbedstedagtilbud.dk/virksomhedsmappe/NYsammenligning.asp

Målet er at alle felter i tabellen er tomme ved load, og man så via tabeltoppen kan tilvælge de punkter man vil se/sammenligne. (loades toppe pga sidebredden)

Jeg har været igennem getElementsByTagName, getElementsByName og getElementById, og fandt at Id er vejen for mig her.

Jeg kan nu godt lave det på min måde som jeg har startet, men som du vil kunne se i min kode, vil scriptet blive ulidelig langt.
Jeg smider lige ID på mine andre felter i tabellen, og ser så om jeg gennemskue dit eksempel.

Hvis du kan se en smartere måde at gøre det vil jeg blive meget glad for at hører det.

mvh
Hopeless

Ok. Har lagt Id på alle mine felter, og afprøvet din model. Smukt smukt!

Så skal jeg lige lave den for de andre overskrifter også.

giver du et svar for point? :)

Start med at sætte linjeskift og indrykninger ind:

elms=document.getElementsByTagName('DIV');
for(i=0;elms.length>i;i++){
  if(elms[i].id.indexOf('adresse')==0){
    e=elms[i].style;
    e.display=(e.display!='block')?'block':'none';
  }
}

Velbekomme '-)

-- og tak for point ;~}

Det er mig som takker. :D
Roenving, hvis du lige tager et kik på siden igen og ser koden.
Jeg har gentaget funktionen for alle mine overskrifter, men er sikker på at jeg kan gøre det smartere ved at kalde een funktion via en variabel afhængig af rækken.
Istedet for:
function skjulAlleAldersgruppe(){
elms=document.getElementsByTagName('P');
for(i=0;elms.length>i;i++){
if(elms[i].id.indexOf('aldersgruppe')==0){
  e=elms[i].style;
  e.display=(e.display!='block')?'block':'none';
  }
}
}
function skjulAlleaabningstid(){
elms=document.getElementsByTagName('P');
for(i=0;elms.length>i;i++){
if(elms[i].id.indexOf('aabningstid')==0){
  e=elms[i].style;
  e.display=(e.display!='block')?'block':'none';
  }
}
}
.....osv osv
Kan jeg så ikke kalde dem via den samme funktion
skjulAlle(aldersgruppe);return false"
eller
skjulAlle(aabningstid);return false"

Hvordan vil sådan en funktion så se ud? :)

Og til sidst. Kan man lave en global funktion som åbner/lukker alle på en gang?

function skjulKolonne(colid){
  elms=document.getElementsByTagName('P');
  for(i=0;elms.length>i;i++){
    if(elms[i].id.indexOf(colid)==0){
      e=elms[i].style;
      e.display=(e.display!='block')?'block':'none';
    }
  }
}

-- og vil det ikke være mere hensigtsmæssigt at skjule td'erne ?-)

-- kaldet bliver så:

skjulAlle('aldersgruppe');return false;

Bemærk gnyfferne, for det skal være en tekststreng !-)

Lækkert. Dejligt at få trimmet koden.

Det med at skjule TD'erne var hvad jeg først gjorde, men da jeg godt ville beholde overskrifterne, rykkede rækkerne sig nedenunder så der kom rod i tabellen. (så hellere vise et tomt felt tænkte jeg) :))

super tak igen. u rule.

ps. hvis du havde et forslag til den globale vis/skjul alle funktion vil jeg gerne hører det. (du får self flere point hvis du mener det)
Jeg har prøvet at lave dette, men den "spejlvender" bare tabellen istedet for at enten vise alle, eller fjerne alle

function showHideAll(action){
  var tabel = document.getElementById('minTabel');
  var tags = tabel.getElementsByTagName('P');
  for(i=0;tags.length>i;i++){
    tags[i].style.display = action;
  }
}

<a href="#" onclick="showHideAll('none');return false;">Gem alle</a>
<a href="#" onclick="showHideAll('block');return false;">Vis alle</a>

selvfølgelig burde jeg jo sige til mig selv :)

Dejligt du har så godt styr på det!

Lukker og slukker herfra.