C++ affecter const à la variable d'environnement ou à la valeur par défaut

Question

Pour une application qui utilise un certain nombre de variables d'environnement, existe-t-il une sorte de convention ou «meilleure pratique» pour saisir des variables d'environnement et les placer dans un tas de const? Évidemment, je veux revenir à une valeur par défaut pour chaque variable d'environnement. En ce moment, en utilisant ce qui suit semble être une façon très désordonnée de le faire:

char* x; 
const SOME_VARIABLE; 
if (NULL == (x = getenv("SOME_VARIABLE"))) 
    SOME_VARIABLE = 5; // default value 
else 
    SOME_VARIABLE = x; 

Je pourrais aussi écrire une fonction qui enveloppe getenv pour renvoyer une valeur par défaut si une variable d'environnement est vide, mais je ne suis pas sûr si c'est même la meilleure façon de le faire. Je pourrais également faire disparaître en utilisant const, mais cela ne semble pas non plus une bonne chose à faire.

Des pensées?

Answer 1

Que diriez-vous:

std::string GetEnvironmentVariableOrDefault(const std::string& variable_name, 
              const std::string& default_value) 
{ 
    const char* value = getenv(variable_name.c_str()); 
    return value ? value : default_value; 
} 

Utilisé comme:

const std::string some_variable = GetEnvironmentVariableOrDefault("SOME_VARIABLE", "5"); 

Answer 2

James McNellis a fourni une great answer, mais voici une opinion alternative:

Je pense que getenv() renvoie la valeur de l'environnement variable au moment du démarrage du programme. Si la variable d'environnement est modifiée après le démarrage du programme, getenv() retournera toujours l'ancienne valeur. Basé sur cela, une approche alternative peut être d'avoir une classe pour capturer toutes les variables d'environnement requises en tant qu'attributs. La classe est remplie au début du programme et fournit uniquement des méthodes d'accès (const-qualifiées). De cette façon, le getenv() est appelé exactement une fois pour chaque variable d'environnement. Du côté négatif, la classe prend de la place.

Dans l'approche alternative, qui n'utilise pas d'espace, getenv() est appelée chaque fois que la valeur d'une variable d'environnement est requise. Ce n'est pas que l'appel getenv() est cher (en fait je ne sais pas), mais il porte une fausse supposition au programmeur que la dernière valeur pourrait être retournée.

Il peut également être utile d'avoir une classe pour abstraire s'il existe une quelconque dépendance à l'utilisation des variables d'environnement, par ex.

if $OS = "SunOs" 
then 
    GCC="/bin/gcc" 
else 
    GCC="/usr/bin/gcc" 

(this is just an example though) 

Maintenant, une fonction qui ne se soucient pas $OS mais n'a besoin que $GCC, peut simplement se référer à envClass->get("GCC");

Juste une pensée.

Answer 3

Je le fais généralement avec une classe qui charge les paramètres de configuration (soit à partir de l'environnement ou d'un fichier de configuration). J'instancie une instance globale unique au démarrage. Les paramètres du programme sont une méthode ou une propriété de l'objet de classe de configuration. Il est simple et quand vous écrivez du code, il est assez clair ce que vous vouliez:

if (configuration.ExitOnFailure && IsError()) 
    exit();