Discussion :

[pepito31]

Bonjour

J'ai un soucis ! J'ai une chaine (un chemin d'acces, en realité) et j'aimerais en extraire un nom de Process que la chaine contient. Lorsque j'utilise le code ci-dessous, l'extraction se fait bien mais il y a un "." (signe qui devient è@ sous un editeur quelconque) à la fin du nom !?! Je suppose que cela vient d'un probleme de pointeur ?
Quelqu'un peut-il m'aider sur le suejt merci !! ??

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
char *Dir;
char *MonProcess;
Dir=malloc(sizeof(char)*strlen(argv[1])); // argv[1] est du type /mnt/hda5/Monchemin/NomDuProcess/NomDuProcess  (ou NomDuProcess est un exe En C et aussi, le repertoire ou il doit se loger).
MonProcess=(char*)malloc(sizeof(char)*100);
strcpy(Dir,argv[1]);
 
int StartProcessName=strlen(Dir)-12;
int EndProcessName=strlen(Dir)-1;
 
MonProcess=str_sub(Dir,StartProcessName,EndProcessName);
 
 
 
// routine str_sub :
/**
Fonction str_sub :
------------------
Goal : This function allows a string to be extracted from another with starting and ending indexes
--------------------------------------------------------------------------------------------------
inputs : const char* s	--> target string to extract from
	 unsigned int start --> starting index 
	 unsigned int end --> ending index
outputs: char* --> the result of extraction
 
*/
 
char *str_sub (const char *s, unsigned int start, unsigned int end)
{
   char *new_s = NULL;
 
   if (s != NULL && start < end)
   {
      new_s = malloc (sizeof (*new_s) * (end - start + 2));
      if (new_s != NULL)
      {
         int i;
         for (i = start; i <= end; i++)
         {
            new_s[i-start] = s[i];
         }
      }
      else
      {
         fprintf (stderr, "Memoire insuffisante\n");
         exit (EXIT_FAILURE);
      }
   }
   return new_s;
}

resultat du code :

Code :

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

1
2
3
 
ARGV1=/mnt/hda5/Local/Travail/VALIDATIONS/TESTS/SimCATDSProc/C1C___00_01/C1C___00_01
MonProcess=C1C___00_01�   // Il y a comme une sorte de "," a la fin qui doit venir d'un \0 j'imagine ?!

Comment faire pour recuperer "NomDeProcess" proprement en C ? (sachant qu'il a toujours la meme taille =12) ?


Merci de m'aider !
pepito

[cheveche4]

Si tu utilise "malloc", tu ne mets pas à '\0' toutes les octets de la zone mémoire que tu alloues, ce qui veux dire que si il y a eu autre chose d'alloué avant, ta chaine ne va pas spécialement avoir de fin (délimité par un '\0') là où tu veux.

Essaie un peu avec calloc à la place de tes malloc?

Et pourquoi allouer MonProcess, si tu le réassigne sans l'avoir utilisé entre-temps? Tu ne fais qu'utiliser une zone mémoire inutilement en plus...

Code :

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

1
2
3
4
 
MonProcess=(char*)malloc(sizeof(char)*100);
//...
MonProcess=str_sub(Dir,StartProcessName,EndProcessName);

puisque c'est au final "new_s = malloc (sizeof (*new_s) * (end - start + 2));" que tu retourne

[Sve@r]

Si tu utilise "malloc", tu ne mets pas à '\0' toutes les octets de la zone mémoire que tu alloues, ce qui veux dire que si il y a eu autre chose d'alloué avant, ta chaine ne va pas spécialement avoir de fin (délimité par un '\0') là où tu veux.

Essaie un peu avec calloc à la place de tes malloc?

Effectivement le '\0' manque. Mais quand on maîtrise un minimum sa gestion mémoire, on reste sur malloc() et on ne met le '\0' que là où c'est nécessaire...

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
 
char *str_sub (const char *s, unsigned int start, unsigned int end)
{
   char *new_s = NULL;
 
   if (s != NULL && start < end)
   {
      new_s = malloc (sizeof (*new_s) * (end - start + 2));
      if (new_s != NULL)
      {
         int i;
         for (i = start; i <= end; i++)
         {
            new_s[i-start] = s[i];
         }
      }
      else
      {
         fprintf (stderr, "Memoire insuffisante\n");
         exit (EXIT_FAILURE);
      }
   }
   return new_s;
}

C'est très mal de faire un exit dans une sous-fonction. La logique de la programmation voudrait que cette fonction renvoie simplement le nouveau pointeur ou NULL si l'allocation échoue et que ce soit l'appelant qui décide d'afficher un message ou pas et qui décide de quitter ou pas en fonction de ce retour. Mais mis à part ce détail, il te suffit je pense de rajouter un '\0' en fin de for() et ça devrait marcher...

[pepito31]

Merci pour vos reponses.
Bon, ca ne marche pas.
J'ai essayé d'ajouter en effet le "\0" au new_s, en fin de boucle for mais je receptionne toujours mon signe cabalistique à la fin de ma chaine ?!
voici le code modifié de la routine str_sub:

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
26
 
char *str_sub (const char *s, unsigned int start, unsigned int end)
{
   char *new_s = NULL;
 
   if (s != NULL && start < end)
   {
      new_s = malloc (sizeof (*new_s) * (end - start + 2));
      if (new_s != NULL)
      {
         int i;
         for (i = start; i <= end; i++)
         {
            new_s[i-start] = s[i];
         }
	 strcat(new_s,"\0"); // Ajout d'un retour chariot en fin de chaine 
      }
      else
      {
         fprintf (stderr, "Memoire insuffisante\n");
         new_s=NULL; // retour de pointeur NULL si depassement memoire
      }
   }
   printf("new_s=%s\n",new_s); // a ce stade, j'ai toujours "new_s="C1C___00_01," au lieu de C1C___00_01 !?? 
   return new_s;
}

Une autre idee svp ??
Merci par avance

(j'ai aussi suivi ton conseil, sve@r, en utilisant un retour NULL plutot que EXIT lors d'un depassement memoire).

[tamiel]

Pourquoi ne pas utiliser la fonction basename de la libc ?

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
 
#include <stdio.h>
#include <string.h>
#include <libgen.h>
 
int main(int argc, char *argv[])
{
  if(argc < 2)
  {
    printf("Need one arg\n");
    return -1;
  }
  printf("ARGV1 : %s\n",argv[1]);
  char *pchr;
  pchr = basename(argv[1]);
  printf("%s\n", pchr);
  return 0;
}

[pepito31]

Basename ne resoudra pas mon probleme.

Mon argv[1] est du type chemin d'acces ( /mnt/hda5/MonProcess/);

Ce que j'aimerais, c'est recuperer le nom "MonProcess" dans ma variable *MonProcess. C'est donc, decouper argv[1]. Pour cela, j'utilise la routine sub_str.
Seulement, j'arrive toujours pas a recuperer ce nom sans avoir de signe bizarre (de retour chariot?) à la fin de mon MonProcess !?!

[pepito31]

pour visualiser le probleme, j'ai integre des printf dans ma fonction str_sub :

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
26
27
28
char *str_sub (const char *s, unsigned int start, unsigned int end)
{
   char *new_s = NULL;
   int lg=sizeof (char) * (end - start8-));
 
   printf("ma chaine =%s\n",s); // visualisation de ma chaine=argv[1] a partir de laquelle je dois recuperer le nom de process (dernier element avant le "/") 
   printf("longueur de chaine =%d\n",lg);
   if (s != NULL && start < end)
   {
      new_s = malloc (lg);
      if (new_s != NULL)
      {
         int i;
         for (i = start; i < end; i++)
         {
            new_s[i-start] = s[i];
	    printf("new_s=%s\n",new_s); // visu de nex_s à l'ecran a chaque incrementation
         }
      }
      else
      {
         fprintf (stderr, "Memoire insuffisante\n");
         new_s=NULL;
      }
   }
   printf("mon new_s=%s\n",new_s); // au final
   return new_s;
}

et voici le resultat obtenu :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
ma chaine =/mnt/hda5/Local/Travail/fferreir/Bouchons_JCB/Bouchons_SMOS/VALIDATIONS/TESTS/SimCATDSProc/C1C___00_01/
longueur de chaine =11
new_s=Cãë0
new_s=C1ãë0
new_s=C1Cë0
new_s=C1C_0
new_s=C1C__
new_s=C1C___
new_s=C1C___0
new_s=C1C___00èãë0
new_s=C1C___00_ãë0
new_s=C1C___00_0ãë0
new_s=C1C___00_01ë0
mon new_s=C1C___00_01ë0

et j'aimerais que new_s = C1C___00_01 à la fin !?

Merci encore pour vos conseils !!

[gerald3d]

Sans vraiment te donner une réponse précise peut être devrais-tu reprendre ton code en pensant "à l'envers".

Je m'explique. Tu disposes d'un char* complet. Pars de la fin avec un while et lis chaque caractère. Tant que le caractère lu n'est pas "/" tu continues. Dés que tu l'atteint tu connais l'adresse+1 du début de ta chaîne à copier/ Il te suffit ensuite de faire un simple fin_de_chaine-debut_de_chaine pour connaître la longueur à allouer.
Tout ceci n'est que du principe. Il existe pas mal de fonctions dans la glib (lib GTK+) te permettant de travailler tout ca assez facilement si tu utilises ces librairies.

[pepito31]

mmh ... peut etre oui, mais j'aimerais quand meme bien comprendre pourquoi je n'arrive pas a extraire cette chaine depuis ma chaine principale ?

[cheveche4]

Pour te répondre... Je cherche mais je ne trouve aucune erreur...
Juste que personnellement, j'aurais fait un pointeur 'c' qui aurait pointé sur (s+start/4) ("/4" vu qu'un int fait 4 octets et qu'un char en fait 1...) et qui aurait été incrémenté au fur et à mesure de la chaine (ce qui au final en théorie n'aurait pas vraiment changé grand chose...)

[tamiel]

Regarde du côté de la fonction strrchr

[Sve@r]

Merci pour vos reponses.
Bon, ca ne marche pas.
J'ai essayé d'ajouter en effet le "\0" au new_s, en fin de boucle for mais je receptionne toujours mon signe cabalistique à la fin de ma chaine ?!
voici le code modifié de la routine str_sub:

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
26
char *str_sub (const char *s, unsigned int start, unsigned int end)
{
   char *new_s = NULL;

   if (s != NULL && start < end)
   {
      new_s = malloc (sizeof (*new_s) * (end - start + 2));
      if (new_s != NULL)
      {
         int i;
         for (i = start; i <= end; i++)
         {
            new_s[i-start] = s[i];
         }
	 strcat(new_s,"\0"); // Ajout d'un retour chariot en fin de chaine 
      }
      else
      {
         fprintf (stderr, "Memoire insuffisante\n");
         new_s=NULL; // retour de pointeur NULL si depassement memoire
      }
   }
   printf("new_s=%s\n",new_s); // a ce stade, j'ai toujours "new_s="C1C___00_01," au lieu de C1C___00_01 !?? 
   return new_s;
}

Putain c'est quoi cette horreur ??? Et il n'y a personne parmi tous les intervenants qui a été capable de voir cette monstruosité ???

Rien que le commentaire associé est déjà assez édifiant !!!

Code :

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

new_s[i-start]='\0';

Ensuite tu devrais te mettre à étudier sérieusement le fonctionnement des strings en C...

Pour te répondre... Je cherche mais je ne trouve aucune erreur...

Ben cherche encore !!!

personnellement, j'aurais fait un pointeur 'c' qui aurait pointé sur (s+start/4) ("/4" vu qu'un int fait 4 octets et qu'un char en fait 1...)

Et personnellement tu serais allé droit au mur vu qu'un pointeur sur char, même si le pointeur (l'adresse) est codé sur 4 octets, quand tu passes à l'adresse suivante cela te place sur l'octet suivant (voir arithmétique des pointeurs dans ton cours de C). C'est comme dans la rue. Ce n'est pas parce que tu passes du 155 rue machin au 156 rue machin que t'as avancé de 3 maisons (en faisant abstraction des n° pairs/impairs de nos rues françaises) !!!

[tamiel]

Un exemple qui pourra t'aider (trouvé dans le dernier article C de ce site ...) :
http://melem.developpez.com/langagec...e=libc#LVI-E-3

[gerald3d]

Deux possibilités peut être :

1. la variable end compte n+1 caractères. Donc dans ta boucle for il faut "<end" et non "<=end".
2. char* est codé en utf8. Je n'y crois pas mais bon au point où tu en es.

[teuf13]

Pour faire de l'extraction de chaînes délimitées par des '/', le code sera exactement le même que ta chaîne soit en UTF-8 ou en iso8859-1. C'est un des avantages de l'UTF-8