Discussion :

[guitz]

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 :

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

[Pouet_forever]

Bonsoir,

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

Code :

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

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 :

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

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

A+

[guitz]

Pour un premier message, tu as fait fort

Merci

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

[diogene]

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.