Va bene mettere "using std :: swap;" in un colpo di testa?

Question

Ho letto che quando sei swap cose Ing in C++, si dovrebbe sempre using std::swap;, quindi chiamare swap qualificato, in modo che seleziona automaticamente la std:: quelli per std:: e incorporate tipi, la vostra abitudine uno per i tipi personalizzati, e su modelli std:: uno per tutto il resto.

Quindi, posso semplicemente inserire using std::swap; nell'intestazione che include ogni file e non doversi preoccupare di ciò?

Comprendo che evitare using in un'intestazione è una pratica comune. Tuttavia, c'è un problema in questo caso particolare?

Answer 1

La guida per lo scambio è using std::swapnella dimensione più locale possibile. Per certo, uno in un file di intestazione che è ampiamente incluso non soddisfa questo requisito. È ancora inquina lo spazio dei nomi globale in modi imprevisti (qualcuno che non si aspetta std::swap verrà importato nello spazio dei nomi globale) e dovrebbe essere evitato come using namespace.

Answer 2

Il problema principale qui è che si presume che le persone non abbiano scritto i propri swap s che si tratti di corrispondenze migliori che fanno cose sottilmente o completamente diverse alla semantica di std::swap e amici correlati. Per un semplice esempio, prendere in considerazione

void swap(int* a, int* b); 

in cui vengono scambiati i contenuti dei puntatori. Ora prova a scambiare un paio di int*. Potresti pensare che i puntatori siano scambiati, ma invece, sorpresa! sono i contenuti che vengono scambiati.

In realtà, la guida per swap è la stessa di qualsiasi altra: solo using con l'ambito più locale possibile e non prima e sicuramente non in un'intestazione.