Utilisation CFile::Seek() et message d'erreur

Version imprimable

Bonjour à tous,

J'utilise un CFile pour lire un fichier texte depuis la fin. Le but est d'éviter de parcourir tout le fichier qui peut-être très grand (plusieurs méga...) pour gagner du temps.
L'utilisation est un peu particulière car le fichier lu est continuellement écrit, mais de toutes façons je ne cherche pas à récupérer les lignes qui ont été écrites entre le moment où j'ai ouvert le fichier et celui où des lignes ont été rajoutées.
Le problème est que j'ai le message d'erreur suivant qui apparaît : "Seek failed on an unnamed file".

J'ai essayé mon code sur un fichier 'statique' et là, pas de problème.

Est-ce que ma méthode est bonne ?
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 CFile f; CFileException e; CString sLine = ""; int iLinesNumber = 0; bool bError =false; if(!f.Open(sLogFilePath,CFile::modeRead,&e)) return -1; int iShiftFromEnd = -1; char cCurrentChar = ' '; CString sTemp; CStringArray oFileNamesArray; oFileNamesArray.RemoveAll(); while(iLinesNumber < m_MaxDefect && !bError) //on lit chaque ligne du fichier en partant de la fin { f.Seek(iShiftFromEnd,CFile::end); if(f.Read(&cCurrentChar,1) == 0) bError = TRUE; if(cCurrentChar == '\n') //fin d'une ligne { if(sLine.Find("&&&",0)==0) //c'est une ligne qui contient un nom de fichier { ReadFileNameInLine(sLine,sTemp); // extraction du nom du fichier oFileNamesArray.Add(m_ImageDirectory + '\\' + sTemp); iLinesNumber ++; } sLine = ""; //on efface la ligne dans tous les cas } else sLine.Insert(0,cCurrentChar); iShiftFromEnd--; } f.Close();

25/01/2006, 17h15
Invité
Donc, pour résumer: ton code fonctionne sur un fichier statique mais pas dynamique (en train d'être écrit).

Ca devrait te mettre la puce à l'oreille: A ton avis le Seek() il est placé sur le pointeur de lecture ou d'écriture ? Car il faut savoir qu'il y a deux pointeurs quand tu joues avec n'importe quelle libstream.

J'essaie de te mettre sur la voie: je n'ai pas exécuté ton code. Au fait:
Code:

1 2 sLine = "";
Je préfère ca:
Code:

1 2 sLine.Empty();
25/01/2006, 17h32
farscape

salut,
je rajouterais CFile::shareDenyWrite a l'open.
ensuite je ferais un catch sur le seek voir (CFileException
et je reouvrerais le fichier en cas d'erreur ...
:D
25/01/2006, 17h37
nmarf

Merci pour vos réponses.

Citation:

Envoyé par rockthat

Donc, pour résumer: ton code fonctionne sur un fichier statique mais pas dynamique (en train d'être écrit).

Tout à fait.

Citation:

Envoyé par rockthat

Car il faut savoir qu'il y a deux pointeurs quand tu joues avec n'importe quelle libstream.

Bon, ça je ne le savais pas. Le problème viendrait donc de la ligne

Code:

f.Seek(iShiftFromEnd,CFile::end);

Citation:

Envoyé par rockthat

Au fait [...]

Ok c'est noté. :D

Citation:

Envoyé par farscape

salut,
je rajouterais CFile::shareDenyWrite a l'open.

Ca je n'ai pas le droit, l'écriture doit être prioritaire sur la lecture.
Justement, je viens de rajouter CFile::shareDenyNone...
A la limite, si la lecture se passe mal, on sort calmement de la fonction et on attend le prochain appel (toutes les 5 secondes)
25/01/2006, 17h48
farscape

il faut donc que tu catch le seek ...
:D

Alors j'ai essayé deux choses :
- le try/catch pour la fonction seek
- positionner le pointeur à la fin et ne plus utiliser CFile::end à chaque boucle.
Depuis le try/catch, Je n'ai plus le message d'erreur, par contre je n'avance plus non plus dans la suite de la fonction. :(

Voici à quoi ressemble le code maintenant :

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
CFile f;
	CFileException e;
	CString sLine;
	int iLinesNumber = 0;
	bool bError =false;
 
	if(!f.Open(sLogFilePath,CFile::modeRead | CFile::shareDenyNone,&e))
		return -1;
 
	int iShiftFromEnd = -1;
	char cCurrentChar = ' ';
	CString sTemp;
	CStringArray oFileNamesArray;
	oFileNamesArray.RemoveAll();
	f.Seek(0,CFile::end);
 
	while(iLinesNumber < m_MaxDefect && !bError)	//on lit chaque ligne du fichier en partant de la fin
	{
		try
		{
			f.Seek(iShiftFromEnd,CFile::current);
			if(f.Read(&cCurrentChar,1) == 0)
				bError = TRUE;
		}
		catch (CFileException* e)
		{
			return -1;
		}
 
		if(cCurrentChar == '\n')			//fin d'une ligne
		{
			if(sLine.Find("&&&",0)==0)		//c'est une ligne de défaut
			{
				ReadFileNameInLine(sLine,sTemp);	// extraction du nom du fichier
 
				oFileNamesArray.Add(m_ImageDirectory + '\\' + sTemp);
 
				iLinesNumber ++;
			}
			sLine.Empty();	//on efface la ligne dans tous les cas
		}
		else
			sLine.Insert(0,cCurrentChar);
 
		iShiftFromEnd--;
	}
 
	f.Close();

25/01/2006, 18h07
farscape

ça serait pas iShiftFromEnd-=2;
pour tenir compte de la lecture ?
:D
25/01/2006, 18h16
nmarf

Citation:

Envoyé par farscape

ça serait pas iShiftFromEnd-=2;
pour tenir compte de la lecture ?
:D

Ca serait étonnant , puisque ça fonctionne avec -1 et un fichier statique.
:?:
25/01/2006, 18h20
farscape

en fait c'est plutot -2 direct .
:D
26/01/2006, 12h21
nmarf

Effectivement, ça marche mieux !

Merci :D