Discussion :

[alfred5]

Bonjour tt le monde;
j'ai fait un petit programme qui fait la comparaison entre 2 valeurs, s'ils sont égaux il retourne 1 sinon il retourne 0; voila le prog :

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
#include <iostream>
using namespace std;
class Valeur{
      public:
             int x,y;
             Valeur(int v1=0):x(v1){}
             int egale(Valeur V);
};
int main(){
    Valeur a,b;
    a=1;
    b=1;
    a.egale(b);
    system("pause");
}
int Valeur::egale(Valeur V){
    if(x == V.x){
         return 1;
    }else {
          return 0;      
    }
}

lorsque je compile mon programme, il me retourne rien !!
SVP de m'aider à corriger l'erreur .

[Joel F]

faudrait voir a afficher quelque chose :E

[koala01]

Salut,

Heu, tu as déjà de la chance que cela compile, grâce au fait que ton constructeur agisse comme un opérateur de conversion, car, dans bien des cas, a = 1 te posera quelques problèmes (a est de type Valeur, et dispose de deux entiers, alors que 1 est associé à... un (et un seul) entier )

Ceci dit, egal renvoie bel et bien quelque chose... Simplement, le programme n'est pas prévu pour te le faire savoir: si tu lui demandais, tout simplement d'afficher le résultat, tu verrais qu'il effectue bel et bien la fonction egal :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
int main()
{
 
    Valeur a(1);
    Valeur b(1);
    Valeur c(2); //pour voir une différence :D
 
    cout<<"a egal b :"<<a.egale(b)<<endl
        <<"a egal c :"<<a.egale(c);
    system("pause");
}

Ceci étant réglé, je me permet quelques remarques générales:

1. L'instruction system a l'énorme inconvénient de ne pas être portable, car elle va faire appel à des applications présentes sur le système... Et pause n'existe pas sous linux par exemple ... tu peux te reporter à cette entrée de la faq pour savoir comment faire une pause portable
2. Je sais que le langage autorise la déclaration de plusieurs variables sur une seule ligne en les séparant par des virgules, mais je ne suis pas partisan de ce genre de pratique: il devient trop facile de "louper" la déclaration de l'une d'elles, et on peut se méprendre sur ce qui est fait lorsque l'un est un pointeur
3. tu crées un constructeur pour ta structure, pourquoi ne pas l'utiliser (cf mon code) L'idéal est de toujours, pour autant que faire se peut, reporter la déclaration de la variable jusqu'à ce que tu sois en mesure de l'initaliser directement (RAII inside).
4. L'idéal est de transmettre les structures aux fonctions par références, constantes si la fonction ne doit pas modifier la variable obtenue par ce biais... Autrement, tu observe une copie de la variable transmise qui peut demander énormément de ressources, tant au point de vue de la mémoire que du point de vue du temps (bien que ce ne soit pas forcément le cas ici)
5. Lorsque tu crées une fonction membre qui ne modifie pas l'objet courent, l'idéal est de déclarer cette fonction constante, de manière à prévenir le compilateur que la fonction s'engage à ne pas modifier l'objet courent... Cela n'a l'air de rien, mais seules les fonctions membres constantes peuvent être appelées au départ d'objets constants (bien qu'elles puissent, aussi, être appelées au départ d'objets non constants)... Cela participe à une meilleure sécurisation et porte le nom anglais de "const correctness"

[alfred5]

OK merci bcp