Ok... I started my career as a web developer many years ago, and have sinced moved on to programming, BUT from my experience here is what I think...
(I work mainly on e-commerce sites, so my viewpoint reflects that.)
1) The flash HAS to go. It is gimmicky and will prevent many people from using the site at all.
2) The navigation is useless. This is what I call "meatloaf surprise" navigation - you don't know what you are going to get until you mouseover/click on it. The navigation should be IMMEDIATELY visible and understandable - it shouldn't take user-interaction to discern what/where/how your nav works.
3) The animated contact us is too distracting, and frankly it's annoying.
4) The colours are bland. From experience, I would suggest primary colours (used minimally as highlights) and black text on a white background. You have this, but the backgrounding makes the text hard to read. Alternatively, for a more "modern" feel, choose a colour and use variations of it to achieve a "monotone" theme. ie choose a beige colour for example and then use several lighter and darker browns/beiges with simple black text for highlights.
5) Flash makes updates harder and time consuming. This should be avoided for information that is subject to change ie prices, contact info. If flash HAS to be used, consider having your developer code the flash to load this information at runtime from an external text file. This will allow you to immediately change something without having to build a new flash file.
Just my 2 cents!