[SOLUCIONADO] Problemas con un if

Borha · #1 (**permalink**) 14/05/2014, 16:47

Tengo una función para hacer que aparezca y desaparezca un texto al pulsar un link. Entonces el texto a aparece pero al pulsar de nuevo no desaparece. Este es el codigo.

Código:

<a href="#" onclick="cambiar('oculto'); return false;" class="l_lorena">Por que no hay dos personas iguales,<br>
no puede haber dos personas iguales.</a>
<div id="oculto" style="display: none;">
<p class="toculto">"Cada persona necesita comunicar algo distinto  con su imagen, que no es ajeno a su entoprno personal o al escenario de tendencias actuales, por eso mi método consiste encrear tendencia en exclusiva en cada cliente"</p>
</div><br>


<a href="#" onclick="cambiar2('oculto2'); return false;" class="l_lorena">Yo no sigo tendencias,<br>
soy la tendencia.</a>
<div id="oculto2" style="display: none;">
<p class="toculto">"Un método que profundiza en la necesidad de cada client de ser uno mismo, de expresarse desde su propia tendencia concebida por Lorena a partir de su fisionomía física y su entorno social. Tú imagen habla de tí, que diga algo único."</p>
</div><br>


<a href="#" onclick="cambiar3('oculto3'); return false;" class="l_lorena">Unos clientes muy especiales.</a>
<div id="oculto3" style="display: none;">
<p class="toculto">"CAMBIAR!!!!"</p>
</div><br>

Cita:

Alexis88 · #2 (**permalink**) 14/05/2014, 17:17

En lugar de hacer la comparación de esta forma:

Código Javascript:

Ver originalif (document.getElementById(oculto2).style.display = '')

Hazlo así:

Código Javascript:

Ver originalif (document.getElementById(oculto2).style.display != 'none')

Ten en cuenta que cuando realizas una comparación, debes de utilizar dos signos de igualdad, porque con uno solo asignas un valor, mas no lo comparas. Por otro lado, para verificar que un elemento es visible, basta con que el valor de la propiedad display sea distinto a none.

Saludos

Borha · #3 (**permalink**) 14/05/2014, 18:27

Ya lo conseguí, muchas gracias :), lo deje así

Código:

function cambiar(oculto)
{
	if(document.getElementById(oculto).style.display == 'none'){
	document.getElementById(oculto).style.display = '';
	document.getElementById(oculto2).style.display = 'none';
	document.getElementById(oculto3).style.display = 'none';

	} else {
	document.getElementById(oculto).style.display = 'none';
	document.getElementById(oculto2).style.display = 'none';
	document.getElementById(oculto3).style.display = 'none';
	}
}



function cambiar2(oculto2)
{
	if (document.getElementById(oculto2).style.display == 'none'){
	document.getElementById(oculto2).style.display = '';
	document.getElementById(oculto).style.display = 'none';
	document.getElementById(oculto3).style.display = 'none';
	

	} else {
	
	document.getElementById(oculto2).style.display = 'none';
	document.getElementById(oculto).style.display = 'none';
	document.getElementById(oculto2).style.display = 'none';
	
}
}

function cambiar3(oculto3)
{
	if(document.getElementById(oculto3).style.display == 'none'){
	document.getElementById(oculto3).style.display = '';
	document.getElementById(oculto).style.display = 'none';
	document.getElementById(oculto2).style.display = 'none';
	} else {
	
	document.getElementById(oculto3).style.display = 'none';
	document.getElementById(oculto).style.display = 'none';
	document.getElementById(oculto2).style.display = 'none';
	
}
}

Y ahora para que al mostrar un párrafo se cierren los demás (en caso de estar abiertos), como seria por que de esta manera al ser 3 funciones diferentes no se podría no? tendría que ser una única función.

Italico76 · #4 (**permalink**) 14/05/2014, 18:49

@Borha : deberias re-factorizar o si sigues agregando enlaces te vas a volver chino

En vez de:

Cita:

onclick="cambiar2('oculto2');"

... podrias hacer:

Cita:

onclick="cambiar2(this);"

Lo que te permitira re-factorizar las N-funciones cambiar_i() a cambiar(enlace)

videlxu · #5 (**permalink**) 15/05/2014, 09:26

Estas usando el metodo super antiguo te recomiendo que leas un poco mas de js, te doy una pista con un hide() y show() lo haces sin ningun problema. salu2

Italico76 · #6 (**permalink**) 15/05/2014, 09:54

Cita:

Iniciado por videlxu

Estas usando el metodo super antiguo te recomiendo que leas un poco mas de js, te doy una pista con un hide() y show() lo haces sin ningun problema. salu2

Eso que dices es con un framework (jQuery) con con JS a secas

Para eso que use toggle() en jQ y no escribe nada

Alexis88 · #7 (**permalink**) 15/05/2014, 10:10

Cita:

Iniciado por videlxu

con un hide() y show() lo haces sin ningun problema.

Esos son métodos de jQuery, mientras que este es el foro de JavaScript, en donde se trabaja con código nativo.

Saludos

Borha · #8 (**permalink**) 18/05/2014, 08:29

Ya lo conseguí, gracias a todos :)