Dove inserisco stringhe costanti in C++: membri di classi statiche o namespace anonimi?

Question

Devo definire alcune stringhe costanti che verranno utilizzate solo da una classe. Mi sembra di avere tre opzioni:

1. Incorporare le stringhe direttamente nei luoghi in cui vengono utilizzate.

2. li definiscono come membri privati ​​statica costante della classe:

   //A.h 
   class A { 
   private: 
       static const std::string f1; 
       static const std::string f2; 
       static const std::string f3; 
   }; 
   
   //A.cpp 
   const std::string f1 = "filename1"; 
   const std::string f2 = "filename2"; 
   const std::string f3 = "filename3"; 
   
   //strings are used in this file 
   

3. definirli in uno spazio dei nomi anonimo nel file cpp:

   //A.cpp 
   namespace { 
       const std::string f1 = "filename1"; 
       const std::string f2 = "filename2"; 
       const std::string f3 = "filename3"; 
   } 
   
   //strings are used in this file 
   

Alla luce di queste opzioni, che uno consiglieresti e perché? Grazie.

Answer 1

Li inserirò nello spazio dei nomi anonimo nel file CPP. Li rende privati ​​dell'implementazione e allo stesso tempo li rende visibili alle funzioni non membri che fanno parte dell'implementazione (come operator<<).

Answer 2

Se le stringhe devono essere viste dagli utenti della classe, inserirle nella classe. Altrimenti, nascondili nello spazio dei nomi senza nome del file di implementazione.

Answer 3

Se vengono utilizzati solo in un singolo file, non è necessario esporli al mondo esterno includendoli nel file di intestazione.

Se vengono utilizzati e verranno sempre utilizzati solo in un singolo luogo, non c'è davvero alcun motivo per non limitarsi a scriverli come valori letterali in cui devono essere utilizzati.

Se sono utilizzati in più posizioni nel cpp, vorrei andare per lo spazio dei nomi anonimo.

Un'altra opzione che non si menziona è definirli come variabili statiche all'interno di cpp. questo è in qualche modo equivalente all'opzione namespace anonimo e più simile a C che a C++.

Answer 4

Membri statici della classe.

Se sono usati in più posti da una classe, di solito è più facile mantenere le cose organizzate - e in seguito trovare dove hai definito tutto - se le tenga definite nella classe che li usa. Definirli sul posto li rende difficili da localizzare e successivamente modificare. E opterei per la classe specifica sullo spazio dei nomi anonimo per una definizione e un uso della classe più puliti.

Answer 5

Delle tre opzioni, l'unica da evitare è la # 1. Non usare i cookie magici nel tuo codice. Inserendo le costanti in uno spazio dei nomi o in una classe, è più semplice estendere e mantenere il codice in futuro.

Se le vostre costanti sono di natura globale, quindi tra 2 e 3, importa poco. Ciò che conta è che tu ne scelga uno e lo segua. Ma se hai delle costanti che si applicano a una particolare classe, allora dovrebbero far parte di quella classe.

Personalmente, userei un namespace per la maggior parte delle cose.

Answer 6

Penso che il vero problema sia: le stringhe sono realmente utilizzate solo internamente durante l'implementazione della classe o vengono utilizzate altrove.

Per essere davvero pignoli, proverei a mantenere l'interfaccia della classe più pulita possibile, quindi se le stringhe del nome file non dovrebbero interessare il mondo "esterno". Li nasconderò internamente solo nel file .cpp. E in tal caso non penso che mi preoccuperei del namespace, ma semplicemente mantenere le cose "statiche" (cioè interne al file .cpp).

Answer 7

Se utilizzato solo nel file cpp della classe, non v'è alcuna necessità di utilizzare uno spazio dei nomi di qualsiasi tipo, dire semplicemente:

const std::string f1 = "filename1"; 
const std::string f2 = "filename2"; 
const std::string f3 = "filename3"; 

L'abuso di spazi dei nomi sembra essere la cosa nuova - non posso vedere personalmente l'attrazione.

Answer 8

Tuttavia, è necessario fare attenzione: non raccomanderei l'uso di oggetti statici std :: string, invece di usare char statico *. La ragione di ciò è dovuta a potenziali problemi con l'ordine di inizializzazione. Supponiamo di avere un'istanza statica di una classe il cui costruttore fa riferimento alla stringa A::f1. Non vi è alcuna garanzia che lo A::f1 sia stato ancora costruito e si verifichi un arresto anomalo, o peggio, nessun arresto anomalo ma dati falsi.

Tracciare i bug di ordine di inizializzazione può essere piuttosto sgradevole, e tutto può sembrare buono in un progetto, ma poi si potrebbe costruire un altro progetto usando le stesse librerie e le sottili differenze nell'ordine di collegamento causeranno questo errore apparire misteriosamente.

Answer 9

Basta avere le stringhe const sull'ambito del file nel file di implementazione, lo spazio dei nomi anonimo non è necessario per limitare il loro uso solo a quella classe.

C++ 003 Standard C.1.2 Clause 3: concetti di base

Change: A name of file scope that is explicitly declared const, and not explicitly declared extern, has internal linkage, while in C it would have external linkage 

Nota: Lo spazio dei nomi anonimo aiuta ridurre i conflitti di denominazione.