optimiser un programme avec inotify

Question

J'écris un programme par inotify pour vérifier le changement de fichier dans la boucle pour saisir toute modification à cela. mais je pense que cela peut être fait mieux. quelqu'un peut-il écrire ce code mieux?

#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); 
} 

Answer 1

Je ne l'ai jamais utilisé inotify, il est donc possible que cela pourrait se tromper mais voici quelques changements que je pense pourrait « améliorer » votre code.

1. Je ne pense pas qu'il y ait quelque raison que ce soit de mettre inotify_init ou inotify_add_watch dans la boucle. Faites juste ce travail d'initialisation une fois, avant la boucle. Je ne sais pas pourquoi vous avez créé la fonction event_check. Vous ne spécifiez pas de délai et vous n'utilisez qu'un seul descripteur de fichier, donc je pense que lire vous donnera la même fonctionnalité.

Answer 2

Je pense que vous avez seulement besoin d'un inotify_init pour un seul programme et un seul inotify_add_watch pour un répertoire. Pouvez-vous s'il vous plaît coller la version "init et add_watch en dehors de la boucle" que vous avez dit ne fonctionne pas?