Discussion :

[Ganondorf]

bonsoir à tous,

J'ai besoin pour mon programme de convertir une chaine de ce style:

moi*!*@mon.host.*

en une expression régulière, ce qui veut dire qu'il faut que j'echappe tout les caractères '*' '?' et '.'

J'ai réussi a écrire cette fonction, cependant je la trouve vraiment dégueulasse
J'aurai donc besoin de votre aide pour l'améliorer

Code :

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

1
2
3
4
5
6
7
 
char *host = strdup("Cory!*@192.*");
char *tmphost = convertForRegexp(host);
 
printf("convert: %s\n", tmphost);
 
free(host);

la fonction:

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
20
21
22
23
24
25
 
static char *convertForRegexp(char *string)
{
	static char result[250];
	char c, *p = string;
	unsigned int i = 0;
 
	while ((c = *p++))
	{
		if (c == '*' || c == '?')
		{
			result[i++] = '.';
			result[i++] = *string++;
		}
		else if (c == '.')
		{
			result[i++] = '\\';
			result[i++] = '\\';
			result[i++] = *string++;
		}
		else result[i++] = *string++;
	}
	result[i] = 0;
	return result;
}

résultat:

convert: Cory!.*@192\\..*

La fonction fonctionne mais je pense qu'il a vraiment moyen de faire mieux,
merci à vous et bonne soirée.

[diogene]

L'utilisation d'un switch() permettrait d'éviter les if..else et rendrait le code plus clair :

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
20
static char *convertForRegexp(char const *string)
{
   static char result[250];
   unsigned int i;
   for(i=0; *string != '\0'; i++, string++)
   {
      switch(*string)
      {
        case '*' :
        case '?' : result[i++] = '.' ;
                   break;
        case '.' : result[i++] = '\\' ;
                   result[i++] = '\\' ;
                   break;
      }
      result[i] = *string;
  }
  result[i] = 0;
  return result;
}

[Ganondorf]

salut diogene et merci pour ta réponse, ca fonctionne comme il faut.
Par contre j'ai un soucis lors de la comparaison entre les chaines, ca ne semble plus matcher correctement

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
	char hostmask[MASKLEN+1];
	char *tmphost = convertForRegexp("havoc!*@192.*");
 
	if (parc < 2)
		return irc_notice(client->nick, "Syntaxe: login <idnick> <password>");
 
 
	sprintf(hostmask, "%s!%s@%s", client->nick, client->user, client->host);
	if (!regexcmp(hostmask, tmphost))
		return irc_notice(client->nick, "Les masques ne correspondent pas. %s - %s", hostmask, tmphost);

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
 
static int regexcmp(const char *s1, const char *s2)
{
	regex_t myreg;
	unsigned int flags = (REG_ICASE|REG_NOSUB|REG_EXTENDED);
 
	if (regcomp(&myreg, s1, flags) != 0)
		return 0;
 
	if (regexec(&myreg, s2, 0, NULL, 0) != 0)
		return 0;
 
	regfree(&myreg);
	return 1;
}

résultat:

[21:34:43] Notice Bot:havoc Les masques ne correspondent pas. havoc!havoc@192.168.1.20 - havoc!.*@192\\..*

la chaine semble pourtant bien converti en expression régulière

[jeroman]

Code :

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

1
2
3
        case '.' : result[i++] = '\\' ;
                   result[i++] = '\\' ;
                   break;

Il y a une ligne en trop.

[Ganondorf]

même comme ca, ca ne fonctionne pas, je comprend pas bien ca fonctionnait avant ...