Alguma dica substituindo o const global char [] por constexpr string_view?

Nossa equipe está trabalhando com uma base de código C ++ com mais de 10 anos e recentemente mudou para um compilador C ++ 17. Então, estamos procurando maneiras de modernizar nosso código. Em uma conferência no YouTube, ouvi a sugestão de substituir const char*as cordas globais por constexpr string_view.

Como temos um número considerável dessas const char*constantes globais de string em nosso código, quero perguntar se há alguma dica ou problema em potencial que precisamos estar cientes?

Vale a pena conhecer esses problemas:

1. std::string_viewnão precisa ser nullterminado. Portanto, se você substituir alguns const char*por string_viewe substituir a construção de uma substring anteriormente nullterminada char*por uma string_viewvia std::string_view::substr, não poderá passar o ponteiro subjacente para uma API que espera uma nullstring terminada. Exemplo (sem UB, mas que também é facilmente construtível):

   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. Enquanto você pode implicitamente construir a std::stringpartir de a const char*, você não pode fazer isso com a std::string_view. A idéia é que uma cópia detalhada não ocorra de forma oculta, mas apenas quando solicitada explicitamente. Exemplo:

   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

   Dependendo do uso existente das const char*instâncias globais em seu projeto, esse comportamento pode exigir intervenção manual em vários locais.