Estoy seguro de que se puede hacer esto de alguna otra forma mas eficiente!

[moro666]

Tengo un código muy simple para hacer que un div se muestre o se oculte:

Código HTML:

<section id="lol1" class="uno">
    <div class="container">
        <div id="sublol1" class="subcontainer boxx" >
            CONTENIDO OCULTO
        </div>
    </div>
</section>
<section id="lol2" class="dos">
    <div class="container">
        <div id="sublol2" class="subcontainer boxx" >
            CONTENIDO OCULTO
        </div>
    </div>
</section>
.
.
.
 

Código Javascript:

$(document).ready(function() {
$('#otro-link').click(function() {
    if($('#sublol1').hasClass('subcontainer')){
        $('#sublol1').removeClass('subcontainer');
    }else{
        $('#sublol1').addClass('subcontainer');
    }
    });
$('#otro2-link').click(function() {
    if($('#sublol2').hasClass('subcontainer')){
        $('#sublol2').removeClass('subcontainer');
    }else{
        $('#sublol2').addClass('subcontainer');
    }
    });
});
 

Según al botón (div con id otroX-link) al que se le de se añade o se quita una clase que lo hace visible o invisible.

Viendo como otra gente programa con javascript y lo que he hecho yo, esto me parece una chapuza monumental. Algún consejo?

Muchas gracias a todos!

[Alexis88]

Si se trata de asignar/quitar clases con métodos de jQuery, está bien así, aunque puedes usar el método toggleClass, con el cual asignas la clase al elemento si no la tiene y se la quitas, si la tiene. De esta manera, evitarías el uso de condiciones.

Saludos