Discussion :

[mohsenuss91]

Voila un petit programme en C (tiré d'un support de cour) que j'essai de l’exécuter sous ubuntu 12.04 LTS.
Il m'affiche des erreurs et j'arrive pas a les comprendre.

Aidez-moi svp.

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
#include <stdlib.h>	// pour exit()
#include <unistd.h>	// pour getpid()
#include <signal.h> 	// pour sleep()
#include <sys/types.h>	// pour fork()
#include <stdio.h>	// pour printf()
int main()
{
	char buf[256];
	int pid;
	int status;
 
printf("%%");	// imprimer un prompt %
	while(fgets(buf,256,0)!= NULL)
	{
		buf[strlen(buf)-1]=0;
 
		if((pid==fork())<0)                                        // erreur de fork
			{	printf("Erreur de fork()");
				exit(127);
 
				break;
			}else if (pid == 0)                                     // Processus Fils
				{
                execlp(buf,buf,(char *)0);
				printf("%s n a pu etre executée\n",);
				exit(127);
			}
							      // Processus Père
				if ((pid==waitpid(pid,&status, 0))<0)
				{   	printf("erreur dans wait\n");
					Exit(127);
				} printf("%%");
 
	}
exit(0);
}

Code :

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

1
2
3
recouvrementdepss.c: In function ‘main’:
recouvrementdepss.c:15:7: attention : incompatible implicit declaration of built-in function ‘strlen’ [enabled by default]
recouvrementdepss.c:25:41: erreur: expected expression before ‘)’ token

Merci d'avance.

[Neckara]

Bonjour, ne manquerait-il pas string.h ?

[ok.Idriss]

Bonjour.

Un code indenté te permettrait d'y voir plus clair. Voici quelques remarques (voir les commentaires "CORRECTION") :

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
#include <stdlib.h>
#include <unistd.h>
#include <signal.h>
#include <sys/types.h>
#include <stdio.h>
#include <string.h> /* CORRECTION : include manquant comme l'a fait judicieusement remarqué Neckara */
 
int main(void){ /* CORRECTION : une fonction sans paramètre doit, en toute rigueur, mettre void entre parenthèse pour indiquer au compilateur qu'elle ne prend pas de paramètres */
	char buf[256];
	int pid;
	int status;
 
	printf("%%");	/* imprimer un prompt % */
	while(fgets(buf,256,0)!= NULL){
		buf[strlen(buf)-1]=0;
 
		/* erreur de fork */
		if((pid==fork())<0) {	
			printf("Erreur de fork()");
			exit(127);
 
			/* CORRECTION : break; => ne sert à rien */
		}else if (pid == 0){ /* Processus Fils */
			execlp(buf,buf,(char *)0);
			printf("%s n a pu etre executée\n",); /* CORRECTION : Oh une virgule qui se balade toute seule, d'où l'erreur ;) */
			exit(127);
		}
 
		/* CORRECTION : Dans la logique des choses, on met un else if ici */
		if ((pid==waitpid(pid,&status, 0))<0){   /* Processus Père */ 
			printf("erreur dans wait\n");
			Exit(127); /* Avec une majuscule ??? */
		} 
 
		printf("%%");
	}
	/* CORRECTION exit(0); => pourquoi exit ici, quand on peux retourner une valeur correspondant à la signature de main() ??? */
	return EXIT_SUCCESS;
}

Cordialement,
Idriss

[mohsenuss91]

comment régler l'erreur de la ligne 25 svp !

Code :

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

printf("%s n a pu etre executée\n",);

[ok.Idriss]

comment régler l'erreur de la ligne 25 svp !

Tu rigoles ou quoi ? T'as lu le commentaire ? Il manque un paramètre à printf ... faut faire davantage d'efforts que ça

[mohsenuss91]

voila le programme qu'en pensez vous !

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
#include <stdlib.h>
#include <unistd.h>
#include <signal.h>
#include <sys/types.h>
#include <stdio.h>
#include <string.h>
 
int main(void){
    char buf[256];
	int pid;
	int status;
 
	printf("%%");	/* imprimer un prompt % */
	while(fgets(buf,256,0)!= NULL){
		buf[strlen(buf)-1]=0;
 
		/* erreur de fork */
		if((pid==fork())<0) {
			printf("Erreur de fork()");
			exit(127);
            break;
		}else if (pid == 0){ /* Processus Fils */
			execlp(buf,buf,(char *)0);
			printf("%s n a pu etre executée\n",&buf[0]);
			exit(127);
		}
	else if((pid==waitpid(pid,&status, 0))<0){   /* Processus Père */
			printf("erreur dans wait\n");
			exit(127);
		}
 
		printf("%%");
	}
	exit(0);
}