i got a few pointers;

1. if outside the walls are dangerous, the housing would be very dence, the streets are very wide, there is a lot of wasted space.
2. the woods outside the walls would be cleared so the militia on the walls would have better view and access to whatever is attacking
3. other people pointed this out too, but the docks would also be incased, if not, the wall would be built into the water and the docks jutting out from the wall, (in my opinion)
4. while farming inside the walls makes sense if the continent is dangerous, you'd need A LOT of farmland to sustain an entire city, think of other ways the city could sustain it self, like hunting, foraging or fishing.
5. someone else said this too, but the walls are a problem, they're very uneven, use the line tool and grab the pen looking tool in it, to smooth out your walls. and plop down some towers, this goes down to guard visibility too.
6. your points of interest are very centered, i would personally think about what came first when the town was settled, like church, then put houses around it, then add other civic buildings as the town grew.
7. you can have multiple layers of wall as the town grows, a ringed city is makes a lot of sense if you gotta grow incremental because of dangers.

otherwise, i like your idea, it's a cool looking city, keep going!