оптимизировать программу с помощью inotify - PullRequest
Новая
       13

оптимизировать программу с помощью inotify

0 голосов
/ 02 августа 2010

Я пишу программу по inotify, чтобы проверить изменение файла в цикле, чтобы получить любые изменения в этом.но я думаю, что это можно сделать лучше.Кто-нибудь может написать этот код лучше? 

#include <stdio.h>
#include <stdlib.h>
#include <errno.h>
#include <sys/types.h>
#include <linux/inotify.h>
#include <sys/select.h>
#define EVENT_SIZE  ( sizeof (struct inotify_event) )
#define EVENT_BUF_LEN     ( 1024 * ( EVENT_SIZE + 16 ) )
int event_check (int fd)
{
  fd_set rfds;
  FD_ZERO (&rfds);
  FD_SET (fd, &rfds);
  /* Wait until an event happens or we get interrupted 
     by a signal that we catch */
  return select (FD_SETSIZE, &rfds, NULL, NULL, NULL);
  }

int main( )
{
  int length, i = 0;
  int fd;
  int wd;
while(1){
i=0;
  fd = inotify_init();

  if ( fd < 0 ) {
    perror( "inotify_init" );
  }

  wd = inotify_add_watch( fd, "/tmp/test", IN_CLOSE_WRITE);
    if (event_check (fd) > 0)
    {
        char buffer[EVENT_BUF_LEN];
        int count = 0;
        length = read( fd, buffer, EVENT_BUF_LEN ); 
        if ( length < 0 ) {
            perror( "read" );
          } 
         while ( i < length ) {     struct inotify_event *event = ( struct inotify_event * ) &buffer[ i ]; 
            printf( "New file %s Editted.\n", event->name );
            i += EVENT_SIZE + event->len;
          }
    }
}
    inotify_rm_watch( fd, wd );
   close( fd );
}

Ответы [ 2 ]

1 голос
/ 05 декабря 2011

Я думаю, что вам нужен только один inotify_init для одной программы и только один inotify_add_watch для одного каталога. Не могли бы вы вставить версию «init и add_watch вне цикла», которая, как вы сказали, не работает?

0 голосов
/ 02 августа 2010

Я никогда не использовал inotify, поэтому возможно, что это может быть неправильно, но вот несколько изменений, которые, я думаю, могут «улучшить» ваш код.

  1. Я недумаю, есть какая-то причина, чтобы поместить либо inotify_init, либо inotify_add_watch в цикл.Просто выполните эту инициализацию один раз, перед циклом.

  2. Я не уверен, почему вы создали функцию event_check.Вы не указываете таймаут и используете только один дескриптор файла, поэтому я думаю, что чтение даст вам ту же функциональность.

...