Meilleure façon de gérer le stockage (éventuellement NULL) char * dans une chaîne std ::

Question

class MyClass 
{ 
public: 
    void setVar(const char *str); 
private: 
    std::string mStr; 
    int maxLength; //we only store string up to this length 
}; 

Quelle est la meilleure approche pour implémenter setVar lorsque le code externe est susceptible de passer en NULL pour une chaîne vide (et ne peut pas être modifié)? Je fais actuellement quelque chose un peu comme:

void MyClass::setVar(const char *str) 
{ 
mStr.assign(str ? str : "",maxLength); 
} 

Mais il semble un peu salissant. des idées?

Answer 1

Le code que vous inscrire est incorrect, car il sera toujours lu maxLength caractères de la chaîne source. En particulier, cela signifie qu'il lira après la fin de la chaîne vide lorsque str est NULL. Cela fonctionnera à la place, en supposant que str est terminé par zéro:

void MyClass::setVar(const char *str) 
{ 
    if (str==NULL) 
     mStr.clear(); 
    else 
     mStr.assign(str, std::min(strlen(str), maxLength)); 
} 

Answer 2

void MyClass::setVar(const char *str) 
{ 
    if (str) { 
     mStr.assign(str, str + std::min(strlen(str), maxLength)); 
    } else { 
     mStr = ""; 
    } 
}