fix segfaults caused by returning invalid references

[ottojo]

fixes https://github.com/SoPra-Team-17/Server/issues/81

getNearFieldsInDist returnt einen vektor by value. Der wird in getRandomNearField einer lokalen variable zugewiesen. Mit getRandomItemFromContainer wird ein zufälliger iterator in den vektor erzeugt. Dann "dereferenziert" getRandomNearField diesen iterator und returnt diese referenz, die in diesem moment ungültig wird da der vektor out of scope geht.

einfacher fix: In den utility funktionen einfach die Points direkt zurückgeben und nicht per referenz. (rechtfertigung: point ist klein, cache ist groß, compiler macht return value optimization, c++17 fordert guaranteed copy elision,...)

[sonarcloud[bot]]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

0.0% Coverage
0.0% Duplication