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 !

[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]

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