jquery - problema da principiante

[crisz78]

salve a tutti,
sto passando da prototype a jquery e non riesco più a far funzionare una cavolatina del menu:

questo è l'html del menu

<ul id="NewTopNav">
    <li onmouseout="mclosetime()" onmouseover="mopen('divCat_44')">Bimbo
        <div style="visibility: hidden;" id="divCat_44">
        Abbigliamento
        Scarpe
        </div>
    
    <li onmouseout="mclosetime()" onmouseover="mopen('divCat_46')">Donna
        <div style="visibility: hidden;" id="divCat_46">
        Maglie
        Pantaloni
        Scarpe
        </div>
    [/list]

e queste sono le funzioni jquery:

function mopen(id)
{	
	// cancel close timer
	mcancelclosetime();

	// close old layer
	if(ddmenuitem != 0) 
		$("#".ddmenuitem).attr("style","visibility:hidden");

	// get new layer and show it
	ddmenuitem = id;
	$("#".ddmenuitem).attr("style","visibility:visible");
}
// close showed layer
function mclose()
{
	if(ddmenuitem != 0) 
		$("#".ddmenuitem).attr("style","visibility:hidden");
}

quindi all'evento "onmouseover" dovrebbe farmi vedere il sottomenu, ma non funziona...
qualcuno riesce ad aiutarmi?

[fcaldera]

la concatenazione in js vuole il '+' non il punto

$("#" + ddmenuitem)

(il codice comunque è migliorabile / ottimizzabile in diversi modi)

[crisz78]

grazie mille!
ovviamente era un errore da meno che principiante (in teoria questa avrei dovuto vederla... )

tutti i suggerimenti su come migliorare il codice sono ben accetti!

[RoBYCoNTe]

be potresti evitare di scrivere codice javascript direttamente nel codice è sfruttare classi o id per la semantica della pagina, successivamente tramite jQuery impostare il comportamento dei vari oggetti, per capirci:

<ul id="NewTopNav">
    <li id="cat44">[/list]

poi in jQuery potresti fare cosi (una delle mille soluzioni che questo fantastico framework mette a dispozione):

$('#NewTopNav li').hover(function() {
    // Fai quello che devi.
    $(this).show();
});

Ovviamente ne guadagni in pulizia del codice e se ci penso hai modo di organizzare davvero molto bene la logica lato client che deve essere applicata alle pagine

[fcaldera]

inanzitutto si potrebbe pensare ad una marcatura tale da non richiedere per nulla l'uso di javascript in modo da sfruttare la gestione dell'hover via css

altrimenti si potrebbe usare sempre javascript anche se ovviamente ciò comporta uno sforzo maggiore per trovare una soluzione unobtrusive (che succede se javascript non è attivo?) performante sia per la marcatura (tutti quegli eventi online e id sono evitabili) sia per il javascript (si potrebbe sfruttare l'event delegation e il metodo hover() di jquery per esempio)

[crisz78]

grazie mille!
comincerò da subito ad applicare i vostri consigli