Discussion :

[LittleWhite]

Bonjour à tous,

Je me retrouve une nouvelle fois face à un code contenant des fonctions retournant des chaînes de caractères. Malheureusement (pour moi et surtout pour les performances) celles-ci sont retournées par copie. Un des soucis qui m'empêche de retourner une référence est le fait qu'il existe des cas où ces fonctions peuvent retourner des chaînes vides : return "";.

Quelles solutions avez-vous pour éviter des copies ?

Mes solutions :

• passer la chaîne de caractère par référence en argument à la fonction. Point noir : modification lourde du code existant ;
• utiliser une chaîne vide statique, qui ne sera pas détruite, même lorsque retournée par la fonction.

Exemple de code problématique :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
std::string myFct()
{
    if (test)
        return member;
 
    return "";
}

[jo_link_noir]

Quel est le problème avec la chaîne vide ? Elle est statique (et devrait être constante) alors la non destruction n'a aucune importance puisqu'elle doit survivre jusqu'à la fin du programme.
Bien sûr, les retours devront être des références constantes, à la charge de l'appelant de réduire les copies en utilisant aussi des références.

PS: le constructeur par défaut s'utilise avec {} et non pas "".

[Bousk]

Dans ce genre de cas j'opte pour la static et le retour par const& s'il n'est pas possible de retourner directement par const char*Les cas les plus tordus, ou s'y prétant, peuvent se contourner en retournant un bool et la string en paramètre out

[Luc Hermitte]

Pareil que Bousk.
C'est ce que j'ai fait il n'y a pas longtemps.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
std::string const& C::f() const {
    static const std::string k_empty;
    if (foo) return m_bar;
    return k_empty;
}

Et si tu veux reconnaitre une chaine vide précise (car elle pourrait vouloir dire "pas trouvé" et pas juste "trouvé mais vide"), il faut la sortir de sa portée locale.

NB: je n'ai pas étudié la pertinence de retourner une string_view dans ces cas là.
Hum... Cela m'a l'air d'être un sacré progrès à vue de nez.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
std::experimental::string_view C::f() const {
    if (foo) return m_bar;
    return std::experimental::string_view{};
}

[LittleWhite]

Ok, merci pour m'avoir éclairci ce point.