[jQuery] Rendere il codice più magro

[blinkshow87]

Salve,
io ho questo semplice codice:

<h3><a id="uno">Uno</a></h3>          
   <ul class="uno">
Prima prova
[*]<h3><a id="due">Due</a></h3>          
   <ul class="due">[*]Seconda prova[/list][*]<h3><a id="tre">Tre</a></h3>          
   <ul class="tre">[*]Terza prova[/list][/list]

<script type="text/javascript">     
$('ul,p').hide();     
$('#uno').click(function() {         
     $('.uno').toggle();    
     });     
$('#due').click(function() {         
     $('.due').toggle();    
     });      
$('#tre').click(function() {        
     $('.tre').toggle();     
     });
</script>

Chiedo qualche consiglio su come diminuire la lunghezza del codice, perché per una cosa così semplice mi sembrano troppi i passaggi.

Grazie a tutti, buona serata

[cavicchiandrea]

Prova cosi

<script type="text/javascript">     
$('ul,p').hide();     
$('#uno, #due, #tre').live('click',function() {         
     $('.uno, .due, .tre').toggle();    
     });     
</script>

[lnessuno]

Oppure (a seconda di cosa ti serve)

<script type="text/javascript">     
$('ul,p').hide();     
$('ol a').live("click", function() {    
     $("." + $(this).attr("id")).toggle();    
});​
</script>

O, un pochino più leggibile, così

$('ul,p').hide();     
$('ol a').live("click", function() {    
    var element = $(this).attr("id");   
     $("." + element).toggle();    
});​

[blinkshow87]

Grazie mille per avermi risposto,
a questo ci avevo pensato pure io:

<script type="text/javascript">
$('ul,p').hide(); $('#uno, #due, #tre').live('click',function() {
$('.uno, .due, .tre').toggle(); });
</script>

ma vorrei che non si visualizzassero tutti allo stesso momento, ma uno alla volta, per questo avevo usato quella soluzione.

Per lnessuno ho provato tutte e due i codici postati ma non funzionano, gli "ul" non si nascondono e cliccando non succede nulla.

Grazie comunque

[cavicchiandrea]

$('ul,p').hide();     
$('#uno, #due, #tre').live('click',function() {         
     $('.'+$(this).attr("id")).toggle();    
     });

[cavicchiandrea]

Originariamente inviato da blinkshow87
... Per lnessuno ho provato tutte e due i codici postati ma non funzionano, gli "ul" non si nascondono e cliccando non succede nulla ...

Nel codice di Inessuno ci sono due spazi che bloccavano il funzionamento tolti funziona benissimo vedi esempio

[lnessuno]

Originariamente inviato da blinkshow87
ma vorrei che non si visualizzassero tutti allo stesso momento, ma uno alla volta, per questo avevo usato quella soluzione.

Per lnessuno ho provato tutte e due i codici postati ma non funzionano, gli "ul" non si nascondono e cliccando non succede nulla.

Non so perchè, ma facendo copia-incolla da jsfiddle si crea qualche strano carattere che genera un errore... togliendo qualche spazio di solito riparte, come diceva cavicchiandrea.

Per quanto riguarda l'aprire un elemento chiudendo gli altri in modo da averne uno solo aperto, prova così: http://jsfiddle.net/rNsRt/

[lnessuno]

In alternativa, se vuoi una roba un po' più pulita (che non ti crei la necessità di usare id e classi per gli elementi dell'accordion, perchè alla fin fine è pur sempre un accordion), tempo fa buttai giù questo... http://jsfiddle.net/C6qBT/

[blinkshow87]

Perfetto!!!

Grazie mille per il vostro aiuto!!