codice per espldoere/nascondere campi al click di un bottone

[frenky7]

Salve, io ho un codice javascript per esplodere 2 campi di un form al click di un bottone e vorrei che se si clicca nuovamente sul bottone stesso questo formettino si richiudesse. Attualmente fa solo l'eslposione dei campi ma poi non li nasconde al click successivo.. il codice è questo ma non capisco il senso sinceramente..

function visualizzaForm(valore){
   if (valore == "primoForm"){
     document.getElementById("secondoForm").style.display="none";
     document.getElementById("primoForm").style.display="block";
   }else{
     document.getElementById("primoForm").style.display="none";
     document.getElementById("secondoForm").style.display="block";
   }
}

c'è il secondoform che non esiste e credo venga usato per nascondere il form vero e proprio.. ma non avrebbe più senso una cosa del genere(premetto fatta da me che non conosco bene il linguaggio) per fare queta cosa?

function visualizzaFormMia(valore){
   if ((valore.value == "primoForm") && (document.getElementById("primoForm").style.display == "none")) {
     document.getElementById("primoForm").style.display="block";
   }else{
     document.getElementById("primoForm").style.display="none";
   }
}

il codice html è questo invece:

<div style=""><a>[img]/images/stories/recru/butt_bp.png[/img]</a></div>

<div id="primoForm" class="primoForm">Primo contenuto</div>
   <div id="secondoForm" class="secondoForm">

Scusate se mi sono dilungato

[Vindav]

intanto leva il parametro che passi alla funzione(non stai passando il valore del div ma dell'immagine). E leva il primo controllo nell'if.

[frenky7]

molto giusto...
grazieeeeeeeee

[cavicchiandrea]

inziamo col dire che un'immagine non ha un value, pertanto inutile passare this.value, al massimo gli puoi passare l'elemento da apri/chiudi cosi:

function visualizzaForm(Div){
   if (document.getElementById(Div).style.display == "none" ||
document.getElementById(Div).style.display=="undefined"){
     document.getElementById(Div).style.display="block";
   }else{
     document.getElementById(Div).style.display="none";
   }
}

<div style=""><a>[img]/images/stories/recru/butt_bp.png[/img]</a></div>

<div id="primoForm" class="primoForm">Primo contenuto</div>
   <div id="secondoForm" class="secondoForm">

[frenky7]

ok..ho aggiunto la seconda parte dell'if...per l'undefined.
Tnx