Qualche gotchas sostituisce const globale [] con constexpr string_view?

Il nostro team sta lavorando con una base di codice C ++ di oltre 10 anni e recentemente è passato a un compilatore C ++ 17. Quindi stiamo cercando modi per modernizzare il nostro codice. In una conferenza su YouTube ho sentito il suggerimento di sostituire le const char*stringhe globali constexpr string_view.

Dal momento che const char*nel nostro codice abbiamo ottenuto un certo numero di costanti di stringa globali di questo tipo, voglio chiedermi se ci sono problemi o potenziali problemi di cui dobbiamo essere a conoscenza?

Potrebbe essere utile conoscere questi problemi:

1. std::string_viewnon ha bisogno di essere nullterminato. Quindi, se ne sostituisci alcuni const char*con string_viewe sostituisci la costruzione di una sottostringa precedentemente nullterminata char*con una string_viewvia std::string_view::substr, non puoi passare il puntatore sottostante a un'API che prevede una nullstringa terminata. Esempio (senza UB, ma anche questo è facilmente costruibile):

   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. Mentre puoi implicitamente costruire a std::stringda a const char*, non puoi farlo con a std::string_view. L'idea è che una copia profonda non dovrebbe avvenire sotto la copertina, ma solo quando esplicitamente richiesto. Esempio:

   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

   A seconda dell'utilizzo esistente delle const char*istanze globali nel progetto, questo comportamento potrebbe richiedere un intervento manuale in vari punti.