Question

阅读Effective Modern C ++后，我试图找出如何在C ++类中为setter传递参数的最佳方法;想象下面的场景：

您正在编写解析输入文件的应用程序（例如以CVS格式），将其提供给模型类，然后将实例列表传递给数据库驱动程序以进行批量插入。

我们可以谈论最新和最好的C ++ 14;我写了一个用一栏代表这个的尝试：

class Person {
    std::wstring _name;
public:
    template<typename T>
    inline void setName(T&& name) { _name = std::forward<T>(name); }
};

我还编写了以下代码片段来测试性能：

const auto start = std::chrono::system_clock::now();
std::vector<std::shared_ptr<Person>> vec;
for (auto i = 0; i < 999999; i++)
{
    auto p = std::make_shared<Person>();
    p->setName(L"John Chester Doe");

    vec.emplace_back(p);
}   

const auto end = std::chrono::system_clock::now();
std::cout << "Took " << std::chrono::duration_cast<std::chrono::milliseconds>(end - start).count() << " ms" << std::endl;

在我的系统上，使用带有/ O2的MSVC2017 Release x64编译约175ms，关闭SDL，无安全检查（/ sdl- / GS-）。出于好奇，我在C＃中尝试了同样的事情：

class Person
{
    public string Name { get; set; }
} 


var sw = System.Diagnostics.Stopwatch.StartNew();
var i = 0;
var vec = new System.Collections.Generic.List<Person>();
for (; i < 999999; i++)
{
    var p = new Person();
    p.Name = "John Chester Doe";
    vec.Add(p);
}
sw.Stop();
System.Console.WriteLine("Took {0}ms", sw.ElapsedMilliseconds);

这在发布模式下产生77毫秒！所以我的问题很自然就是我做错了什么？为什么我的C ++代码在基本相同的操作上比C＃代码慢？

我尝试在C ++中使用struct（没有区别），通过const ref（较慢）传递值，使用std :: move（较慢）传递值。我也试过使用noexcept（没有大的区别）甚至std :: string（没有太大的区别）。

谢谢！

修改

感谢大家的建议，真的很感激。看起来我希望实现的最佳方式是这样的：

std::vector<Person> vec;
vec.reserve(999999);
for (auto i = 0; i < 999999; i++)
{
    vec.emplace_back();
    vec.back().setName(L"John Chester Doe");
}

这是75分钟的预分配，没有vec.reserve它是130毫秒。

Answer 1

＆＃34;将橙子与苹果进行比较＆＃34;是我能在这里给出的最佳描述。您的C＃代码最接近C ++，如下所示：

#include <iostream>
#include <deque>
#include <chrono>
#include <string>

struct Person {
    std::wstring _name;

    Person() = default;
    Person(Person&& p) = default;
    Person(const Person& p) = default;
    Person(const std::wstring& n) : _name(n) {}
};

int main()
{
    std::deque<Person> vec; 
    const std::wstring n =  L"John Chester Doe";
    const auto start = std::chrono::system_clock::now();
    for (auto i = 0; i < 999999; i++)
    {
        vec.emplace_back(n);
    }   

    const auto end = std::chrono::system_clock::now();
    std::cout << "Took " << std::chrono::duration_cast<std::chrono::milliseconds>(end - start).count() << " ms" << std::endl;
}

至少比您的变体快3倍。也许我可以让它更快，将所有内存的分配转移到deque的初始化，但这会将循环更改为远程循环。是的，C＃和java一样。它试图将大部分可预测的工作卸载到编译时间，例如收集类似的文字，投射可用的内存并更有效地分配它等等。主要是因为CLR的工作原理类似于java机器。

Answer 2

我的理解是C＃编译器在后台执行的操作大致转换为以下C ++代码：

class Person {
    std::wstring _name;
public:
    template<typename T>
    inline void setName(T&& name) { _name = std::forward<T>(name); }
};

对于插入部分：

std::vector<Person> vec;
vec.reserve(999999);
for (auto i = 0; i < vec.size(); i++)
{
    vec.emplace_back();
    vec.back().setName(L"John Chester Doe");
}

再次感谢您的时间，如果有任何其他建议如何改进，我会很高兴听到它。

编写高性能的C ++ setters

2 个答案: