Problème de sprintf

Version imprimable

Problème de sprintf

Bonjour!

pourquoi est ce quand je fais 2 sprintf à la suite sur deux char* différents, j'obtiens une concaténation des deux dans le même char * ?

Voici une partie partielle de la fonction
Code:

1 2 3 4 5 6 char request[MAX_SIZE_REQUEST];/*250*/ char select[MAX_SIZE_REQUEST]; sprintf(request,"insert into %s values (NULL,'%s','%s','%s','%s')",tableName,values->pair[mapString_isset(*values,LABEL_GENRE)]->value, values->pair[mapString_isset(*values,LABEL_DIRECTOR)]->value, values->pair[mapString_isset(*values,LABEL_ACTOR)]->value, values->pair[mapString_isset(*values,LABEL_PRODUCER)]->value ); sprintf(select,"select max(id_profile)+1 from %s",tableName);
j'obtiens
request :
insert into profile values (NULL,'4,8.91;17,9.65;16,7.02;3,7.94;12,8.19;2,9.56;15,8.44;6,7.05','3630,0.1;3177,0.1;4016,0.77;3649,0.1;3871,4.28','12679,0.1;12680,0.1;11754,0.1;12681,0.1;12682,0.1;11074,0.1;11075,0.1;11076,0.1;11077,0.1;11078,0.1select max(id_profile)+1 from profile

select :
select max(id_profile)+1 from profile

Voila merci!
et au sinon un moyen de passer en dynamique request et select?
j'avais essayé char ** request;
request=malloc(sizeof (char*));
sprintf(*request,[..]);
mais ça me faisait une erreur de segmentation.

02/07/2012, 18h27
Trademark
Salut,

En fait c'est parce que ton buffer est trop petit, il fait 250 alors que la chaine que tu nous montres fait 272.

Comme tu le sais, chaque chaine de caractère est terminée par le caractère '\0', sprintf l'ajoute lui même à la fin.

Tes deux buffers sont situés un à côté de l'autre, et dans ce cas, tu as alloué 500 caractères de mémoire contigües. Si tu n'avais pas déclaré le deuxième buffer juste après tu aurais pu avoir une segfault car tu accédais une mémoire que tu n'avais pas réservée.

Ce qui se passe c'est que les 22 caractères en trop sont stockés dans le 2ème buffer, par conséquent lors du 2ème sprintf tu les écrases (légitimement) avec les nouvelles données. Le 2ème printf ajoute un '\0' qui sera le délimiteur pour le premier et deuxième buffer !!

Citation:

et au sinon un moyen de passer en dynamique request et select?
j'avais essayé char ** request;
request=malloc(sizeof (char*));
sprintf(*request,[..]);
mais ça me faisait une erreur de segmentation.

C'est normal, dans ce code, tu réserves un pointeur sur char, autrement dit, une zone mémoire pouvant contenir une adresse. Donc faisant 8 octets.

Je ne vais tout ré-expliquer ici, je te conseille d'aller voir les nombreux tutoriels sur les pointeurs. Néanmoins, si tu veux éviter une segfault une fois pour toute (dans cette exemple), il faudra que tu réserves assez de mémoire pour toute ta chaine, peu importe sa taille. Donc via l'allocation dynamique tu feras :
Code:

1 2 3 char* request; // simple pointeur suffisant request = malloc(strlen(chaine1) + strlen(chaine2) + ... + taille_chaine_statique + 1) ; // +1 pour le '\0' sprintf(request, ...);
Ce que j'appelle chaine "statique" c'est ce que tu écris en "dur" dans le sprintf.

Voila, j'espère que j'ai été assez clair ;)
03/07/2012, 08h52
Kimly

Merci! je vais essayer de faire ça :)
04/07/2012, 09h21
nowahn

Salut,

Citation:

Envoyé par Kimly

et au sinon un moyen de passer en dynamique request et select?
j'avais essayé char ** request;
request=malloc(sizeof (char*));
sprintf(*request,[..]);
mais ça me faisait une erreur de segmentation.

Il y a asprintf pour ça, mais je ne sais pas si c'est standard ou une extension GNU/linux.
04/07/2012, 09h57
Bktero

Jolie fonction :)

Sauf que :

Citation:

Envoyé par http://man.developpez.com/man3/asprintf.3.php

NOTES
Ces fonctions sont des extensions GNU et sont absentes des standards C et POSIX. Elles sont également disponibles sous *BSD. L'implantation FreeBSD affecte à strp la valeur NULL en cas d'erreur.

Bon j'en reviens à vous pour m'aider à résoudre mon problème!

Voici mon code (très partiel) :
Code:

1 2 3 4 5 char * request; int sizeR2=1+strlen("insert into (id_product, status, id_genres, title, id_countries, id_directors, id_actors, id_producers, duration, short_synopsis, long_synopsis, id_tags, date) values ('', '', '', '', '', '', '', '', '', '', '', '', '')")+strlen(tableName)+strlen(values->pair[idx_idProduct]->value)+strlen(values->pair[idx_status]->value)+strlen(values->pair[idx_genre]->value)+strlen(values->pair[idx_title]->value)+strlen( values->pair[idx_country]->value)+strlen(values->pair[idx_director]->value)+strlen(values->pair[idx_actor]->value)+ strlen(values->pair[idx_producer]->value)+strlen( values->pair[idx_duration]->value)+strlen(values->pair[idx_shortSynopsis]->value)+strlen(values->pair[idx_longSynopsis]->value)+strlen(values->pair[idx_tags]->value+strlen(values->pair[idx_date]->value)); request=malloc(sizeR2); snprintf(request,sizeR2,"insert into %s (id_product, status, id_genres, title, id_countries, id_directors, id_actors, id_producers, duration, short_synopsis, long_synopsis, id_tags, date) values ('%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s')", tableName, values->pair[idx_idProduct]->value, values->pair[idx_status]->value, values->pair[idx_genre]->value, values->pair[idx_title]->value, values->pair[idx_country]->value, values->pair[idx_director]->value, values->pair[idx_actor]->value, values->pair[idx_producer]->value, values->pair[idx_duration]->value, values->pair[idx_shortSynopsis]->value, values->pair[idx_longSynopsis]->value, values->pair[idx_tags]->value, values->pair[idx_date]->value);
Mon problème est que la taille sizeR retounée est trop petite. Un exemple :
Pour une chaine de taille 300+1;

Code:

insert into kuest_product (id_product, status, id_genres, title, id_countries, id_directors, id_actors, id_producers, duration, short_synopsis, long_synopsis, id_tags, date) values ('1', 'available', '12,13,14', '', '', '2100,3649', '11000,11076,12681,13000', '250,342', '4', '', '', '', '2012/7/10')

j'ai sizeR=292 :

Code:

insert into kuest_product (id_product, status, id_genres, title, id_countries, id_directors, id_actors, id_producers, duration, short_synopsis, long_synopsis, id_tags, date) values ('1', 'available', '12,13,14', '', '', '2100,3649', '11000,11076,12681,13000', '250,342', '4', '', '', '', '20

Donc je comprends que le problème vient du calcul de la taille mais je ne sais pas comment j'ai pu omettre quelquchose.

Merci d'avance et désolé si c'est une erreur stupide.

Petit soucis de parenthèse

Code:

strlen(values->pair[idx_tags]->value+strlen(values->pair[idx_date]->value));

Par contre, tu t'en es peut-être rendu compte, mais c'est quand même pas très beau comme système...

Pour améliorer un peu, j'aurais plutôt fait quelque chose comme :
Code:

1 2 3 4 5 6 7 8 9 10 11 char* raw_request = "insert into %s (id_product, status, id_genres, title, id_countries, id_directors, id_actors, id_producers, duration, short_synopsis, long_synopsis, id_tags, date) values ('%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s')"; size_t request_size = strlen(raw_request) - (nb_variable * strlen("%s")); // on retire les "%s" de la taille. request_size += strlen(tableName); request_size += strlen(values->pair[idx_idProduct]->value); // ... ++request_size; // Pour le \0 final char* request = malloc(...); snprintf(request, request_size, raw_request, tableName, ...);
Parfois, on évite des erreurs et c'est plus clair sur plusieurs lignes ;)

A part ça j'ai l'impression que ta table n'est pas en 3ème forme normal (ou 4ème). Vérifie quand même, tu auras surement des requêtes plus digestes après ?

10/07/2012, 14h11
Bktero

A noter une utilisation intéressante de snprintf() :

Citation:

Envoyé par http://man.developpez.com/man3/snprintf.3.php#L13

lorsque snprintf est appelée avec un argument size=0 [...] alors [...] C99 autorise str à être NULL dans ce cas, et réclame en valeur de retour (comme toujours) le nombre de caractères qui auraient été écrits si la chaîne de sortie avait été assez grande.

On peut ainsi connaitre la taille de la chaine à écrire.

PS : 2000 points \o/
10/07/2012, 14h16
ternel

Effectivement, c'est bon à savoir. Merci pour cette précision.
10/07/2012, 15h47
Kimly

Merci, c'était bête désolé j'aurai du aller sur un autre éditeur de texte...
J'ai bien changé en séparant les lignes .

J'ai pas compris ça

Citation:

A noter une utilisation intéressante de snprintf() :

Citation:
Envoyé par http://man.developpez.com/man3/snprintf.3.php#L13
lorsque snprintf est appelée avec un argument size=0 [...] alors [...] C99 autorise str à être NULL dans ce cas, et réclame en valeur de retour (comme toujours) le nombre de caractères qui auraient été écrits si la chaîne de sortie avait été assez grande.
On peut ainsi connaitre la taille de la chaine à écrire.

Mais pas grave!

Ah et je voulais savoir, (j'ai étudié ces histoires de 3FN mais je m'en rappelle plus vraiment. ) En quoi ne serai elle pas en 3FN?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
int main(int argc, char *argv[])
{
    const char * cf = "a vaut %d";
    int a = 123456;
    int longueur = snprintf(NULL, 0, cf, a);
 
    char * resultat = malloc(longueur + 1);
 
    snprintf(resultat, longueur+1, cf, a);
    printf("'%s' de longueur %d (comme prevu %d)", resultat, strlen(resultat), longueur);
 
    free(resultat);
 
    return 0;
}

Essaye ce code et tu comprendras.