High elf citadel

Hi folks. I'm new here at wc3c. I came over this place when I was searching for terraining tutorials and ideas, and this looks like a very nice place to get some tips etc. But anyway, here's my first terrain. Tell me what you think and I'd greatly appreciate suggestions to improve it.

That's pretty :D

I'm no expert at terraining and all, but the light seems to come from two directions (and not cover certain areas, like near the left), which makes it look really weird.

First of all the rocks are really ugly, second the light is messy, third the tree combination you used looks bad.
Suggestions:

1.For the rocks use some different ones, like barrens.
2.The light should go in 1 direction not how you made it.
3. There shouldn't be green trees in my opinion and they rather look spammed.

For the Castle, it isn't anything special could use more work....those towers look more spammed then placed.
Oh, almost forgot, the balcony looks large from the size of the castle, you might want to make it smaller.

Maybe try some different angles? I know I'm a huge hypocrite by saying that, but it's still worth a shot.