Utilisation d'une classe aléatoire d'une manière statique

Question

Je fais une Random classe simple:

class Random 
{ 
public: 
    static bool seeded = false; 

    static void SeedRandom(int number) 
    { 
     srand(number); 
    } 
    static int GetRandom(int low, int high) 
    { 
     if (!seeded) 
     { 
      srand ((int)time(NULL)); 
     } 
     return (rand() % (high - low)) + low; 
    } 
}; 

Il est évident que C++ ne permet pas une classe entière à être déclarée comme static (qui est ce qui rend ce si facile en C#). J'ai plutôt fait tous les membres comme static. Il n'y a pas non plus de constructeur static donc je n'ai aucun moyen d'initialiser mon bool seeded sauf si j'appelle une fonction à la main, ce qui va à l'encontre du but. Je peux utiliser un constructeur normal dans lequel je devrais créer une instance de Random, ce que je ne veux pas faire.

Est-ce que quelqu'un sait si le nouveau standard C++ 0x autorisera des classes statiques et/ou des constructeurs statiques?

Answer 1

C++ ne permet pas de déclarer une classe comme statique

Bien sûr, il le fait. Vraiment, il n'y a aucune raison de mettre cela dans une classe. C++ ne vous force pas à tout mettre en classe - pour de bonnes raisons. En C# vous êtes obligé de tout mettre dans une classe et de déclarer les choses dans des méthodes statiques et autres, mais qui n'est pas idéomatique C++.

Vous ne pouvez vraiment pas simplement prendre le code C# idéomatique, et écrivez cela en C++, et attendez-vous à ce qu'il fonctionne bien. Ce sont des langages très différents avec des exigences et des caractéristiques de programmation très différentes.

Si vous voulez une méthode C++ idéomatique, ne créez pas de classe du tout. Appelez srand l'intérieur de votre main et définir une fonction qui fait votre serrage:

int RandomInteger(int high, int low) 
{ 
    return (std::rand() % (high - low)) + low; 
} 

EDIT: Bien sûr, il serait préférable pour vous d'utiliser la nouvelle installation de génération de nombres aléatoires et uniform_int_distribution pour obtenir votre gamme pincé au lieu de rand. Voir rand() considered harmful.

Answer 2

Votre static bool seeded devra de toute façon être défini dans un fichier cpp, et vous devrez l'initialiser là.

bool Random::seeded = false; 

Answer 3

Essayez ceci:

class Random 
{ 
public: 
    static bool seeded; 

    static void SeedRandom(int number) 
    { 
     srand(number); 
     seeded = true; 
    } 

    static int GetRandom(int low, int high) 
    { 
     if (!seeded) 
      SeedRandom(time(0)); 
     return (rand() % (high - low)) + low; 
    } 
}; 

bool Random::seeded; 

staticbool s initialisés à false par défaut, donc pas besoin de le faire explicitement. Notez que la logique de votre classe était également erronée, car vous n'avez jamais défini seeded sur true.

Answer 4

Vous pouvez toujours approcher ceci avec un motif singleton, en ayant le constructeur privé et un accesseur statique pour l'objet.

class Random 
{ 
public: 
    static Random& instance() 
    { 
    static Random instance; 

    return instance; 
    } 

    // your other functions here 

private: 
    Random() 
    { 
    // your seed code here 
    }  
}; 

Cela vous permettra de n'avoir une seule instance de la classe et chaque fois que vous avez besoin d'une fonction aléatoire, vous avez juste besoin d'appeler l'instance de Random().()

Answer 5

Quelle est la raison pour vous avoir une classe avec des méthodes statiques?Peut être qu'il existe un remplaçant pour atteindre votre objectif. Cherchez-vous quelque chose de similaire à singleton? BTW si vous avez des variables membres statiques dans votre déclaration de classe, comme indiqué par d'autres réponses avant moi, vous devez les initialiser en dehors de la classe (de préférence dans le fichier cpp) avant de les utiliser. Donc, si vous suivez ceci et initialisez votre comme ceci:

bool Random::seeded = false

il sera automatiquement initialisé à false comme vous voulez.

Answer 6

Vous n'avez pas besoin de tout faire une classe en C++.

namespace Random 
{ 
    bool seeded = false; 

    void SeedRandom(int number) 
    { srand(number); } 

    int GetRandom(int low, int high) 
    {   
     if (!seeded) 
     { srand((int)time(NULL)); } 

     return (rand() % (high - low)) + low; 
    }  

}