Partage
  • Partager sur Facebook
  • Partager sur Twitter

Language c

besoin d'aide

23 mars 2020 à 11:08:58

Bonjour
Nombre d'occurance de x dans un tableau . 
Probleme resulta toujour =1
#include<stdio.h>
#include <stdlib.h>
#define MAX 100
int tab [MAX];
  remplir (int tab[] , int *n,int *x)
  { int i;
   do 
         {
         printf("donnez la taille du tableau: \n ");
         scanf ("%d",n);
          } while (!( *n>0 && *n<MAX));
          for (i=1;i<=*n;i++)
          {
         printf("donnez t[%d] : ",i);
         scanf ("%d",&tab[i]);
  }
  printf("donnez x ");
         scanf ("%d",x);
  }
 int occurance (int  tab[] , int n ,int x,int i,int *occ=0)
 { 
   while ((i!=n))
   {
   if (tab[i]=x)  {occ++ ;}
   return  occurance (tab,n,x,i+1);
   }
   return (*occ)++; ;
 }
 int main ()
{  int n,x,*occ;
remplir(tab,&n,&x);
 printf("%d",occ);
return 1 ;
}

-
Edité par KarimZrouga 23 mars 2020 à 12:45:29

  • Partager sur Facebook
  • Partager sur Twitter
23 mars 2020 à 11:16:57

Bonjour,

Le message qui suit est une réponse automatique activée par un membre de l'équipe. Les réponses automatiques leur permettent d'éviter d'avoir à répéter de nombreuses fois la même chose, ce qui leur fait gagner du temps et leur permet de s'occuper des sujets qui méritent plus d'attention.
Nous sommes néanmoins ouverts et si vous avez une question ou une remarque, n'hésitez pas à contacter la personne en question par Message Privé.

Pour plus d'informations, nous vous invitons à lire les règles générales du forum

Merci de colorer votre code à l'aide du bouton Code

Les forums d'Openclassrooms disposent d'une fonctionnalité permettant de colorer et mettre en forme les codes source afin de les rendre plus lisibles et faciles à manipuler par les intervenants. Pour cela, il faut utiliser le bouton Code de l'éditeur, choisir un des langages proposés et coller votre code dans la zone prévue. Si vous utilisez l'éditeur de messages en mode Markdown, il faut utiliser les balises <pre class="brush: cpp;">Votre code ici</pre>.

Merci de modifier votre message d'origine en fonction.

Mauvais titre

Le titre est un élément important qui ne doit pas être négligé. N'oubliez pas cette règle simple : le titre idéal résume la question que vous allez poser en une petite phrase. Il doit permettre aux visiteurs de se repérer facilement dans le forum visité et d'identifier le sujet à sa seule lecture.

Vous pouvez utiliser divers préfixes comme [Erreur], [MySQL], [Compatibilité], etc... Aussi, pensez à consulter les règles propres à chaque forum (visibles dans les topics épinglés en haut des sections).

De plus, choisir un bon titre permet de rendre plus faciles les recherches des autres membres.

Les titres de type "besoin d'aide" ou "problème" ne sont pas tolérés.

Pour modifier votre titre, éditez le premier message de votre sujet.

(titre originel : Language c)

Liens conseillés

  • Partager sur Facebook
  • Partager sur Twitter
23 mars 2020 à 13:45:15

Bonjour.

Premièrement, ta fonction 'remplir' ne possède pas de type. Je te conseilles de lui donner un type (pourquoi pas un void vu qu'elle ne retourne rien). Dans cette même fonction, lorsque tu demandes la valeur de x, tu dois envoyer l'adresse de x à scanf. Faudra donc écrire :

scanf("%d",&x);

Deuxièmement, tu n'appelles jamais ta fonction 'occurance' dans le programme. C'est donc normal que tu n'obtiennes pas ce que tu espères. Dans cette même fonction tu initialises le paramètre occ à 0. En C ,ce n'est pas possible (sauf si je ne suis pas à jour).

J'ai également le pressentiment que tu n'as pas compris l'utilisation des pointeurs, rien qu'en voyant ta fonction 'occurance'. Quel intérêt d'avoir un pointeur sur occ si c'est pour après retourner sa valeur ?

Pour revenir à ton problème, tu devrais écrire ta fonction main comme ceci :

int main(){
    int n, x, occ=0;
    int i=0;//Tu as oublié de déclarer i. Même si le mieux serait de le faire à l'intérieur de la fonction occurance et retirer la récursivité
    int *tab = malloc(n*sizeof(int));//Si tu comprends pas, supprimes la variable n et déclare un tableau normalement
    remplir(tab,n,x);//Enlèves les pointeurs sur n et x, à mon avis ils servent à rien
    printf("%d",occurance(tab,n,x,i,occ));
    return 0;//Je sais pas pourquoi tu as mis 1 mais je vais pas te blâmer
}


-
Edité par Abel Rudy 23 mars 2020 à 13:45:55

  • Partager sur Facebook
  • Partager sur Twitter
23 mars 2020 à 20:22:10

Abel Rudy a écrit:

Dans cette même fonction, lorsque tu demandes la valeur de x, tu dois envoyer l'adresse de x à scanf. Faudra donc écrire :

scanf("%d",&x);

Non, x est déjà un pointeur, pas besoin de donner l'adresse d'une adresse. Et tu aurais dû faire le même commentaire pour n ;)

@KarimZrouga: ne me dis pas que ton compilateur n'a pas donné une erreur pour int *occ=0 !

-
Edité par edgarjacobs 23 mars 2020 à 20:22:51

  • Partager sur Facebook
  • Partager sur Twitter

On écrit "j'ai tort", pas "tord" qui est le verbe "tordre" à la 3ème personne de l'indicatif présent