- DIV align=center
- テーブルレイアウト
- インラインスタイル使用
- img要素に画像のサイズと代替テキストの指定がない
- 見出しに<Hn>を使ってない
- ていうか<li>閉じてない
- リスティングすべき箇所を<br>でごまかし
- （div要素の中に直接テキスト挿入）
- （メニューのli要素のクラスのネーミングが手抜き）
- （タグ大文字小文字の混同）
総評：もっとがんばれ。
ソースを綺麗に仕立て直しました。（中略）そのことによって益があるの？
HTMLから見た目の要素を排除し論理要素でのみページの構造をマークアップする事により、PCモバイル問わず全ての環境でそのWebページの情報を得る事が出来るようになります。また機械がページを理解しやすくなり、自然とSEO化されることになります。さらに言えば、見た目に関する記述を全てCSSに移す事によってサイトのリニューアル作業が容易になります（HTML自体に手を加える必要が無くなるので）。