首先看下面的函数
<span style="font-size:18px;">void writelog(char* file, char* msg)
{
FILE*fp=NULL;
int nDataLen = strlen(msg);
fp=fopen(file,"ab+");//只供读取
if(fp!=NULL)
{
fwrite(msg, nDataLen, 1, fp);
}
fclose(fp);//关闭文件
}</span>
这个有什么问题呢?当fp为空的时候,是不是也会执行fclose(fp)?
修改如下:
<span style="font-size:18px;">void writelog(char* file, char* msg)
{
FILE*fp=NULL;
int nDataLen = strlen(msg);
fp=fopen(file,"ab+");//只供读取
if(fp!=NULL)
{
fwrite(msg, nDataLen, 1, fp);
fclose(fp);//关闭文件
}
}</span>
这样是不是就没问题了呢?
当多线程中调用这个日志函数的时候,会不会有问题?
所以有一种方法就是加锁,但这样效率可能会低一些。
另外一种方法就是用一个线程来接收其他线程的写日志的消息,有一个线程来专门写日志。
这样工作线程不会阻塞,可提高并发效率。
编程和设计的过程,就是一个逐步完善的过程,没有人一开始会想的很全面,也没有绝对的错误的代码和设计。如果有明显的功能错误,在设计过程中,就会改正。就比如最开始的那个函数,fp为空的概率有多少?所以一般不会出错。当日志函数只在单线程中使用,那考虑加锁和多线程消息也是多余。所以,要看应用场景。
程序员都很反感需求的变化。需求,就是应用场景。只有在需求确认了,才可以设计出相对匹配的方案出来,不一定是完美的,但肯定是在综合各种因素的情况下,是最优的。周期是要考虑的,成本是要考虑的。
转载请注明原创链接:http://blog.csdn.net/wujunokay/article/details/38310669