calloc + strcpy

Version imprimable

calloc + strcpy

Bonjour ,

J'ai un problème au niveau du calloc et du strcpy, un warning qui semble pouvoir être contourner avec un cast foireux , mais j'ai bien peur que ce ne soit pas la solution :)
A ce que j'ai compris , calloc alloue un pointeur vers un tableau en mémoire , je me trompe ?

Dans le code qui suit, je ne fais aucune vérification sur le retour des fonctions, c'est juste à titre d'exemple pour le calloc :

voici l'erreur : warning: passing argument 1 of ‘strcpy’ makes pointer from integer without a cast
Code:

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 #include <stdio.h> #include <stdlib.h> #include <string.h> char* split(char * ); int main() { int i; char buff[100] = "azerty.qsdfg.wxcvb"; char * p = split(&buff[0]); for(i=0;i<3;i++) printf("%s\n",p[i]); free(p); return 0; } char* split(char *buff) { int i = 0; char *chaine = (char*) calloc(3,sizeof(char)*20); char *pseek = strtok(buff,"."); while( pseek!= NULL ) { strcpy(chaine[i++],pseek); pseek = strtok(NULL,"."); } return chaine; }
Merci d'avance de votre aide :)

01/04/2008, 13h58
crocodilex

strcpy() attend une chaine de caractère en premier argument, alors que toi tu lui passes un caractère....c'est pas cool....
01/04/2008, 14h07
Leole

Ok donc j'ai mal compris le calloc ...
Comment accède-ton au premier élément du tableau ?

Voici ton code avec quelques modifs:

Code:

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define SPLIT_SIZE 20
 
char ** split(char *buff, size_t *n_splits);
void split_delete(char ***ppp);
 
int main(void)
{
    size_t i;
    char buff[100] = "azerty.qsdfg.wxcvb";
    size_t n_splits = 0;
    char ** pp = split(buff, &n_splits);
 
    for (i = 0; i < n_splits; i++)
    {
        printf("%s\n", pp[i]);
    }
 
    split_delete(&pp);
    return 0;
 
}
char** split(char *buff, size_t *n_splits)
{
    char **chaines = NULL;
 
    /* -tc- vaidation des parametres */
    if (buff != NULL && n_splits != NULL)
    {
        size_t i = 0;
        size_t n = 1;
 
        for (i = 0; buff[i] != '\0'; i++)
        {
            if (buff[i] == '.')
            {
                n++;
            }
        }
 
        chaines = malloc(n * sizeof *chaines);
        if (chaines != NULL)
        {
            chaines[0] = malloc(n * sizeof *chaines[0] * SPLIT_SIZE);
            if (chaines[0] != NULL)
            {
                char *pseek = NULL;
 
                for (i = 1; i < n; i++)
                {
                    chaines[i] = &chaines[0][i * SPLIT_SIZE];
                }
 
                pseek = strtok(buff, ".");
                i = 0;
 
                while (pseek != NULL)
                {
                    chaines[i][0] = '\0';
                    strncat(chaines[i], pseek, SPLIT_SIZE - 1);
                    ++i;
                    pseek = strtok(NULL,".");
                }
 
                /* -tc- retour du nombre de splits */
                *n_splits = n;
            }
            else
            {
                /* -tc- Erreur d'allocation: on fait le menage! */
                free(chaines), chaines = NULL;
            }
        }
    }
 
    return chaines;
}
 
void split_delete(char ***ppp)
{
    if (ppp != NULL && *ppp != NULL)
    {
        free((*ppp)[0]);
        free(*ppp), *ppp = NULL;
    }
}

Thierry

02/04/2008, 00h04
Sve@r

Citation:

Envoyé par Leole

Ok donc j'ai mal compris le calloc ...

Tout à fait. La fonction calloc veut en premier paramètre le nombre d'éléments (toi tu lui passes "3") et en second paramètre la taille d'un élément (là tu lui passes "20 * sizeof(char)"). Le premier paramètre peut se comprendre, le second...

Citation:

Envoyé par Leole

Comment accède-ton au premier élément du tableau ?

Par la syntaxe standard [0]. Une fois qu'une zone a été allouée, on s'y comporte exactement comme avec un tableau...
02/04/2008, 01h12
Leole

Bonjour , réponse un peu tardive désolé , je viens de rentrer chez moi .
@Thierry Chappuis: Merci ! Je regarde ça de plus près dès maintenant. J'ai déjà manipuler des pointeurs de pointeurs , mais il ya qq temps . Apparement ici ce sont des pointeurs de pointeurs de pointeurs ^^ je crois que je vais prendre un papier :)
@Sve@r: En effet , pour le 2e parametre, c'était un copié collé d'un autre de mes programmes où c'était compréhensible , c'est ma faute :) Sinon pour ma question pour accéder au premier élément du tableau alloué par calloc , c'est en effet ce que je faisais dans le code que j'ai posté par chaine[i] mais j'ai posé la question au vu de la réponse de crocodilex.

Merci à tous pour vos réponses , je check tout ça :D
02/04/2008, 01h41
Thierry Chappuis

Citation:

Envoyé par Leole

Apparement ici ce sont des pointeurs de pointeurs de pointeurs ^^ je crois que je vais prendre un papier :)

Seulement pour le destructeur split_delete(). Mais le code de cette fonction n'est pas bien sorcier. Un pointeur sur pointeur ou un pointeur sur pointeur sur pointeur, ce n'est finalement rien d'autre qu'un... pointeur.

Thierry
02/04/2008, 02h58
Leole

Yep ! J'ai adapté tout ça à mon projet , cela fonctionne et j'ai compris :) c'est vrai que ce n'est qu'un pointeur :aie:

Bonne nuit et merci :D
02/04/2008, 22h07
Sve@r

Citation:

Envoyé par Leole

et j'ai compris :)

C'était la seule et unique chose à comprendre en C. Félicitations !!!