Erreurs dont "expected expression before ‘)’ token" dans mon fichier source

[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 :

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 :

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 :

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 :

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

[ok.Idriss]

Envoyé par mohsenuss91

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 :

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);
}

[Neckara]

Je pense qu'on souhaite afficher le nom du programme qui n'a pas pu être lancé ie buf.

[mohsenuss91]

Envoyé par Neckara

Je pense qu'on souhaite afficher le nom du programme qui n'a pas pu être lancé ie buf.

voila ou je suis arrivé mais une fois le programme exécuter je remarque rien de spéciale!!

Code :

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);
}

[Neckara]

Peux-tu être un peu plus précis?

"Rien de spécial" ne nous avance pas tellement pour comprendre ce qui ne va pas.

[edgarjacobs]

Code :

1
2
3
4
5
...
if((pid==fork())<0) // --> if((pid=fork())<0)
...
if((pid==waitpid(pid,&status, 0))<0) // --> if((pid=waitpid(pid,&status, 0))<0)
...

Non?

[ok.Idriss]

Si c'est bien des opérateurs d'affectation qu'il faut utiliser. Bien vu, ça m'avait échappé

Sinon je constate que certaines remarques que Neckara et moi-même avons émises ne sont toujours pas prises en compte.