Concaténation de chaînes en C++

Question

Je suis un programmeur C++ plutôt inexpérimenté, donc cette question est probablement plutôt basique. Je suis en train d'obtenir le nom de fichier pour mon copule:

string MonteCarloBasketDistribution::fileName(char c) 
{ 
    char result[100]; 
    sprintf(result, "%c_%s(%s, %s).csv", copula.toString().c_str(), left.toString().c_str(), right.toString().c_str()); 
    return string(result); 
} 

qui est utilisé dans:

MonteCarloBasketDistribution::MonteCarloBasketDistribution(Copula &c, Distribution &l, Distribution &r): copula(c), left(l), right(r) 
{ 
    //..... 
    ofstream funit; 
    funit.open (fileName('u').c_str()); 

    ofstream freal; 
    freal.open (fileName('r').c_str()); 
} 

Cependant, les fichiers créés ont des noms, des ordures composées principalement de personnages étranges. Une idée de ce que je fais mal et comment le réparer?

Answer 1

sprintf a 4 place détenteurs alors que vous donnez seulement 3 paramètres.

Je suggère:

string MonteCarloBasketDistribution::fileName(char c) { 
    std::ostringstream result; 
    result << c <<"_"<<copula<<'('<<left<<", "<<right<<").csv"; 
    return result.str(); 
} 

Votre sprintf n'est pas sûr pour dépassement de mémoire tampon, utilisez plutôt C99 snprintf ou std::stringstream

Answer 2

Vous avez quatre spécificateurs dans votre chaîne de format, mais vous ne fournissez que trois arguments supplémentaires.

Answer 3

La seule chose qui semble tout à fait cassé est que vous ne passer 3 paramètres de données à votre sprintf, mais il attend 4 (% c,% s,% s,% s)

Answer 4

Puisque toutes les choses que vous punaiser ensemble sont basées sur des caractères, en utilisant sprintf pour c'est un peu bête.

Quel programmeur C++ devrait faire il y a quelque chose de plus le long des lignes de:

std::string result = copula.toString() + "(" + left.toString() + "," 
        + right.toString() + ")"; 

Answer 5

Puisque vous semblent travailler sur std :: string, vous ne devez pas utiliser sprintf du tout. std :: string a simple à utiliser des opérateurs surchargés, donc vous pouvez concaténer les chaînes en utilisant + =. Je pense que + fonctionne aussi, donc juste "ajouter" des chaînes std :: ensemble.

Answer 6

En supposant que toString() retourne un std::string,

Ce:

sprintf (résultat, "% s c_% (% s,% s) .csv", copula.toString(). c_str(), left.toString(). c_str(), right.toString(). c_str());

... devrait être:

sprintf (résultat, "% s _% s (% s,% s) .csv", copula.toString(). c_str(), left.toString(). c_str(), right.toString(). c_str());

Answer 7

Boost a un formatting library qui est plus sûr que printf et ses amis.

#include <boost/format.hpp> 
string MonteCarloBasketDistribution::fileName(char c) 
{ 
    return boost::str(boost::format("%c_%s(%s, %s).csv") 
     % c % copula.toString() % left.toString() % right.toString()); 
} 

ou bien:

#include <boost/format.hpp> 
string MonteCarloBasketDistribution::fileName(char c) 
{ 
    return boost::str(boost::format("%1%_%2%(%3%, %4%).csv") 
     % c % copula.toString() % left.toString() % right.toString()); 
} 

Avec ce dernier exemple, Boost sait que% 1% est un caractère et% 2% thru% 4% sont des chaînes par "regarder" les types des arguments transmis via l'opérateur%.