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 !?!