Discussion :

[edgarjacobs]

Bonjour à tous,

Je fais appel appel à vos lumières pour le motif suivant: je souhaite inverser un fichier, càd que le dernier octet devient le premier, etc, jusqu'au premier qui devient le dernier, et ce quelque soit le type de fichier en entrée.
Le programme ne se plante pas, mais si je le teste avec un fichier binaire (un mp3 par ex.) dont la taille d'entrée est de 26Mb, le fichier en sortie ne contient que 13Mb (la moitié, mais c'est par hasard).

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
 
#define BUFLEN 4096
 
/* fout, fin : descripteurs fichiers sortie et entrée */
void reverse(int fout,int fin)
{
	unsigned char bufin[BUFLEN],bufout[BUFLEN];
	unsigned char *pi,*po;
	int sstop=0;
	long pseek,rlen=BUFLEN;
 
	pseek=filelength(fin)-BUFLEN-1; /* -1:filelength() compte aussi EOF */ /* Provisoire */
	if(pseek<0l) {
		rlen=BUFLEN+pseek;
		pseek=0l;
		sstop=1;
		}
	printf("Processing .");
	lseek(fin,pseek,SEEK_SET);
	while(sstop<2) {
		read(fin,bufin,(unsigned)rlen); 
		setmem(bufout,BUFLEN,0);
		pi=bufin+(unsigned)rlen-1;
		po=bufout;
		while(pi>=bufin) *po++=*pi--;
		write(fout,bufout,(unsigned)rlen);
		pseek-=2l*BUFLEN;
		if(pseek<0l) {
			rlen=BUFLEN+pseek;
			pseek=0l;
			sstop++;
			}
		if(sstop<2) {
			putchar('.');
			lseek(fin,pseek,SEEK_SET);
			}
		}
	printf("\n");
}
 
void main(int argc,char *argv[])
{
	int fhin,fhout;
	char tmp[80];
	fhin=open(argv[1],O_BINARY|O_RDONLY,S_IREAD|S_IWRITE);
	fhout=open(argv[2],O_BINARY|O_WRONLY|O_CREAT|O_TRUNC,S_IREAD|S_IWRITE);
	reverse(fhout,fhin);
	close(fhout);
	close(fhin);
}

Je passe sur tous les contrôles que j'ai retirés pour faire court.

Merci de m'aider,
Edgar.

[fregolo52]

tu peux m'expliquer le principe de ton algo (qui à l'air d'écrire par bloc de 4k) ?

j'ai un peu de mal avec la variable pseek :
pseek-=2l*BUFLEN;
tu copies BUFLEN et après tu te décales de 2 fois cette taille !

Je ne comprends pas ton critère d'arrêt. Pourquoi ne pas tester le fin du fichier ?

[sam1507]

Apparemment, le décalage de pseek sert à repositionner le pointeur de lecture sur le fichier d'entrée de deux blocs (celui qu'on vient de lire et celui qu'on va lire).

[edgarjacobs]

Apparemment, le décalage de pseek sert à repositionner le pointeur de lecture sur le fichier d'entrée de deux blocs (celui qu'on vient de lire et celui qu'on va lire).

@sam1507: tu as bien compris ce que je fais, ça recule la position courante du pointeur de deux blocs. J'ai encore (un peu) bossé dessus ce matin, je ne vois pas pourquoi j'ai ce problème.

qui à l'air d'écrire par bloc de 4k

Oui, ça c'est provisoire, un malloc() est prévu dans la version définitive.
Le principe: je lis le dernier bloc du fichier d'entrée, j'inverse les octets et les écris, je recule mon pointeur de 2*(la taille d'un bloc), et on recommence le cycle, jusqu'à ce que le dernier bloc à lire soit plus petit que la taille BUFLEN.

[Invité]

EDIT :
Comme l'a fait remarquer fregolo52, c'est une mauvaise solution qui suit, ne lisez pas. Désolé.

Ta solution me paraît compliquée à première vue, est-ce qu'il ne serait pas plus simple de :
- mapper le fichier en mémoire avec mmap
- inverser les octets un à un dans une boucle grâce à deux compteurs, un qui partirait du début et l'autre de la fin, et ce jusqu'au moment où ils se rejoignent :

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
 
int           fin;
int           dbt;
char        tmp;
 
dbt = 0;
fin = strlen(str) - 1;
while (dbt <= fin)
   {
     tmp = str[dbt];
     str[dbt] = str[fin];
     str[fin] = tmp;
     fin--;
     dbt++;
 }

[fregolo52]

C'est quoi cette horreur !!!
edgarjacobs donne un exemple de fichier mp3, strlen est à banir !!!!

Ta condition de "stop" est moche !! Utilise feof sur fin (ou le code de retour de la fonction read, c'est sûrement mieux) pour savoir si tu as fini de lire le fichier source.