Some Pros: You're following a good bonus value system universally Your lakes allow a couple instances of territories not connecting because they do not touch (which gives a little variety) Your clearly thinking about effective bonus colors (I see the dotted lines)
Overall though the map looks sloppy. Bonus Objects are too big and too far away from their territories. It also looks like you drew each territory individually, and therefor they look spaced out and crudely fit together. One easy way to avoid this in the future is in how you create your territories. You can draw the entire landmass all at once and 'cut out' your territories. They'll stay snug together and the whole thing will look clean. There is a map making Discord I recommend you look at if you have any questions: https://discord.gg/34K6VMJcgK
By all means keep making maps. You'll get better over time, and can always come back to this map to improve it if desired.