Discussion :

[Basicnav]

Bonjour,
voilà, je programme un µc PIC en C dont l'une des fonctions est de retourner un paramétre à une autre; ma question est la suivante :
peut-on renvoyer deux paramètres (à 2 fonctions différentes) lors de l'appel?
Voici mon idée :

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
 
int speed_select (void)
{
    char select_speed; // variable locale
    int speed;
    int speeddec;
 
    if(PORTC.F0==0)
    {
        select_speed=PORTA.F4+PORTA.F5;
 
        {
            if (PORTA.F1!=PORTA.F2){
                switch(select_speed){
                    case 0b00:
                        speed=100;
                        speeddec=300;
                        break;
                    case 0b01:
                        speed=700;
                        speeddec=1200;
                        break;
                    case 0b10:
                        speed=1700;
                        speeddec=2500;
                        break;
                    default:
                        speed=toto;
                        break;
                }
            }
            else 
                speed=80;
 
            return speed,speeddec; // retour de mes param
        }
    }
}

je ne retourne apparament qu'un seul paramète
La syntaxe est-elle la bonne?

Merci de votre aide

[homeostasie]

Non une seule variable peut être retournée avec "return".
Ce que tu peux faire, c'est retourner l'adresse d'un tableau contenant tes deux valeurs:
-> deux solutions:
1) retourner un pointeur comme valeur de retour de ta fonction:

Code :

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

1
2
3
4
5
6
7
int *exemple(){
    int *p;
    p = malloc(2*sizeof(int));
    p[0] = 1;
    p[1] = 2;
    return p;
}

2) avoir un pointeur sur un tableau de deux éléments en paramètre entrant et sortant de ta fonction.

Code :

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

1
2
3
4
void exemple(int *p){
    p[0] = 1;
    p[1] = 2;
}

Dans ce cas, attention a bien allouer la mémoire dans dans ta fonction appelante ou appelée. Tu peux aussi passer la dimension de ton tableau en paramètre.

[ram-0000]

Super, du code pas indenté et sans la balise de code, c'est plus facile à lire...

Sinon, plus sérieusement, ce que tu veux faire n'est pas possible.

Une fonction ne retourne qu'une et une seule valeur à son appelant.

Par contre, la syntaxe de ton return, bien que syntaxiquement correcte, est vraiment tordue (et à proscrire ?).

En fait, je ne sais même pas dire a priori ce qui est retourné comme valeur : speed ou speeddec ? Après un test, je le sais, c'est speeddec qui est retourné.

[Basicnav]

Ram > Excuses moi pour la syntaxe, cela ne fait qu'un petit moi que je m'essaie à la programmation...je sais que de grosses lacunes sur la présentation, mais à force de voir les forum, j'apprendrais bien qqchose non?
En tout cas merci à vous deux pour votre aide.

[diogene]

En fait, je ne sais même pas dire a priori ce qui est retourné comme valeur : speed ou speeddec ? Après un test, je le sais, c'est speeddec qui est retourné.

La "," n'est pas ici un séparateur mais l'opérateur , (virgule).
Avec cet opérateur, le premier terme est évalué en premier et sa valeur est écartée. Le second terme est évalué et l'expression prend pour valeur le résultat de cette évaluation. Par conséquent, pour que cet opérateur serve vraiment à quelque chose, il faut que la première évaluation provoque des effets de bords (ce qui n'est pas le cas ici). On le trouve assez souvent dans des expressions du type :

Code :

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

for(i=0, j=0;...)

[ram-0000]

Manitenant que le code est indenté avec un balise , une deuxième erreur saute au yeux : si PORTC.F0 est différent de 0, le code de retour de la fonction est indéfini.

Soit il n'y a pas tous le code dans ton post et dans ce cas, oublie ma réponse, soit il faut rajouter du code pour prendre en compte ce cas de figure.

Les warning du compilateurs devraient le dire aussi (si le niveau de verbosité est suffisemment élevé)

[Basicnav]

Ram_0000, pour ta dernière réponse, cette fonction n'est appelée sous certaine conditions dont celle où PORTC.F0 serait égal à 1 :j'avance petit à petit sur mon code parceque la gestion de mon clavier peut prendre beaucoup de conditions.
Mais merci de me le faire remarquer.

Homeostasie propose :

int *exemple(){
int *p;
p = malloc(2*sizeof(int));
p[0] = 1;
p[1] = 2;
return p;
}

cette fonction doit-elle être appelée par ma fonction ou une partie doit être intégrée à la mienne?
J'ai un peu de mal avec les pointeurs là...je vais relire les tutos du site; je pense en avoir grand besoin !

[homeostasie]

Code :

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

1
2
3
4
5
6
7
int *exemple(){
int *p;
p = malloc(2*sizeof(int));
p[0] = 1;
p[1] = 2;
return p;
}

cette fonction doit-elle être appelée par ma fonction ou une partie doit être intégrée à la mienne?

Tu peux faire ces modifications dans ta fonction. A toi de bien adapter. C'est une méthode permettant entre autres de retourner tes deux valeurs.
Par contre, avec cette méthode, tu devras libérer la mémoire allouée dans la fonction appelante (utilisation de free()).

D'autre part, d'autres solutions sont disponibles:
- l'autre méthode que j'ai présenté plus haut
- retourner une structure avec deux champs contenant tes deux valeurs de retour
- passer en paramètre de ta fonction un pointeur sur une structure

[Médinoc]

Ou plus simple s'il n'y a que deux entiers à retourner:

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
/* La valeur de retour est 0 si OK, -1 si erreur.
   En cas d'erreur, les pointeurs sont ignorés. */
int exempleDivision(int dividende, int diviseur, int *pQuotient, int *pReste)
{
	/* Vérification des paramètres */
	if(diviseur == 0)
		return -1;
 
	/* Corps de la fonction */
	if(pQuotient != NULL)
	{
		*pQuotient = dividende/diviseur;
	}
	if(pReste != NULL)
	{
		*pReste = dividende%diviseur;
	}
	return 0;
}