Te sobran los ; del if. Además el operador = es para asignar, si queres una comparación tenes que usar == o ===.
Aplicamos un tabulado más propicio:
Código Javascript
:
Ver originalfunction ver(nombre){
if(document.getElementById('nombre'+nombre).style.display == "none"){
document.getElementById('nombre'+nombre).style.display =" ";
} else {
document.getElementById('nombre'+nombre).style.display =" none";
}
}
Extraemos en variables aquellas expresiones que se repiten:
Código Javascript
:
Ver originalfunction ver(nombre){
var element = document.getElementById('nombre' + nombre);
if(element.style.display == "none"){
element.style.display = "";
} else {
element.style.display = "none";
}
}
Renombramos las variables para una mejor semantica y la hacemos un poco más génerica:
Código Javascript
:
Ver originalfunction toggle(id){
var element = document.getElementById(id);
if(element.style.display == "none"){
element.style.display = "";
} else {
element.style.display = "none";
}
}
Y finalmente, si queres mantener el mismo comportamiento, haces una funcion específica para tu caso:
Código Javascript
:
Ver originalfunction ver(nombre) {
toggle('nombre' + nombre);
}