change usage of ClosestCity

[kermit10000000]

Descrpition

Make better use out of it than hard-coded Poddebice (that are just to prove it works)

---

Release affected

Yes it is part of 0.4.0 release but it is blocked( @JaroslawWiosna could you please comment when it will be possible ? )

---

Priority

Medium Prio - part of 0.4.0 release, no impact on other contributors work

---

Is there possible contributor who could fix the issue

Wojciech Mielczarek

---

[JaroslawWiosna]

@kermit10000000 It will be possible when #87 is closed

[JaroslawWiosna]

87 closed, so this task is not blocked anymore.

[kermit10000000]

based on the console output do you have any idea where should we put the call of this function?

[JaroslawWiosna]

hmm..., just put it below the name of the new elected capital Like:

The best capital: Poddebice
which is near Zgierz->Lodz->Warszawa

Please, take care of this task ASAP

[kermit10000000]

is this acceptable:
std::cout << "Which is near: " << ClosestCity::getAllBiggestCitiesInOrder(DatabaseReader::readCitiesFromFile("../database/cities_data"),subregion.getCapital()) << std::endl;

Or should it be hidden inside of some method or something?

[kermit10000000]

Current look:

[kermit10000000]

Second look:

• faster since we avoid getting the same whole vector x times

[kermit10000000]

I am also not sure about "which is near" . It looks like this: lodz: which is near Lodz... cos tam(w sensie zdoublowane dziwnie wyglada):

[kermit10000000]

Task will be moved to done but it might be improved when the change from fx will be done

[JaroslawWiosna]

Starting from scratch with 7e13971