Requesting feedback on my InDesign project

thanks alot everyone:) i ended up changing Almost all of it.

i will post it in a bit

Three things I'd suggest:

1) You're right -- lose the display font. Part of advertising design is to develop a story which appeals to your audience. I personally feel your display font choice doesn't work here. I'd find something a bit more classic in the way of a serifed font. And the yellow sans-serifed tag really doesn't work against that background. Also think about changing the copy font at bottom-left. Or at least the ALL CAPS tag to lead in with that font. Think about who your target audience is, and then make font choices which you think would best make your appeal to that audience.

2) The flopped photo is obviously a flopped photo, because the Ray-Ban logo on the sunglasses is wrong-reading. That's fixable, though by copying that information -- right reading, from the original shot -- and rotating/placing it carefully above the wrong-reading type.

3) This is a little harder, in that part of your purpose is to appeal to your class instructor as much as it is to your target audience. A good ad should tell a good story, and your photo choice doesn't need any dramatic enhancement with shadows/filters etc. to pull that off. But if you're instructor is really into that kind of stuff, you may have to balance that against appealing to your target audience to boost your grade.

Now that I've finished nitpicking, let me compliment you on your sense of balancing -- symmetrical and non-symmetrical -- in your ad. The photo choice is a lot of fun. And while I might soften the intensity of the can reflection off your bottom shape, it's a nice piece of stylistic flair. Good luck on your project!