schreibe eine CSS-Datei und füge das ein:
td:hover
{
background-color: #FF0000;
}funktioniert super in allen Browsern (IE ist keiner)
das was du willst kann man imho für den IE nicht mit weniger Code-Aufwand als den den du jetzt schon hast durchführen (ein paar Zeichen könnten vielleicht schon eingespart werden)
leider kann das der scheiss ie nicht und das benutzen die meisten, die meine hp besuchen werden. 
Da der IE td:hover nicht kann....
Workaround:
<style type="text/css">
a {
text-decoration:none;
color: black
}
a:hover {
background-color: black;
color: white;
}
</style>und dann in jede <td> einen leeren link einfügen
<td><a href="">was auch immer hier steht</a></td>Ist nicht der Weisheit letzter Schluss aber.....
Die Eventhandler onmouseover und onmouseout kann man nicht auslagern. Sie müssen da stehen, wo der Event auftritt.
Davon abgesehen haben die Zeichenfolgen <!-- und //--> nichts in einer .js Datei zu suchen.