Является ли замена всех const std::string & на std::string_view хорошим выбором?

std::string_view::data() возвращает указатель, который не обязательно заканчивается нулем.
std::string::data() возвращает указатель, который всегда завершается нулем (начиная с C++11).
std::stringКонструктор 9, который принимает char*, требует, чтобы он указывал на строку с нулевым завершением.

Поскольку вы все равно создаете временный std::string, делайте это правильно: std::string{v.front()} создаст строку правильной длины из std::string_view.

Или, если это единственное использование Split, не используйте std::string_view вообще, не будет большой выгоды, если вы будете использовать string_view только для того, чтобы построить из него string.

Кроме того, помимо вопроса, но, как заметил Ботье в комментарии , вы используете ссылки неправильно. Оба объекта, на которые они ссылаются, умирают в конце соответствующей строки ;, и у вас остаются висящие ссылки.

Основная проблема здесь заключается в том, что вы создаете string из string_view с единственной целью использовать его с map.find, и в этом случае вы, вероятно, захотите вызвать конструктор string, который принимает string_view вместо передачи char*, чтобы он не передавал strlen. нужно вызвать std::string{the_string_view}. (сделайте std::unordered_map явно), но это не настоящее решение проблемы.

Обычно find имеет find, а компаратор и хеш не допускают других типов, поэтому string_view не работает для Key, если std::string есть std::string, вам придется определить свой собственный хэшер и компаратор, и вам понадобится C++20 для перегрузки, которая принимает любые введите, аналогичный ключу для найти

#include <unordered_map>
#include <string>
#include <string_view>
#include <iostream>

struct Equality
{
    using is_transparent = std::true_type;

    bool operator()(const auto& lhs, const auto& rhs) const { 
        auto result = lhs == rhs;
        return result;
    }
};
struct Hash
{
    using is_transparent = std::true_type;

    template <typename T>
    size_t operator()(const T& obj) const { 
        auto result = std::hash<std::decay_t<T>>{}(obj);
        return result;
    }
};

int main()
{
    std::unordered_map<std::string, int, Hash, Equality> m{ {"hel",1} };
    std::string s = "hello";
    std::string_view sv{ s };
    sv = sv.substr(0, 3);
    auto it = m.find(sv);
    if (it != m.end())
    {
        std::cout << "found!";
    }
}

Обратите внимание, что для запуска этого кода вам понадобится компилятор C++20, в противном случае вам придется использовать версию контейнера Boost, которая уже имеет эту перегрузку для более низких версий C++, или любую другую реализацию, имеющую эту перегрузку.

Последний вариант, если вы застряли на C++17 и не можете перейти на C++20 или использовать boost или любую другую библиотеку, — это создать собственный контейнер, который хранит std::string_view в наборе и соответствующий string_view в ключах карты, поэтому что вы можете использовать find в качестве аргумента string... но это расточительно для памяти (дополнительные 16 байт на ключ)