Improve join method.
- pop_back() was removing only one character even though the separator might be longer than one character
- Loop using const reference, there's no need to pass a copy