Aide avec les fork()

**Metalman** · 28/03/2013, 12h30

omg omg...
J'avais aussi zappé le fgets....

Vérifie si dans les caractères que tu as lu, tu as bien de l'alphanumérique....

man ascii avec des if/then/else pour recopier tout sauf les caractères que tu ne veux pas...

EDIT : le '\n' final doit trainer, du coup le strcmp de ls renvoi un "non" ou des trucs dans ce genre là...
Pas besoin de rediriger les flux, par défaut le fils écrira aussi dans le même STDOUT que le père (héritage des fd ouverts)

**Raikyn** · 28/03/2013, 13h21

Comment je fais pour vérifier ça ?

**imperio** · 28/03/2013, 13h27

Code :

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

char **mots = malloc(i + 1);

Et voici le meilleur moyen au monde de segfaulter ! La macro sizeof existe bien pour une bonne raison !

**Raikyn** · 28/03/2013, 13h29

Il faudrait mettre sizeof(char)*(i+1) tu veux dire ?

**imperio** · 28/03/2013, 13h35

Il faudrait mettre sizeof(char)*(i+1) tu veux dire ?

Non plus ! Petite leçon sur la taille des variables et pourquoi il faut toujours utiliser sizeof dans un malloc :

char -> 1 octet
char* -> 4 octets (sur 32 bits) / 8 octets (sur 64 bits)

Conclusion, quand tu fais un malloc et que tu alloues 3 octets (par exemple) à ton tableau de pointeur, tu n'as même pas la place pour en avoir un seul en fait ! Ce serait équivalent a un char[3] (je grossis les bords).

Pour éviter ce problème, on fait toujours (tout du moins du moins on devrait !) :

Code :

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

1
2
 
void *ptr = malloc(sizeof(*ptr) * (taille_tableau));

**Raikyn** · 28/03/2013, 13h39

Ok merci ^^

**Metalman** · 28/03/2013, 14h08

Essaye déjà tout ça, puis si ça ne marche toujours pas, vérifies les arguments que tu envoies.

Le bon vieux :

Code :

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

1
2
3
4
5
 
if ((str[i] < ' ') || (str[i] > '~'))
  /* caractère pas affichable */
else
  /* caractère affichable */

Te permettra de recopier ce qu'il faut ! (et un grand "(!(if...))" pour inverser la condition évidemment)

**Raikyn** · 28/03/2013, 15h32

Je ne comprend pas, cela me permettra-t-il de vérifierr la validité d'un mot avant de le mettre dans mon tableau ?

**Metalman** · 28/03/2013, 15h40

Non... vas lire la table ASCII...
Ca te permet de savoir si c'est un caractère affichable ou non...
Du coup tu vas recoder un strdup qui va copier le caractère SI et SEULEMENT SI il est affichable... et tu vas le placer dans ton code pour copier les arguments correctement

**Raikyn** · 28/03/2013, 16h48

Désolé je n'y arrive pas ><

J'avais trouvé une fonction pour remplacer un char (pour remplacer le "\n" par un "\0"), mais ça me fait un segmentation fault :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void chng(char *buff, char old, char new)
{
 char *ptr;
 for(;;)
 {
  ptr = strchr(buff, old);
  if(ptr==NULL) break; 
     buff[(int)(ptr-buff)]=new;
 }
 
}

**Metalman** · 28/03/2013, 17h03

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
 
char *my_strdup_alphanum(char *str)
{
  int max, j, i = 0;
  char *tmp = NULL;
 
  if (str == NULL)
    return (NULL);
 
  max = 0;
  while (str[i] != '\0')
  {
     if ((str[i] => ' ') && (str[i] <= '~'))
        max++;
     i++;
  }
 
  if (max == 0)
    return (NULL);
 
  tmp = malloc((max + 1) * sizeof (char));
  i = 0;
  j = 0;
  while (str[i] != '\0')
  {
     if ((str[i] => ' ') && (str[i] <= '~'))
     {
        tmp[j] = str[i];
        j++;
     }
     i++;
  }
  tmp[max] = '\0';
  return (tmp);
}

**Raikyn** · 28/03/2013, 17h42

Whow, ça marche impecc !

Je vais reprendre mes cours et faire des td plus simples sur les chars avant de continuer je crois bien !

Merci metalman (et tous les autres).

**Metalman** · 28/03/2013, 18h58

Le plus intéressant c'est en effet la partie avec les interfaces système, vu que cela réagi "vraiment" avec les périphériques

Mais pour travailler avec il faut bien comprendre les typages, pointeurs.... et le fonctionnement des chaînes de caractère.

Enfin t'inquiète pas trop... comme "exercices" je peux te conseiller de recoder la plupart des fonctions de la libC (toutes les str* les strn*... atoi, itoa...), les tester avec tout ce que tu peux, et passer un coup de valgrind dessus pour vérifier qu'il n'y a aucun problème de mémoire.

Ensuite, tu pourras faire le fou avec toute la partie système et réseau !
(et l'excellent livre avec des exemples qui marchent : "Linux : Programmation Système et Réseau")

**Raikyn** · 31/03/2013, 11h37

Merci pour la référence ^^ Ca va bien m'aider pour mon cours de système et réseau.

Sinon j'avais une petite question, là j'essaye de donner le temps d'exécution d'une commande. Je place donc un premier clock() pour récupérer l'heure au départ, et un second à la fin pour l'arrivée, puis je fais la différence des deux.
J'ai toujours un temps de 0 donc il y a un problème quelque part. Comment je dois plcer mes clock() ? Pas dans le fils j'imagine ?
J'ai mis le 1er avant le parsing et le second dans ma boucle avec le wait().

**Metalman** · 31/03/2013, 12h54

Dans un vieux minishell à moi que j'ai retrouvé...
Il y a ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
  is_fils = fork();
  if (is_fils == 0)
  {
    ret = the_exec(tab); /* appel a execvp */
    if (ret != 0)
      exit(127);
  }
  else
  {
    fils = wait(&status);
    ret = WEXITSTATUS(status);
  }

Tu dois mettre ton clock après le wait(); et avant le fork(); !
Hmmm... mais si ta différence reste à 0.... quel type utilises-tu pour ta clock ?

EDIT : et n'oublies pas :
"La valeur renvoyée est le temps CPU écoulé, en unités d'horloge clock_t, pour obtenir une durée en secondes, divisez-la par CLOCKS_PER_SEC. "

**Raikyn** · 31/03/2013, 13h00

D'accord, alors j'ai fais :

Code :

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

clock_t t1, t2;

Je les ai placé comme tu me l'a dis, mais j'ai toujours 0, ça vient peut-être du type de la différence ?

Code :

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

1
2
3
4
wait();
      t2 = clock();
      temps = (float)(t2-t1)/CLOCKS_PER_SEC;
      printf("\n Temps d'exécution : %f\n",temps);

**Metalman** · 31/03/2013, 13h11

Hmmm...

Essaye :

Code :

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

1
2
3
 
  t2 = (t2 - t1) / CLOCKS_PER_SEC;
  printf("\n Temps d'exécution : %f\n", (long) t2);

De ce que je vois, le type "clock_t" n'est pas très très défini...
D'autre sont conseillé ça :

Code :

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

  t2 = (double) (t2 - t1) / (double) CLOCKS_PER_SEC;

**Raikyn** · 31/03/2013, 13h16

Toujours pareil, sauf qu'avec long j'ai qu'un 0 alors qu'avant c'était 0.00000.

Je vais essayé de trouver une autre fonction que clock peut-être.

**Metalman** · 31/03/2013, 14h07

gettimeofday !

Code :

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

1
2
3
4
5
6
7
 
long                  res;
struct timeval        tv;
 
gettimeofday(&tv, NULL);
res = tv.tv_sec;
printf("long : %ld\n", res);

Ce que je peux ajouter : si tu veux construire un char* avec, il te faudra un itoa qui gère les long....
Personnellement j'ai du le recoder avec le flag ansi...

**Raikyn** · 31/03/2013, 14h42

C'est quoi comme format ça ?
J'ai des chiffres du genre 1364733626 à l'exécution de ce morceau de code.

Aide avec les fork()

C

Discussions similaires

Partager

Partager