surcharge d'une fonction par une methode

**sprunk** · 19/10/2010, 14h58

Bonjour à tous,

Malgré mes recherches, je n'ai pas trouvé la solution précise à mon problème. J'ai bien trouvé quelques pistes s'en rapprochant, mais je ne les ai pas très bien comprises.

Mon but est d'implémenter une méthode random "static int random(int)", privée, à l'intèrieur de ma classe "MyObject". Cette méthode utilise la fonction random "long random()" de la librairie standard.

voila le code:

MyObject.h

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
 
#ifndef MYOBJECT_H_
#define MYOBJECT_H_
 
class MyObject
{
 private:
  int   _id;
 
 public:
  MyObject();
 
 private:
  static int    random(int);
};
 
#endif  // MYOBJECT_H_

MyObject.cpp

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
 
#include <stdlib.h>
#include "MyObject.h"
 
MyObject::MyObject()
{
  // appel de la methode MyObject::random
  _id = random(42);
}
 
int     MyObject::random(int max)
{
  // appel de la fonction random de la lib standard
  return ((int)(random() % max));
}

main.cpp (sait-on jamais :p)

Code :

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

1
2
3
4
5
6
7
8
 
#include "MyObject.h"
 
int             main(void)
{
  MyObject      foo();
  return (0);
}

Qui me donne le message d'erreur suivant à la compil' :

> In static member function `static int MyObject::random(int)':
> error: no matching function for call to `MyObject::random()'
> note: candidates are: static int MyObject::random(int)

Je n'ai pas trouvé la manière de dire que la fonction qui doit être appelée, est celle de la librairie standard et non la méthode. La signature de la fonction ne suffisant apparement pas.

Merci d'avance pour votre aide

**omc24** · 19/10/2010, 15h09

Bonjour, la fonction random de la lib standart n'est-elle pas plutôt rand() ?

**sprunk** · 19/10/2010, 15h11

Envoyé par omc24

Bonjour, la fonction random de la lib standart n'est-elle pas plutôt rand() ?

Je ne crois pas, mais je me trompe peut être. Toujours est-t-il que je veux bien utiliser la fonction random() -> (cf man 3 random).

**seeme** · 19/10/2010, 15h20

Avec un #include <cstdlib> ?

**sprunk** · 19/10/2010, 15h31

Envoyé par seeme

Avec un #include <cstdlib> ?

Toujours pas, même problème. Dommage.

**omc24** · 19/10/2010, 15h39

Heu... Je persiste...
Mieux vaut utiliser rand(), qui est le "portage" C++ de random() de la libc.

**sprunk** · 19/10/2010, 15h46

Envoyé par omc24

Heu... Je persiste...
Mieux vaut utiliser rand(), qui est le "portage" C++ de random() de la libc.

Code :

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

1
2
3
4
5
6
7
8
#include <cstdlib>
#include <iostream>
 
int     main(void)
{
  std::cout << random() << std::endl;
  return (0);
}

Chez moi ca marche, et c'est bien cette fonction que je veux utiliser. Même si rand() existe aussi, ca ne résoud pas mon problème

.

EDIT: je n'avais pas vu ton édit, oui tu as raison. Mais c'est quand même random() que je veux utiliser. Ou si tu préfères, le problème est le même si ma méthode s'appelle MyObject::rand() :p

**omc24** · 19/10/2010, 15h52

Code :

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

1
2
3
4
5
6
7
8
9
 
#include <cstdlib>
#include <iostream>
 
int main () 
{
  std::cout << rand() << std::endl;
  return 0;
}

Chez moi ça marche, mais effectivement ça ne résoudrait pas ton problème

. Typiquement tu as besoin des namespaces, cela permet justement d'éviter les conflits de nommage.

**sprunk** · 19/10/2010, 16h04

Il me semble que la classe rempli déjà ce rôle dans le cas de mon exemple. Non?

**Astraya** · 19/10/2010, 16h12

Bonjour,

Je n'ai pas trouvé la manière de dire que la fonction qui doit être appelée, est celle de la librairie standard et non la méthode

il n'y as pas de random() dans la std. Mais bien std::rand().
cf : <cstdlib>

Pour le code de ton main retire les parenthèses à ton foo, sinon tu dois avoir un avertissement sous visual studio du type:

warning C4930: 'MyObject ff(void)' : fonction prototypée non appelée (était-ce la définition de variable souhaitée ?)

Et ton objet ne sera pas crée.

**sprunk** · 19/10/2010, 16h27

Envoyé par Astraya

Bonjour,

il n'y as pas de random() dans la std. Mais bien std::rand().
cf : <cstdlib>

Je parlais bien de random (man 3 random), et le problème reste le même. cf les messages si dessus.

Envoyé par Astraya

Pour le code de ton main retire les parenthèses à ton foo, sinon tu dois avoir un avertissement sous visual studio du type:
Et ton objet ne sera pas crée.

Certes, mais tout le monde n'utilise pas visual

**Astraya** · 19/10/2010, 16h30

random (man 3 random)

C'est du standard C et pas du standard C++ ici:

et
http://en.wikipedia.org/wiki/Stdlib.h

En standard:

Code :

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

1
2
3
4
 
#include <cstdlib>
 
std::rand();

Standard c++ : http://www.cplusplus.com/reference/clibrary/cstdlib/

Quel compilateur utilises tu?

**sprunk** · 19/10/2010, 16h42

Envoyé par Astraya

C'est du standard C et pas du standard C++

Oui. Je sais, mais c'était un exemple. Si tu préfères, changes mon exemple en remplacant la méthode random(int) par rand(int) et la fonction appelée par rand(). Mon problème reste le même.

Envoyé par Astraya

Quel compilateur utilises tu?

gcc/g++. Mais là encore, je ne vois pas le rapport.

**Arzar** · 19/10/2010, 16h42

Il faut probablement lever l'ambigüité :

Code :

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

1
2
3
4
5
int MyObject::random(int max)
{
  // appel de la fonction random de la lib standard
  return ((int)(::random() % max));
}

:: permet d'indiquer au compilateur qu'il doit chercher random() dans le namespace global.

**sprunk** · 19/10/2010, 16h44

Envoyé par Arzar

Il faut probablement lever l'ambigüité :

Code :

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

1
2
3
4
5
int MyObject::random(int max)
{
  // appel de la fonction random de la lib standard
  return ((int)(::random() % max));
}

:: permet d'indiquer au compilateur qu'il doit chercher random() dans le namespace global.

Merci, c'est tout à fait ce que je cherchais. Ca compile et ca marche.

**Astraya** · 19/10/2010, 16h48

gcc/g++. Mais là encore, je ne vois pas le rapport.

Car cela compile sur mon compilateur.

Oui. Je sais, mais c'était un exemple. Si tu préfères, changes mon exemple en remplacant la méthode random(int) par rand(int) et la fonction appelée par rand(). Mon problème reste le même.

En C++ pour éviter justement les ambiguïté du ::random du C, la namespace standard à été créer, Il te faut mettre std::rand(). Ici aussi cela compilera et sera standard C++ et non du mélange C/C++.

A bon entendeur.

**sprunk** · 19/10/2010, 16h53

Envoyé par Astraya

En C++ pour éviter justement les ambiguïté du ::random du C, la namespace standard à été créer, Il te faut mettre std::rand(). Ici aussi cela compilera et sera standard C++ et non du mélange C/C++.

Je comprends mieux avec ces précisions. Cependant, je cherchais la réponse à cet exemple précis, avec l'utilisation de random() et je ne connaissais pas la possibilité d'utiliser le "::" de cette manière.

Sinon j'aurais effectivement pu utiliser std::rand() ou plus simplement changer le nom de ma méthode

Merci pour les précisions.

**Chessmaster1966** · 20/10/2010, 08h19

Je n'ai pas eu le temps de lire tous les messages alors peut-être que je répète une solution.

Deux points sont à considérer dans ton code qui génère des messages d'erreur :
1) Tu as oublier dans le fichier ".cpp" de mettre le mot clé 'static' alors que tu as déclaré la fonction 'random' comme ceci : static int random dans le fichier d'entête.

2) C'est la fonction 'rand' du standard qu'il faut utiliser. Tu utilises la fonction 'random' qui est ta propre fonction donc il y a un appel récursif extrêmement dangereux.

Conclusion :
Les messages d'erreurs sont générés seulement parce que tu as oublié le mot clé 'static' dans le fichier source et non par la fonction 'random' qui existe bien puisque c'est celle que tu as créée.

**Jean-Marc.Bourguet** · 20/10/2010, 09h11

1/ random() vient de POSIX, pas de C ni de C++.
2/ pour reduire l'intervalle, voir la FAQ (ou la FAQ C, je sais plus)
3/ meme probleme que quand on ne trouve pas la definition d'une fonction dans une classe de base quand on a surcharge le nom dans une classe derivee: les classes sont des portees et quand on cherche les definitions d'un identificateur, on s'arrete a la premiere portee qui le fournit, donc ici la classe et on ne va pas voir dans le namespace global. Pour utiliser celle du namespace global, il faut donc le qualifier: ::random().

**Astraya** · 20/10/2010, 13h08

random() vient de POSIX, pas de C ni de C++.

pourrais tu m'éclaircir se point s'il te plais? je n'ai pas l'habitude de travailler avec C et donc je ne connais pas trop ses librairies. POSIX est une libraire de C et pas du standard?

surcharge d'une fonction par une methode

C++

Discussions similaires

Partager

Partager