besoin d'avis sur ce code

15 réponses

did

04/12/2003 à 21:53

Salut
j'aimerai avoir vos avis sur ce code. Je me doute qu'il est pas
terrible. Dites moi ce qu'il ne va pas. Normalement, il devrait compiler ;)
J'ai zappe les parties ne concernant pas le C standard (enfin je crois :) )

did

ps: le but de ce programme est de creer un ordonnanceur, par le biais
d'une liste circulaire...

--------------------------------

#include <stdio.h>

#define NB_PID_MAX 100
#define PID_T 6

typedef struct liste_s {
struct liste_s* suiv ;
int pid ;
} ordo_s ;

int main(void)
{
int nb_pid = 0, i=0 ;
int nb_maillon_crees = 0 ;
ordo_s* liste, *debut_liste ;
int tab_pid[NB_PID_MAX] ;
char pid_name[PID_T] ;
FILE* fic ;

if ((fic = fopen("pid.txt", "r")) == NULL) {
perror("erreur") ;
return -1 ;
}

while(1) {
while (fgets(pid_name, PID_T, fic)) {
tab_pid[i] = atoi(pid_name) ;
i++ ;
nb_pid++ ;
}

if (nb_pid == 0) {
int c = getchar() ;
}
else {
liste = (ordo_s*) malloc(sizeof* liste) ;
liste->pid = tab_pid[nb_maillon_crees] ;
liste->suiv = NULL ;
nb_maillon_crees++ ;
if (nb_maillon_crees == 1) {
debut_liste = liste ;
}
while(nb_maillon_crees < nb_pid ) {
liste->suiv = (ordo_s*) malloc(sizeof* liste) ;
liste->suiv->pid = tab_pid[nb_maillon_crees] ;
nb_maillon_crees++ ;
liste = liste->suiv ;
}
}
}

liste->suiv = debut_liste ;
i=0 ;
while (i<=nb_maillon_crees) {
printf("%d\n", debut_liste->pid) ;
debut_liste = debut_liste->suiv ;
i++;
}

return 0 ;
}

5 réponses

1 2

ZeGrunt

06/12/2003 à 14:31

Salut.

did wrote:

Bernard Bandiera a écrit:
je pense que si on t'a répondu, contrairement à ce qu'a vécu notre
cher , c'est que tu as d'abord cherché, et proposé ton
code. Si ce cher cyber (ou cryptono) avait fait un minimum
d'effort, il n'aurait pas eu les réponses qu'il a eues...

En conclusion (là, je parle d'expérience) : même si on est
débutant, même si on se trompe, même si on n'est pas très bon, il
suffit d'amorcer la pompe (bosser un peu par soi même) pour que les
réponses affluent. Et faut pas se priver, il y a ici des tas de
gens compétents qui veulent bien prendre de leur temps temps à
aider les autres. Seulement, faut pas les prendre pour des c***s...

Mais j'ai prit personne pour un con moi... :/

Je pense que Bernard voulait dire que tu avais eu le bon comportement
(travail personnel d'abord, question ensuite) et que c'est pour ça que
tu avais eu des réponses contrairement à cryptono qui avait demandé des
réponses sans avoir montrer un minimum de réflexion personnelle.

En bref, c'est cryptono qui a pris les gens d'ici pour des cons pas toi. ;)

A+

ZeGrunt.

did

06/12/2003 à 14:51

ZeGrunt a écrit:

Je pense que Bernard voulait dire que tu avais eu le bon comportement
(travail personnel d'abord, question ensuite) et que c'est pour ça que
tu avais eu des réponses contrairement à cryptono qui avait demandé des
réponses sans avoir montrer un minimum de réflexion personnelle.

En bref, c'est cryptono qui a pris les gens d'ici pour des cons pas toi. ;)

A+

ZeGrunt.

ah ok. Je suis rassure. Bon, je retourne a mon pitoyable programme... :)

Bernard Bandiera

06/12/2003 à 17:35

did écrivait dans le message :

ZeGrunt a écrit:

Je pense que Bernard voulait dire que tu avais eu le bon
comportement (travail personnel d'abord, question ensuite) et que
c'est pour ça que
tu avais eu des réponses contrairement à cryptono qui avait demandé
des réponses sans avoir montrer un minimum de réflexion personnelle.

En bref, c'est cryptono qui a pris les gens d'ici pour des cons pas
toi. ;)

A+

ZeGrunt.

ah ok. Je suis rassure. Bon, je retourne a mon pitoyable programme...
:)

Re-salut.
désolé si je me suis mal exprimé, c'est en effet dans le sens de la réponse
de ZeGrunt qu'il fallait me lire...

Bon courage pour ton "pitoyable" programme (les miens le sont sans doute
aussi parfois malgré quelques années de métier...)
--
Bernard

Gloire à qui, n'ayant pas d'idéal sacro-saint
Se borne à ne pas trop emmerder son voisin
Georges Brassens - Don Juan

did

06/12/2003 à 18:11

Bernard Bandiera a écrit:

Re-salut.
désolé si je me suis mal exprimé, c'est en effet dans le sens de la réponse
de ZeGrunt qu'il fallait me lire...

Bon courage pour ton "pitoyable" programme (les miens le sont sans doute
aussi parfois malgré quelques années de métier...)

l'exercice en lui meme est tres interessant, mais le programmeur (moi)
est pas tres doue de ses doigts :)
Merci :)

Bernard Bandiera

06/12/2003 à 18:26

did écrivait dans le message :

Bernard Bandiera a écrit:

Re-salut.
désolé si je me suis mal exprimé, c'est en effet dans le sens de la
réponse de ZeGrunt qu'il fallait me lire...

Bon courage pour ton "pitoyable" programme (les miens le sont sans
doute aussi parfois malgré quelques années de métier...)

l'exercice en lui meme est tres interessant, mais le programmeur (moi)
est pas tres doue de ses doigts :)
Merci :)

Boh, tu sais, être doué ou pas c'est pas forcément rédhibitoire. Si tu es
motivé, que tu bosses, que tu as un minimum de logique, avec de l'expérience
tu peux construire des programmes qui tiennent la route. Et ça quel que soit
le langage.
Donc : bon courage !
--
Bernard

Gloire à qui, n'ayant pas d'idéal sacro-saint
Se borne à ne pas trop emmerder son voisin
Georges Brassens - Don Juan

1 2

besoin d'avis sur ce code

5 réponses

Veuillez sélectionner un problème