tableau avec des chaines

Version imprimable

Bonsoir,

Voilà j'essaie d'écrire dans un fichier des index aléatoires d'un tableau contenant des chaînes. Mais mon code juste après la compilation, à l'ouverture de la console, fait planter mon programme.
Le voici :

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
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <time.h>
#include <string.h>
 
int main(int argc, char *argv[]){
 
    FILE* fichier = NULL;
    fichier = fopen("test.txt", "a");
 
    if (fichier != NULL)
    {
 
        char *tableau[] = {'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', '\n', NULL};
        int i;
        srand(time(NULL));
 
    for(i = 0; i <= 1000; i++){
 
            fputs(tableau[rand() % 9], fichier);
 
        }
 
         // Ecriture du caractère A
        fclose(fichier);
    }
 
    return 0;
}

Selon vous le plantage est dû à quoi ?

16/10/2009, 21h14
Pouet_forever

Bonsoir,

Il faut faire attention quand tu déclares cette ligne là :

Code:

char *tableau[] = {'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', '\n', NULL};

Car là tu déclares un tableau de pointeur mais les arguments ne sont pas bons :)
Ils sont considérés comme des nombres, donc autant faire un tableau de char tout court ;)
Il faut que tu déclares tes arguments comme des chaînes de caractère, comme ça tu pointera sur des chaînes de caractères constantes ;)

Essaye avec ça :

Code:

char *tableau[] = {"A", "B", "C", "D", "E", "F", "G", "H", "\n", NULL};

A+
16/10/2009, 21h33
guitz

Pour un premier message, tu as fait fort ;)

Merci:ccool:
17/10/2009, 00h46
TheDrev

le NULL a la fin du tableau est inutile si tu l'utilise pour determiner a fin de la chaine, il faut utiliser '\0'
mais comme tu la crée "en dur" au moment de l'initalisation, ca caractere est deja présent et automatiquement ajouté par le compilateur (mais peu etre met tu NULL pour une autre raison).
17/10/2009, 11h47
diogene

Citation:

Envoyé par TheDrev

le NULL a la fin du tableau est inutile si tu l'utilise pour determiner a fin de la chaine, il faut utiliser '\0'
mais comme tu la crée "en dur" au moment de l'initalisation, ca caractere est deja présent et automatiquement ajouté par le compilateur (mais peu etre met tu NULL pour une autre raison).

Il s'agit d'un tableau de chaînes, pas d'une chaîne. L'utilisation de NULL pour terminer ce tableau est donc justifiée.