Langage C : lire un tableau depuis un fichier [Résolu]

Bonjour Char Snipeur,

scanf("%\n") ?
Tu souhaites faire quoi ?
Ensuite, j'imagine que tu voulais mettre fscanf au lieu du scanf ?
Et pourquoi une boucle j<2 ?
Sinon, attention au redirection...
Il n'y a pas de solution portable pour restaurer le flux sur la sortie standard.
Pour réussir, il faut jouer avec dup / dup2 et sauvegarder le descripteur par dup(fileno(stdout));
Bref rien de très joli. Il vaut mieux éviter ce genre de redirection.

Cdlt,

ouai, c'est plein de merde. j'ai oublié le fscanf.
Je voulais juste donner l'algorithme général. La boucle sur "j", c'est car tu as deux grilles. Mais après relecture, il y en a 4 : 2 pour chaque joueur.
Merci de ta réponse sur stdout, ça me semblait trop simple aussi.

une solution si tu veux faire qu'une routine, c'est de modifier ta fonction aff_grille et mettre :
aff_grille(Grille A,Grille B,FILE * s=stdout);
vérifier qu'en C on peut bien mettre un argument par défaut.
Ensuite, tu modifie tout les printf de ta fonction par des fprintf(s,...)
Dans le reste du code, la seule modification à faire sera alors d'appeler aff_grille avec en argument l'identifiant du fichier.

Nop, pas d'argument par défaut en C..

Pour écrire dans un fichier, tu peux simplement utliser un fprintf.

Sinon, tu as deux joueurs dans une bataille navale, il faut donc 2 grilles. Chacune des grilles contiendra 4 valeurs (0 si aucun bateau et pas de missile lancée sur cette zone, 1 si présence bateau et pas de missile lancée, ...). Tu peux faire un enum pour plus de clarté.

Ensuite, pour ta grille, il te faut une autre dimension pour stocker les deux grilles. Par exemple : int grille[2][N][N]; Sinon tu peux utiliser une structure aussi.

Ensuite plutôt que t'embêter avec fprintf/fscanf, un simple fread/fwrite te fera la lecture/écriture en une passe.

Cordialement,

t'inquiète pas pr les grilles, j'ai déja fait une grande partie du programme du jeu. Simplement, j'aimerai savoir si cette version de load, codée à partir de vos commentaires, est correcte. Je vous mets aussi les structs de départ pr pouvoir comprendre.

#define N 10

typedef char Grille[N][N];
typedef struct{
char typeB;
int Nbcases;
int absOrig;
int ordOrig;
int Orientation[2];
}bateau;
typedef struct{
int Nbship;
bateau tabShip[N];
}listeBateau;



void load(Grille A,Grille B,Grille A2,Grille B2,listeBateau lp,listeBateau lo){
FILE* fi=fopen("save.txt","r");
for(n=0;n<N;++n){
for(m=0;m<N;++m){
fscanf("%c",&A[n][m]);
}
fscanf("\n");
}
for(n=0;n<N;++n){
for(m=0;m<N;++m){
fscanf("%c",&B[n][m]);
}
fscanf("\n");
}
fscanf("\n");
for(n=0;n<N;++n){
for(m=0;m<N;++m){
fscanf("%c",&A2[n][m]);
}
fscanf("\n");
}
for(n=0;n<N;++n){
for(m=0;m<N;++m){
fscanf("%c",&B2[n][m]);
}
fscanf("\n");
}
fscanf("%d",&lp.Nbship);
for(i=0;i<N;i++){
fscanf(" %c,%d,%d,%d,%d,%d",&((lp.tabShip)[i].typeB),&((lp.tabShip)[i].Nbcases),&((lp.tabShip)[i].absOrig),&((lp.tabShip)[i].ordOrig),&((lp.tabShip)[i].Orientation[0]),&((lp.tabShip)[i].Orientation[1]));
}
fscanf("\n");
fscanf("%d",&lo.Nbship);
for(i=0;i<N;i++){
fscanf(" %c,%d,%d,%d,%d,%d",&((lo.tabShip)[i].typeB),&((lo.tabShip)[i].Nbcases),&((lo.tabShip)[i].absOrig),&((lo.tabShip)[i].ordOrig),&((lo.tabShip)[i].Orientation[0]),&((lo.tabShip)[i].Orientation[1]));
}
fclose(fi);
}

Je sais que niveau lisibilité c'est pas la fête mais bon si ça marche sa me suffira pour le moment. Quant à save je prévois de faire la même chose en remplaçant les fscanf par des fprintf.

Effectivement, c'est pas clair du tout.
Quand tu postes du code, utilise les balises "code" (à droite du bouton souligné), ça sera beaucoup plus lisible pour nous.
Sinon, tu as mis des fscanf("\n"), ça ne correspond à rien.
Ensuite, ton code est bien trop compliqué...
En une ligne, tu peux faire ta fonction load et open avec fread et fwrite. Pas besoin de faire de boucle etc.

Cdlt,

ya pas besoin de lire les retours a la ligne? ah ok....mais par contre je vois pas comment en une ligne je peux faire ma fonction load ou save vu que fread est assez compliqué à utiliser et j'ai 4 grilles et 2 structures à charger/sauvegarder.
On a pas encore vraiment étudié les pbs de fichier et mémoire en c donc je comprends pas tout...

fwrite(grilleA,sizeof(grilleA),1,fs); et hop tu écrit une grille, en binaire en plus je crois : donc gain de place.

le probleme avec fscanf est que cette fonction lit mot apres mot
Sauf que fscanf n'est pas une fonction qui lit mot à mot. Cela dépend du format passé en argument. Dans nos exemples, on utilisait fscanf(fp,"%c",...) et dans ce cas ça lit caractère par caractère comme ton fgetc... Un petit lien pour voir ce que fait scanf : http://www.linux-france.org/article/man-fr/man3/scanf-3.html

alors que tu conviens avec moi que cela est un peut rude parcontre avec le cas de caractere par caractère tu gere ton contenue comme tu veux
Oui je conviens. Par contre, tu conviendras avec moi qu'un simple fread est encore moins rude que la gestion du caractère par caractère...

Ce n'est pas la fonction fget mais fgetc.
Et c'est un peu le même problème que la fonction scanf("%c",...), cela rend le code compliqué car il faut user de boucle for et traduire la "complexité" des structures.
Un simple fread suffit...