首页
学习
活动
专区
工具
TVP
发布
社区首页 >问答首页 >编写性能良好的C++设置器

编写性能良好的C++设置器
EN

Stack Overflow用户
提问于 2018-05-30 05:19:19
回答 2查看 125关注 0票数 -5

在阅读了高效的现代C++之后,我正在尝试找出如何在C++类中传递setter参数的最佳方法;想象一下下面的场景:

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

我们可以只讨论最新和最伟大的C++14;我用一个专栏来表示我的尝试:

代码语言:javascript
复制
class Person {
    std::wstring _name;
public:
    template<typename T>
    inline void setName(T&& name) { _name = std::forward<T>(name); }
};

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

代码语言:javascript
复制
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;

在我的系统上,这需要大约175ms的时间,使用带有/O2的MSVC2017 Release x64编译,关闭SDL,没有安全检查(/sdl- /GS-)。出于好奇,我在C#中尝试了同样的方法:

代码语言:javascript
复制
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);

在释放模式下,这会产生77ms!所以我的问题自然是我做错了什么?为什么在本质上相同的操作上,我的C++代码比C#代码慢?

我尝试在C++中使用结构(没有区别),通过const ref传递值(较慢),使用std::move传递值(较慢)。我也尝试过使用noexcept (没有大的区别),甚至std::string (没有大的区别)。

谢谢!

编辑

感谢大家的建议,真的很感谢。对于我想要实现的目标,最好的方法看起来是这样的:

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

这是75ms的预分配,没有vec.reserve的是130ms。

EN

回答 2

Stack Overflow用户

回答已采纳

发布于 2018-05-30 06:56:36

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

代码语言:javascript
复制
class Person {
    std::wstring _name;
public:
    template<typename T>
    inline void setName(T&& name) { _name = std::forward<T>(name); }
};

并且对于插入部分:

代码语言:javascript
复制
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");
}

再次感谢您的时间,如果有任何其他建议如何改善这一点,我将很高兴听到。

票数 0
EN

Stack Overflow用户

发布于 2018-05-30 06:44:38

“把桔子比作苹果”是我能在这里给出的最好的描述。您的C#代码获得了最接近的C++等效项,如下所示:

代码语言:javascript
复制
#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机器。

票数 0
EN
页面原文内容由Stack Overflow提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接:

https://stackoverflow.com/questions/50593102

复制
相关文章

相似问题

领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档