Thats a nice map - welcome to the guild.

The key to doing cities that make sense is that they grow. You got to think what would somebody want at the point in the town / city and then build that. Then keep going adding to it constantly. The people will exploit the resources from the city. It will make more sense since each item works with all the buildings and structures around it. For example the farms will not cluster because it makes more sense to have somebody strike some new land and expand their farm or take a whole new plot that was unused. People build onto existing buildings and along existing roads until the space is exhausted. Where there is a lot of footfall shops and taverns will spring up and it goes on like that. So its just a minor mental shift and then it will all just flood out.

I also think its a wise idea to get in the heights of the land before starting any buildings and roads. Its not too clear from your map whether that headland to the right is flat or hilly. If it were flat then I am sure fishermen would be all along there with boats. That kind of thing.

The art side of your map is fine.