Discussion :

[vincent.mbg]

Bonjour,

j'ai un bug avec la fonction strncpy.
je cherche à récupérer le préfix de chaque mot d'une liste de mots.
Le problème, c'est que à un moment, strncpy ne veut plus copier correctement.

Voici un extraie de code.

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
 
 
    for ( i = 0; i < size_of_list ; i++ ) {
        int taille_du_mot ;
        mot_a_inserer = PyString_AsString( PyList_GetItem( liste_de_mots, i ) ) ;
 
        [...]
 
        index = 1 ;
        strncpy( prefix, mot_a_inserer, index ) ;  //prefix = mot_a_inserer[ : index ]
        printf("pour etre sur que ca bug : \n") ;
        printf("mot %s copié sur %d\n", mot_a_inserer, index );
        printf("prefix %s \n", prefix ) ;
        printf("-----------------------------\n" );       
        insertion_termine = FALSE ;

Et voici ce que ça affiche.

pour etre sur que ca bug :
mot art copié sur 1
prefix a
-----------------------------
pour etre sur que ca bug :
mot artiste copié sur 1
prefix a
-----------------------------
pour etre sur que ca bug :
mot article copié sur 1
prefix artis
-----------------------------

Pourquoi à la dernier ligne copier "article" sur 1 me renvoie "artis" et non "a" ?

D'avance, merci beaucoup.

[gl]

Pourquoi à la dernier ligne copier "article" sur 1 me renvoie "artis" et non "a" ?

Si la copie s'arrête parce que la taille est atteinte, strncpy() ne met pas le caractère de fin de chaîne '\0'. Donc si le tableau dans lequel tu copie n'est pas suffisamment grand pour contenir ce \0 ou n'est pas correctement initialisé, la chaîne n'est pas correctement terminée (et donc n'est pas une chaîne C) et ne s'affichera donc pas correctement.

[vincent.mbg]

Merci beaucoup pour cette réponse rapide.
Si je comprend bien j'ai juste à faire :

Code :

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

1
2
3
 
strncpy( prefix, mot_a_inserer, index ) ;
prefix[ index + 1 ] = '\0' ;

Je vais tester ça demain matin là je suis un peu fatigué.
Encore merci.

[Sve@r]

Merci beaucoup pour cette réponse rapide.
Si je comprend bien j'ai juste à faire :

Code :

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

1
2
3
 
strncpy( prefix, mot_a_inserer, index ) ;
prefix[ index + 1 ] = '\0' ;

Presque. N'oublie pas que les tableaux sont numérotés à partir de 0 donc si tu copies "index" caractères, le \0 devra être stocké à prefix[index] et non prefix[index + 1]...

[vincent.mbg]

C'est bon ça marche

Merci beaucoup.