回调函数中写日志崩溃

小卡拉米 2021-01-13 07:21:19
调用一个dll , 里面会回调我的函数, 我看回调函数与主线程是同一个线程
问题:
1. 回调函数里写日志会崩溃
2. 是否应该用一个死循环防止main 退出??


MyLog logger = MyLog(); // 全局

void runstr(void* pSty) // 回调函数
{
Strategy* s = (Strategy*)pSty;
std::cout << "callback thread id: " << std::this_thread::get_id() << std::endl;
logger.info("Test"); // 会崩溃
}

int main() {
CGetValue cgvObj;
cgvObj.Subscribe(runstr); // 注册回调函数 runstr, 它会周期性地被调用, 比如 每秒一次
std::cout << "Main thread id: " << std::this_thread::get_id() << std::endl;
cgvObj.addcl(s, s->allMetaData[0], s->allMetaData.size());
cgvObj.Init(); // 开始执行

while(true) { // 这里是不是要用一个死循环防止退出?

}

}
...全文
252 11 打赏 收藏 转发到动态 举报
写回复
用AI写文章
11 条回复
切换为时间正序
请发表友善的回复…
发表回复
赵4老师 2021-01-14
  • 打赏
  • 举报
回复
引用 7 楼 管理員 的回复:
[quote=引用 4 楼 赵4老师 的回复:]把你的Mylog logger换成下面我的MyLog:

在回调函数第一行加一句 system("cls"); 也会导致崩溃, 这是什么原因?[/quote] 列夫·托尔斯泰:“幸福的家庭都一样,不幸的家庭各有各的不幸.” zhao4zhong1:“不崩溃的程序都一样,崩溃的程序各有各的崩溃原因.”
小卡拉米 2021-01-14
  • 打赏
  • 举报
回复
引用 4 楼 赵4老师 的回复:
把你的Mylog logger换成下面我的MyLog:

在回调函数第一行加一句 system("cls"); 也会导致崩溃, 这是什么原因?
小卡拉米 2021-01-14
  • 打赏
  • 举报
回复
引用 1 楼 ningto点com 的回复:
简单点可以while(true) { sleep(5); } sleep几秒钟防止退出,但是你这个回调函数在主线程里调用就不合适了,因为sleep会阻塞主线程,所以你应该单独起一个线程来执行上面的逻辑。
是写日志导致的; 在回调函数第一行加一句 system("cls"); 也会导致崩溃, 这是什么原因?
小卡拉米 2021-01-14
  • 打赏
  • 举报
回复
只要代码中有 logger.info() 这种写日志的语句 , 就会在 cgvObj.Init(); 后面 崩溃, 不加写日志, 就没事
赵4老师 2021-01-14
  • 打赏
  • 举报
回复
把你的Mylog logger换成下面我的MyLog:
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#ifdef _MSC_VER
    #pragma warning(disable:4996)
    #include <windows.h>
    #include <io.h>
#else
    #include <unistd.h>
    #include <sys/time.h>
    #include <pthread.h>
    #define  CRITICAL_SECTION   pthread_mutex_t
    #define  _vsnprintf         vsnprintf
#endif
//Log{
#define MAXLOGSIZE 20000000
#define MAXLINSIZE 16000
#include <time.h>
#include <sys/timeb.h>
#include <stdarg.h>
char logfilename1[]="MyLog1.log";
char logfilename2[]="MyLog2.log";
static char logstr[MAXLINSIZE+1];
char datestr[16];
char timestr[16];
char mss[4];
CRITICAL_SECTION cs_log;
FILE *flog;
#ifdef _MSC_VER
void Lock(CRITICAL_SECTION *l) {
    EnterCriticalSection(l);
}
void Unlock(CRITICAL_SECTION *l) {
    LeaveCriticalSection(l);
}
#else
void Lock(CRITICAL_SECTION *l) {
    pthread_mutex_lock(l);
}
void Unlock(CRITICAL_SECTION *l) {
    pthread_mutex_unlock(l);
}
#endif
void LogV(const char *pszFmt,va_list argp) {
    struct tm *now;
    struct timeb tb;

    if (NULL==pszFmt||0==pszFmt[0]) return;
    _vsnprintf(logstr,MAXLINSIZE,pszFmt,argp);
    ftime(&tb);
    now=localtime(&tb.time);
    sprintf(datestr,"%04d-%02d-%02d",now->tm_year+1900,now->tm_mon+1,now->tm_mday);
    sprintf(timestr,"%02d:%02d:%02d",now->tm_hour     ,now->tm_min  ,now->tm_sec );
    sprintf(mss,"%03d",tb.millitm);
    printf("%s %s.%s %s",datestr,timestr,mss,logstr);
    flog=fopen(logfilename1,"a");
    if (NULL!=flog) {
        fprintf(flog,"%s %s.%s %s",datestr,timestr,mss,logstr);
        if (ftell(flog)>MAXLOGSIZE) {
            fclose(flog);
            if (rename(logfilename1,logfilename2)) {
                remove(logfilename2);
                rename(logfilename1,logfilename2);
            }
        } else {
            fclose(flog);
        }
    }
}
void Log(const char *pszFmt,...) {
    va_list argp;

    Lock(&cs_log);
    va_start(argp,pszFmt);
    LogV(pszFmt,argp);
    va_end(argp);
    Unlock(&cs_log);
}
//Log}
int main(int argc,char * argv[]) {
    int i;
#ifdef _MSC_VER
    InitializeCriticalSection(&cs_log);
#else
    pthread_mutex_init(&cs_log,NULL);
#endif
    for (i=0;i<10000;i++) {
        Log("This is a Log %04d from FILE:%s LINE:%d\n",i, __FILE__, __LINE__);
    }
#ifdef _MSC_VER
    DeleteCriticalSection(&cs_log);
#else
    pthread_mutex_destroy(&cs_log);
#endif
    return 0;
}
//1-78行添加到你带main的.c或.cpp的那个文件的最前面
//81-85行添加到你的main函数开头
//89-93行添加到你的main函数结束前
//在要写LOG的地方仿照第87行的写法写LOG到文件MyLog1.log中
小卡拉米 2021-01-14
  • 打赏
  • 举报
回复
我的场景是这样: 这个回调不是执行一次就完了, 是事件触发的, 每隔一段时间被调用
小卡拉米 2021-01-14
  • 打赏
  • 举报
回复
引用 1 楼 ningto点com 的回复:
简单点可以while(true) { sleep(5); } sleep几秒钟防止退出,但是你这个回调函数在主线程里调用就不合适了,因为sleep会阻塞主线程,所以你应该单独起一个线程来执行上面的逻辑。
哦哦, 要放在线程里执行是吗? 我现在这种方式 , 走到 while(true) 那里也会崩溃
小卡拉米 2021-01-14
  • 打赏
  • 举报
回复
是同事写的动态库的bug ,
小卡拉米 2021-01-14
  • 打赏
  • 举报
回复
引用 9 楼 管理員 的回复:
[quote=引用 7 楼 管理員 的回复:][quote=引用 4 楼 赵4老师 的回复:]把你的Mylog logger换成下面我的MyLog:

在回调函数第一行加一句 system("cls"); 也会导致崩溃, 这是什么原因?[/quote] 没有任何信息留下, 会在cgvObj.Init(); 下一句崩溃, 难找原因
引用 8 楼 赵4老师 的回复:
[quote=引用 7 楼 管理員 的回复:][quote=引用 4 楼 赵4老师 的回复:]把你的Mylog logger换成下面我的MyLog:

在回调函数第一行加一句 system("cls"); 也会导致崩溃, 这是什么原因?[/quote] 列夫·托尔斯泰:“幸福的家庭都一样,不幸的家庭各有各的不幸.” zhao4zhong1:“不崩溃的程序都一样,崩溃的程序各有各的崩溃原因.” [/quote] 老湿, 根据你的经验, 给点方向啊, 别这么水[/quote] 好像是 boost::format 格式化字符串 导致, 因为 logger.info() 内用的 boost::format 格式化字符
小卡拉米 2021-01-14
  • 打赏
  • 举报
回复
引用 7 楼 管理員 的回复:
[quote=引用 4 楼 赵4老师 的回复:]把你的Mylog logger换成下面我的MyLog:

在回调函数第一行加一句 system("cls"); 也会导致崩溃, 这是什么原因?[/quote] 没有任何信息留下, 会在cgvObj.Init(); 下一句崩溃, 难找原因
引用 8 楼 赵4老师 的回复:
[quote=引用 7 楼 管理員 的回复:][quote=引用 4 楼 赵4老师 的回复:]把你的Mylog logger换成下面我的MyLog:

在回调函数第一行加一句 system("cls"); 也会导致崩溃, 这是什么原因?[/quote] 列夫·托尔斯泰:“幸福的家庭都一样,不幸的家庭各有各的不幸.” zhao4zhong1:“不崩溃的程序都一样,崩溃的程序各有各的崩溃原因.” [/quote] 老湿, 根据你的经验, 给点方向啊, 别这么水
ningto.com 2021-01-13
  • 打赏
  • 举报
回复
简单点可以while(true) { sleep(5); } sleep几秒钟防止退出,但是你这个回调函数在主线程里调用就不合适了,因为sleep会阻塞主线程,所以你应该单独起一个线程来执行上面的逻辑。

64,648

社区成员

发帖
与我相关
我的任务
社区描述
C++ 语言相关问题讨论,技术干货分享,前沿动态等
c++ 技术论坛(原bbs)
社区管理员
  • C++ 语言社区
  • encoderlee
  • paschen
加入社区
  • 近7日
  • 近30日
  • 至今
社区公告
  1. 请不要发布与C++技术无关的贴子
  2. 请不要发布与技术无关的招聘、广告的帖子
  3. 请尽可能的描述清楚你的问题,如果涉及到代码请尽可能的格式化一下

试试用AI创作助手写篇文章吧