taille du fichier

Version imprimable

taille du fichier

bonsoir
j'ai crée un nombre de fichier dans un dossier,
dans mon application et dans une boite dilaog j'affiche les noms de ces fichiers dans un Listbox
je veux que lorsque je clique sur un nom , j'ouvre ce fichier et quand je clique sur un button creer ,je dois modifier les valeurs existant ds mon fichier(incrementer chaque valeur de 1)
voici les sous programmes:
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 OOL CTatoCreePatchDlg::OnInitDialog() { CDialog::OnInitDialog(); // TODO: Add extra initialization here CFileFind f; CString str; // str = m_pParent->m_repApp+"\\Marques\\BM*.*"; str = m_pParent->m_repApp+"\\patchwork\\test*.*"; BOOL bMoreFiles = f.FindFile(str); while (bMoreFiles){ bMoreFiles = f.FindNextFile(); if (!f.IsDirectory()) m_mfiles.AddString((LPCSTR)f.GetFileName() );; }; return TRUE; // return TRUE unless you set the focus to a control // EXCEPTION: OCX Property Pages should return FALSE }
et la fonction qui select un nom:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 void CTatoCreePatchDlg::OnSelchangeList1() { // TODO: Add your control notification handler code here if(!m_marq.IsWindowEnabled()) m_marq.EnableWindow(TRUE); CString str1,str2; int nindex = m_mfiles.GetCurSel(); if((nindex != LB_ERR)) m_mfiles.GetText( nindex,str1); FILE *f; str2.Format("c:\\tatouage\\patchwork\\%s",str1); f=fopen(str2,"w" ); }
et dernierement la fonction qui dois modifient ces valeurs
mais j'a pas encore traier, car je sais pas comment determiner la taille du fichier et incrementer ses valeurs entiers !!
les données dsmes fichiers sont comme suit par exemple:
Code:

1 2 3 4 5 6 (5,98)-(180,11) (434,74)-(529,95) (993,48)-(974,90) (437,72)-(102,29) (101,10)-(604,74)
donc je dois incrementer chaque valeur de 1
puis j'ai le choix entre sauvgarder mon fichier par son non( çàd remplacer par le nouvel) soit creer un autre fichier et sauvegareder les nouvelle valeurs dedans
est ce qq1 a une idée??

bonjour
j'essaie d'écrire ce code
est ce qq1 peut me le corriger
( supposons que lorsque je clique sur le nom du fichier sur la ListBox je l'ouvre
)
voici la fonction qui fait les traitements:

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
 
void CTatoCreePatchDlg::OnCree() 
{
	// TODO: Add your control notification handler code here
 
// premirement on ouvre notre ficher en lecture( dans la fonction precedante)
	int caractere = 0;
	int cmp =0; //compteur qui va compter le nombre du caractere ds mon fichier
	if (f != NULL)
	{
    do
        {
            caractere = fgetc(f); // On lit le caractère
            cmp++;; // On compte
        } while (caractereActuel != EOF); // On continue tant que fgetc n'a pas retourné EOF (fin de fichier)
 
        fclose(f);
    }
	int nb;
	f=fopen(str2,"r+" );  // ici je sais pas comment j'ouvre ce meme fichier! est ce str2 reste memorisée 
		if (f != NULL)
	{
			for( int i=0;i<cmp;i++)
			{
			fscanf ( f,"%d", &nb) ;     // ici je lis juste les nombres ds mon fichier
			fprintf( f, "%d",&nb+1);     // ici je fais mon traitement, incrementer chaque nombre de 1
			}
		fclose(f);
		}
}

je sais pas si correcte ou non??

Ton code, réindenté et quelque peu commenté:

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
31
32
33
34
35
36
37
38
39
40
41
42
43
void CTatoCreePatchDlg::OnCree() 
{
	// TODO: Add your control notification handler code here
 
	// premirement on ouvre notre ficher en lecture( dans la fonction precedante)
	/* Med: Tu veux dire f est une donnée membre de la classe?
	   Ça ne se voir pas assez. 
	   Et d'ailleurs, je pense que ça ne sert à rien. */
	int cmp =0; //compteur qui va compter le nombre du caractere ds mon fichier
	if(f != NULL)
	{
		/* Med: Déplacé. */
		int caractere;
		do
		{
			caractere = fgetc(f); // On lit le caractère
			cmp++; // On compte
			/* Med: corrigé le nom de variable. */
		} while (caractere != EOF); // On continue tant que fgetc n'a pas retourné EOF (fin de fichier)
 
		fclose(f);
	}
 
 
	int nb;
	// ici je sais pas comment j'ouvre ce meme fichier! est ce str2 reste memorisée 
	/* Med: Je ne sais pas, c'est ton code, ça dépend de toi. */
	f=fopen(str2,"r+" );
	if(f != NULL)
	{
		for(int i=0 ; i<cmp ; i++)
		{
			fscanf ( f,"%d", &nb) ;     // ici je lis juste les nombres ds mon fichier
			/* Med:
			   1. Pas de & ici: C'est un nombre que tu écris, pas son adresse.
			   2. Très mauvais. La taille du nombre peut changer. 
			      Tu DOIS écrire dans un autre fichier, pas dans le même
			      (ou alors, en forçant en toutes circonstances la taille du nombre). */
			fprintf( f, "%d",&nb+1);     // ici je fais mon traitement, incrementer chaque nombre de 1
		}
		fclose(f);
	}
}

PS: Essaie de bien indenter ton code quand tu postes... Dans l'état actuel, c'est très désagréable.

13/11/2006, 14h25
meera

donc je dois d'abord copier les donnés de mon fichier dans un autre
puis les modifient dedans ce dernier
n'est ce pas???
je vais esaier de nouveau
13/11/2006, 14h41
Médinoc

Ce qu'on fait généralement, c'est écrire directement une copie modifiée du fichier sous un autre nom (en passant par un buffer en mémoire ou en écrivant directement au fur et à mesure), puis supprimer l'ancien fichier et renommer le nouveau...

moi j'ai copiée les données du mon fichier ds un autre( je le cree)
puis je modifie dedans ce dernier
j'ai pas suprimer l'ancien ( juste pour verifier ces les traitement que je fait est juste ou bien non)
voici le code

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
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
 
void CTatoCreePatchDlg::OnCree() 
{
CString str1,str2,str3;
int n= static_cast< CListBox * >(GetDlgItem(IDC_LIST1))->GetCurSel();
// ici je reccupere le nom du fichier pour l'ouvrir
 
if(n!=LB_ERR) static_cast< CListBox * >(GetDlgItem(IDC_LIST1))->GetText(n,str1);
 
 
FILE *f,*g; 
str2.Format("c:\\tatouage\\patchwork\\%s",str1); // ouvre premier fichier en lecture
f=fopen(str2,"r" );
if(!f)
{
printf("impossible d'ouvrir le fichier"); 
}
 
str3.Format("c:\\tatouage\\patchwork\\nov%s",str1); //cree un nouveau fichier qui porte le nom novstr1
f=fopen(str3,"w" ); //ouvre le deuxieme fichier en ecriture
if(!g)
{
printf("impossible d'ouvrir le fichier");
}
 
 
int cmp =0; //compteur qui va compter le nombre du caractere ds mon fichier
if((f != NULL) & (g!=NULL))
{
/* Med: Déplacé. */
int caractere;
do
{
caractere = fgetc(f); // On lit le caractère
fputc(caractere,g); //on ecrit ds le fichier g
cmp++; // On compte
/* Med: corrigé le nom de variable. */
} while (caractere != EOF); // On continue tant que fgetc n'a pas retourné EOF (fin de fichier)
 
fclose(f);
fclose(g);
}
 
 
int nb;
 
g=fopen(str3,"r+" );
if(g != NULL)
{
for(int i=0 ; i<cmp ; i++)
{
fscanf ( g,"%d", nb) ; // ici je lis juste les nombres ds mon fichier
/* Med:
1. Pas de & ici: C'est un nombre que tu écris, pas son adresse.
2. Très mauvais. La taille du nombre peut changer. 
Tu DOIS écrire dans un autre fichier, pas dans le même
(ou alors, en forçant en toutes circonstances la taille du nombre). */
fprintf( g, "%d",nb+1); // ici je fais mon traitement, incrementer chaque nombre de 1
}
fclose(g);
}

est ce que vous pouvez me le corriger?

13/11/2006, 16h11
meera

quand j'execute mon code,mon programme se bloque , je sais pas pourquoi?
le nouveau fichier a créée mais vide (0kb):?

j'initialise ts mes variables par 0
donc est ce que j'ai mis une faute lors de copie des donnees!!!
13/11/2006, 16h46
Médinoc
Tu ne dois pas lire et écrire dans un même fichier alors que tu augmentes potentiellement la taille. Les gourous eux-mêmes ne font pas ça (il est possible aux gourous de le faire quand il n'y a aucun risque d'augmenter la taille).

Tu dois lire de f et écrire sur g, et tu peux le faire au fur et à mesure (en une fois).
PS: d'ailleurs, tu n'as même pas besoin de calculer à l'avance la taille du fichier, puisque:
1. Elle risque d'augmenter
2. Tu ne l'utilises pas...

j'ai le pobleme lors de copier d'un fichier à à l'aute
j'ai fait tt en commentaire juste la partie qui me donne l'erreur
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 int cmp =0; //compteur qui va compter le nombre du caractere ds mon fichier if((f != NULL) && (g!=NULL)) { int caractere=0; do { caractere = fgetc(f); // On lit le caractère fputc(caractere,g); //on ecrit ds le fichier g cmp++; // On compte } while (caractere != EOF); // On continue tant que fgetc n'a pas retourné EOF (fin de fichier) fclose(f); fclose(g); }
je sais pas pourquoi!!
puis la deuxieme partie du mon code c d'ouvrir g, et lire juste les entiers pour puisse les modifient apres ( incrementer de 1)
et je ferme ts
mon fichier d'origine porte des donnees comme celle la:
Code:

1 2 3 4 (0,17)-(14,10) (17,11)-(8,16) (6,11)-(3,17)
j'espere que mon nouveau fichier conserve la meme format sauf les entiers qui seront incrementés

13/11/2006, 17h22
Médinoc

On peut savoir quelle est l'erreur ? :roll:

De plus, je ne connais pas le format de ton fichier. Si les données sont formattées avec des parenthèses etc., il faudra faire mieux qu'un simple fscanf("%d") pour les lires.
PS: Je croyais t'avoir dit que la mesure de la taille était inutile... :roll:
13/11/2006, 17h29
meera

il ne m'affiche aucun erreur
juste quand je clique sur le button modifier, tt bloque et la page d'exccution se ferme

pour la taille oui je souviens:oops:
je vais le modifie mnt

j'ai modifiée un petit peu le code, et j'ai eliminée le calcul du taille
voici le code:

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
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
 
void CTatoCreePatchDlg::Onmodifier() 
{
	// TODO: Add your control notification handler code here
 
	CString str1,str2,str3;
	int n= static_cast< CListBox * >(GetDlgItem(IDC_LIST1))->GetCurSel();
	// ici je reccupere le nom du fichier pour l'ouvrir
 
	if(n!=LB_ERR) static_cast< CListBox * >(GetDlgItem(IDC_LIST1))->GetText(n,str1);
 
 
	FILE *f,*g; 
	str2.Format("c:\\tatouage\\patchwork\\%s",str1); // ouvre premier fichier en lecture
	f=fopen(str2,"r" );
	if(!f)
	{
	printf("impossible d'ouvrir le fichier"); 
	}
 
	str3.Format("c:\\tatouage\\patchwork\\nov%s",str1); //cree un nouveau fichier qui porte le nom novstr1
	g=fopen(str3,"w" ); //ouvre le deuxieme fichier en ecriture
	if(!g)
	{
	printf("impossible d'ouvrir le fichier");
	}
 
 
	if((f != NULL) && (g!=NULL))
	{
 
	int caractere=0;
	while(caractere = fgetc(f)!= EOF) // On continue tant que fgetc n'a pas retourné EOF (fin de fichier)
	{
 
	fputc(caractere,g); //on ecrit ds le fichier g
 
 
	}  
 
	fclose(f);
	fclose(g);
	}
 
		int nb=0;
 
	g=fopen(str3,"w" );
	if(g != NULL)
	{
	while(nb = fgetc(g)!= EOF)
	{
 
	fprintf( g, "%d",nb+1); // ici je fais mon traitement, incrementer chaque nombre de 1
	}
	fclose(g);
	}
}

j'ai resolu le probléme de blocage mais le fichier crée restera vide ( 0kb)

par contre qd j'excute l'ancien

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
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
 
 
void CTatoCreePatchDlg::Onmodifier() 
{
	// TODO: Add your control notification handler code here
 
	CString str1,str2,str3;
	int n= static_cast< CListBox * >(GetDlgItem(IDC_LIST1))->GetCurSel();
	// ici je reccupere le nom du fichier pour l'ouvrir
 
	if(n!=LB_ERR) static_cast< CListBox * >(GetDlgItem(IDC_LIST1))->GetText(n,str1);
 
 
	FILE *f,*g; 
	str2.Format("c:\\tatouage\\patchwork\\%s",str1); // ouvre premier fichier en lecture
	f=fopen(str2,"r" );
	if(!f)
	{
	printf("impossible d'ouvrir le fichier"); 
	}
 
	str3.Format("c:\\tatouage\\patchwork\\nov%s",str1); //cree un nouveau fichier qui porte le nom novstr1
	g=fopen(str3,"w" ); //ouvre le deuxieme fichier en ecriture
	if(!g)
	{
	printf("impossible d'ouvrir le fichier");
	}
	int cmp =0; //compteur qui va compter le nombre du caractere ds mon fichier
	if((f != NULL) && (g!=NULL))
	{
 
	int caractere=0;
	do
	{
	caractere = fgetc(f); // On lit le caractère
	fputc(caractere,g); //on ecrit ds le fichier g
	cmp++; // On compte
 
	} while (caractere != EOF); // On continue tant que fgetc n'a pas retourné EOF (fin de fichier)
 
	fclose(f);
	fclose(g);
	}
 
	int nb=0;
 
	g=fopen(str3,"w" );
	if(g != NULL)
	{
	for(int i=0 ; i<cmp ; i++)
	{
	fscanf ( g,"%d", nb) ; // ici je lis juste les nombres ds mon fichier
 
	fprintf( g, "%d",nb+1); // ici je fais mon traitement, incrementer chaque nombre de 1
	}
	fclose(g);
 
	}
		OnOK();
	}

cette fois çi le fichier ne restras pas vide mais rempli par:

Code:

1
2
 
ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح1ح

je sais pas pourquoi!!!!

13/11/2006, 21h15
Médinoc

Là encore, tu fais deux écritures.
Ça ne sert à rien!
13/11/2006, 21h38
meera

2 écritures!! comment??
franchement j'ai pas compris ce que vous me demander de faire
conçernant la taille, je l ' ai éliminée
et d'autres chose je sais pas:?

Tiens, voilà un exemple de code supposé faire ce que je pense que tu veux faire.
Comme c'est un programme MFC, je n'ai même pas essayé de le compiler, car je n'ai pas que ça à faire, non mais.

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
void PurgeStream(FILE *f)
{
int c;
while((c = fgetc(f)) != '\n' && c != EOF)
	{}
}
 
 
void IncrementeFichier(LPCTSTR str)
{
	FILE *pfIn = NULL;
	FILE *pfOut = NULL; 
	CString strIn;
	CString strOut;
 
	strIn.Format("c:\\tatouage\\patchwork\\%s",str); // ouvre premier fichier en lecture
	pfIn = _tfopen(strIn, _T("r"));
	if(pfIn == NULL)
	{
		printf("impossible d'ouvrir le fichier"); 
	}
 
	strOut.Format("c:\\tatouage\\patchwork\\nov%s",str);
	pfOut = _tfopen(strOut, _T("w")); //ouvre le deuxieme fichier en ecriture
	if(pfOut == NULL)
	{
		printf("impossible d'ouvrir le fichier");
	}
 
	//Copie les nombres du premier fichier en les incrémentant tous de 1
	if((pfIn != NULL) && (pfOut!=NULL))
	{
		int a, b, c, d;
		while(fscanf(pfIn, "(%d,%d)-(%d,%d)", &a, &b, &c, &d)==4)
		{
			PurgeStream(pfIn);
 
			//Traitement:
			//Incrémente
			a++; b++; c++; d++;
 
			//Écrit
			fprintf(pfOut, "(%d,%d)-(%d,%d)\n", a, b, c, d);
		}
	}
	//Fermeture du ou des fichiers dont l'ouverture a réussi
	if(pfIn)
		fclose(pfIn);
	if(pfOut)
		fclose(pfOut);
}

Normalement, si ton fichier d'origine est comme ça:

Code:

1
2
3
4
(0,17)-(14,10) 
(17,11)-(8,16) 
(5,99)-(9,42)
(6,11)-(3,17)

Le fichier de sortie devrait ressembler à ceci:

Code:

1
2
3
4
(1,18)-(15,11) 
(18,12)-(9,17)
(6,100)-(10,42)
(7,12)-(4,18)

Note l'augmentation de taille pour la troisième ligne. C'est pour ça qu'on ne peut pas modifier "sur place"

Pose des questions si tu ne comprends pas mon code, mais n'utilise pas le tien, il est bourré d'erreurs.

encore une fois, j'ai modifiée les nstructions qui font le copier
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 if((f != NULL) && (g!=NULL)) { int caractere=0; while( !feof(f)) { fscanf(f,"%d",&caractere); fprintf(g,"%d",caractere); } fclose(f); fclose(g); }
mais lors de l'execution , le programme se bloque:? :?

13/11/2006, 21h58
Médinoc

As-tu vu le code non-testé de mon dernier post?
13/11/2006, 22h09
meera

vraiment vraiment je sais pas comment je vous remercier
je l'exécute, il ça marche trés bien
le code est tres clair, le le comprends
encore une fois je te remercie du mon coeur:P