Discussion :

[lea.soineca]

Bonjour

Y a-t-il un moyen de lire plus rapidement dans un pipe que cela :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
typedef struct {
	char * string;
	int pfdio[2];
} strpfd;

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
	unsigned sizeMsg = 300000;
	char * buffer = malloc(sizeof(char) * sizeMsg);
	unsigned int n;
	while ((n=read(strpfdturnread->pfdio[0], buffer, sizeof(char)*sizeMsg)) > 0)
	{
		strncat(strpfdturnread->string, buffer, n);
	}

strpfdturnread est un struct strpfd, ce struct est obligatoire.

[Obsidian]

Bonjour,

strncat() est faite pour gérer des chaînes de caractères. Tu risques d'avoir des ennuis avec si tu t'en sers pour traiter des données binaires et que celles-ci contiennent des zéros.
Par ailleurs, strcat() et strncat() déterminent elle-même la position de la fin de la chaîne à chaque fois que tu les appelles, en partant du début et en la remontant entièrement jusqu'à rencontrer le zéro final. Il est évident que c'est dramatique pour les performances.

Essaie plutôt de conserver le nombre total de caractères reçus à travers read() et de te servir de ce nombre comme index. Tu indiques alors directement à read() à quel endroit déposer la suite plutôt que la recevoir dans un buffer fixe et la concaténer ensuite avec le reste.

[lea.soineca]

Merci pour ta réponse

Mais je traite des chaines de caractère, et pas du binaire.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Essaie plutôt de conserver le nombre total de caractères reçus à travers read() et de te servir de ce nombre comme index

Par contre, je ne sais pas d'avance le nombre total de caractères reçus.

[LittleWhite]

Bonjour,

Vous pouvez savoir la taille de la chaîne à l'avance si le programme émetteur l'envoie.

[dinobogan]

J'ajouterai que passer par un buffer intermédiaire n'a pas d'intérêt dans l'exemple présenté.
De plus, "strpfdturnread->string" n'est pas alloué mais je suppose que l'exemple est incomplet.

Au final, le code devrait ressembler à quelque chose comme ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
  typedef struct
  {
    char * string;
    int pfdio[2];
  } strpfd;
 
  unsigned sizeMsg = 300000;
  strpfd strfdturnread;
  if( ( strfdturnread.string = malloc( sizeMsg ) == NULL ) { sortir du programme }
 
  ssize_t n;
  size_t position_append = 0;
  while( ( n = read( strfdturnread->pfdio[ 0 ], strfdturnread->string + position_append, sizeMsg - position_append ) ) > 0 )
  {
    position_append += n;
  }
 
  * ( strfdturnread->string + position_append ) = NULL;

Le code n'est pas testé mais tu as la trame algorithmique. Ce code sera beaucoup plus rapide que le tien pour plusieurs raisons :

1. comme déjà dit, strcat cherche à chaque fois depuis le début de chaîne pour trouver le caractère NULL
2. il n'y a pas de copie dans un buffer intermédiaire, les données sont directement copiées dans le buffer final
3. on prend en compte le nombre de caractère lus pour se positionner immédiatement au bon emplacement mémoire dans le buffer

[lea.soineca]

Merci !

Cela a largement amélioré la performance !

Merci !