Discussion :

[leCcsympas]

Bonjour,
Voila j'ai deux petits problemes :
a la compil :
test.c:14: attention : passing argument 1 of «time» from incompatible pointer type

a l'execution, ca affiche rien ! a part les -- , surement que mon bound est mal passé a la fonction et mal modifier.

Voici le 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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
 
#include <time.h>
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include <malloc.h>
 
/* Fonction qui permet de passer un boundary */
int genererBoundary(char *tab){
	time_t now;
	int taille;
	struct tm now_here;
 
	char *entete="boundary=";
	now=time(&now_here);
	taille=strlen(entete)+10;
	tab=malloc(sizeof(char)*(taille+1));
 
	if(!tab) {
		fprintf(stderr,"--erreur de generation de boundary--\n");
		return -1;
	}
	snprintf(tab,taille,"%s%10d",entete,now);
	return 0;
}
 
/* Generer un boundary */
 
int main (int argc, char* argv[]){
	char *bound;
	genererBoundary(bound);
	fprintf(stdout,"--%s\n",bound);
}

Sinon, si vous avez d'autres remarques sur mon code, n'hesitez car he debute et je dois surement coder desfois un peu "sale", genre eviter le +10 qui correspond au nombre de caractere dans la var now que je veux afficher.
Ciao,

[coca25]

bonjour,
pour l'erreur:

Code :

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

1
2
3
 
 #include <time.h>
 time_t time(time_t *T);

time attend un time_t * comme argument, tu lui fournis un struct tm

[hiko-seijuro]

d'apres ce que j'ai lu time à la signature suivante :

Code :

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

time_t time(time_t *tod);

or now_here n'est pas un time_t.....

[Emmanuel Delahaye]

a la compil :
test.c:14: attention : passing argument 1 of «time» from incompatible pointer type

a l'execution, ca affiche rien ! a part les -- , surement que mon bound est mal passé a la fonction et mal modifier.

<...>
Sinon, si vous avez d'autres remarques sur mon code, n'hesitez car he debute et je dois surement coder desfois un peu "sale", genre eviter le +10 qui correspond au nombre de caractere dans la var now que je veux afficher.

• <malloc.h> n'est pas standard. <stdlib.h> fera l'affaire.
• Comme il se doit en C, le paramètre 'tab' de la fonction est passé par valeur. Modifier cette valeur dans la fonction ne fera pas changer la valeur originale. Il faut donc trouver une autre méthode. Je suggère de retourner le valeur. En cas d'erreur, il suffit de retourner NULL et de le tester avant usage dans la fonction appelante. Penser à libérer le bloc après usage.
• Le paramètre de time() est erroné. Un simple NULL suffit ici.
• sizeof (char) vaut 1 par définition.
• now_here n'est pas utilisé.
• Il est inutile de séparer la définition des variables de leur initialisation quand celle-ci est évidente.
• Le terme 'taille' inclue le 0 final de chaine. C'est la nombre de bytes qu'occupe un chaine en mémoire.
• snprintf() ne fait pas ce que tu crois... En cas de saturation, il ne met pas le 0 final. Le comportement est indéterminé. On peut utiliser sprintf() en vérifiant que la valeur retournée est <= à la taille...
• Les paramètres de main() ne sont pas utilisés.
• Si on doit utiliser stderr, ça ne peut être qu'avec fprintf(), et non printf() qui va sur stdout.

Ton code corrigé. Pose des questions si tu ne comprends pas.

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
#include <time.h>
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#include <assert.h>
 
/* Fonction qui permet de creer un boundary */
char *genererBoundary (void)
{
   char *tab = NULL;
   char const *entete = "boundary=";
   int taille = strlen (entete) + 10 + 1;
 
   tab = malloc (taille);
 
   if (tab != NULL)
   {
      time_t now = time (NULL);
      int n = sprintf (tab, "%s%10d", entete, (int) now);
      assert (n <= taille);
   }
   return tab;
}
 
/* Generer un boundary */
int main (void)
{
   char *bound = genererBoundary ();
   if (bound != NULL)
   {
      fprintf (stdout, "--%s\n", bound);
      free (bound), bound = NULL;
   }
   else
   {
      fprintf (stderr, "--erreur de generation de boundary--\n");
   }
   return 0;
}

Code :

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

1
2
3
4
 
--boundary=1165359070
 
Press ENTER to continue.

[leCcsympas]

Merci je vais tester tout ca, mais en fait au debut, je voulais faire un passage par reference, je pensais pas que ca aller faire du passage par copie.
Ciao,

[Emmanuel Delahaye]

Merci je vais tester tout ca, mais en fait au debut, je voulais faire un passage par reference, je pensais pas que ca aller faire du passage par copie.

Il n'y a pas de passage par référence en C.

[Médinoc]

• snprintf() ne fait pas ce que tu crois... En cas de saturation, il ne met pas le 0 final. Le comportement est indéterminé. On peut utiliser sprintf() en vérifiant que la valeur retournée est <= à la taille...

Le snprintf() standard C99 le fait.
C'est le _snprintf() de Micro$oft qui ne le fait pas.