这几个构造函数应该选哪一个啊？

[youg]

我有一个类
Class A
{
public:
A();
~a();
private:
string m_s1;
string m_s2;
}

我写构造函数

1,
A(string s1,string s2):m_s1(s1),m_s2(s2){};
2,
A(const string &s1,const string &s2):m_s1(s1),m_s2(s2){};
3,
A::A(string s1,string s2)
{
m_s1 = s1;
m_s2 = s2;
}
4,
A(const string &s1,const string &s2):m_s1(s1),m_s2(s2){};
{
m_s1 = s1;
m_s2 = s2;
}
上面4种选哪一种好？

[solar]

the 2nd. the 1st is also ok. but not use 3rd or 4th

[cococut]

2

[hz129]

2

[tokens]

2

[echolover]

2,原因可以在《深入探索c++对象模型》中找到。

[career]

2

[davidlxm]

2

[veinstone]

2nd

[maofeng206]

第二种最好。只传引用，没有构造、析构成本。不用产生临时对象。第一种在传参数时会引起拷贝构造函数的调用。第三种的成本更高，传参数，赋值时都要调用构造函数。第四在赋值时都要调用构造函数，并却赋值是多余的

[tmp2002]

我倒！
四种方法中，只有4是对的！
所以，我说用4！！！

[tmp2002]

错了！
只有3是对的！
选3！

[Lemon-sai]

同意楼上的
只有3是对的

[d_selenite]

A::A(const string &s1,const string &s2):m_s1(s1),m_s2(s2){};
首先就效率来说，传递引用比传值好：没有构造，析构成本，再者class string有一个构造函数
string(const string& )，在initialize list中已经调用了string的构造函数，而又在函数体内又调用了string::operator=(const string&);函数，这一步是多余的，
所以就对错而言，四个都对，就效率而言，第二个效率较高