Yo no lo veo tan errado. Y el ejemplo me parece similar a los de desarrolloweb.com
El tema de hacer menues es dificil. Hay algunos aquí en el foro:
Lo que sí, yo encabecería todo con esto, para no estar repitiendo codigo:
Cita: * {margin: 0; padding: 0; border: 0; outline: none;}
body{ font-family: Georgia, "Times New Roman", Times, serif; }
a{ text-decoration: none; font-size: 11px; color: #FFFFFF; }
ul { text-transform: capitalize; text-align: center; }
List-style no es necesario porque al hacer display-inline ya se borran los marcadores.
Segundo, en el codigo que publicaste estas usando una class .menu que ya no existe (la habrás borrado): ul.menu
No tengo tiempo ahora para terminar de ajustarlo, pero te recomiendo uses uno de los que te pasó @masterojitos.