Discussion :

[deubelte]

j'ai un exo qui demande d'écrire un programme consiste à echanger les contenus de 2 variables de type int fournies en argument en transmettant l'adress des variables

j'ai fait 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
 
void SWAP(int a, int b){
    int *a1=&a;
    int *b1=&b;
 
	int*swap;
 
	swap=a1;
     a1=b1;
	b1=swap;
 }
 
 
 void main(){
    SWAP(1,2);
 }

est-ce que ca répond à la question?

[poukill]

est-ce que ca répond à la question?

Non !!!!

Code :

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

1
2
3
4
5
6
7
8
int main()
{
     int a = 2;
     int b = 3;
     swap(a, b); // ou swap (&a, &b) suivant ton approche
 
     return 0;
}

est le main que tu veux créer! Il faut que tu échanges deux variables qui existent avant que tu appeles ta fontion, sinon ça n'a aucun sens!

C'est plus du forum C que C++...

2 solutions pour coder ta fonction:

• En C :

  Code :

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

  void swap (int *a, int *b)

  sera le bon prototype
• En C++ :

  Code :

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

  void swap (int & a, int & b)

  pourrait en être un autre...

Je te laisse écrire le reste !

[Ksempac]

J'ai failli repondre a ta question, puis je me suis rendu compte que ton main ne fait rien : Tu dois inverser 2 variables, et tu n'as aucune variable dans ton main, tu ne regardes pas les valeurs obtenues apres l'appel de la fonction swap...

Bref visiblement tu n'as meme pas essayé de faire fonctionner la chose...

Quelquechose me dit que tu souhaites qu'on fasse tes devoirs a ta place.

EDIT : Tiens poukill a ete plus rapide et plus gentil que moi

[deubelte]

excuse moi, mais je ne vois pas vraiment de différence entre ton main et le mien.

Par contre pour le swap, pourquoi tu passes les variables par adresse?
Je ne vois pas quel est le problème avec ma fonction?

[deubelte]

j'aime bien l'agressivité (et la paranoia) du forum, mais bon on se calme.


J'ai essayé ma fonction sur VC2005 et ca marche:

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
 
void SWAP(int a, int b){
    int *a1=&a;
    int *b1=&b;
 
	int*swap;
 
	swap=a1;
     a1=b1;
	b1=swap;
	cout<<*a1;
 }
 
 
 void main(){
	 int c=1,d=22;
    SWAP(c,d);
 }

[Ksempac]

Le but est d'inverser les variables qui sont dans le main. Tu n'affiches rien, tu peux donc pas savoir si cela fonctionne

Essaye ce main :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
 void main()
{
    int c=1;
    int d=22;
    cout << "Valeur de c :" << c;
    cout << "Valeur de d :" << d;
    SWAP(c,d);
    cout << "Valeur de c :" << c;
    cout << "Valeur de d :" << d;
 }

Tu verras que ton code ne fonctionne pas : Comme tu passes les arguments par valeur et non par adresse, les a et b modifiés par swap sont des copies

[Laurent Gomila]

j'aime bien l'agressivité (et la paranoia) du forum, mais bon on se calme.

Il ne faut pas non plus que tu oublies pourquoi tu as le rang "Débutant".

J'ai essayé ma fonction sur VC2005 et ca marche

Pas vraiment non. Une fois sorti de la fonction les variables c et d seront inchangées.
Tu devrais lire ceci, tu devrais comprendre où est le problème :
http://c.developpez.com/faq/cpp/?page=references

[deubelte]

Il ne faut pas non plus que tu oublies pourquoi tu as le rang "Débutant".

ce n'est pas ce qui me gene le plus.

Mais je viens de comprendre mon erreur que j'ai faite dans mon code.

Maintenant, est ce que ca:

Code :

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

1
2
3
4
5
6
7
 
void g(int *a, int *b){....}
 
void main(){
....
g(&a,&b)...
}

c'est équivalent à ca :

Code :

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

1
2
3
4
5
6
7
8
 
void g(int c,int d){...}
 
void main(){
...
g(a,b)
...
}

[Ksempac]

Non, rajoute des cout dans ton main comme je te l'ai suggéré, tu verras tout de suite que ca n'a rien a voir

Le premier est un passage par valeur qui provoque une copie des variables dans de nouvelles variables (locales a la fonction appelée)

Le second est un passage par adresse qui se fait sans copie

[poukill]

Non !
Mais alors pas du tout !!!!

EDIT : ce coup ci, c'est toi qui a été plus rapide et plus gentil !

[Laurent Gomila]

Code :

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

void g(int *a, int *b){....}

Code :

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

void g(int c,int d){...}

Je t'ai donné un lien vers la section "références" de la FAQ (qui commence d'ailleurs par une fonction swap...), ce n'était pas pour faire joli

[deubelte]

ok, je croyais que & et * étaient des opérations "mathématiquement" inversibles.
C'est a dire que *(&v)=v.
Mais si vous dites le crontraire, je vous crois

Je t'ai donné un lien vers la section "références" de la FAQ (qui commence d'ailleurs par une fonction swap...), ce n'était pas pour faire joli

j'ai un livre de C++ avec moi, mais on apprends toujous mieux en échangant avec les autres.

[Laurent Gomila]

ok, je croyais que & et * étaient des opérations "mathématiquement" inversibles.
C'est a dire que *(&v)=v.

*&v = v est vrai (littéralement : la valeur contenue à l'adresse de v est v).
Par contre le contraire n'est pas syntaxiquement correct, à savoir que *v ne signifie rien si le type de v ne définit pas un opérateur *.

j'ai un livre de C++ avec moi, mais on apprends toujous mieux en échangant avec les autres.

Va dire ça aux membres bénévoles qui passent des heures à écrire des FAQ et des tutoriels.
Bon je pense qu'à l'époque tu l'avais compris mais je te le rappelle tout de même : on cherche d'abord, on pose les questions ensuite si on n'a pas trouvé de solution satisfaisante.

[deubelte]

J'utilise aussi les FAQ comme source d'info complémentaire.


voici ma solution:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
void SWAP(int &a, int &b){
	int swap;
	swap=a;
	a=b;
	b=swap;
      }
 
 void main(){
	 int c=1,d=8;
	 cout<<c<<" "<<d<<endl;
 SWAP(c,d);
 cout<<c<<" "<<d;
 }

mais ici, je passe par référence et non par adresse (oui?)

[poukill]

j'ai un livre de C++ avec moi, mais on apprends toujous mieux en échangant avec les autres.

Je ne suis pas d'accord...
On apprend mieux en faisant les deux en parallèle.
Je viens de me taper 3 semaines à ne faire que lire de la conception orienté objet, c'était pas juste par pur plaisir!


Aide toi, et le forum DVP.com t'aidera !

[Laurent Gomila]

mais ici, je passe par référence et non par adresse (oui?)

Oui.