Discussion :

[virtuadrack]

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
 
#include <ctype.h>
#include <stdio.h>
 
isLow(char *str)
{
        if(str != NULL)
        {
                char *p;
                for(p = str; *p != '\0'; p++)
                {
                        *p = islower((unsigned) (*p));
                }
        }
}
 
 
int main(void){
 
char str[] = "mot";
 
if(isLow(str)) printf("%s is in lowercase", str);
return 0
}

Cette fonction est pour verifier si le mot est en minuscule. Ya t-il qqchose a corriger dans la fonction isLow ?

[Gruik]

Salut,

On dirait que tu te melanges completement les pinceaux entre islower() et tolower(). Le role de ces 2 fonctions sont dans leur nom : "is" pour savoir si [...] et "to" pour transformer en [...].
Ici, tu modifies la chaine passée à isLow(), pourtant, tu testes le retour de isLow() donc tu voulais bien savoir si [...]. Donc decide toi : tu veux convertir ta chaine en minuscules ou savoir si elle est en minuscules?

[Thierry Chappuis]

A quoi ça te sert de convertir la valeur passée à *lower() en un entier de type unsigned int? La manière correcte de procéder est de convertir en (unsigned char), non?

Thierry

[virtuadrack]

Salut,

On dirait que tu te melanges completement les pinceaux entre islower() et tolower(). Le role de ces 2 fonctions sont dans leur nom : "is" pour savoir si [...] et "to" pour transformer en [...].
Ici, tu modifies la chaine passée à isLow(), pourtant, tu testes le retour de isLow() donc tu voulais bien savoir si [...]. Donc decide toi : tu veux convertir ta chaine en minuscules ou savoir si elle est en minuscules?

Je voudrais savoir si elle est en miniscule...

Thierry, j'ai pas trop compris quand tu parle de convertir en unisgned char, il me semble que c'est qui a ete fait... ?

[Thierry Chappuis]

Voici mes corrections:

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 <ctype.h>
#include <stdio.h>
 
/*-tc- Attention au prototype. Tu dois déclarer le type de la valeur 
   retournee par isLow() */
int isLow(char const *str)
{
    int rv = 0;
 
    if (str != NULL)
    {
        char const *p;
 
        for (p = str; *p != '\0' && islower((unsigned char) *p); p++)
        {
        }
 
        rv = (*p == '\0');
    }
 
    return rv;
}
 
 
int main(void)
{
 
    char str[] = "mot";
 
    if (isLow(str))
    {
        printf("%s is in lowercase", str);
    }
    return 0;
}

Thierry

[virtuadrack]

Merci,

Et si on voulait modifier la fonction isLow() de facon a ce qu'elle verifie si une seule lettre ou toutes sont minuscules ?

Si la chaine de charactere "str" est definie en tant que "rent", isLow(str) sera vraie. Si str est definie en tant que "Rent" elle est fausse. Comment faire pour qu'elle retourne vraie dans ce sens ?


VD

[Thierry Chappuis]

Merci,

Et si on voulait modifier la fonction isLow() de facon a ce qu'elle verifie si une seule lettre ou toutes sont minuscules ?

Si la chaine de charactere "str" est definie en tant que "rent", isLow(str) sera vraie. Si str est definie en tant que "Rent" elle est fausse. Comment faire pour qu'elle retourne vraie dans ce sens ?


VD

Si tu désires que isLow() retourne vrai dans le cas où toutes les lettres sont minuscules et dans le cas ou seule la première lettre est majuscule, il te suffit de commencer le test au 2e caractère:

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
int isLow(char const *str)
{
    int rv = 0;
 
    if (str != NULL)
    {
        char const *p;
 
        for (p = str + 1; *p != '\0' && islower((unsigned char) *p); p++)
        {
        }
 
        rv = (*p == '\0');
    }
 
    return rv;
}

Si tu désires accepter la présence d'une (et une seule) lettre majuscule dans le mot:

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
int isLow(char const *str)
{
    int rv = 0;
 
    if (str != NULL)
    {
        char const *p;
        int nmaj = 0;
 
        for (p = str; *p != '\0' && nmaj < 2; p++)
        {
            if (islower((unsigned char) *p) == 0)
            {
                ++nmaj;
            }
        }
 
        rv = (*p == '\0');
    }
 
    return rv;
}

ATTENTION toutefois à ne pas créer de confusion avec l'identifiant de ta fonction. isLow() signifie pour moi "retourne vrai si le mot pointé par str est écrit en minuscules". Je ne m'attend donc pas à ce que isLow() retourne vrai si le premier ou n'importe quel autre caractère est majuscule.

Thierry

[virtuadrack]

Dans le meme contexte, si j'ai la fonction suivante qui rend toute la chaine en majuscule

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void uppercase(char *str)
{
        if(str != NULL)
        {
                char        *p;
                for(p = str; *p != '\0'; p++)
                {
                        *p = toupper((unsigned) (*p));
                }
        }
}

Si je veux ne rendre que la premiere lettre en majuscule ?

[Emmanuel Delahaye]

Si je veux ne rendre que la premiere lettre en majuscule ?

Bah tu ne fais pas de boucles, tu ne modifies qu'une fois... C'est un peu évident, non ?

[virtuadrack]

Bah tu ne fais pas de boucles, tu ne modifies qu'une fois... C'est un peu évident, non ?

Enlever la boucle et ne modifier qu'une fois, la console m'affiche une Segmentation fault.

[Gruik]

Si tu enleves le for entier, p n'est pas initialisé et pointe sur ce que j'appelle "l'espace intersideral incommensurable".

[virtuadrack]

Si tu enleves le for entier, p n'est pas initialisé et pointe sur ce que j'appelle "l'espace intersideral incommensurable".

Merci