Discussion :

[scary]

Bonjour à tous,

Alors voila j'ai un problème avec ce 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
#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)

[emprex]

dans ta deuxieme boucle l'une des conditions n'est -elle pas incomplète ?

[droggo]

Gao,

Moi je vois surtout ceci

Code :

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

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

[nicolas.sitbon]

Code :

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

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

[scary]

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

Pour gets je l'ai remplacer par scanf.

[nicolas.sitbon]

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

Pour test oui mais pas pour essai!

[droggo]

Hie,

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

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

[droggo]

Feo,

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.

[scary]

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

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

[Thierry Chappuis]

Voici quelques 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
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

[scary]

Merci Thierry ça marche super comme je voulais

[Emmanuel Delahaye]

Attention au vocabulaire...

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

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