不用智能指针，怎么处理生产者消费者中的内存管理问题比较优雅？

ccnyou 2016-09-19 10:13:19 


class Animal

{

};



class Cat : public Animal

{

};



// 生产者

Animal* GetAnimal()

{

	Animal* animal = new Cat();

	return animal;

}



// 消费者

void MakeFood(Animal* animal)

{

	delete animal;

}


这种逻辑应该很常见，总担心万一消费者因为需求变更被改了逻辑，没有delete到，头疼。
...全文
567 32 打赏 收藏 转发到动态 举报
写回复
用AI写文章
32 条回复
切换为时间正序
请发表友善的回复…
发表回复
ID870177103 2016-09-23
  • 打赏
  • 举报
 回复
引用 25 楼 ccnyou 的回复:
[quote=引用 23 楼 ID870177103 的回复:] [quote=引用 21 楼 ccnyou 的回复:] [quote=引用 20 楼 ID870177103 的回复:] [quote=引用 18 楼 ccnyou 的回复:] [quote=引用 16 楼 ID870177103 的回复:] 可以使用分配器的逻辑，给生产者加个clear函数，强制清除没有消费的对象
能否详细点？[/quote] 简单来说就是内存池啊，如果运行到某一步能确定已经不再消费，那就让内存池回收对象，而且最好嵌到生产者里面，有必要的话可以把生产的对象状态化就很容易能检查出错误[/quote] 这样感觉问题还是没有变，只不过把谁delete变成谁丢回内存池。还把系统内存管理工作给多做了一遍。[/quote] 这不一样，因为不释放内存池对象也不会造成内存泄露，而且你可以任意时间析构掉内存池 如果这还不行就没办法 不使用原生指针，无非也就是间接访问（内存池型容器），移动语义（auto智能指针），和引用计数（shared智能指针）[/quote] 如果你丢到内存池里面没办法使用也没有办法delete，到程序退出才释放，那么也跟泄漏没有什么区别了。内存泄漏的问题本质不在于程序退出了这块内存没法回收，而是随着程序越跑越久而占用内存飙升，导致程序没法一直跑下去。[/quote] 你非要这样理解，我也没办法，但你可以想一下真的“到程序退出才能释放”？
ccnyou 2016-09-23
  • 打赏
  • 举报
 回复
引用 26 楼 CharlesSimonyi 的回复:
在纯C语言的项目中，怎么处理内存管理问题比较优雅？ 总担心malloc了没有free，头疼。
所以应该是怎么处理的？
barrassed 2016-09-23
  • 打赏
  • 举报
 回复
建立一个中间类叫仓库，把所有生产的产品放在这统一管理，有点内存池的概念。在外围封装下，与拥有者进行一个绑定。
dongfangbuba 2016-09-23
  • 打赏
  • 举报
 回复
C++ effictive里面的说法是你把数据封装成对象，对象自己来析构
dongfangbuba 2016-09-23
  • 打赏
  • 举报
 回复
为什么不用内存池？？？？？
encoderlee 版主 2016-09-23
  • 打赏
  • 举报
 回复
引用 27 楼 ccnyou 的回复:
[quote=引用 26 楼 CharlesSimonyi 的回复:] 在纯C语言的项目中，怎么处理内存管理问题比较优雅？ 总担心malloc了没有free，头疼。
所以应该是怎么处理的？[/quote] 我也很疑惑，这个问题困扰我好多年了，同问
encoderlee 版主 2016-09-23
  • 打赏
  • 举报
 回复
在纯C语言的项目中，怎么处理内存管理问题比较优雅？

总担心malloc了没有free，头疼。
ID870177103 2016-09-22
  • 打赏
  • 举报
 回复
引用 18 楼 ccnyou 的回复:
[quote=引用 16 楼 ID870177103 的回复:] 可以使用分配器的逻辑，给生产者加个clear函数，强制清除没有消费的对象
能否详细点？[/quote] 简单来说就是内存池啊，如果运行到某一步能确定已经不再消费，那就让内存池回收对象，而且最好嵌到生产者里面，有必要的话可以把生产的对象状态化就很容易能检查出错误
ccnyou 2016-09-22
  • 打赏
  • 举报
 回复
引用 17 楼 ri_aje 的回复:
为啥不用智能指针呢？
几个理由： 1，会污染，这里是返回值，然后导致外面调用和传参都需要用智能指针。 2，用指针还是很多地方要传个void*，智能指针传不进去。 3，智能指针可以变回裸指针，但是这个是不可逆的，会遇到很多麻烦问题。
ccnyou 2016-09-22
  • 打赏
  • 举报
 回复
引用 16 楼 ID870177103 的回复:
可以使用分配器的逻辑，给生产者加个clear函数，强制清除没有消费的对象
能否详细点？
ri_aje 2016-09-22
  • 打赏
  • 举报
 回复
为啥不用智能指针呢？
ID870177103 2016-09-22
  • 打赏
  • 举报
 回复
可以使用分配器的逻辑，给生产者加个clear函数，强制清除没有消费的对象
赵4老师 2016-09-22
  • 打赏
  • 举报
 回复
引用 7 楼 ccnyou 的回复:
[quote=引用 4 楼 zhao4zhong1 的回复:] 不要企图优雅的结束（因为这是不可能办到的） 而要在烂的不能再烂的摊子上也能重整河山！ http://bbs.csdn.net/topics/390787357
太抽象看不懂 /微笑[/quote] 上士闻道，勤而行之；中士闻道，若存若亡；下士闻道，大笑之。不笑不足以为道。
ccnyou 2016-09-22
  • 打赏
  • 举报
 回复
引用 13 楼 Rewen 的回复:
问题点在于消费者不按照约定删除生产者制造的对象，有时候删除，有时候不删除。 感觉也就是3个思路。 1， 是不用指针，写拷贝构造函数，使用对象传递或者是值传递。 2， 另外就是类似于垃圾回收的机制，利用另外的类来定期(或者按照某个逻辑)删除。 3， 强制消费者删除，在代码里面作为强逻辑，比如写个共同的父类，然后全部的消费者继承 这个父类，在这个使用这个类的时候，某些条件下就必须删除。否则就怎么怎么样。
目前我用了第一种，稍微不太一样的是我是写个clone方法。由于返回的是基类的指针，谁要用谁clone一份，不用就自己delete。
ccnyou 2016-09-22
  • 打赏
  • 举报
 回复
引用 23 楼 ID870177103 的回复:
[quote=引用 21 楼 ccnyou 的回复:] [quote=引用 20 楼 ID870177103 的回复:] [quote=引用 18 楼 ccnyou 的回复:] [quote=引用 16 楼 ID870177103 的回复:] 可以使用分配器的逻辑，给生产者加个clear函数，强制清除没有消费的对象
能否详细点？[/quote] 简单来说就是内存池啊，如果运行到某一步能确定已经不再消费，那就让内存池回收对象，而且最好嵌到生产者里面，有必要的话可以把生产的对象状态化就很容易能检查出错误[/quote] 这样感觉问题还是没有变，只不过把谁delete变成谁丢回内存池。还把系统内存管理工作给多做了一遍。[/quote] 这不一样，因为不释放内存池对象也不会造成内存泄露，而且你可以任意时间析构掉内存池 如果这还不行就没办法 不使用原生指针，无非也就是间接访问（内存池型容器），移动语义（auto智能指针），和引用计数（shared智能指针）[/quote] 如果你丢到内存池里面没办法使用也没有办法delete，到程序退出才释放，那么也跟泄漏没有什么区别了。内存泄漏的问题本质不在于程序退出了这块内存没法回收，而是随着程序越跑越久而占用内存飙升，导致程序没法一直跑下去。
赵4老师 2016-09-22
  • 打赏
  • 举报
 回复
我这个算不算优雅？ 
//循环向a函数每次发送200个字节长度（这个是固定的）的buffer,
//a函数中需要将循环传进来的buffer，组成240字节（也是固定的）的新buffer进行处理，
//在处理的时候每次从新buffer中取两个字节打印
#ifdef _MSC_VER
    #pragma warning(disable:4996)
#endif
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#ifdef _MSC_VER
    #include <windows.h>
    #include <process.h>
    #include <io.h>
    #define  MYVOID             void
    #define  vsnprintf          _vsnprintf
#else
    #include <unistd.h>
    #include <sys/time.h>
    #include <pthread.h>
    #define  CRITICAL_SECTION   pthread_mutex_t
    #define  MYVOID             void *
#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);
}
void sleep_ms(int ms) {
    Sleep(ms);
}
#else
void Lock(CRITICAL_SECTION *l) {
    pthread_mutex_lock(l);
}
void Unlock(CRITICAL_SECTION *l) {
    pthread_mutex_unlock(l);
}
void sleep_ms(int ms) {
    usleep(ms*1000);
}
#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}
#define ASIZE    200
#define BSIZE    240
#define CSIZE      2
char Abuf[ASIZE];
char Cbuf[CSIZE];
CRITICAL_SECTION cs_HEX;
CRITICAL_SECTION cs_BBB;
struct FIFO_BUFFER {
    int  head;
    int  tail;
    int  size;
    char data[BSIZE];
} BBB;
int No_Loop=0;
void HexDump(int cn,char *buf,int len) {
    int i,j,k;
    char binstr[80];

    Lock(&cs_HEX);
    for (i=0;i<len;i++) {
        if (0==(i%16)) {
            sprintf(binstr,"%03d %04x -",cn,i);
            sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
        } else if (15==(i%16)) {
            sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
            sprintf(binstr,"%s  ",binstr);
            for (j=i-15;j<=i;j++) {
                sprintf(binstr,"%s%c",binstr,('!'<buf[j]&&buf[j]<='~')?buf[j]:'.');
            }
            Log("%s\n",binstr);
        } else {
            sprintf(binstr,"%s %02x",binstr,(unsigned char)buf[i]);
        }
    }
    if (0!=(i%16)) {
        k=16-(i%16);
        for (j=0;j<k;j++) {
            sprintf(binstr,"%s   ",binstr);
        }
        sprintf(binstr,"%s  ",binstr);
        k=16-k;
        for (j=i-k;j<i;j++) {
            sprintf(binstr,"%s%c",binstr,('!'<buf[j]&&buf[j]<='~')?buf[j]:'.');
        }
        Log("%s\n",binstr);
    }
    Unlock(&cs_HEX);
}
int GetFromRBuf(int cn,CRITICAL_SECTION *cs,struct FIFO_BUFFER *fbuf,char *buf,int len) {
    int lent,len1,len2;

    lent=0;
    Lock(cs);
    if (fbuf->size>=len) {
        lent=len;
        if (fbuf->head+lent>BSIZE) {
            len1=BSIZE-fbuf->head;
            memcpy(buf     ,fbuf->data+fbuf->head,len1);
            len2=lent-len1;
            memcpy(buf+len1,fbuf->data           ,len2);
            fbuf->head=len2;
        } else {
            memcpy(buf     ,fbuf->data+fbuf->head,lent);
            fbuf->head+=lent;
        }
        fbuf->size-=lent;
    }
    Unlock(cs);
    return lent;
}
MYVOID thdB(void *pcn) {
    char        *recv_buf;
    int          recv_nbytes;
    int          cn;
    int          wc;
    int          pb;

    cn=(int)pcn;
    Log("%03d thdB              thread begin...\n",cn);
    while (1) {
        sleep_ms(10);
        recv_buf=(char *)Cbuf;
        recv_nbytes=CSIZE;
        wc=0;
        while (1) {
            pb=GetFromRBuf(cn,&cs_BBB,&BBB,recv_buf,recv_nbytes);
            if (pb) {
                Log("%03d recv %d bytes\n",cn,pb);
                HexDump(cn,recv_buf,pb);
                sleep_ms(1);
            } else {
                sleep_ms(1000);
            }
            if (No_Loop) break;//
            wc++;
            if (wc>3600) Log("%03d %d==wc>3600!\n",cn,wc);
        }
        if (No_Loop) break;//
    }
#ifndef _MSC_VER
    pthread_exit(NULL);
#endif
}
int PutToRBuf(int cn,CRITICAL_SECTION *cs,struct FIFO_BUFFER *fbuf,char *buf,int len) {
    int lent,len1,len2;

    Lock(cs);
    lent=len;
    if (fbuf->size+lent>BSIZE) {
        lent=BSIZE-fbuf->size;
    }
    if (fbuf->tail+lent>BSIZE) {
        len1=BSIZE-fbuf->tail;
        memcpy(fbuf->data+fbuf->tail,buf     ,len1);
        len2=lent-len1;
        memcpy(fbuf->data           ,buf+len1,len2);
        fbuf->tail=len2;
    } else {
        memcpy(fbuf->data+fbuf->tail,buf     ,lent);
        fbuf->tail+=lent;
    }
    fbuf->size+=lent;
    Unlock(cs);
    return lent;
}
MYVOID thdA(void *pcn) {
    char        *send_buf;
    int          send_nbytes;
    int          cn;
    int          wc;
    int           a;
    int          pa;

    cn=(int)pcn;
    Log("%03d thdA              thread begin...\n",cn);
    a=0;
    while (1) {
        sleep_ms(100);
        memset(Abuf,a,ASIZE);
        a=(a+1)%256;
        if (16==a) {No_Loop=1;break;}//去掉这句可以让程序一直循环直到按Ctrl+C或Ctrl+Break或当前目录下存在文件No_Loop
        send_buf=(char *)Abuf;
        send_nbytes=ASIZE;
        Log("%03d sending %d bytes\n",cn,send_nbytes);
        HexDump(cn,send_buf,send_nbytes);
        wc=0;
        while (1) {
            pa=PutToRBuf(cn,&cs_BBB,&BBB,send_buf,send_nbytes);
            Log("%03d sent %d bytes\n",cn,pa);
            HexDump(cn,send_buf,pa);
            send_buf+=pa;
            send_nbytes-=pa;
            if (send_nbytes<=0) break;//
            sleep_ms(1000);
            if (No_Loop) break;//
            wc++;
            if (wc>3600) Log("%03d %d==wc>3600!\n",cn,wc);
        }
        if (No_Loop) break;//
    }
#ifndef _MSC_VER
    pthread_exit(NULL);
#endif
}
int main() {
#ifdef _MSC_VER
    InitializeCriticalSection(&cs_log);
    InitializeCriticalSection(&cs_HEX);
    InitializeCriticalSection(&cs_BBB);
#else
    pthread_t threads[2];
    int threadsN;
    int rc;
    pthread_mutex_init(&cs_log,NULL);
    pthread_mutex_init(&cs_HEX,NULL);
    pthread_mutex_init(&cs_BBB,NULL);
#endif
    Log("Start===========================================================\n");

    BBB.head=0;
    BBB.tail=0;
    BBB.size=0;

#ifdef _MSC_VER
    _beginthread((void(__cdecl *)(void *))thdA,0,(void *)1);
    _beginthread((void(__cdecl *)(void *))thdB,0,(void *)2);
#else
    threadsN=0;
    rc=pthread_create(&(threads[threadsN++]),NULL,thdA,(void *)1);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
    rc=pthread_create(&(threads[threadsN++]),NULL,thdB,(void *)2);if (rc) Log("%d=pthread_create %d error!\n",rc,threadsN-1);
#endif

    if (!access("No_Loop",0)) {
        remove("No_Loop");
        if (!access("No_Loop",0)) {
            No_Loop=1;
        }
    }
    while (1) {
        sleep_ms(1000);
        if (No_Loop) break;//
        if (!access("No_Loop",0)) {
            No_Loop=1;
        }
    }
    sleep_ms(3000);
    Log("End=============================================================\n");
#ifdef _MSC_VER
    DeleteCriticalSection(&cs_BBB);
    DeleteCriticalSection(&cs_HEX);
    DeleteCriticalSection(&cs_log);
#else
    pthread_mutex_destroy(&cs_BBB);
    pthread_mutex_destroy(&cs_HEX);
    pthread_mutex_destroy(&cs_log);
#endif
    return 0;
}
ID870177103 2016-09-22
  • 打赏
  • 举报
 回复
引用 21 楼 ccnyou 的回复:
[quote=引用 20 楼 ID870177103 的回复:] [quote=引用 18 楼 ccnyou 的回复:] [quote=引用 16 楼 ID870177103 的回复:] 可以使用分配器的逻辑，给生产者加个clear函数，强制清除没有消费的对象
能否详细点？[/quote] 简单来说就是内存池啊，如果运行到某一步能确定已经不再消费，那就让内存池回收对象，而且最好嵌到生产者里面，有必要的话可以把生产的对象状态化就很容易能检查出错误[/quote] 这样感觉问题还是没有变，只不过把谁delete变成谁丢回内存池。还把系统内存管理工作给多做了一遍。[/quote] 这不一样，因为不释放内存池对象也不会造成内存泄露，而且你可以任意时间析构掉内存池 如果这还不行就没办法 不使用原生指针，无非也就是间接访问（内存池型容器），移动语义（auto智能指针），和引用计数（shared智能指针）
angel6709 2016-09-22
  • 打赏
  • 举报
 回复
可不可以这样搞： 生产者做一个全局的循环队列，开始时创建，到最后，程序退出时销毁。
ccnyou 2016-09-22
  • 打赏
  • 举报
 回复
引用 20 楼 ID870177103 的回复:
[quote=引用 18 楼 ccnyou 的回复:] [quote=引用 16 楼 ID870177103 的回复:] 可以使用分配器的逻辑，给生产者加个clear函数，强制清除没有消费的对象
能否详细点？[/quote] 简单来说就是内存池啊，如果运行到某一步能确定已经不再消费，那就让内存池回收对象，而且最好嵌到生产者里面，有必要的话可以把生产的对象状态化就很容易能检查出错误[/quote] 这样感觉问题还是没有变，只不过把谁delete变成谁丢回内存池。还把系统内存管理工作给多做了一遍。
张小飞Official 2016-09-21
  • 打赏
  • 举报
 回复
放到析构函数里边释放不就可以了么 RAII思想
加载更多回复（12）
【C++学习（24）】生产者消费者模式 future、promise、condition_variable
本次课程我们学习了生产者和消费者模式，这在我们模型推理过程中非常常见，生产者线程负责读取图片，拿到推理结果进行画框可视化，消费者线程负责模型推理，将结果返回到生产者线程。我们在学习生产者和消费者模式过程中，解决了三个问题：首先是共享资源访问问题。
Linux 线程：线程同步、生产者消费者模型
Linux 线程同步、生产者消费者模型
Rust SmartPointer
Rust的所有权(ownership)和智能指针(smart pointers)是它的两个重要特征。理解它们是掌握Rust特性和编写Rust程序的基础。理解所有权与智能指针的工作方式,可以让我们编写无bug,高效,优雅的Rust代码。它们是Rust的核心概念,是学习和掌握Rust的基石。Rust禁止在没有所有权的情况下使用值,这可以有效地防止“双乘出错”等问题。智能指针是封装了原始指针的结构体,可以轻松实现自动内存管理。:用于“钉住”数据的智能指针,防止被move。8. Cow:拥有或借用值的智能指针。
【Linux】多线程深度剖析（二）
文章目录1. 死锁1.1 死锁的概念1.2 死锁四个必要条件1.3 避免死锁1.4 简单案例2. 生产者消费者模型2.1 三二一原则2.2 为何要使用生产者消费者模型2.3 生产者消费者模型优点2.4 基于BlockingQueue的生产者消费者模型2.5 C++ queue模拟阻塞队列的生产消费模型3. POSIX信号量3.1 信号量的接口3.1.1 初始化信号量3.1.2 销毁信号量3.1.3 等待信号量3.1.4 唤醒信号量3.2 基于环形队列的生产消费模型3.3 用信号量进行多线程间的同步过程4.
NCRE全国计算机四级网络工程师
使用虚拟页式存储管理时，需要在页表中增加: 页号、有效位(留位)，页框号、访问位、修改位、保护位、禁止缓存位等，有效位决定是否产生缺页中断其中某进程运行时若将磁盘中的一个页面调入内存，对应页表表项中的内存块号、驻留位和访问位修改。在虚拟页式存储管理系统，调入策略、置业策略、置换策略与页面调度有关当系统采用虚拟页式存储管理方案时，需要处理：运行时决定将哪些页面装入内存运行中无空闲页框时选择置换掉某些页面将暂时不需要的页面清除出内存将进程按页框大小划分为页面创建交换分区暂存换出的页面。
C++ 语言

64,654

社区成员

250,484

社区内容

发帖
与我相关
我的任务
社区描述
C++ 语言相关问题讨论，技术干货分享，前沿动态等
c++ 技术论坛（原bbs）
社区管理员
  • C++ 语言社区
  • encoderlee
  • paschen
加入社区
  • 近7日
  • 近30日
  • 至今

加载中

查看更多榜单
社区公告
  1. 请不要发布与C++技术无关的贴子
  2. 请不要发布与技术无关的招聘、广告的帖子
  3. 请尽可能的描述清楚你的问题，如果涉及到代码请尽可能的格式化一下

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

+ 用AI写文章