Discussion :

[mistick1400001]

Bonjours j'ai créer une fonction compression mais je voudrai savoir si je peux la modifier pour qu'elle soit plus courte mais toujours fonctionnels ;

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
int decompression(unsigned char source[], int nb, unsigned char dest[])
{
int i,j,l,dec;
i,j,l,dec = 0;
 
for (i=0;i<nb;i++)
{
if (source[i]== 0xFD)
{
if((source[i+1] == 0xFD)||(source[i+1] == (0xFE)))
{
dest[1-dec]=source[i+1];
i=i+2;
dec = dec+1;
}
}
if (source[i]==0xFE)
{
for (j=0;j<(source[i+1]);j++)
{
dest[i-dec] = source[i+2];
dec = dec-1;
}
dec = dec+3;
i= i+2;
}
else
{
dest[i-dec] = source[i];
}
}
return nb;
}

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 main
{
unsigned char toto[50]={0x78,0x56,0x74,0xFE,0x02,0x10,0x41,0x78,0xFD,0xFE,0x36,0xFD,0xFD,0xFE,0x07,0x25};
unsigned char test[50];
int bn;
bn=50;
 
 
printf("\n");
printf("Tableau decompression :\n");
bn =(decompression(toto, bn, test));
dump(test,bn);
printf(("le nombre d'octets est : %d", bn));
}

et j'avais ceci à respecter :

[Matt_Houston]

Compression ou décompression ? J'ai lu en diagonale mais déjà source[i+1] avec i de 0 à nb - 1, ça pète. dest[1-dec], ça pète. Aucune détection d'erreur, ça pète.

As-tu écrit l'algorithme avant de te lancer dans l'implémentation C ? Il te faut une machine à états pour gérer ça.

Utilise un switch, ce sera plus lisible.

[mistick1400001]

Décompression est cette fonction

[mistick1400001]

Quand tu dis ça pète ça veux dire que c'est bien?

[mistick1400001]

et en fait la fonction dump, voilà le code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
void dump(unsigned char t[], int nb)
{
  int i;
  for (i=0; i < nb; i++)
  {
  if ((i % 16) == 0)
    printf("%s%04d", i ?"\n" :"", i); //?v1 :v2 valant v1 si "expression" est vraie et "v2" si "expression" est fausse
    printf(" %02x", t[i]);
  }
  printf("\n");
}

et pour la décompression j'ai suivi ceci :



et donc voici mon main :

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
int main()
{
  unsigned char toto[50]={0x78,0x56,0x74,0xFE,0x02,0x10,0x41,0x78,0xFD,0xFE,0x36,0xFD,0xFD,0xFE,0x07,0x25};
  unsigned char test[50];
  int bn;
  bn=50;
  printf("Fonction Dump :\n");
  dump(toto,bn);
 
  printf("\n");
  printf("Tableau decompression :\n");
  bn =(decompression(toto, bn, test));
  dump(test,bn);
  printf(("le nombre d'octets est : %d", bn));
 
}

[mistick1400001]

Donc as tu des idées pour que mon code soit plus court ?
je t'ai donnée tous mon code

[mistick1400001]

Ah 😂

[gerald3d]

Quand il dit que ça pète au contraire !

Tu ne prends pas le temps de regarder le déroulé de ton algorithme.

Source[i+1] donne quoi quand i=nb-1, c’est à dire la dernière itération de ta boucle ?

[mistick1400001]

Foetus ma fonction dump est bonne ce que je cherche c'est réduire mon int decompression car je trouve qu'il est trop long

[mistick1400001]

Quand il dit que ça pète au contraire !

Tu ne prends pas le temps de regarder le déroulé de ton algorithme.

Source[i+1] donne quoi quand i=nb-1, c’est à dire la dernière itération de ta boucle ?

Si je fais i = nb-1 ; j'ai n'importe quoi dans la console



alors que mon int donne :