Discussion :

[Linquisiteur]

Bonjour j'ai un exercice ou je dois créer une fonction permettant de cacher un code simple dans des pixel d'une image numérique. Bon ce n'est que un énoncer pour débutant je suis sur que en vrai je devrais procéder autrement

la prof me donne la fonction lsb je ne doit que lire ce qu'elle à besoin et ce qu'elle va faire. J'ai fait l'exercice mais je ne suis pas sur de mon code. J'ai surement mal utiliser les pointeurs d'où je viens vous demander de me faire remarquer mes bourdes.

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
 
typedef struct
{
	char bleu;
	char rouge;
	char vert
	}
pixel;
 
typedef struct
{
	image pixel[12]
}
tabImages;
 
//pré: c pointe sur un octet et x == 0 ou x == 1
//post: x est inchangé et le bit de poids faible de *c est mis à x, les autres bits de *c sont inchangés.
void lsb(char* c, char x)
{
	if(x == 1)  *c = *c | x;
	else 		*c = *c & 0xFE;
}
 
//pré: tabImage contient un tableau de pixel de 12 pixel et i est un entier.
//post: tabImage contient des pixels modifié et int est inchangés
//résultat: les bit de int sont inserer dans le lsb des pixel de tabImage
void modification( struct tabImages, int i)
{
	char tabBit[32];	int cpt;	int modulo=1;
	for(cpt=0;cpt<32;cpt++){
	tabBit[cpt] = (i & modulo);   //on stock les bits dans un tableau (32 bit == 32 case)
	modulo *= 2;    //modulo 1..2..4..8..
	}
 
	int cptP;	//compteur des pointeurs par pixel
	cpt=0;
	while(cpt<12){
	char * couleur = tabImages[cpt].image.rouge;
	char * (couleur+1) = tabImages[cpt].image.vert;
	char * (couleur+2) = tabImages[cpt].image.bleu;
	cpt++;
 
		for(cptP=0;cptP<3;cptP++){
		color = *(couleur+cptP);
		lsb(color,tabBit[cpt]);
		}
	}

hormis la procédure lsb tout le code vient de moi. Je ne suis pas habitué à manier les pointeur mais c'est le but de l'exercice.

merci d'avance !

[Bysbobo]

Coucou,

Est-ce que ça fonctionne au moins?
J'ai l'impression que tu n'as pas compiler, il y a des oublis à droite et à gauche

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
typedef struct
{
	char bleu;
	char rouge;
	char vert
	}
pixel;

typedef struct
{
	image pixel[12]; // manque le point virgule. Et c'est quoi image? Peut etre est ce pixel image[12];??
}
tabImages;
[...]
//pré: tabImage contient un tableau de pixel de 12 pixel et i est un entier.
//post: tabImage contient des pixels modifié et int est inchangés
//résultat: les bit de int sont inserer dans le lsb des pixel de tabImage
void modification( struct tabImages, int i)
// tu as une structure tabImages déjà existante -> Utilises juste en parametres tabImages monTableau
{
	char tabBit[32];	int cpt;	int modulo=1;
	for(cpt=0;cpt<32;cpt++){
	tabBit[cpt] = (i & modulo);   //on stock les bits dans un tableau (32 bit == 32 case)
	modulo *= 2;    //modulo 1..2..4..8..
	} 
	
	int cptP;	//compteur des pointeurs par pixel
	cpt=0;
	while(cpt<12){
	char * couleur = tabImages[cpt].image.rouge; 
	char * (couleur+1) = tabImages[cpt].image.vert;
	char * (couleur+2) = tabImages[cpt].image.bleu;
        // que vaut image dans les 3 lignes? A quoi correspond-il?
	cpt++;
	
		for(cptP=0;cptP<3;cptP++){
		color = *(couleur+cptP);
		lsb(color,tabBit[cpt]);
		}
	}

[deletme]

Salut,

Avant de te proposer une solution, je veux être sûr d'avoir bien compris ton objectif.
Tu as un code contenu dans un entier et tu souhaites l'intégrer dans une image en modifiant la valeur du bit de poids faible des pixels c'est bien ça ?

[Sve@r]

J'ai surement mal utiliser les pointeurs d'où je viens vous demander de me faire remarquer mes bourdes.

Salut

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
typedef struct
{
	char bleu;
	char rouge;
	char vert
} pixel;
 
typedef struct
{
	image pixel[12]
} tabImages;

Essaye de nommer tes types "t_qqchose" plutôt que de leur donner des mnémoniques ressemblant à des noms de variables. tabImage c'est un tableau d'images, pas un type !!!

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
typedef struct
{
	char bleu;
	char rouge;
	char vert
} t_pixel;
 
typedef struct
{
	image pixel[12]
} t_image;

Code c :

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
//pré: tabImage contient un tableau de pixel de 12 pixel et i est un entier.
//post: tabImage contient des pixels modifié et int est inchangés
//résultat: les bit de int sont inserer dans le lsb des pixel de tabImage
void modification( struct tabImages  //tabImages est un type, pas une structure. Et où est la variable ??? Hé oui, c'est ça quand on ne nomme pas bien les choses...,  int i)
{
	char tabBit[32];	int cpt;	int modulo=1;
	for(cpt=0;cpt<32;cpt++){
		tabBit[cpt] = (i & modulo);   //on stock les bits dans un tableau (32 bit == 32 case)
		modulo *= 2;    //modulo 1..2..4..8..
		// modulo <<= 2;    => modulo 1..2..4..8.. mais en plus rapide
	}
 
	int cptP;	//compteur des pointeurs par pixel
	cpt=0;
	while(cpt<12){
	char * couleur = tabImages[cpt].image.rouge;
	char * (couleur+1) = tabImages[cpt].image.vert;
	char * (couleur+2) = tabImages[cpt].image.bleu;
	// Hum, "rouge" est un char, pas un char étoile. Pareil pour les autres
	cpt++;
 
		for(cptP=0;cptP<3;cptP++){
		color = *(couleur+cptP);
		lsb(color,tabBit[cpt]);
		}
	// Indentation à revoir. Ecrire du code ce n'est pas seulement écrire du code qui fonctionne mais aussi du code lisible. Es   t-ce q    u    e t                  u t                      ro                      u                         v                          es                     ce                 t                   t                          ep                                     h                           ra                          se                                       fac                                                                                   i                                                                         l                                  ea                         li               re ???
	}

hormis la procédure lsb tout le code vient de moi. Je ne suis pas habitué à manier les pointeur mais c'est le but de l'exercice.

merci d'avance !

Mes autres remarques sont dans ton code...

[Bysbobo]

Code :

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

// Indentation à revoir. Ecrire du code ce n'est pas seulement écrire du code qui fonctionne mais aussi du code lisible. Es   t-ce q    u    e t                  u t                      ro                      u                         v                          es                     ce                 t                   t                          ep                                     h                           ra                          se                                       fac                                                                                   i                                                                         l                                  ea                         li               re ???

J'adore ^^

Code :

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

1
2
3
4
5
6
7
8
9
10
11
typedef struct
{
	char bleu;
	char rouge;
	char vert
} pixel;
 
typedef struct
{
	image pixel[12]
} tabImages;

Tu oublies le "image pixel[12];" le point virgule et image c'est pas un type alors que pixel oui

[gavarax]

Bonjour,
Je t'envoie un lien sur l'indentation, http://fr.wikipedia.org/wiki/Style_d%27indentation c'est succin mais ça reste compréhensible.
Lorsque l'on débute il est important de très vite indenter correctement et ce pour deux raisons : la première est la correction de ton code, une bonne indentation de permet de savoir rapidement si une accolade n'est pas matché. Pour cela, si tu en a l'occasion utilise un éditeur de texte qui te permet d'avoir l'indentation automatique, par exemple emacs le fait très bien(il utilise le premier type d’indentation du lien que je t'ai envoyé par défaut.) il ne reste que a appuyer sur tab dès que tu change de ligne (ou a naviguer dans les menu pour faire "indenter tout le document") et les parenthèse ou accolades mal matchées apparaissent beaucoup mieux comme ça. De plus tu peut savoir exactement quand tu commence/ finit une fonction ou une boucle (et ça détecte même les ";" manquant, si ce n'est pas la belle vie !!!) la deuxième raison est qu'une bonne indentation, tout comme les commentaires permettent aux autres de relire ton code plus facilement, et te permettent a toi de reprendre ton code même après un long moment sans avoir jeté un œil dessus !!

[Linquisiteur]

Oups désoler de la réponse tardive mais je n'était pas là.

Merci beaucoup
J'ai indenter sur notepad++ Ce n'est pas un code que je pourrais compiler. Un pur code pour cacher un message dans une information visuelle n'est pas de mon niveau..

Par contre c'est le résultat que j'ai obtenu en accomplissant cet exercice mais je n'était pas sur de moi..

Je vais ré indenter et mettre les ; mais ces erreur ne sont pas important du fait que je ne compilerai pas ^^'


EDIT:

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
51
52
53
 
typedef struct
{
	char * bleu;
	char * rouge;
	char * vert;
	}
t_pixel;
 
typedef struct
{
	t_pixel image[12];
}
t_GroupeImages;
 
//pré: c pointe sur un octet et x == 0 ou x == 1
//post: x est inchangé et le bit de poids faible de *c est mis à x, les autres bits de *c sont inchangés.
void lsb(char* c, char x)
{
	if(x == 1)  *c = *c | x;
	else 	*c = *c & 0xFE;
}
 
//pré: tabImage contient un tableau de pixel de 12 pixel et i est un entier.
//post: tabImage contient des pixels modifié et int est inchangés
//résultat: les bit de int sont inserer dans le lsb des pixel de tabImage
void modification( t_GroupeImages monTableau, int i)
{
	char tabBit[32];	int cpt;	int modulo=1;
 
	for(cpt=0;cpt<32;cpt++)
	{
	   tabBit[cpt] = (i & modulo);   //on stock les bits dans un tableau (32 bit == 32 case)
	   //modulo *= 2;    //modulo 1..2..4..8..
	   modulo <<= 2;
	}
 
	int cptP;	//compteur des pointeurs par pixel
	cpt=0;
	while(cpt<12)
	{
     	char *  couleur = monTableau[cpt].image.rouge;
	    char * (couleur+1) = monTableau[cpt].image.vert;
	    char * (couleur+2) = monTableau[cpt].image.bleu;
	    cpt++;
 
		for(cptP=0;cptP<3;cptP++)
		{
		   color = *(couleur+cptP);
		   lsb(color,tabBit[cpt]);
		}
	}
}

J'ai corriger les fautes et tenter de faire se que vous m'avez expliquer. Disons que je n'ai jamais manipuler des pointeurs dans des structures.. Donc je ne connais pas encore le "mode d'emploi" apparemment c'est la même chose.

[Sve@r]

J'ai indenter sur notepad++ Ce n'est pas un code que je pourrais compiler. Un pur code pour cacher un message dans une information visuelle n'est pas de mon niveau..

...
Je vais ré indenter et mettre les ; mais ces erreur ne sont pas important du fait que je ne compilerai pas ^^'

Bizarre. Si on te fait faire cet exercice, c'est qu'il est de ton niveau. Au pire on t'indique comment le faire (en clair) et tu traduis ces instructions en C.
Quand à "compiler" au-moins ça te permet de voir si, à défaut de l'algo, la syntaxe est bonne...

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
51
52
53
54
55
56
57
58
59
60
61
62
typedef struct
{
	char * bleu;
	char * rouge;
	char * vert;
	}
t_pixel;

typedef struct
{
	t_pixel image[12];
}
t_GroupeImages;

//pré: c pointe sur un octet et x == 0 ou x == 1
//post: x est inchangé et le bit de poids faible de *c est mis à x, les autres bits de *c sont inchangés.
void lsb(char* c, char x)
{
	if(x == 1)  *c = *c | x;
	else 	*c = *c & 0xFE;
}

//pré: tabImage contient un tableau de pixel de 12 pixel et i est un entier.
//post: tabImage contient des pixels modifié et int est inchangés
//résultat: les bit de int sont inserer dans le lsb des pixel de tabImage
void modification( t_GroupeImages monTableau,
/* Ca marche... mais étant donné qu'un paramètre est recopié depuis la variable passée, tu comprendras qu'il est plus pratique de passer par un pointeur.
En effet, recopier une adresse (même d'une structure) ira plus vite que recopier la structure elle-même
Toutefois c'est pas primordial de piger ça de suite...   */
 int i)
{
	char tabBit[32];	int cpt;	int modulo=1;
	
	for(cpt=0;cpt<32;cpt++)
	{
	   tabBit[cpt] = (i & modulo);   //on stock les bits dans un tableau (32 bit == 32 case)
	   //modulo *= 2;    //modulo 1..2..4..8..
	   modulo <<= 2;
	}
	
	int cptP;	//compteur des pointeurs par pixel
	cpt=0;
	while(cpt<12)
	{
     	char *  couleur = monTableau[cpt].image.rouge;
	    char * (couleur+1) = monTableau[cpt].image.vert;
	    char * (couleur+2) = monTableau[cpt].image.bleu;
        /* Là c'est pas bon. C'est comme si t'avais écrit
        int i=123;
        int(i+1)=512;
        Tu peux pas créer de variable nommée "(couleur+1)" quoi... */

	    cpt++;
	
		for(cptP=0;cptP<3;cptP++)
		{
		   color = *(couleur+cptP);
		   lsb(color,tabBit[cpt]);
		}
	}
}

Remarques dans le code en rouge...

Disons que je n'ai jamais manipuler des pointeurs dans des structures.. Donc je ne connais pas encore le "mode d'emploi" apparemment c'est la même chose.

Un pointeur c'est juste un type destiné à stocker une adresse. Tout comme un int est un type destiné à stocker un entier. Simplement quand tu mets une adresse dans un pointeur tu dois juste t'assurer que l'adresse est valide.
Maintenant que le pointeur soit "encapsulé" dans une structure importe peu.

Toutefois, revenons à ton t_pixel. Pour toi, ce type est dédié à stocker 3 adresses. Que représentent ces adresses pour toi ??? Parce que pour moi, un pixel c'est juste une couleur décomposée en 3 composantes RVB donc en 3 entiers quoi...

[edgarjacobs]

Le membre de t_groupeImages est déclaré comme un tableau. Donc l'écriture devrait être

Code :

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

monTableau[cpt].image[indice].vert;

@Sve@r

Code :

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

1
2
		modulo *= 2;    //modulo 1..2..4..8..
		// modulo <<= 2;    => modulo 1..2..4..8.. mais en plus rapide

Si c'est pour multiplier par 2, le code est

Code :

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

 modulo<<=1;

Mais le compilateur ne va-t-il pas de lui-même traduire le *=2 par un shift? Je trouve qu'écrire *=2 est plus compréhensible que <<=1

[Neckara]

Bonjour,

@edgarjacobs : Cela dépend des optimisations du compilateur, mais il y a de très fortes chances que *=2 soit traduit par <<= 2.


Personnellement, je pense que c'est une histoire de contexte, quand on manipule des octets comme un entier, j'utiliserais *=2.
En revanche, quand je manipule des octets comme une suite de bits (ex. j'utilise des masques, des opérations bits à bits) j'utiliserais plutôt <<=2.