Discussion :

[jayfaze]

Bonjour, je dois écrire une fonction qui met une chaîne de caractère passée en argument en majuscule.
J'ai donc fait un calcul par rapport à la valeur ascii. Comme a -> 97 et A ->65, alors je fais - 32 à chaque fois. Mais ça ne fonctione pas.

voici la fonction

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
char    *upcase(char *str)
{
  int   i;
  char  *tp;
 
  i = 0;
  while (str[i] != '\0')
    {
      tp[i] = str[i] - 32;
      i++;
    }
    return (tp);
}

et le main

Code :

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

1
2
3
4
5
6
7
8
9
10
 
int     main()
{
  char  *p;
  char  *a;
 
  a = "hello";
  p = upcase(p);
  printf(p);
}

erreur : segmentation fault

Je ne vois pas trop où est le problème

[Bruno Gillet]

Code :

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

1
2
3
4
 
char  *tp;
<...>
tp[i] = str[i] - 32;

Tu n'a pas alloué tp, donc tu écris n'importe ou dans la mémoire.

[jayfaze]

Selon l'exercice que je dois faire, j'ai pas le droit d'utiliser malloc. En plus l'erreur vien du main, pas de la fonction.

[Bruno Gillet]

Selon l'exercice que je dois faire, j'ai pas le droit d'utiliser malloc. En plus l'erreur vien du main, pas de la fonction.

J'ai pas dis que tu devais utiliser malloc moi... j'ai dis que utilisais une chaine non allouée et que tu allais écrire n'importe ou dans la mémoire.

[jayfaze]

ok !
J'ai fait tp = "";
du coup j'ai

Code :

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

main.c:9: warning: assignment makes integer from pointer without a cast

comme erreur !

[Bruno Gillet]

J'ai des questions pour toi...
As tu besoin d'utiliser tp dans uppercase ?
Si tu passe ta chaine en pointeur c'est bien pour pouvoir la modifier non ?

Autre chose, ton code ne marche que si tu passe une chaine de caractère en minuscule et dans aucun autre cas.

[rutabagas]

Bonjour, je dois écrire une fonction qui met une chaîne de caractère passée en argument en majuscule.
J'ai donc fait un calcul par rapport à la valeur ascii. Comme a -> 97 et A ->65, alors je fais - 32 à chaque fois. Mais ça ne fonctione pas.

voici la fonction

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
char    *upcase(char *str)
{
  int   i;
  char  *tp;
 
  i = 0;
  while (str[i] != '\0')
    {
      tp[i] = str[i] - 32;
      i++;
    }
    return (tp);
}

et le main

Code :

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

1
2
3
4
5
6
7
8
9
10
 
int     main()
{
  char  *p;
  char  *a;
 
  a = "hello";
  p = upcase(p);
  printf(p);
}

erreur : segmentation fault

Je ne vois pas trop où est le problème

Bonjour,

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
 
#include <stdio.h>
/* upcase1.c */
 
void upcaset(char str[])
{
	int i;
	i = 0;
	while (str[i] != '\0')
	{
		str[i] = str[i] - 32;
		i++;
	}
}
 
int main(void)
{
 
	int i; 
	char tab[] = "hello";
	upcaset(tab);
	for (i = 0; i < 5; i++)
		printf("%c", tab[i]);
	printf("\n");
	return 0;
}

cela fonctionne mais j'ai dû modifier ton code car tu écrits des chose s étranges ? (on doit pouvoir faire mieux, suis pas un expert ;-))

[Bruno Gillet]

et le main

Code :

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

1
2
3
4
5
6
 
<...>
char  *a;
<...>
  a = "hello";
<...>

Bon, le problème vien de cette ligne

En faisant comme ca, tu définis une chaine de caractère "hello" et tu affecte le pointeur à a.
Or, cette chaine est constante, c'est a dire que tu n'a pas le droit de la modifier.

[Thierry Chappuis]

Moi, je ferais comme cela, avec une gestion d'erreur sommaire:

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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
 
int upcase(char *dest, char const *src, size_t dest_size);
 
int main(void)
{
    /* -tc- La chaine a copier*/
    char const *s = "Bonjour, developpez.net!";
    /* -tc- tampon pour accueillir la copie */
    char s_up[20] = "";
    int err = 0;
 
    err = upcase(s_up, s, sizeof s_up);
 
    /* -tc- eventuellement verifier le code d'erreur retourne */
    printf("original string: %s\nprocessed string: %s\nerr: %d\n", 
            s, s_up, err);
 
    return EXIT_SUCCESS;
}
 
int upcase(char *dest, char const *src, size_t dest_size)
{
    int err = 0;
 
    if (dest != NULL && src != NULL)
    {
        if (dest_size > 1)
        {
            size_t i;
 
            for (i = 0; i < dest_size - 1 && src[i] != 0; i++)
            {
                dest[i] = toupper((unsigned char) src[i]);
            }
            dest[i] = '\0';
 
            if (src[i] != '\0')
            {
                /* -tc- Avertissement: la chaine a ete tronquee */
                err = 3;
            }
        }
        else
        {
            /* -tc- Erreur: la taille du tampon de destination doit etre 
               superieue a 1 */
            err = 2;
        }
    }
    else
    {
        /* -tc- Erreur: argument invalide */
        err = 1;
    }
 
    return err;
}

Thierry

[jayfaze]

rutabagas, la modification par ton code ne met aucune erreur, mais n'affiche rien. Il ne faut pas faire un truc du genre

Code :

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

p = strupcase[tab]

?

Bruno Gillet, merci pour tes explications. Donc la seule façon de la modifier c'est de passer par un 2nd pointeur ? (d'où mon utilisation de *tp)

[Thierry Chappuis]

Si cela ne te gène pas de modifier ta chaine en place, tu peux procéder comme cela:

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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
 
char * upcase(char *s)
{
    if (s != NULL)
    {
        size_t i = 0;
 
        while (s[i] != 0)
        {
            s[i] = toupper((unsigned char) s[i]);
            i++;
        }
    }
 
    return s;
}
 
int main(void)
{
    char chaine[] = "Bonjour, developpez!";
 
    printf("%s\n", upcase(chaine));
 
    return EXIT_SUCCESS;
}

La solution qui utilise un tampon pour la copie est dans mon ancien post précédent. Les deux fonctions sont portables.

Thierry

[rutabagas]

[QUOTE=jayfaze;2571123]rutabagas, la modification par ton code ne met aucune erreur, mais n'affiche rien. Il ne faut pas faire un truc du genre

Code :

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

p = strupcase[tab]

Pour moi cela fonctionne, mais mon OS est linux Debian Sid et j'utilise gcc-4.2,
mon code n'est pas portable(à windows ;-)) tu dois l'arranger pour ton OS ...
De plus comme le signale Bruno il faut tenir compte du cas où le mot comporte des majuscules ...
Le code de Thierry mérite plus d'attention que le mien.

Bon courage.

[jayfaze]

Ok merci pour vos explications !
J'ai compris !

alors ça donne ça

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
 
int     main()
{
  int   i;
 
  i = 0;
  char tab[] = "HeLLoLppmmMM";
  strupcase(tab);
  while (i <= my_strlen(tab))
    {
      my_putchar(tab[i]);
      i++;
    }
  return (0);
}

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
 
char    *strupcase(char str[])
{
  int   i;
 
  i = 0;
  while (str[i] != '\0')
    {
      if (str[i] >= 97 &&  str[i] <= 122)
        {
          str[i] = str[i] - 32;
          i++;
        }
      else
        {
          i++;
        }
    }
  return (str);
}

Et la ça fonctionne. Je suis sous FreeBSD

[Emmanuel Delahaye]

alors ça donne ça

Code :

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

1
2
char    *strupcase(char str[])
{

Les noms commençant par str, suivis d'une minuscule sont réservés pour les évolutions futures du langage. str_upcase() est correct.

http://emmanuel-delahaye.developpez....htm#id_reserve

Attention, le code posté est incomplet :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:2: warning: function declaration isn't a prototype
main.c: In function `main':
main.c:7: warning: implicit declaration of function `strupcase'
main.c:8: warning: implicit declaration of function `my_strlen'
main.c:10: warning: implicit declaration of function `my_putchar'
main.c: At top level:
main.c:17: error: conflicting types for 'strupcase'
main.c:7: error: previous implicit declaration of 'strupcase' was here
Process terminated with status 1 (0 minutes, 4 seconds)
2 errors, 4 warnings