à l'aide, j'ai un problème dans un programme

Gwenalizee -  
varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   -
Bonjour,
J'ai fait un programme pour pouvoir comparer 3 nombres définis qui est le suivant

#include <stdio.h>
#include <math.h>

int Comparaison (int X, int Y)
{
int A;
if (X>Y)
{
X=A;
}
else
{
Y=A;
}
return A;
}

int Comparaison2 (int X, int Y, int Z)
{
int A;
Comparaison (X, Y);
if (A>Z)
{
printf ("Le plus grand nombre est %d.\n", A);
}
else
{
printf ("Le plus grand nombre est %d.\n", Z);
}
return 0;
}

int main ()
{
int X, Y, Z;
X=23;
Y=236;
Z=4;
Comparaison2 (X, Y, Z);
return 0;
}


Et il m'affiche comme résultat que le plus grand nombre est 4

Je ne trouve pas où est la faille. On pourrait m'aider, s'il vous plait?
Merci d'avance

18 réponses

Résumé de la discussion

Le souci principal réside dans une variable A non initialisée et une attribution incorrecte dans Comparaison, qui ne renvoie pas le maximum de X et Y mais une valeur indéterminée. Dans Comparaison2, le résultat de Comparaison n’est pas capté dans A et la comparaison s’effectue sur une valeur non initialisée, ce qui conduit à afficher 4 comme plus grand. La correction consiste à affecter A = Comparaison(X, Y) et à comparer ensuite A avec Z pour afficher le maximum réel, c’est-à-dire que A doit représenter le maximum de X et Y avant la comparaison avec Z. Le fil évoque aussi brièvement d’autres sujets (conversion majuscule/minuscule, usage de fonctions séparées pour le PGCD), mais le problème central porte sur la gestion des valeurs et le flux de retour entre les fonctions.

Bobot (l'IA à votre service)
  1. loupius Messages postés 789 Statut Membre 148
     
    int A;
    Comparaison (X, Y);

    Il manque quelque chose ;)
    Bonne réflexion.
    0
    1. Gwenalizee
       
      Oui , mais qu'est-ce qui manque?
      0
  2. scaravenger Messages postés 2691 Date d'inscription   Statut Membre Dernière intervention   625
     
    Revois ta variable locale A. Ya comme un problème de transmission de valeur.
    0
    1. Gwenalizee
       
      Je ne vois pas trop. il faut que je change le premier ou le deuxième A?
      0
    2. scaravenger Messages postés 2691 Date d'inscription   Statut Membre Dernière intervention   625
       
      Pour simplifier ce n'est pas le "même A".
      Conserve une seule variable LOCALE A et renomme la deuxième variable LOCALE différemment (B par exemple).
      Ton erreur te sera peut etre plus flagrante.
      0
    3. Gwenalizee
       
      C'est ce que j'ai fait mais cela me donne toujours 4 comme réponse alors que c'est faux.
      0
    4. scaravenger Messages postés 2691 Date d'inscription   Statut Membre Dernière intervention   625
       
      Bon j'ai pitié pour toi
      corrige tes ligne erronées par:
      A=X; (et non X=A;)
      A=Y;
      A=Comparaison (X, Y);
      Ca devrait marcher.
      0
    5. Gwenalizee
       
      Merci!
      0
  3. Gwenalizee
     
    Sorry, j'ai un autre problème: en fait je dois transformer un caractère qui est en majuscule en minuscule et inversement. Je n'ai que le mode minuscule à majuscule

    #include <stdio.h>
    #include <math.h>
    #include <stdlib.h>

    void minuscule(char s[])
    {
    int mot;
    /* la chaîne est parcourue par index */
    for (mot= 0; s[mot] != '\0'; mot++)
    /* si le caractère à la position index est majuscule*/
    if (s[mot] >= 'A' && s[mot] <= 'Z')
    /* le remplacer par son équivalent en minuscule */
    s[mot]= s[mot] - 'A' + 'a';
    }

    int main ()
    {
    int salut;
    minuscule(salut);
    return 0;
    }


    Il ne veut pas marcher. Il y a quoi comme erreur?
    0
  4. scaravenger Messages postés 2691 Date d'inscription   Statut Membre Dernière intervention   625
     
    C'est une question académique ou estudiantine?
    Etant un peu fainéant j'aurais essayé "strlwr" et "strupr"
    qui font tout par simple appel de fonction.
    0
    1. Gwenalizee
       
      En fait c'est une série d'exos que je fais seule, parce que je suis pas trop douée pour le langage C.
      Et comment tu utilises "strlwr" et "strupr" ?
      0
    2. Bobleponge
       
      RTFM !
      Tape "man strlwr" et "man strupr" sur Google ;p
      Autre question, je vois que tu fais tout a la main.. A la mode Epitech.. Tu t'autorises a utiliser les fonctions systemes ou tu fait tout toute seule ?
      Un moyen simple de convertir "a la main" une chaine en majuscule/minuscule et inversement, c'est de passer par la table ASCII (man ascii sur google ;D)
      0
    3. Gwenalizee
       
      Je fais tout toute seule ou j'utilise d'autres fonctions
      J'ai ajouté #include <string.h>
      mais j'ai comme message
      fonction main
      ligne 20: warning passing argument 1 of 'minuscule' makes pointer from interger without a cast
      je ne comprends pas trop pourquoi il y a ce warning
      0
    4. Bobleponge
       
      C'est normal regarde ton code :

      int main ()
      {
      int salut;
      minuscule(salut);
      return 0;
      }

      Et le prototype de ta fonction :

      void minuscule(char s[]);

      Ta fonction indique qu'elle attend un tableau de char comme argument (donc une chaine de caractère..)
      Or, dans ton main, tu déclares une variable qui s'appelle salut (que tu n'initialises jamais déjà) mais qui est surtout du type int...
      Tu envoies donc a ta fonction un entier au lieu d'une chaine de caractère..
      Comprends-tu ton erreur et comment y remédier ?
      0
    5. Gwenalizee
       
      et si je mets

      int main ()
      {
      char salut;
      char tab[];
      minuscule(salut);
      return 0;
      }

      est-ce qu'il faut que je mette quelque chose dans char tab?
      0
  5. Vous n’avez pas trouvé la réponse que vous recherchez ?

    Posez votre question
  6. Gwenalizee
     
    Donc je remets après plusieurs modifs

    #include <stdio.h>
    #include <math.h>
    #include <stdlib.h>
    #include <string>

    void minuscule(char s[])
    {
    int mot;
    /* la chaîne est parcourue par index */
    for (mot= 0; s[mot] != '\0'; mot++)
    /* si le caractère à la position index est majuscule*/
    if (s[mot] >= 'A' && s[mot] <= 'Z')
    /* le remplacer par son équivalent en minuscule */
    s[mot]= s[mot] - 'A' + 'a';
    printf ("%s", s);
    }

    int main ()
    {
    char SALUT;
    char tab["SALUT"];
    minuscule(tab["SALUT"]);
    return 0;
    }


    c'est bon? Ou j'ai pas encore compris dans la fonction main?
    0
    1. Gaunts Messages postés 1390 Statut Membre 116
       
      Non reprend ce que tu avais avant, et regarde bien uniquement la première ligne de ton main.
      char A=SALUT.

      Chico a dit un peu plus haut qu'un tableau devait être entourée de guillemets.
      Il a aussi précisé que tu ne déclares qu'un char, qui ne représente qu'un caractère alors qu'il te faut un tableau.
      Suis les deux conseils qu'il t'a donné, et essaye de changer cette ligne pour qu'elle soit fonctionnelle.
      0
  7. chico200987 Messages postés 856 Statut Membre 144
     
    Nooooooooooooon :(

    On est revenu au même point qu'hier...

    Gwenalizee.. Prends le temps de lire calmement ce que je (on maintenant ?) t'ecris(/t) .. Et prends le temps de réfléchir sans vouloir tout essayer..

    Reprends ton dernier code :

    int main ()
    {
    char A=SALUT;
    minuscule(A);
    return 0;
    } 
    


    Relis ce que Gaunts et moi t'avons écris, et reposte la bonne réponse :p
    0
  8. Gwenalizee
     
    Je suis vraiment pas douée (sorry), bon alors

    int main ()
    {
    char A=SALUT;
    char tab["A"];
    minuscule(tab["A"]);
    return 0;
    }


    c'est bon?
    0
    1. chico200987 Messages postés 856 Statut Membre 144
       
      Comme l'a dit Gaunts sous ton message, tu n'as que la ligne

      char A=SALUT;
      


      qui est fausse dans ce code :

      int main ()
      {
         char A=SALUT;
         minuscule(A);
         return 0;
      } 
      


      Pour rappel, encore une fois, tu n'as besoin de rajouter AUCUNE ligne, nous attendons juste la déclaration un tableau de chaine de caractère a la place de celle d'un caractère que tu appelleras toujours A.. Et pour affecter une chaine, a un tableau de caractère tu dois entourer sa valeur par des guillemets..

      Courage courage !
      0
    2. Gaunts Messages postés 1390 Statut Membre 116
       
      Quelle aventure
      0
    3. chico200987 Messages postés 856 Statut Membre 144
       
      J'te l'fais pas dire :p
      0
    4. Gwenalizee
       
      Ok, je réessaie:

      int main ()
      {
      char A=tab["SALUT"];
      minuscule(A);
      return 0;
      }


      c'est bon?
      0
    5. chico200987 Messages postés 856 Statut Membre 144
       
      Presque..

      Encore une fois, regarde le prototype :

      void minuscule(char s[]) 
      


      Il faut que ce soit EXACTEMENT le même type déclaration de ce qu'il y a entre les parenthèses..

      Comme dans mon exemple tout a l'heure on avait :

      afficherNombre(int nb)

      et dans le main : int monChiffre = 5

      Des deux cotes on avait "int"

      Ici on doit non seulement avoir char a gauche.. mais aussi autre chose a droite, de la même manière qu'entre les parenthèses :)

      Et tableau, ne veut pas forcement dire qu'il faut rajouter tab quelque part, cela veut juste dire crochet ici ;)
      0
  9. Gwenalizee
     
    Yep, ça marche. (et la fonction majuscule, il faut que j'inverse (A=>a, Z=>z, a=>A)

    J'en ai un autre de problème (sorry de vous déranger pour ça)
    alors voilà je dois faire le PGCD de deux nombres que l'on peut choisir

    #include <stdio.h>
    #include <math.h>
    #include <stdlib.h>

    int PGCD (int X, int Y)
    {
    int z;
    printf ("Choisir deux nombres entiers X et Y, de telle façon que X>Y.\n");
    scanf ("X=%d\n Y=%d\n", &X, &Y);
    z=X/Y;
    printf("Le PGCD de X=%d et de Y=%d est %d\n", X, Y, z);
    return PGCD (X, Y);
    }

    int main ()
    {
    int X;
    int Y;
    PGCD (X, Y);
    return 0;
    }

    je prends X=12346 et Y=22 et on me dit
    "Floating point exception"
    ça veut dire quoi?
    0
  10. chico200987 Messages postés 856 Statut Membre 144
     
    Chez moi ton code m'écris :

    Choisir deux nombres entiers X et Y, de telle fan que X>Y.
    Le PGCD de X=123456 et de Y=22 est 5611

    A l'infini !

    Tu sais pourquoi ?

    Parceque tu écris dans ton return : return PGCB (X,Y)

    Ce qui veut dire que tu rappelles ta fonction infiniment, une fois arrivé au return, tu retournes en haut redemander les valeurs etc.. et donc ca fait une boucle infinie..

    Ce phénomène s'appelle la récursivité, tu peux en avoir besoin parfois, mais pas ici !

    Quand une fonction fini bien, comme dans le main, tu dois retourner 0 :)

    Par contre, tu ne trouves pas le PGCD la.. tu fais une simple division, l'algo n'est pas bon !
    0
    1. Gwenalizee
       
      Bah en fait qu'en j'utilise que le main, je n'ai pas soucis parce que je met ça:

      int main ()
      {
      int X=12346;
      int Y=22;
      int z;
      z=X/Y;
      printf ("Le PGCD de X=%d et de Y=%d est %d.\n", X, Y, z);
      return 0;
      }

      Or le problème je dois utiliser une fonction (autre que le main). Et dans cette fonction on nous demande de choisir deux valeurs.
      Et je ne vois pas où est mon problème (à part celui que tu m'as dis, chico200987).
      0
    2. Gaunts Messages postés 1390 Statut Membre 116
       
      Il y a plusieurs façons de calculer le PGCD de deux nombres..
      Pour celle que j'aurais utilisé si je veux faire un programme simple mais pas forcément optimisé, il faut uniquement savoir que le PGCD est le plus grand entier possible pour lequel a%PGCD = 0 et b%PGCD = 0
      0
    3. chico200987 Messages postés 856 Statut Membre 144
       
      Gwenalizee, le code que tu donnes dans ton main seul ne donne pas le PGCD, il fait simplement la division de X par Y..

      Tu peux le remarquer toi même tu marques :

      z=X/Y; 


      La piste de Gaunts est bonne.. Réfléchis a l'algorithme qui va avec ;)
      0
    4. Gwenalizee
       
      Arg non, je me suis trompée (la gourde)
      Je viens de le modifier:

      #include <stdio.h>
      #include <math.h>
      #include <stdlib.h>

      int PGCD (int X, int Y)
      {
      int X1;
      int Y1;
      X=X1;
      Y=Y1;
      printf ("Choisir deux nombres entiers X et Y, de telle façon que X>Y.\n");
      scanf ("X=%d\n Y=%d\n", &X, &Y);
      int z;
      while (z==0)
      {
      int r;
      z=X/Y;
      r=X%Y;
      printf("%d=%d*%d+*d\n", X, Y, z, r);
      X=z;
      Y=r;
      }
      printf("Le PGCD de X=%d et de Y=%d est %d\n", X1, Y1, z);

      return 0;
      }

      int main ()
      {
      int X;
      int Y;
      /*int X=12346;
      int Y=22;*/
      PGCD (X, Y);
      return 0;
      }


      mais il m'affiche comme warning "too man y arguments for format"
      ça veut dire quoi?
      0
    5. chico200987 Messages postés 856 Statut Membre 144
       
      Alors deja pour l'erreur ""too man y arguments for format" " ce la veut dire littéralement "Trop d'argument pour le format"

      Si tu regardes la ligne du Warning, c'est celle-ci :

      printf("%d=%d*%d+*d\n", X, Y, z, r);    
      


      C'est juste une petite erreur d'inattention.. trouve l'erreur ;)

      Ensuite il y a plusieurs problèmes dans ton algo.. les vois-tu ?
      0
  11. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
     
    chico va me tuer, mais bon:

    Voici le code que tu devrais avoir, commenter pour que tu comprenne bien toute tes erreurs.
    Je me suis inspirer de l'explication de l'algorithme pour le faire et de ton code.
    Je n'ai pas de compilateur C donc il peut y avoir des petites erreurs toute bete.

    #include <stdio.h>
    #include <math.h>
    #include <stdlib.h>
    
    int PGCD ()
    {
    	int X1, X;
    	int Y1, Y;
    	printf ("Entrez X et Y.\n");
    	scanf ("X=%d\n Y=%d\n", &X, &Y);		//tu demande les valeur des deux nombres dont on veut calculer le PGCD
    	
    	X1 = X;			//tu initialise X1
    	Y1 = Y;			//Tu inialise Y1
    	int r;			// tu declare le reste r
    	do				// on execute ce qui suit en boucle
    	{
    		r = X1%Y1;	// on divise X1 par Y1 et on met le reste dans r
    		if (r != 0)	// si le reste n'est pas nul
    		{
    		X1=Y1;		// on met Y1 dans X1
    		Y1=r;		// on met le reste r dans Y1
    		}
    	}while (r>0);	// tant que le reste n'est pas nul
    	printf("Le PGCD de X=%d et de Y=%d est %d\n", X1, Y1, z);
    
    	return 0;
    }
    
    int main ()
    {
    	PGCD (X, Y);
    	return 0;
    } 
    
    0
    1. Gwenalizee
       
      Stop!
      Pourquoi faire un do suivi de while, ça n'aurait pas suffit un simple while?
      0
    2. chico200987 Messages postés 856 Statut Membre 144
       
      Tu n'avais pas l'droit varfendell :o
      0
    3. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      Je t'avais poster ça tout a l'heure.
      En effet tu as dit ne pas savoir comment initialiser ton reste r (enfin z tout à l'heure).
      Et bien l'avantage c'est qu'il va d abord rentrer dans la boucle, et dans cette boucle r sera initialisé.

      en gros:

      int r; 
      
      do 
      { 
       r = 4; 
      } 
      while (r !=4) 
      


      fonctionne alors que

      int r; 
      
      while (r !=4) 
      { 
       r = 4; 
      } 
      
      

      ne marchera pas vu que r est inconnu

      edit:

      t'en fais pas chico, y a toujours qque erreur (comme dans le scanf)
      0
    4. chico200987 Messages postés 856 Statut Membre 144
       
      Le code de varfendell effectivement comporte effectivement quelques erreurs mais il marche dans l'ensemble.

      Par contre celui de Gaunts ne fonctionne pas :p
      0
    5. Gwenalizee
       
      Pourquoi r doit diminuer?
      0
  12. Gwenalizee
     
    Non, je suis paumée.
    On peut me ré-expliquer pourquoi Gaunts? fait diminué r.
    0
    1. chico200987 Messages postés 856 Statut Membre 144
       
      Pas étonnant que vous l'ayez paumé en donnant le code complet alors qu'elle galère avec les affectations !

      Donner la réponse a ce stade la ne peut que l'embrouiller et la pousser a apprendre le code par coeur sans comprendre une ligne.

      Gwen, sais-tu ce qu'est exactement le PGCD ?
      0
    2. Gwenalizee
       
      Bah c'est le plus grand diviseur commun à deux nombres choisis, pourquoi?
      0
    3. chico200987 Messages postés 856 Statut Membre 144
       
      Pour savoir comment tu procèderais A LA MAIN pour le calculer (sur des petits nombres)

      Ensuite, une fois que tu établis les règles qui te permettent de trouver le PGCD de ces deux nombres, il ne te reste plus qu'a essayer de l'adapter en code.
      0
  13. Gaunts Messages postés 1390 Statut Membre 116
     
    Non oublie mon code et mon r, c'était une autre technique et ce r était différent de celui que tu utilises. Garde le code complet de Varfendell, et essaye de bien décortiquer chaque étape, essaye de le faire marcher, et si tu as des questions pose les nous :).
    Si tu veux une fois que tu auras compris celui la, je reposterai ma méthode pour te l'expliquer.
    0
    1. Gwenalizee
       
      J'ai compiler le code de Varfendell et ça me donne que des erreurs (sorry pour Varfendell) et ça me donne ça:

      In function `PGCD':
      c:24: error: `z' undeclared (first use in this function)
      c:24: error: (Each undeclared identifier is reported only once
      c:24: error: for each function it appears in.)
      In function `main':
      c:31: error: `X' undeclared (first use in this function)
      c:31: error: `Y' undeclared (first use in this function)


      Je crois qu'il a oublié d'utilisé 'z' et il a oublié d'enlever dans le main X et Y.
      Mais je ne vois pas pourquoi il n'utilise pas 'z'.
      0
    2. Gaunts Messages postés 1390 Statut Membre 116
       
      C'est normal, son code n'était pas complet, il ne t'a montré que l'algo, et il y a quelques trucs à changer (il a codé sans pouvoir tester, c'est pas simple).
      Comme chico a dit, reprend le code que tu avais avant, et essaye de voir comment tu trouves un PGCD à la main. Quand on aura bien avancé comme ça, et que tu auras bien compri le fonctionnement du programme, on essaiera d'utiliser le code de Varfendell ainsi que le mien, pour te donner de nouveaux exemples à comprendre :), et on corrigera les erreurs que tu as obtenues avec son code.
      Mais pour l'instant garde ce que tu avais, et pense à l'algo que tu vas utiliser.
      Regarde la site que Varfendell t'avait donné, c'est assez bien expliqué
      0
    3. chico200987 Messages postés 856 Statut Membre 144
       
      J'ai supprime la correction !
      0
    4. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      Pour ces deux erreurs, enleve ce qui est dans les parenthese de PGCD et remplace z par r:

      ligne 24
      printf("Le PGCD de X=%d et de Y=%d est %d\n", X1, Y1, r);

      ligne 31
      PGCD (X, Y);

      Si tu n'as pas été capable de comprendre ces deux erreur, je te vois mal coder ne serait ce qu'un programme :s
      0
    5. Gwenalizee
       
      Bon, je reprends mon code:
      mon code marche pour le scanf (jusque là, ça va) mais après, il travaille (un peu trop d'ailleurs)
      donc c'est mon while qui me fait des misères, non?
      0
  14. Gaunts Messages postés 1390 Statut Membre 116
     
    Tu as toujours deux problêmes dont on avait déja parlé avant de partir sur l'algo:
    scanf ("%d %d\n", &X, &Y);

    Quelque chose ne va pas sur cette ligne, chico t'en avait parlé.
    Ensuite :
     int z;  
    int r;  
    while (z>0)  
    
    Ici toujours pareil, tu compares z à 0 alors qu'il n'a pas de valeur.
    Il faut soit lui en donner une avant le while, soit utiliser un do..while comme avait dit varfendell, que tu peux apprendre à utiliser sur le site que l'on t'a donné.
    0
    1. Gwenalizee
       
      Alors, j'ai mis une boucle do while et j'ai compiler, ça me fait un truc bizarre:
      ça me donne le contenu de chaque calcul mais je n'ai pas mon dernier printf contenant "Le PGCD de X et de Y est ..." et à la fin j'ai floating pion exception (je ne vois toujours pas ce qui pourrait poser problème avec ce float alors que j'en mets pas).
      0
    2. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      tu divise quelque pars par 0...et il aime pas (ça se comprend, moi non plus j'ai pas diviser par 0)
      0
    3. Gwenalizee
       
      Attends, c'est quand même pas mon r qui est égal à 0 et qui me divise par 0, si?
      0
    4. chico200987 Messages postés 856 Statut Membre 144
       
      Prends de petits nombres pour tester sur papier avec une calculette, et effectue le traitement de ton algorithme..

      Tu verras si un moment tu divises par 0 !
      0
    5. chico200987 Messages postés 856 Statut Membre 144
       
      Et pour te donner quelque chose de plus concret avec ton exemple 123456 et 22

      La derniere ligne que tu obtiens est : 6=2*3+0 avant ton exception.

      Ce qui veut dire que :

      X1 = 6
      Y1 = 2
      z = 3
      r = 0

      Ensuite tu fais :

       X1=Y1; 
       Y1=r; 
      


      Donc :

      X1 = 2
      Y1 = 0

      Puis tu remontes en haut puisque z > 3 et tu fais :

      z=X1/Y1; 
      


      Soit z = 2 / 0 ...

      Et ca.. il aime pas du tout !

      Edit : Ah lol Gaunts, je complétais ta réponse :p
      0
  15. Gwenalizee
     
    c'est bon, j'ai trouvé mais ça m'affichait toujours un truc, donc j'ai du mettre un break.
    Bon, bah je ne vais pas me reposer sur mes lauriers, j'ai déjà un nouveau code qui me pose problème: je dois faire la fonction factorielle de n.
    Il marche mais il ne m'affiche pas les résultats que je veux voir.
    J'envoie le code:

    #include <stdio.h>
    #include <math.h>
    #include <stdlib.h>

    /*Écrire un sous-programme en C pour calculer n!. Écrire alors l'appel.*/

    int Factorielle (int n)
    {
    if (n>1)
    {
    return n*Factorielle (n-1);
    }
    else
    {
    return 1;
    }
    return Factorielle (n);
    }

    int main ()
    {
    int n;
    printf ("Vous voulez faire factorielle de n=");
    scanf ("%d", &n);
    printf("\n");
    Factorielle (n);
    printf("\n");
    return 0;
    }
    0
    1. Gaunts Messages postés 1390 Statut Membre 116
       
      Avant de continuer, je pense que ton programme serait plus propre si tu initialisais les valeurs dans le main(), que tu fasses le calcul dans ta fonction PGCD puis que tu retournes le résultat pour l'afficher dans le main.
      0
    2. chico200987 Messages postés 856 Statut Membre 144
       
      Je confirme pour la fonction PGCD que tu devrais demander les numeros dans ton main..

      Sinon les déclarer avant et les envoyer vide ne sert a rien..

      De plus, un break dans une boucle.. Ça existe bel et bien et ca marche, mais c'est super moche !
      0
    3. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      Tu as plusieur probleme:

      tu doit faire une boucle:

      int Factorielle (int n)
      {
      int resultat = 1
      for (i = n; i >1; 1--)
      {
      resultat = n*Factorielle (n-1);
      }
      
      return resultat;
      }
      0
    4. Gwenalizee
       
      Pour ma fonction PGCD, c'est bon elle marche avec mon break, mais j'ai pas trouvé une autre solution que d'utiliser le break.
      @varfendell: j'ai modifié mon programme mais il y a un truc qui cloche, il n'arrête pas de m'afficher la question sans afficher le résultat.
      0
    5. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      montre moi ton code qui fait ça.
      0
  16. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
     
    Tu as un serieux probleme de logique mon pauvre

    essaye comme ça:

    #include <stdio.h>
    #include <math.h>
    #include <stdlib.h>
    
    /*Écrire un sous-programme en C pour calculer n!. Écrire alors l'appel.*/
    
    int Factorielle (int n)
    {
    	int i;
    	int resultat;
    	for (i=n; i>1; i--)
    	{
    		resultat=n*Factorielle (n-1);
    	}
    	return resultat;
    }
    
    int main ()
    {
    	int n;
    	printf ("Vous voulez faire factorielle de n=");
    	scanf ("%d", &n);
    	printf("\n");
    	int resultat = Factorielle (n);
    	printf("Le résultat est: %d\n", resultat);
    	return 0;
    }
    
    0
    1. scaravenger Messages postés 2691 Date d'inscription   Statut Membre Dernière intervention   625
       
      Si c'est une fille, qu'elle est blonde et débute en programmation, t'as pas fini de plancher sur ses problèmes.
      Au 2ème exo j'ai laissé tomber.
      0
    2. Gwenalizee
       
      @varfendell
      Désolée, de ne pas être assez logique.
      Je l'ai modifié de nouveau, mais il m'affiche -6. (je crois que la fonction factorielle n'affiche que des entier positifs, non?
      @scaravenger: je ne suis pas BLONDE, merci (pas besoin de t'imaginer que je suis ce genre de filles)
      0
    3. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      Perso une fille ou un mecs je m'en fiche.

      Enfin bref, quand tu dit que tu l'a modifié, tu parle de ton code qui ne marchait pas ou du code que je t'ai posté?
      0
    4. Gwenalizee
       
      non varfendell, c'est pas toi mais c'est scaravenger.
      Enfin bref, quand j'ai dit que je l'ai modifié, c'est que j'ai compilé ton code et ça m'a donné ce résultat là (-6).
      0
    5. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      tu as mis quoi en entré?
      0
  17. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
     
    Autant pour moi, j'avais oublié un petit truc, essaye ceci et calcul le factorielle de 5 (ça devrait te donner 120)

    #include <stdio.h>
    #include <math.h>
    #include <stdlib.h>
    
    /*Écrire un sous-programme en C pour calculer n!. Écrire alors l'appel.*/
    
    int Factorielle (int n)
    {
    	int i;
    	int resultat = 1;
    	for (i=n; i>1; i--)
    	{
    		resultat = n * Factorielle (n - 1);
    	}
    	return resultat;
    }
    
    int main ()
    {
    	int n;
    	printf ("Vous voulez faire factorielle de n=");
    	scanf ("%d", &n);
    	printf("\n");
    	int resultat = Factorielle (n);
    	printf("Le résultat est: %d\n", resultat);
    	return 0;
    }
    


    0
    1. chico200987 Messages postés 856 Statut Membre 144
       
      Pourquoi se compliquer avec un for ?

      Un simple if comme elle avait fait marche très bien... et c'est plus compréhensible pour elle...
      0
    2. Gwenalizee
       
      Par contre il ne marche plus à n=17 parce que je crois que la mémoire ou l'espace de calcul est trop petit(e). Je fais comment pour augmenter la mémoire de calcul?
      0
    3. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      il faut que tu change le type:
      Nom du type	Nombres stockables
      char 		-128 à 127
      int 		-2 147 483 648 à 2 147 483 647
      long 		-2 147 483 648 à 2 147 483 647
      float 		-3.4 x 10 puissance 38 à 3.4 x 10 puissance 38
      double 		-1.7 x 10 puissance 308 à 1.7 x 10 puissance 308
      0
    4. Gwenalizee
       
      je met double tout seul ou un double int?
      0
    5. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      tu met double tout seul
      0
  18. Gwenalizee
     
    Merci chico200987, il marche
    Alors voici le programme que je dois faire (enfin l'énoncé):
    je dois calculer la solution de x=cos (x) par la suite [je mets les indices]
    (grande accolade)
    X[0]=0
    X[i+1]=cos X[i]
    avec pour condition d'arrêt telle que valeur absolue de (X[i+1]-X[i]) est plus petit que ?.
    Je dois alors écrire l'appel.

    En fait là, je n'ai pas d'idée pour l'algo. Je ne vois pas comment démarrer.
    0
    1. chico200987 Messages postés 856 Statut Membre 144
       
      Là.. Je t'avoue que je ne comprends pas bien l'énoncé..

      C'est l'intitule exact ?
      0
    2. Gwenalizee
       
      Oui, c'est l'intitulé exact: en fait je dois calculer x=cosx par le biais d'une suite.
      cette suite commence avec X[0]=0 et on a X[i+1]=cosX.
      Jusque là, ça va?
      0
    3. chico200987 Messages postés 856 Statut Membre 144
       
      Yes jusque la ca va
      0
    4. Gwenalizee
       
      Mais il y a en plus une condition d'arrêt: (X[i+1]-X[i]) est plus petit que petit epsilon.
      ça va pour cette condition?
      0
    5. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      petit epsilon est une notation mathématique pour dire "tres petit", donc declare une variable epsilon que tu initialise a 0.00001 par exemple.

      Ensuite tu creer une boucle while ou tu effectue ton algorithme.

      Tu peut choisir avec la boucle while:

      #include <stdio.h>
      #include <math.h>
      #include <stdlib.h>
      
      /**/
      
      int CosEgaleX (int n)
      {
      	float epsilon = 0.0001;
      	float X = 0, temp;
      	while (diff > espilon)
      	{
      		temp = X;
      		X = cos(X);
      		diff = X - temp;
      	}
      	return X;
      }
      
      int main ()
      {
      	float resultat = CosEgaleX ();
      	printf("Le résultat est: %f\n", resultat);
      	return 0;
      }


      ou avec un boucle do{}while();
      0
  19. chico200987 Messages postés 856 Statut Membre 144
     
    Gwen on en est ou ?

    Il me reste une heure !
    0
    1. Gwenalizee
       
      Salut tout le monde et désolée de n'être pas assez présente ces temps-ci, il y a eu une quantité de boulot que je devais gérer.
      Alors j'ai réfléchi à programme et je crois qu'en fait je pense qu'il faut poser le problème doit se poser comme ça:
      comme i=0; X=0
      i=1; X=cos0=1
      i=2; X=cos1
      i=3; X=cos(cos1)
      ...
      Est-ce que tu vois comment le problème doit être résolu (et en fait je voulais mettre un i et non un X au départ pour demander à quel niveau on doit avoir le résultat)?
      Je refais des modifs et je reposte le programme
      0
    2. chico200987 Messages postés 856 Statut Membre 144
       
      Salut ! D'accord no soucy.

      J'attends ton post de ton code ou t'en es, je fais une petite sieste d'une demi-heure et je suis la entre deux lignes de codes (et oui moi aussi je dois taffer :p)
      0
    3. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      Ce que tu propose est de planifier le nombre d'itération...dans ce cas remplace le while par un for.

      Cependant ce que tu essaye ne sera pas en accord avec le sujet étant donné qu'il ne t'ai pas demandé de planifié le nombre d'itération, mais de planifié la marge d'erreur (epsilon)
      0
    4. Gwenalizee
       
      Je vois ce que tu veux dire varfendell, mais c'est en plus.
      Je pense que je vais le faire sous forme de tableau avec 1 ligne correspondant à i, une autre à la valeur de X et une dernière pour la valeur de epsilon.
      Bon j'y retourne le travailler, ce programme
      à toute!
      0
    5. varfendell Messages postés 3259 Date d'inscription   Statut Membre Dernière intervention   707
       
      Si tu veut faire les deux en meme temps c'est pas dur, il te suffit juste de garder ta boucle while, de rajouter comme condition de sorti iteration < i (i sera definit par toi) et d'initialiser itération à 0 et de l'incrémenter a chaque passage dans la boucle while.
      0