Identifiants valides

Version imprimable

Bonjour à tous:),

Alors voila j'ai un problème avec ce 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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
 
typedef struct
{
    char *essai;
}nb;
 
int main(void)
{
    int N, i;
    nb *test;
 
    printf("Donnez le nombre de variable a tester: ");
    scanf("%d", &N);
    test = malloc(N*sizeof(nb));
    if (test == NULL)
    {
        printf("Erreur: allocation échoué");
    }
    for (i=0;i<N;i++)
    {
        printf("Entrez vos propositions: ");
        gets(test[i].essai);
    }
    for (i=0;i<N;i++)
    {
        if (test[i].essai[i] == '-' || isdigit(test[i].essai[1]) || test[i].essai[i] < 0)
        {
            printf("NO");
        }
        else
        {
            printf("YES");
        }
    }
    free(test);
    return 0;
}

Pourquoi quand je l'exécute il m'affiche deux fois d'un coup "Entrez vos propositions". Merci d'avance:) (ce code sert à savoir si le nom taper est valide ou non pour une variable ou une fonction)

09/05/2008, 12h41
emprex

dans ta deuxieme boucle l'une des conditions n'est -elle pas incomplète ?
09/05/2008, 12h56
droggo

correction balises
Gao,

Moi je vois surtout ceci
Code:

1 2 3 4 5 for (i=0;i<N;i++) { printf("Entrez vos propositions: "); gets(test[i].essai); }
alors que les pointeurs test[i].essai ne sont pas initialisés, pas de mémoire réservée... :)
09/05/2008, 13h31
nicolas.sitbon

Code:

gets(test[i].essai);

gets() est officiellement obsolète et déprécié par la norme, elle ne doit plus être utilisé (et n'aurait jamais dû l'être d'ailleurs).
09/05/2008, 15h18
scary

Citation:

Envoyé par droggo

alors que les pointeurs test[i].essai ne sont pas initialisés, pas de mémoire réservée... :)

J'ai pourtant alloué la mémoire qu'il fallait avec le malloc non ? je comprend pas ce que tu veux dire:oops:

Pour gets je l'ai remplacer par scanf.
09/05/2008, 15h25
nicolas.sitbon

Citation:

Envoyé par scary

J'ai pourtant alloué la mémoire qu'il fallait avec le malloc non ?

Pour test oui mais pas pour essai!
09/05/2008, 16h00
droggo

Hie,

Citation:

Envoyé par scary

J'ai pourtant alloué la mémoire qu'il fallait avec le malloc non ? je comprend pas ce que tu veux dire

Pour le tableau de structures, mais pas chaque pointeur dans chaque structure !!
09/05/2008, 16h14
scary

A d'accord et comment je peux faire cela ? je fais un malloc pour chaque possibilité ? vous auriez pas un petit exemple que lequel je pourrais m'appuyer svp?
09/05/2008, 17h43
droggo

Feo,

Citation:

Envoyé par scary

A d'accord et comment je peux faire cela ? je fais un malloc pour chaque possibilité ? vous auriez pas un petit exemple que lequel je pourrais m'appuyer svp?

Standard :

Pour chaque structure de ton tableau :

- commencer par mettre les pointeurs à NULL, pour éviter l'utilisation d'un pointeur invalide

- puis, AVANT d'utiliser un pointeur, réserver de la place mémoire (SAUF si appel d'une fonction qui réserve elle-même la place !!)

- quand les données ne sont plus nécessaires, LIBÉRER la mémoire.

Bon j'ai préférer faire comme ça car je ne voyais vraiment pas comment faire ce que tu m'as dis.

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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
 
typedef struct
{
    char essai[100];
}nb;
 
int main(void)
{
    int N, i, j;
    nb *test;
 
    printf("Donnez le nombre de variable a tester: ");
    scanf("%d", &N);
    test = malloc(N*sizeof(nb));
    if (test == NULL)
    {
        printf("Erreur: allocation échoué");
    }
    for (i=0;i<N;i++)
    {
        printf("Entrez vos propositions: ");
        scanf("%s", test[i].essai);
    }
    for (i=0;i<N;i++)
    {
        for (j=0;j<N;j++)
        {
            if (test[i].essai[j] == '-' || isdigit(test[i].essai[1]) || test[i].essai[j] < 0)
            {
                printf("NO\n");
            }
            else
            {
                printf("YES\n");
            }
        }
    }
    free(test);
    return 0;
}

La ça marche mais j'aimerais bien que vous me montriez avec l'allocation svp. Puis la vérification ne marche pas:(

Voici quelques corrections:

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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
 
typedef struct
{
    char essai[100];
} nb;
 
/* -tc- Elimine les caracteres non lus par scanf() */
void purge(void)
{
    int c;
 
    while ((c = fgetc(stdin)) != '\n' && c != EOF)
    {
    }
}
 
int main(void)
{
    int N;
    int i, j;
    nb *test;
    int rv;
    int ret = 0;
 
    do
    {
        printf("Donnez le nombre de variable a tester: ");
        /* -tc- Pour etre portable, il est conseille de forcer l'affichage
           ici en purgeant explicitement le flux de sortie standard */
        fflush(stdout);
        rv = scanf("%d", &N);
        purge();
    }
    while (rv != 1);
 
    test = malloc(N * sizeof(nb));
    if (test != NULL)
    {
        /* -tc- ATTENTION: les lignes qui suivent ne doivent pas etre executees
           lorsque l'allocation n'a pas reussi */
        for (i = 0; i < N; i++)
        {
            printf("Entrez vos propositions: ");
            fflush(stdout);
            /* -tc- Tu n'utilises pas correctement scanf(). Pour la saisie de
               chaines, je prefere fgets(). Au minimum, limite le nombre de
               caracteres pouvant etre saisis par scanf(). */
            scanf("%99s", test[i].essai);
            purge();
        }
 
        for (i = 0; i < N; i++)
        {
            int ok = 1;
 
            for (j = 0; ok == 1 && test[i].essai[j] != 0; j++)
            {
                /* -tc- Attention a tes conditions. Ceci devrait fonctionner si
                   la locale est celle par defaut (i.e. "C") */
                if ((j == 0 && isdigit((unsigned char) test[i].essai[j])) ||
                        (isalnum((unsigned char) test[i].essai[j]) == 0 &&
                         test[i].essai[j] != '_'))
                {
                    ok = 0;
                }
            }
 
            printf("%s\n", (ok == 1) ? "Oui" : "Non");
        }
 
        free(test), test = NULL;
    }
    else
    {
        fprintf(stderr, "Erreur: allocation impossible\n");
        ret = EXIT_FAILURE;
    }
    return 0;
}

Thierry

10/05/2008, 12h23
scary

Merci Thierry:) ça marche super comme je voulais:mrgreen:
10/05/2008, 12h40
Emmanuel Delahaye

Attention au vocabulaire...

Citation:

Envoyé par droggo

- commencer par mettre les pointeurs à NULL, pour éviter l'utilisation d'un pointeur invalide

Mettre les pointeurs à NULL (pointeur invalide) permet d'éviter l'utilisation de pointeurs indéterminés.

Citation:

- puis, AVANT d'utiliser un pointeur, réserver de la place mémoire (SAUF si appel d'une fonction qui réserve elle-même la place !!)

Un pointeur, comme n'importe quelle variable, doit avoir été initialisé avant usage. Une bonne façon d'initialiser un pointeur est de lui donner comme valeur, l'adresse d'un bloc alloué par malloc().