在C++中连接字符串

Question

我是一个缺乏经验的C++程序员，所以这个问题可能很基本。我想为我的系词获取文件名：

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); 
} 

这是在使用：

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()); 
} 

然而，创建的文件有垃圾的名字，从怪异的人物为主。任何想法我做错了什么，以及如何解决它？

Answer 1

sprintf的有4个地方而您仅提供3个参数。

我建议：

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

你sprintf不是缓冲区溢出的安全，使用，而C99 snprintf或std::stringstream

Answer 2

您的格式字符串中有四个说明符，但只提供三个附加参数。

Answer 3

，看起来完全破碎的唯一的事情是，你只有通过3个数据参数传送到您的sprintf，但它预计4（％C，％S，％S，S）

Answer 4

因为所有的东西你都黏合在一起，是基于字符的，使用的sprintf为它有点傻。

什么是C++程序员应该做的还有线沿线的东西更多：

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

Answer 5

因为你似乎性病工作::字符串，你不需要用sprintf的。 std :: string有简单的重载操作符，所以你可以使用+ =来连接字符串。我认为+也可以，所以只需将“std :: strings”添加到一起即可。

Answer 6

假设toString()返回std::string，

此：

的sprintf（结果， “％C_％S（％S，％S）的.csv”， copula.toString（）。 c_str（）， left.toString（）。c_str（）， right.toString（）。c_str（））;

...应该是：

的sprintf（结果， “％S _％S（％S，S）的.csv”， copula.toString（）。 c_str（）， left.toString（）。c_str（）， right.toString（）。c_str（））;

Answer 7

Boost有一个formatting library比printf和朋友更安全。

#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()); 
} 

，或者可选地：

#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()); 
} 

对于后者例如，升压知道％1％在类型是字符，并且％2％直通％4％是字符串由 “寻找”通过％运算符传入的参数。