Your layout is actually quite good. It looks planned, but in a medieval way rather than the post-industrial cities most of us are familiar with. Three things to point out though: first, you only seem to have two gates (I'm assuming the road in the northeast leads out of the city). Ordinarily, you'd expect many more roads, especially along the edges of the river.

Second, which direction does the river flow? If it flows E-W no problems, but if it is W-E you've got a few difficulties: docks would normally be located downstream, closer to the sea. Even more of an issue, in your city it looks like you're going to be putting bridges between the docks and the coast, which would make it very difficult for ships. None of that matters, of course, if they are only trading upriver, but I would presume a large city in this position would have an extensive maritime trade.

And third, are you going to add suburbs? Historically, all cities would have had some suburbs. They'll follow the lines of roads and rivers. Farmers bringing their goods to market will generally prefer to sell their produce without paying the fee to enter the town, and so small-scale markets will form at gates.

Also you might want to take a look at maps of London, because they looked into planning the town after the Great Fire of 1666, and it shares some similarities to what you have described for your city.

Anyway, looking good so far.