Discussion :

[aimad41]

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 :

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
 
 
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'));
}

[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 :

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

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 :

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

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")

[homeostasie]

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()

[Médinoc]

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

[aimad41]

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 :

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
 
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
je sait que dans ces cas la je doit utiliser malloc?

[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

[aimad41]

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 :

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
 
#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..........

[zooro]

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 :

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

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;
}

[ronan99999]

"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 :

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
 
//-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.

[zooro]

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

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
 
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;
}

[Emmanuel Delahaye]

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 :

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

"hello"

un tableau de char initialisé

Code :

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

char s[] = "hello";

ou un pointeur sur une chaine littérale

Code :

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

char const *p = "hello";

le prototype de la fonction est donc

Code :

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

void f (char *p);

ou

Code :

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

void f (char const *p);

et l'appel se fait respectivement comme ceci :

Code :

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

1
2
3
4
 
   f ("hello");
   f (s);
   f (a);

[Emmanuel Delahaye]

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

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

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...

[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!

[aimad41]

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 :

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
 
#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....

[Rits]

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')
{
....
}

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

[aimad41]

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 :

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
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..

[zooro]

Code :

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

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 !!!

[Rits]

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

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

[aimad41]

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 :

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
 
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));
}

[aimad41]

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 :

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
 
#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]));
}