Lire dans un fichier en C

Au minimum :

#include <string.h>
#include <stdlib.h>
#include <stdio.h>

int main()
{
	char* x;
	printf("Entrez x:");
	scanf("%s",x);

	FILE* fichier = NULL;
	int caractereActuel = 0;
	fichier = fopen(x, "r");

	if (fichier == NULL){
		printf("fichier inexistant\n");
	}

	while (caractereActuel != EOF)
	{
		caractereActuel = fgetc(fichier);
		printf("%c", caractereActuel);
	}

	fclose(fichier);
	return 0;
}

juste quelques observations :-)

si on n'alloue pas de mémoire pour le pointeur

x

, il ne va rien arriver de bon quand

scanf()

tentera d'y mettre quelque chose ;-) ... c'est même certainement l’origine de l'erreur expérimentée par momoadmo.

l'inclusion de

#include <string.h>

n'est pas nécessaire, pareil pour

#include <stdlib.h>

.

en C, le prototype correct de main sans arguments est

int main(void)

sur le forum, la balise de code correcte pour la coloration syntaxique du C est :

<code c>

code ici

</code>


Dal

Ah oui effectivement pas bien de ne pas avoir malloc, je me demande d'ailleurs pourquoi ça fonctionne...
Bon après pour le int main (void) c'est un peu le cadet de ses soucis ahah, mais tu as raison.
Autant pour moi pour le %s je pensais que ça venait de string.h, je l'inclus toujours quand j'utilise des string, j'avoue que je ne m'étais jamais posé la question !
Malheureusement je ne peux pas éditer mon code ...

YCN-

pour éviter les complications de

malloc()

, momoadmo pourrait réserver la mémoire nécessaire au stockage du nom de fichier en déclarant un tableau de char, par exemple :

char x[255];

.

pour faire un code plus robuste, Il serait bien, aussi, de s'assurer que la saisie ne dépasse pas cette limite, faire quelque chose si c'est le cas, et purger le flux stdin.

Par exemple avec scanf et un spécificateur de taille

#include <stdio.h>

int main(void) {
    char x[255];
    char c;

    printf("Veuillez saisir une chaîne de 254 caractères "
            "au plus.\n");
    if (scanf("%254[^\n]",x) == 1) {
        printf("J'ai récupéré : [%s]\n", x);
        if (getchar() != '\n') {
            printf("Erreur : je n'ai pas pu tout récupérer"
                    "car votre saisie est trop longue\n");
            while ((c = getchar()) != '\n' && c != EOF)
                /* purger le flux si le programme doit continuer*/ ;
            return 1;   /* dans cet exemple, je termine */
        }
    }

    return 0;
}

tu dois inclure

string.h

quand ton code a besoin d'appeler une fonction dont le prototype est déclaré dans

string.h

(par exemple

strlen()

)

http://www.cplusplus.com/reference/cstring/

ou le type

size_t

ou la macro

NULL

, qui sont aussi définis dans cet entête.

Dal