Discussion :

[laracroft87]

salut tout le monde :

SVP est ce qu'il y a quelqu'un qui peut m'aider ,d'abord comment je remet une chaîne de caratères a NULL . elle est désactivée cette ligne dans le source.


//for(int r=0;r<20;r++) chaine=NULL;**********************


et 2iément si vous pouvez me rectifier mon programme
car j'ai essayé pas mal de fois, pas d'erreurs de compilation, mais erreur de segmentation pendant l' exécution, je ne comprends pas d'où viens ça.
merci d'avance

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
51
52
53
54
#include <stdio.h>
#include <string.h>
#include <unistd.h>
#include <stdlib.h>
 
int main ()
{//mettre les lien de "PATH" dans le tableau tlien
   char *path = getenv("PATH");
   char *p,*tampon;
   p = strtok_r(path,":",&tampon);
   char **tlien[10];int k=0;
   while (p != NULL)
   { 
      tlien[k]=&p;
      //printf("%s\n",*tlien[k]);
      k++;
      p = strtok_r(NULL,":",&tampon);
   } //lire la ligne de commande et mettre chaque commande dans le tableau tab;
   char comand[80],*buffer,*c,sep[]="\t;|\n";
   printf("my shell >>");
   scanf("%[^\n]",&comand);
   c = strtok_r(comand,sep,&buffer);
   char **tab[10];int i=0;
   while (c != NULL)
   {
      tab[i]=&c;
      //printf("%s\n",c);
      //printf("%s\n",*tab[i]);
      i++;
      c = strtok_r(NULL,sep,&buffer);
   }
   //vérification de l'existance des commandes tapées
   int j=0,s=0,acces=1;
   char chaine[20];
   while (j != i)
   {
      while ((s != k)&&(acces == 1))
      {
         strcat(chaine,*tlien[j]);
         strcat(chaine,"/");
         char *p,*buf,*dup;
         dup = strdup(*tab[s]);
         if (dup==NULL) dup = *tab[s];
         p = strtok_r(dup," ",&buf);
         strcat(chaine,p);
         printf("%s\n",chaine);
         if (access(chaine,F_OK)==0) acces= 0; //la commande existe
         //for(int r=0;r<20;r++) chaine=NULL; *************************
         }
      if (acces == 0) { j++; s=0;acces = 1;}
      else {printf("commande inexistante \n");break;}
   }
   return 0;
}

[Médinoc]

Code C :

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

1
2
for(int r=0;r<20;r++)
    chaine[r]='\0';

[rilou]

Code C :

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

1
2
for(int r=0;r<20;r++)
    chaine[r]='\0';

En effet, sinon la fonction :

Code :

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

void *memset(   void *dest,   int c,   size_t count );

marche relativement bien il me semble (en tout cas je n'ai jamais eu de problème avec).

[crocodilex]

Est-ce qu'il ne suffit pas de faire :

Code :

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

chaine[0] = '\0';

?

[laracroft87]

re

merci à vous ,moi j'ai utiliser la fontion : strcpy(chaine,""); je ne sais pas est ce que ça marche ou pas ,

le pblme que j'ai c'est que pas d'erreurs de compilations,mais erreur de segmentation , et ça se provoque au niveau du premier appel de strcat je ne sais pas pquoi ?? si vous pouvez m'aider sachant bien que je travaille sous linux .

et ce qu'il y'a une autre fonction qui peut me servir ,et qui m'evite ce genre de pblme .

merci encore une fois

[souviron34]

merci à vous ,moi j'ai utiliser la fontion : strcpy(chaine,""); je ne sais pas est ce que ça marche ou pas ,

oui ça marche, et c'est strictement équivalent à ce qu'a écrit crocodilex.

Il faut cependant être conscient que ça n'initialise pas l'ensemble des caractères. Pour ça, c'est soit memset (le plus court) soit une boucle.

• Déjà, outre le fait que déclarer les variables en plein milieu du code est illisible, sauf si c'est dans des blocs (entourés par des accolades), ce n'est autorisé que en C99, et c'est rigoureusement interdit avant.
  
  
• Ensuite, main est une fonction qui prend soit 0 argument, auquel cas on la déclares int main(void), soit 2 arguments, auquel cas on la déclare int main(int argc, char **argv)

Je n'ai regardé le reste qu'en diagonale, bien qu'il me semble qu'il y aie un certain nombre d'erreurs majeures (dont char **tlien[10] mais un effort de présentation serait souhaitable tout d'abord.

[rilou]

re

merci à vous ,moi j'ai utiliser la fontion : strcpy(chaine,""); je ne sais pas est ce que ça marche ou pas ,

le pblme que j'ai c'est que pas d'erreurs de compilations,mais erreur de segmentation , et ça se provoque au niveau du premier appel de strcat je ne sais pas pquoi ?? si vous pouvez m'aider sachant bien que je travaille sous linux .

et ce qu'il y'a une autre fonction qui peut me servir ,et qui m'evite ce genre de pblme .

merci encore une fois

Un erreur de segmentation viens du fait que tu dois surement dépasser la capacité de ton tableau.

Déjà, outre le fait que déclarer les variables en plein milieu du code est illisible, sauf si c'est dans des blocs (entourés par des accolades), ce n'est autorisé que en C99, et c'est rigoureusement interdit avant.

En effet, c'est pas bien beau tout ça :p

De plus essaye de donner des "NOM" a tes variable on comprendrai mieux ce qu'elles doivent faire :p

Aussi j'ajouterais que tu dois avoir la possibilité de "virer" quelques variables, j'imagine que t'en a une ou deux que tu n'utilise qu'une fois ?


PS : Est-on sûr que le STRCPY soit vraiment plus rapide et compréhensible qu'un bon vieux chaine[0]='\0' ??

[gl]

le pblme que j'ai c'est que pas d'erreurs de compilations,mais erreur de segmentation , et ça se provoque au niveau du premier appel de strcat je ne sais pas pquoi ??

chaine n'est pas initialisée, il y a donc peu de chance que strcat() trouve un '\0' d'où un probable débordement de buffer.