Closed JaroslawWiosna closed 6 years ago
By the way, brute force for choosing two regions is not possible when parsing cities_data
.
There are just too many cities.
...so, the solution would be to limit brute force to...., I don't know, first* 20 cities?
I mean, in line 72:
for (; it != vec.end() ; ++it ) {
should be
for (; it != vec.begin()+20 ; ++it ) {
another loops could iterate over the whole vector just like it is done in this patchset
first 20 cities - what I mean is that the vector should be sorted. The city with the greatest population should come first.
I will prepare a new patch set soon.
I think this is ready to be merged.
I mean that calculateHappinessLevelForTwoRegionsByBruteForce
represents proof of concept here.
...but still Cities separation to two regions needs to be done in more elegant way.
It is more like an idea of
How to choose two regions and their capitals
than actual solution. I am open to write down a clarification for any part of the code. ...well I have to do this anyway.