Любые ошибки заменить глобальный const char [] на constexpr string_view?

Наша команда работает с базой кода C ++ более 10 лет и недавно перешла на компилятор C ++ 17. Поэтому мы ищем способы модернизировать наш код. В ходе конференции на YouTube я услышал предложение заменить const char*глобальные строки на constexpr string_view.

Поскольку const char*в нашем коде есть довольно много таких глобальных строковых констант, я хочу спросить, есть ли какие-то ошибки или потенциальные проблемы, о которых нам нужно знать?

Эти вопросы, возможно, стоит знать:

1. std::string_viewне должен быть определен null. Так что, если заменить некоторые const char*на string_viewи заменить конструкцию ранее null-завершённых char*подстрок по с string_viewпомощью std::string_view::substr, вы не можете передать указатель , лежащий на API, ожидающий null-завершённую строку. Пример (без UB, но это тоже легко построить):

   void legacy(const char *str) {
      std::printf("%s\n", str);
   }
   
   constexpr std::string_view sv1 = "abcde";
   constexpr std::string_view sv2 = sv1.substr(0, 2); // view on "ab"
   
   legacy(sv2.data()); // Not intended: prints "abcde" 
   

2. Хотя вы можете неявно создать a std::stringиз a const char*, вы не можете сделать это с помощью a std::string_view. Идея заключается в том, что глубокая копия не должна происходить под прикрытием, а только при явном запросе. Пример:

   std::map<std::string, int> m;
   constexpr std::string_view sv = "somekey";
   constexpr const char *old = "somekey";
   
   m[old] = 42; // works as expected
   m[sv] = 42; // fails to compile
   m[std::string(sv)] = 42; // be explicit, this is ok
   

   В зависимости от существующего использования глобальных const char*экземпляров в вашем проекте, такое поведение может потребовать ручного вмешательства в различных местах.