多线程数据竞争和代码规范

目录

问题

解决方法

代码重构

---

Rester是基于eventloop和epoll的并发服务器，实现过程中发现了数据竞争的问题，对其进行分析并规范了原始代码。

问题

1. 经常性的发生 段错误，segment fault，段错误通常是由内存访问越界引起的；
2. 在本机用 ab测压工具测试时，qps仅达到一两千的水平，一直以为是虚拟机性能的问题；

解决方法

1. 重新梳 理工作线程事件循环中管理连接的容器，发现了多线程对同一内存进行读写，原始用一个互斥量进行管理存储连接的一系列信息，并且某些位置未进行lock，有些位置过度lock。后对容器分别设置互斥量。段错误再也没有出现。
2. 性能过低的问题同 样由第一个问题引起，互斥量保护区过大，各线程竞争更加激烈。最后qps达到了九千多，并发数一万的情况下延迟在一秒左右。

​

​

代码重构

1. 基于google代码规范对 代码进行重构，修改命名，添加注释，组织代码；
2. 管理好类， 对不可复制的类进行uncopyable的处理。

    Config config;
    config.Update();
    cout<<config;
    Rester server(config);

    //close_log > 0 for close log
    if(!Log::get_instance()->init("log",0,8192,5000000,50000))
    {
        return -1;
    }

    Router file_worker("/");//default router
    auto on_get= [](RequestPtr request_ptr,ResponsePtr response_ptr)
    {
        response_ptr->SetStatusCode(200);
        response_ptr->SetHeader("Content-Type", "text/html"); //application/x-zip-        compressed
        response_ptr->SetHeader("Connection", "keep-alive");
        response_ptr->PrintResponse();
        char *buf2 = nullptr;
        int length = ReadFileAll("", buf2);
        response_ptr->SetData(buf2, length);
        response_ptr->CombineResponse();        
    };
    file_worker.SetGet(on_get);
    
    server.AddWorker(file_worker);    
    server.Init();
    return 0;