aide error pointeur

Version imprimable

Voir 40 message(s) de cette discussion en une page

aide error pointeur

Bonjour!

voila mon probleme d'aujourd'hui, je realise une calculatrice gerant les nombres infinis, seul bemol type int interdit, il faut manipuler uniquement les chaines de caracteres et les 2 fonction suivantes malloc et putchar et putstr, les boucles while, fonction strlen voila, je doit gerer les operateurs suivants -+*/% les priorites de calcul, bref tres dure pour un novice comme moi, alors voila j'ai commence par la fonction addition seul bemol une erreur qui m'empeche de continuer , je ne veut pas avancer sans comrpendre cette erreur, pour certain cela relevera de l'evidence pour d'autre le debut d'une comprehension de warnings: voici mon code j'ai commence avec une fonction addition , lors de la cc il me retourne cette erreur ;

addition2.c: In function `conversion':
addition2.c:18: error: subscripted value is neither array nor pointer
addition2.c:20: error: subscripted value is neither array nor pointer
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 void conversion(char nb) { int i; i = 0; while (nb[i] != '\0') { nb[i] -= '0'; printf("Conversion ok!\n"); } } char addition(char nb1, char nb2) { conversion(nb1); } int main() { my_putstr(addition('5', '5')); }

25/10/2006, 15h56
ronan99999
Attention je pense que tu as confondu un char et un pointeur sur char.
De plus tu n'incrémente pas ton i, tu ne sortiras jamais de ta boucle.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void conversion(char *nb) { int i; i = 0; while (nb[i] != '\0') { nb[i] -= '0'; i++; printf("Conversion ok!\n"); } }
De meme dans addition tu manipules des chaines donc des pointeurs.
Code:

1 2 void addition(char *nb1,char *nb2);
Enfni dans ton main est tu sures de vouloir manipuler des char ou des chaines.

addition('5','5') ou addition("5","5")
25/10/2006, 15h59
homeostasie

Citation:

ll faut manipuler uniquement les chaines de caracteres...

Je pense que tu peux donc utiliser les fonctions pour manipuler les chaines de caractères:
-> Pour convertir une chaine de caractères en nombre: utilisation de strtol() par exemple.
-> Pour convertir un nombre en une chaine de caractères: Utilisation de sprintf()
25/10/2006, 16h06
Médinoc

Mais là, il est question de nombres infinis: En clair, les calculs doivent être faits chiffre par chiffre dans les chaînes.

aide

merci, effectivement , il faut gerer des nbres infinits, je le repete aucun droit d'utiliser des fonctions autre que putstr putchar free malloc , je doit realise les conversions moi meme,

pourrai-je avoir de plus amples informations concernant mon erreur avec les pointeurs a chaque fois que j'envoie une chaine de caracteres a une fonction elle doit etre du type pointeur de char?? pour quel raison, j 'aimerai tout comprendre.....

merci encore

nouveau code a jour :
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 void conversion(char *nb) { int i; i = 0; while (nb[i] != '\0') { nb[i] -= '0'; printf("Conversion ok!\n"); i++; } } char addition(char *nb1, char *nb2) { int i; i = 0; conversion(nb1); conversion(nb2); if (nb1[i] < 10) { my_putstr("conversion reussit!"); } } int main() { my_putstr(addition("55", "55")); }
erreur segmentation fault
:roll: je sait que dans ces cas la je doit utiliser malloc?

25/10/2006, 17h17
Rits

un char en C represente un caractere et c tout

si tu veux affciher plusieurs caractere il te faut passer par un char *
c'est a dire un pointeur sur char!

le pointeur te permet de creer plusieurs 'char' à la suite grace au malloc ce qui te donne au final une suite de char donc une "chaine de caractere"

char *p;
p = malloc(12*sizeof(char));

p peut contenir 11 caractere le 12e etant le marqueur de fin de lignes '\0'
et ne pas oublier de librerer p avec free(p) qd tu en as plus besoin

ok merci j'ai compris l'utilite d'un pointeur avec un char , merci bien.
donc je doit faire un malloc a chaque fois aue j'envoie une chaine de caracteres a une fonction??

peut tu modifier mon code pour que je comprenne , j'ai mis des malloc mais ca segfault encore....
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 #include <stdlib.h> void conversion(char *nb) { int i; i = 0; nb = malloc(sizeof (*nb) * strlen(nb)); while (nb[i] != '\0') { nb[i] -= '0'; printf("Conversion ok!\n"); i++; } } char addition(char *nb1, char *nb2) { int i; i = 0; conversion(nb1); conversion(nb2); nb1 = malloc(sizeof (*nb1) * strlen(nb1)); nb2 = malloc(sizeof (*nb2) * strlen(nb2)); if (nb1[i] < 10) { my_putstr("conversion reussit!"); } } int main() { my_putstr(addition("55", "55")); }
execution:
(harit_a@muette 152)./a.out
conversion reussit!Segmentation fault

de plus je comprend meme pas pk il ne m'affiche pas a l'ecran "conversion ok"
encore une fois j'ai besoin de comprendre tout ca , pour avancer plus vite... j'insiste pour eviter les reponses du type vas voir des tutorials ou autres..........

malloc sert à créer une zone mémoire où sera stockée ta chaîne. Il faut donc l'appeler tout au début, pour initialiser le char*, pas dans la fonction où il va écraser sa valeur !

Dans ton cas, tu peux utiliser strdup aussi, à la place de malloc pour initialiser tes char*.
Code:

1 2 3 4 5 6 7 8 9 10 11 int main() { char* nombre1 = strdup("55"); char* nombre2 = strdup("55"); my_putstr(addition(nombre1, nombre2)); free(nombre1); nombre1 = NULL; free(nombre2); nombre2 = NULL; }

"Back to the roots" regarde un peu ton cours ou cherche je suis sure que tu devrais trouver d'excellent tutoriel sur les chaines de caractéres en C.

Sinon pour ton segmentation fault:

Ta fonction addition renvoie un char??????!!!!!!!, non elle doit renvoyer une chaine de caractére donc elle doit etre du type: char* x char* -> char*
De plus dans ton code il n'ya aucun return.
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 //-1- methode char * addition(char*nb1,char*nb2) { char* pcRet; int iSize; //-------calcul de la taille de la zone à allouer if(pcRet=calloc(iSize,sizeof(char)) { //------calcul de pc Ret } return pcRet; } //effectivement à la sortie une fois que tu auras fini l'utilisation de pcRet tu devras liberer la memoire allouée //-2- Methode void addition(char*nb1,char*nb2,char*nbRes) { //-----calcul de nbRes } //ici tu fais un passage par reference tu peux gerer toute la memoire en statique
Fait attention au typage des fonctions que tu utilises.

Citation:

Envoyé par zooro

En utilisant la première méthode donnée par ronan99999 :

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
 
char * addition(char*nb1,char*nb2)
{
   char* pcRet = NULL;
   int     iSize = 0;
 
  //-------calcul de la taille de la zone à allouer
  ...
 
  // --- allocation de la mémoire pour le résultat (calloc renvoie NULL en cas d'erreur, ce qui est très bien dans notre cas
  if(pcRet=calloc(iSize,sizeof(char))
 {
    //------calcul de pc Ret
  }
 
  return pcRet;
}
 
int main()
{
  char* nombre1 = strdup("55");
  char* nombre2 = strdup("55");
  char* resultat = addition(nombre1, nombre2);
 
  if (resultat != NULL) {
    my_putstr(resultat);
    free(resultat); resultat = NULL;
  }
 
  free(nombre1); nombre1 = NULL;
  free(nombre2); nombre2 = NULL;
}

25/10/2006, 17h44
Emmanuel Delahaye
Citation:

Envoyé par aimad41

pourrai-je avoir de plus amples informations concernant mon erreur avec les pointeurs a chaque fois que j'envoie une chaine de caracteres a une fonction elle doit etre du type pointeur de char??

Hum, ça fait partie des bases du langage C...

Petit rappel :
- Une chaine C est une séquence de caractères terminée par un 0
- Une chaine C est implémenté par un tableau de char initialisé.
- Pour "passer une chaine à une fonction", la regle des tableaux s'applique : il faut passer l'adresse de son premier élément, c'est à dire le nom de la chaine. Pour celà, le paramètre doit être un pointeur du même type de l'élément, soit ici char (éventuellement qualifé const, c'est à dire non modifiabe, si la fonction s'engage à ne pas écrire dans la chaine), soit char * ou char const *.
Donc, que ce soit pour une chaine littérale

Code:

"hello"

un tableau de char initialisé

Code:

char s[] = "hello";

ou un pointeur sur une chaine littérale

Code:

char const *p = "hello";

le prototype de la fonction est donc

Code:

void f (char *p);

ou

Code:

void f (char const *p);

et l'appel se fait respectivement comme ceci :
Code:

1 2 3 4 f ("hello"); f (s); f (a);
25/10/2006, 17h47
Emmanuel Delahaye

Citation:

Envoyé par aimad41

ok merci j'ai compris l'utilite d'un pointeur avec un char , merci bien.

Je n'en suis pas si sûr...

Citation:

donc je doit faire un malloc a chaque fois aue j'envoie une chaine de caracteres a une fonction??

Non. J'ai donné 3 contre-exemples. Mais, oui, c'est possible de le faire. Il faut que ce soit justifié, car ce n'est pas une opération légère...
25/10/2006, 17h52
Rits

ya du boulot ;)

je pense que tu devrais essayer de faire des exercices plus simple pour commencer
ou on manipule des chaines de caracteres!

sinon malloc sert a creer une variable qui ne contient rien pour ensuite mettre ce que l'on a besoin! mais une fois cree pas besoin de faire de recrée toi tu en met partout!

par exxemple qd tu appelles addition("55,"55");

ensuite dans ta fontion addition(char *nb1.....)
nb1 et nb 2 contiennent deja quelque chose nb1 = "55" et nb2 = "55"
donc pas besoin de faire de malloc!

ok mais aucun droit d'utiliser la fonction strdup seulement malloc et free pour la memoire,
dans l'ensemble je pense avoir compris, j'aimerait juste comprendre encore une chose , la cc se deroule bien , execution ok mais il n'affiche pas a l'ecran
"conversion ok!"
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 #include <stdlib.h> void conversion(char *nb) { int i; i = 0; nb = malloc(sizeof (char) * strlen(nb)); while (nb[i] != '\0') { nb[i] -= '0'; printf("Conversion ok!\n"); i++; } } char *addition(char *nb1, char *nb2) { int i; i = 0; conversion(nb1); conversion(nb2); if (nb1[i] < 10) { my_putstr("conversion reussit!"); } return(nb1, nb2); } int main() { my_putstr(addition("55", "55")); }
il affiche a l'ecran 55 vraiment tres bizarre vous trouvez pas? comme si les autres fonction n'existe pas

PS: j'ai coder 70% des fonctions maipulant des chaines de caracteres
strstr strcmp strncmp revstr strcpy strncpy strupcase strlowcase strcapitalize show_mem putnbr_base getnbr_base

donc voila le pb c'est que je code les fonctions , mais je retient pas mes erreurs passes.
d'ou l'interet que je porte q la comprehension de chaque erreur, je peut la finir ma fonction en bricolant les fonctions, mais j'en ai marre de faire ca , je veut tout comprendre, pour arreter tout le temps d'avoir des warnings....

25/10/2006, 18h01
Rits

Citation:

mais il n'affiche pas a l'ecran
"conversion ok!"

essai de comprendre ce que tu fais....

nb = malloc(sizeof (char) * strlen(nb)); // qu'est ce qui passe ici ? que vos nb apres malloc

while (nb[i] != '\0')
{
....
}

Citation:

PS: j'ai coder 70% des fonctions maipulant des chaines de caracteres
strstr strcmp strncmp revstr strcpy strncpy strupcase strlowcase strcapitalize show_mem putnbr_base getnbr_base

elle fonctionne au moins 8O

ecoute je suis pas la pour polluer le forum je suis la pour progresser, si je te dit que j'ai coder les fonctions de chaine de caracteres , c'est que c'est vrai et qu'elle fonctionne!

epargne moi ce genre de remarque s'il te plait c'est decourageant et vexant.

merci.

au depart j'etait partit dans ce programme addition mais j'ai abandonner pour tout reprendre a zero maintenant j'avance pas et ca me frustre
voila le code

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
 
#include <stdlib.h>
 
int  conversion(char *nb)
{
  int   i;
 
  for (i = 0; nb[i]; i++)
    {
      nb[i] -= '0';
      if (nb[i] < 10)
        {
          my_putstr("Conversion ok!\n");
        }
    }
}
 
void    putzero(int *nb, int size)
{
  for (; size > 0; size--)
    {
      nb[size] = 0;
      printf("Affectation de putzero :%d\n", nb[size]);
    }
}
 
int     add_carry(int *nb)
{
  int   pos;
 
  for (pos = 0; nb[pos] >= 10; pos++)
    {
      nb[pos + 1] += nb[pos] / 10;
      nb[pos] %= 10;
    }
}
 
int     *do_oplus(char *nb1, int len1, char *nb2, int len2)
{
  int   pos1;
  int   pos2;
  int   *res; // initialisation tab entier pour stocker les resultats                                                                        
  int   i;
 
  pos1 = len1 - 1; // curseur en derniere position                                                                                           
  pos2 = len2 - 1;
  res = malloc(sizeof (*res) * (len1 + len2)); // allocation de la memoire                                                                   
  putzero(res , len1 + len2); // affectation de 0000000 sur le resultat                                                                      
  for (i = 0; i < len1+len2; i++)
    {
      printf("%d",res[i]);
    }
  for (pos2 = len2 - 1; pos2 >= 0; pos2--)
 {
      for (pos1 = len1 - 1; pos1 >= 0; pos1--)
        {
          i = (len2 - pos2 - 1) + (len1 - pos1 - 1);
          res[i] = nb1[pos1] + nb2[pos2];
          printf("\nres[%d] = %d",i, res[i]);
        }
    }
  add_carry(res + i);
  return(res);
}
 
int     addition(char *nb1, char *nb2)
{
  int   *res;
  int   len1;
  int   len2;
 
  len1 = strlen(nb1);
  len2 = strlen(nb2);
  conversion(nb1);
  conversion(nb2);
  res = do_oplus(nb1, len1 , nb2, len2);
}
 
int     main(int ac, char **av)
{
  if ( ac != 3)
    my_putstr("Bad usage of function\nFunction usage: ./a.out nb1 nb2");
  else
    printf("\nresultat de nb1 * nb2 = %d",(addition(av[1], av[2])));
}

en esperant que tu comprenne que je sache en partie ce que je fait, seulement desfois je suis perdu, ca fait 15jours que j'apprend ce langage......
j';ai decider de reprendre a zero pour mieux comprendre toutes les parties..

25/10/2006, 18h11
zooro
Code:

1 2 3 4 5 6 7 void conversion(char *nb) { int i; i = 0; nb = malloc(sizeof (char) * strlen(nb)); ...
Comme je te le disais plus haut, en faisant un malloc ici, tu écrases la valeur actuelle de nb, en le faisant pointer sur une nouvelle zone mémoire non initialisée.
Donc quand ensuite tu regardes nb[i], impossible de prévoir ce qu'il y aura à cette adresse !!!
25/10/2006, 18h17
Rits

excuse moi je veux ni te vexer ni te decourager
mais qd tu demandes

Citation:

a chaque fois que j'envoie une chaine de caracteres a une fonction elle doit etre du type pointeur de char?

et que tu ensuites tu balances j'ai coder strcpy, strncpy, strstr.......qui prennent des pointeurs sur char!

sur ce bonne continuation

ca sert a rien de s'enerver sur un debutant, faut passer tes nerfs ailleurs, si t'as pas la patience d'expliquer pas a pas les choses, ca sert a rien de nous aider....
mais merci qd meme des elements que tu as poste ca m'as qd meme aide....

donc pour en revenir au code voila ce que j'ai fait:
#include <stdlib.h>
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 void conversion(char *nb) { int i; i = 0; while (nb[i] != '\0') { nb[i] -= '0'; printf("Conversion ok!\n"); i++; } } char *addition(char *nb1, char *nb2) { int i; i = 0; conversion(nb1); conversion(nb2); if (nb1[i] < 10) { my_putstr("conversion reussit!"); } return(nb1, nb2); } int main() { char *nb1; char *nb2; nb1 = malloc(sizeof (char) * 100); nb2 = malloc(sizeof (char) * 100); nb1 = "55"; nb2 = "55"; my_putstr(addition(nb1, nb2)); }

sa fonctionne

bon alors maintenant ca fonctionne , j'ai juste ajouter les arguments taper sur la ligne de commande et ca fonctionne pas d'utilisation de malloc ici.
difficile a comprendre je dois dire

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
 
#include <stdlib.h>
 
void    conversion(char *nb)
{
  int   i;
 
  i = 0;
   while (nb[i] != '\0')
    {
      nb[i] -= '0';
      printf("Conversion ok!\n");
      i++;
    }
}
 
char    *addition(char *nb1, char *nb2)
{
  int   i;
 
  i = 0;
  conversion(nb1);
  conversion(nb2);
  if (nb1[i] < 10)
    {
      my_putstr("conversion reussit!");
    }
  return(nb1, nb2);
}
 
int     main(int ac, char **av)
{
  if (ac != 3)
    my_putstr("bad usage of function\n");
  else
    my_putstr(addition(av[1], av[2]));
}

Voir 40 message(s) de cette discussion en une page