numéro de chaîne strncpy char lors de l'ajout longueur

Question

Je vais avoir un problème avec la comparaison 2 chaînes char qui sont à la fois le même:

char string[50]; 

strncpy(string, "StringToCompare", 49); 

if(!strcmp("StringToCompare", string)) 
//do stuff 
else 
//the code runs into here even tho both strings are the same...this is what the problem is. 

Si j'utilise:

strcpy(string, "StringToCompare");

au lieu de:

strncpy(string, "StringToCompare", 49);

il résout le problème, mais je préfère insérer la longueur de la chaîne plutôt que de l'obtenir elle-même.

Qu'est-ce qui ne va pas ici? Comment puis-je résoudre ce problème?

Answer 1

Vous avez oublié de mettre un caractère NUL de terminaison à string, alors peut-être que strcmp est exécuté à la fin. Utilisez cette ligne de code:

string[49] = '\0'; 

pour résoudre votre problème.

Answer 2

Vous devez définir le terminateur null manuellement lorsque vous utilisez strncpy:

strncpy(string, "StringToCompare", 48); 
string[49] = 0; 

Answer 3

strcopy et strncpy: dans cette situation, ils se comportent de manière identique !!

Vous ne nous dit pas la vérité ou l'image (par exemple: la chaîne est au moins 49 caractères)

Answer 4

Beaucoup de suppositions apparentes dans les autres réponses, mais une suggestion rapide. Tout d'abord, le code tel qu'il est écrit devrait fonctionner (et fonctionne en fait dans Visual Studio 2010). La clé est dans les détails de 'strncpy' - il ne sera pas implicite ajouter un caractère de fin null sauf si la longueur de la source est inférieure à la longueur de destination (ce qui est dans ce cas). D'autre part, inclut le terminateur null dans tous les cas, ce qui suggère que votre compilateur ne gère pas correctement la fonction strncpy.

Donc, si cela ne fonctionne pas sur votre compilateur, vous devriez probablement initialiser votre mémoire tampon temporaire comme ceci:

char string[50] = {0}; // initializes all the characters to 0 

// below should be 50, as that is the number of 
// characters available in the string (not 49). 
strncpy(string, "StringToCompare", 50); 

Cependant, je soupçonne que cela est probablement juste un exemple, et dans le monde réel votre chaîne source est 49 (encore, vous devez passer 50 à strncpy dans ce cas) caractères ou plus, auquel cas le terminateur NULL n'est pas copié dans votre chaîne temporaire.

Je ferais écho aux suggestions dans les commentaires pour utiliser std::string si disponible. Il prend soin de tout cela pour vous, afin que vous puissiez vous concentrer sur votre implémentation plutôt que sur ces détails banals.

Answer 5

Le paramètre de nombre d'octets dans strncpy indique à la fonction le nombre d'octets à copier et non la longueur du tampon de caractères. Donc, dans votre cas, vous demandez de copier 49 octets de votre chaîne constante dans le tampon, ce que je ne pense pas être votre intention!

Cependant, cela n'explique pas pourquoi vous obtenez le résultat anormal. Quel compilateur utilisez-vous? Quand je cours ce code sous VS2005 j'obtiens le comportement correct.

Notez que strncpy() a été dépréciée en faveur de strncpy_s, qui ne veut la longueur du tampon qui lui est passé:

strncpy_s (string,sizeof(string),"StringToCompare",49)