Discussion :

[BriocheNC]

Bonjour,
Cela fait un mois que je suis dessus et je pense avoir pas mal avancer mais la il y a certain petit problème que je n'arrive pas à régler.
Je fais une communication TCPIP multi-clients avec un pic18F97J60 sur une carte PICDEM.net2 que je programme à l'aide d'un ICD2 sous MPLAB (v8.30 et stack v4.55).
J'aimerai pouvoir:

- Changer ma MAC address que j'arrive à afficher sur mon LCD (en déimal malheureusement). Je la change avec TCPConfigWizard mais elle est toujours identique lorsque je vais voir avec Ethernet Discover

- Je fais des actions (basiques pour l'instant afficher des trucs su le LCD allumer les leds etc...) mais j'aimerai pouvoir passer à l'octet suivant sans refaire un switch (qui serrai simple mais super mega sale)...donc j'ai fait une fonction qui ressemble çà ROTL en assembleur mais je me fais jeter...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	char Verlan(char var[50],unsigned int lg)
	{
		char new_var[50];
		BYTE i;
 
		memset(new_var,' ',32);
		for(i=0;i<lg;i++)
		{
			new_var[lg-i]=var[i];
			new_var[i]=var[i++];		 
		}
		return new_var;
	}

Je bugg totalement à ce point la...

Merci de votre aide.
Brieuc

[Invité(e)]

Bonjour,

La fonction Verlan retourne un type char, or, on lui demande de retourner un tableau de char...

Solution : Passer la chaine à inverser en paramètre :

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
void Verlan(const char s_in[50], char s_out[50],unsigned int lg)
{
    BYTE i;
    /* vérification du paramètre lg pour éviter les débordements */
    if(lg > 49) {
        lg = 49;
    }
    /* on remplit la destination avec des 0*/
    memset(s_out, '\0', lg+1);
    /* calcul du mot inversé */
    for(i=0;i<lg;i++)
    {
        s_out[lg-i-1]=s_in[i];
    }
}

[souviron34]

et en plus de ce qu'a dit mabu,

donc j'ai fait une fonction qui ressemble çà ROTL en assembleur mais je me fais jeter...

parfaitement normal que ça crash (même si le compilo te jette à cause de ce qu'a dit mabu) :

tu retournes un tableu lol (new_var) qui n'a plus d'existence à l'extérieur....

[BriocheNC]

le CONST!!c'est le const que je trouvais plus...
Merci bien
Par contre l'appel je le fais normal du style :
Verlan(chaine,eniahc,7);
ou y a un truc avec le const...c'est le genre de truc a la con que j'oublie a chaque fois...
Merci encore
C'est bon j'ai réussi, je m'étais gourré si je mets un lg supérieur à chaine ça me renvoi un truc vierge...
Merci encore bcp...
bon weekend.

[Invité(e)]

Par contre l'appel je le fais normal du style :
Verlan(chaine,eniahc,7);
ou y a un truc avec le const...c'est le genre de truc a la con que j'oublie a chaque fois...

Le code donné attend queque chose du genre :

Code :

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

1
2
3
4
char entree[] = "mon message";
char sortie[50];
/* strlen(entree) est 11 ici. */
Verlan(entree, sortie, strlen(entree));

[BriocheNC]

Salut,

Après avoir passé un bon moment à essayer de faire exactement ce que je voulais càd choisir le nombre de rotation que je souhaite je finis avec un code qui ressemble à ceci:

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
 
	//fonction Verlan
	// mot: char à rotationner
	// tom: char rotationné 
	// lg : nombre de rotation
	void Verlan(const char mot[50], char tom[50],unsigned int lg)
	{
	    BYTE i,j;
		char interm[50];
	    // vérification du paramètre lg pour éviter les débordements 
	    if(lg > strlen(mot)) 
	    {
	        lg = strlen(mot);
	    }
	    // on remplit la variable intermédiare et la destination avec des 0
	 	memset(interm, '\0', sizeof(interm));
		memset(tom, '\0', sizeof(tom));
	    // calcul du mot inversé
		strcpy((char*)tom,(char*)mot);
		do
		{ 
			for(i=0;i<(strlen(mot)-1);i++)
		    {
				interm[strlen(mot)-1]=tom[0];
		        interm[i]=tom[i+1];
		    }
			strcpy((char*)tom,(char*)interm);
			lg--;
		}while(lg > 0);
 
	}

J'ai perdu du temps parce que je ne voulais pas utiliser de variable intermédiaire au début puis avec ça marche donc bon...
Par contre pour moi il est optimiser à mort.
Merci pour votre aide.

Bonne continuation.
Brieuc

[Invité(e)]

Par contre pour moi il est optimiser à mort.

Hum :

Code :

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

1
2
3
4
5
	for(i=0;i<(strlen(mot)-1);i++)
		    {
				interm[strlen(mot)-1]=tom[0];
		        interm[i]=tom[i+1];
		    }

A chaque itération de la boucle for, on appelle deux fois strlen, ce n'est pas vraiment optimisé.
De plus à chaque itération, on affecte toujours la même case dans le tableau interm.
Le code suivant fait la même chose que la boucle for que tu avais écrit, mais évite certaines redondances.

Code :

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

1
2
3
4
5
6
size_t len = strlen(mot)-1;
interm[len]=tom[0];
for(i=0;i<len;i++)
{
    interm[i]=tom[i+1];
}

PS : on pourrait même faire sortir le size_t len = strlen(mot)-1; de la boucle do while.