自己思考调用函数会出现这样的问题吗？

光速

在服务器端写了一个日志函数：

参数：fileName是日志文件名称；
        content为写入的内容；
WriteLogFile(char* fileName,char *content)
{
     FILE *fp;
     if((fp=fopen(fileName,"at+"))!=NULL)
     {
            while (*content!='\0')
            {
             fputc(*content,fp);
             content++;
            }
     }
     fclose(fp);
}

在网络编程中我想到这样一个问题：当多个客户端连接到服务器端后，然后给服务器发送信息，只要信息发送的服务器端，那么服务器端就调用上面的日志函数。假如有多个客户端发送的信息同时到达服务器，那么这个函数被同时调用会发生冲突吗？也就是说，服务器端正在将客户端a发送过来的信息写入日志文件中，但还没有写完，这个时候，客户端b也发送过来信息，那么服务器端又要调用日志函数写入日志，可是这个时候，客户端a的信息还有完全写入到日志中。这样会出现写日志冲突吗？或者说会出现错误码？
不知道自己表达清楚了没有。大概意思就是这样。

kwaz

兄弟，加上写锁就好了。
给你一个代码。

#include <stdio.h>
#include <unistd.h>
#include <fcntl.h>
#include <errno.h>

int main(int argc,char* argv[])
{
    int fd;
    struct flock lock;
    int count = 0;

    if(argc != 2)
    {
        printf("Usage: %s filename.\n",argv[1]);
        return 1;
    }

    fd = open(argv[1],O_RDWR);
    if(fd<0)
    {
        printf("Open file failed.\n";
        return 1;
    }
    lock.l_type = F_WRLCK;
   lock.l_whence = 0;   
   lock.l_start = 0;     
    lock.l_len = 0;      

    while(fcntl(fd,F_SETLK,&lock)<0)
    {
       if(errno == EAGAIN||errno ==  EACCES)  //被其他进程加锁了
       {
          if(++count<5)
              sleep(1);  //加锁申请最多持续5s
          else
          {
             fcntl(fd,F_GETLK,&lock);
             printf("id: %ld process find pid %ld process lock the file %s.\n",
                     (long)getpid(),(long)lock.l_pid,argv[0]);
             return 1;
          }
       }
       else
       {
          printf("Error: exec function fcntl failed.\n";
          return 1;
       }
   }
   printf("id: %ld process locked the file.\n",(long)getpid());
   sleep(;   //占用文件的时间，这里可以是对文件的一些操作
   printf("id: %ld process release the file.\n",(long)getpid());
   return 0;
}

[ 本帖最后由 kwaz 于 2009-9-16 21:28 编辑 ]

光速

非常感谢！革命尚未成功，我还将继续努力！

ideawu

再说了, 为什么要用fputc? 用 fputs(content, fp) 不是可以吗? 另外, 这个函数设计得很不好, 频繁打开关闭文件非常影响性能.

[ 本帖最后由 ideawu 于 2009-9-16 23:10 编辑 ]

光速

原帖由 ideawu 于 2009-9-16 23:08 发表
再说了, 为什么要用fputc? 用 fputs(content, fp) 不是可以吗? 另外, 这个函数设计得很不好, 频繁打开关闭文件非常影响性能.

能说的具体一点吗？谢谢！

光速

原帖由 ideawu 于 2009-9-16 23:08 发表
再说了, 为什么要用fputc? 用 fputs(content, fp) 不是可以吗? 另外, 这个函数设计得很不好, 频繁打开关闭文件非常影响性能.

多个客户端都给服务器传递信息，那如何才能不频繁打开关闭文件从而达到自己的结果？谢谢！

ideawu

可以把日志功能做成一个单独的进程, 或者线程. 如果是线程, 通过队列和日志的使用者进行通信. 无论进程或者线程, 文件只打开一次, 关闭一次. 而且, 不用文件锁, 但可能需要对队列进行加锁.

或者, 在程序初始化的时候打开文件, 在程序退出时关闭文件.

光速

原帖由 ideawu 于 2009-9-17 12:20 发表
可以把日志功能做成一个单独的进程, 或者线程. 如果是线程, 通过队列和日志的使用者进行通信. 无论进程或者线程, 文件只打开一次, 关闭一次. 而且, 不用文件锁, 但可能需要对队列进行加锁.

或者, 在程序初始 ...

频繁打开与关闭文件确实不是好办法。
对于你所说的：队列和日志使用者进行通信。我感觉还是比较模糊。