Discussion :

[chris7522]

Bonjour a toutes et a tous ,
Je n'arrive pas a resoudre ce probleme d' erreur de segmentation . Merci de votre aide .

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
#include <iostream>
#include <string>
using namespace std;
int const N=5;
 
int compare (const void * a, const void * b) {
  return ( *(int*)a - *(int*)b );
}
 
int main ()
{
 int valeurs[N];
 
 cout <<"Vous avez "<< N << " essai pour trouver le nombre mystere " << endl;
 
 for(int i=0,nb=5;i<N;i++,nb--) 
 {
  cout <<"Entrez une valeur entre 0 et 50 :" << endl;
  cin >> valeurs[i];
  cin.ignore();
  cout <<"Il vous reste " << nb-1 << " nombre a entrer" << endl;
 }
 
  int element=0;
  srand(time(nullptr));
  element = rand()%50;
 
  int *pos;
  pos = (int*) bsearch (&element, valeurs, 7, sizeof (int), compare);
 
  cout <<"Le nombre magique etait : " << *pos << endl;
  if ( pos != NULL )
    cout << *pos << " est dans votre choix" << endl;
  else
    cout << element << " n'est pas dans votre choix" << endl;
 
  return 0;
}

[jo_link_noir]

Ce n'est absolument pas du C++, mais du C. En C++ il existe std::lower_bound et std::sort ; bien plus sûr que qsort et bsearch. De plus, bsearch et std::lower_bound ne fonctionnent qu'avec des séquences triées, ce qui ne sera absolument pas le cas avec des donnés utilisateurs.

rand est aussi à remplacer par ce qui se trouve dans l'en-tête <random>.

[chris7522]

Merci beaucoup d'avoir pris le temps de repondre a mon petit probleme . Je vais essayer de trouver des exemples pour apprendre a m'en servir .
Pour en revenir a mon probleme de segmentation , je pense que l'erreur vient de ma boucle for . Qu'en penses tu ?

[jo_link_noir]

Non, c'est votre utilisation de bsearch qui est mauvaise. Ceci ne serait pas arrivé avec une utilisation correcte des classes et fonctions C++.

[Bousk]

Code :

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

1
2
3
4
5
int const N = 5;
...
int valeurs[N];
...
bsearch(,,7,,);

Sinon, utiliser cout et cin ne suffisent pas à faire du C++.
std::find
std::find_if

[chris7522]

Tu as raison . Il n'y a pas trop d'interet a faire du C++ a la C .☺