mauvaise creation d'objet

Version imprimable

salutation à tous je suis super novice en C++ et apparament y'a un truc qui cloche au niveau de la création de l'objet mm

Code:

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
39
40
41
42
43
44
45
#include <string>
#include <iostream>
class MM
{
    public:
      int som1;
      int n1;     

    MM::MM(){
    som1=0;
    n1=0;
    }
    double fact(int n)
{
     if (n > 0) return n*fact(n-1);
     else return 1;
}
int puiss (int x,int n2)
{
    if (n2>0) return x*puiss(x,n2-1);
    else return 1;
}
};
   int main()
{
   
   MM mm();
    int b,c,y,a,i;
    int tab[100];
   printf("donner la valeur de a\n ");
   scanf("%i",&a);
   printf("donner la valeur de b\n");
   scanf("%i",&b);
   printf("donner la valeur de c\n");
   scanf("%i",&c);
   y=a/b;
   for (n1=0 ; n1<s ; i++){
   tab[i]=mm.puiss(y,n);
   
   
    system("PAUSE");
    
    return EXIT_SUCCESS;
}

ce qui est marqué en gras c'est la partie qui pose probléme ...
merci

Citation:
Envoyé par onlytime
salutation à tous je suis super novice en C++ et apparament y'a un truc qui cloche au niveau de la création de l'objet mm
Code:

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 39 40 41 42 43 44 45 #include <string> #include <iostream> class MM { public: int som1; int n1; MM::MM(){ som1=0; n1=0; } double fact(int n) { if (n > 0) return n*fact(n-1); else return 1; } int puiss (int x,int n2) { if (n2>0) return x*puiss(x,n2-1); else return 1; } }; int main() { MM mm(); int b,c,y,a,i; int tab[100]; printf("donner la valeur de a\n "); scanf("%i",&a); printf("donner la valeur de b\n"); scanf("%i",&b); printf("donner la valeur de c\n"); scanf("%i",&c); y=a/b; for (n1=0 ; n1<s ; i++){ tab[i]=mm.puiss(y,n); system("PAUSE"); return EXIT_SUCCESS; }
ce qui est marqué en gras c'est la partie qui pose probléme ...
merci
Première chode, ton constructeur de être déclaré con suit:
Code:

1 2 3 4 MM(){ som1=0; n1=0; }
La syntaxe MM::MM() ne vaut que lorsque que tu déclare le constructeur en dehors de celle-ci.
Ensuite, pour instancier un objet, il faut faire:

Code:

MM mm;

sans parenthès.
Enfin, pour ta boucle, je ne comprends pas ce que tu veux faire
- d'ou viens ce n1???
- d'ou viens le n que tu utilise juste après???

En esperant que ça aide :D

19/01/2008, 01h21
Pacorabanix
ça n'a pas de rapport avec le problème, mais juste pour compléter : en C++ on utilisera plutot :
Code:

1 2 std::cout << "Donnez la valeur de a" << std::endl; std::cin >> a;
à la place de :
Code:

1 2 3 printf("donner la valeur de a\n "); scanf("%i",&a);
19/01/2008, 10h44
Invité

Hello Onlytime,

Pour t'aider à comprendre ce qui ne va pas, regarde à nouveau ta ligne MM mm(); - elle ressemble comme deux gouttes d'eau à une déclaration de fonction. En écrivant cela, tu déclares effectivement une fonction qui s'appelle "mm", qui ne prend pas de paramètre, et qui renvoie un objet de type "MM". Relis le message d'erreur du compilateur, qui devrait te sembler plus clair maintenant.

Donc pour construire un objet avec le constructeur sans paramètre, ne pas mettre de parenthèses.

Cheers,

Carl
19/01/2008, 10h54
zais_ethael

Pour faire court: oui.
Ce n'est pas évident pour un débutant de comprendre l'utilité de cette nouvelle syntaxe un peu déconcertante, on se sent souvent mieux avec son bon vieux printf. Mais les streams apportent des avantages non négligeables.
D'ailleurs, de manière générale, il est fortement déconseillé d'utiliser des fonctions C depuis du C++ tant que tu n'as pas quelques années d'expérience. Mélanger les deux sans prendre moulte précautions peut facilement rendre les programmes instables pour des raisons que tu n'imagines même pas au début. Le meilleur réflexe du débutant (et que très peu ont) est de systématiquement chercher une biblio pure C++ dés que tu veux faire quelque chose, même si tu connais une fonction C qui saurait le faire.
19/01/2008, 12h31
flyweight

Alors c'est résolu ?
19/01/2008, 18h35
Mat007
Salut,
Citation:
Envoyé par bountykiler

Première chode, ton constructeur de être déclaré con suit:

Code:

1 2 3 4 MM(){ som1=0; n1=0; }
En fait il devrait même utiliser la liste d'initialisation :
Code:

1 2 3 4 MM() : som1( 0 ) , n1( 0 ) {}
MAT.
19/01/2008, 19h16
Davidbrcz

Il devrait pour plein de raisons.
Mais il débute, alors bon pour le moment , ne pas utiliser la liste d'initialisation pour 2 entiers ce n'est pas vraiment grave.

Quand il sera plus expérimenté, il comprendra pourquoi il faut mieux l' utiliser, pour 2 entiers.
20/01/2008, 01h51
coyotte507

Citation:

Envoyé par Davidbrcz

Quand il sera plus expérimenté,il comprendra pourquoi il faut mieux utiliser la liste d'initialisation,même pour 2 entiers.

Salut,
je ne suis pas un expert mais ça m'intéresse, c'est optimisé de le mettre dans la liste d'initialisation?

coyotte507
20/01/2008, 09h02
zais_ethael

Pour deux entiers non, ou alors seulement en mode debug.
Pour des classes oui. Cela s'explique par le fait que quand tu n'utilises pas la liste d'initialisation c'est d'abord le constructeur par défaut qui est appelé. Par exemple pour un std::string, c'est plus rapide de lui passer un const char* dans le constructeur prévu à cet effet plutot que d'appeler son constructeur par défaut qui va initialiser une chaine vide puis son opérateur d'affectation prenant un const char* (un seul new dans le premier cas, deux dans le second pour le même résultat).
Bien entendu, certaines classes ne possèdent pas de constructeur par défaut, dans ce cas il est alors indispensable d'utiliser la liste d'initialisation sous peine que ça ne compile jamais.
20/01/2008, 17h54
coyotte507

ok merci beaucoup
20/01/2008, 18h40
onlytime

J'ai du éffectuer quelques corrections au niveau des fichiers qui sont les suivants:
1)le premier au nom MM.h qui contient le nom de mes methodes avec leurs paramétres seulment.

2)le second MM.cpp qui contient les methodes developpées avec leurs code

3)le troisiéme qui est le main.cpp qui est le programme principale qui utilise les différentes methodes...etc

pour la declaration de l'objet j'ai juste utilisé MM();
pour le créer MM mm;
merci à tous...
21/01/2008, 00h41
Invité

Citation:

Envoyé par onlytime

3)le troisiéme qui est le MM.main qui est le programme principale qui utilise les différentes methodes...etc

Oups, mauvaise idée, ça! Tu vas te mettre à dos tous tes outils de développement. Si ton fichier contient du code C++, mieux vaut lui donner l'extension ".cpp". Main.cpp me paraît un bon choix.

Carl
21/01/2008, 18h24
onlytime

Oui c'est vrai,erreur de frappe c'est "main.cpp" effectivement