tausch(&b,&a);

heißen müßte. Schließlich erwartet die Funktion "tausch" doch die Adresse zweier Variablen und nicht deren Inhalt, oder?
(s. dazu auch Zeile 43)

#include <iostream>
using namespace std;

void tausch(int *wert1,int*wert2)
{
  int tmp=*wert1;
  *wert1=*wert2;
  *wert2=tmp;
};

void swab(int *a,int *b,int *c)
{
   for(;(*a>*b) || (*a>*c) || (*b>*c);)
    {
      if(*b<*a)
       {
         tausch(b,a);
       }

      if(*c<*b)
       {
         tausch(c,b);
       }

      if(*c<*a)
       {
         tausch(c,a);
       }
    }
};

int main()
{
    int a,b,c;
    cout<<"Bitte geben sie den ersten Wert ein: ";
    cin>>a;
    cout<<"Bitte geben sie den zweiten Wert ein: ";
    cin>>b;
    cout<<"Bitte geben sie den dritten Wert ein: ";
    cin>>c;
    cout<<"Die eingegebenen Werte werden nun sortiert, bitte etwas Geduld."<<endl;

    swab(&a,&b,&c);
    cout<<"a:"<<a<<endl;
    cout<<"b:"<<b<<endl;
    cout<<"c:"<<c<<endl;
    return 0;
};

Ich verstehe leider nicht, warum es z.B. in Zeile 17 nicht

tausch(&b,&a);

heißen müßte.

Weil a und b in diesem Scope bereits vom Typ int* sind. Und lese dich in das Kapitel "Referenzen" deines Buches ein.

P.S.: Erst jetzt aufgefallen: Normalerweise schreibt man "swap" und nicht "swab"

Schließlich erwartet die Funktion "tausch" doch die Adresse zweier Variablen und nicht deren Inhalt, oder?

Ja, das ist richtig. Aber in a und b in Zeile 17 sind ja Adressen, denn sie sind ja

void swab(int *a,int *b,int *c)

so definiert und nicht etwas so

void swab(int a,int b,int *c)

P.S.: Erst jetzt aufgefallen: Normalerweise schreibt man "swap" und nicht "swab"

Die Funktion swab sollte sort heissen und die Funktion tausch sollte man swap nennen.

PS: Sorry wegen den drei Beiträgen nacheinander.

Nicht zu vergessen ist, dass die STD auch Funktionen für das Austauschen ( std::swap aus <algorithm> ) und sogar für das Sortieren ( std::sort aus <algorithm> ) oder wenn die Sortierung stabil sein soll kann man auch die stabile Sortierung der STD nutzen ( std::stable_swap aus <algorithm> ).

Ja, wollte ganz gerne den Umgang mit Zeigern im Zusammenhang mit Funktionen lernen, das Sortierprogramm ist da eher zufällig.

Nexus schrieb:

P.S.: Erst jetzt aufgefallen: Normalerweise schreibt man "swap" und nicht "swab"

"swab" habe ich deshalb geschrieben, damit es nicht zu Verwechslungen mit der gleichnamigen, in der iostream-Bibliothek enthaltenen Funktion kommt.

EOutOfResources schrieb:

Übrigens schreibt man am Ende der Funktionsdefinition kein Semikolon!

Ok.
In dem Tutorial( http://www.c-plusplus.net/cms/modules.php?op=modload&name=Downloads&file=index&req=getit&lid=15&nocache=1 )wird das so praktiziert.

redrew99 schrieb:

Ich verstehe leider nicht, warum es z.B. in Zeile 17 nicht

tausch(&b,&a);

heißen müßte.

Weil a und b in diesem Scope bereits vom Typ int* sind.

Ok,danke.

EOutOfResources schrieb:

Und lese dich in das Kapitel "Referenzen" deines Buches ein.

Werde ich auf jeden Fall noch machen. In dem Tutorial wird von der Verwendung von Referenzen in diesem Zusammenhang abgeraten, deshalb habe ich Zeiger genommen.

"swab" habe ich deshalb geschrieben, damit es nicht zu Verwechslungen mit der gleichnamigen, in der iostream-Bibliothek enthaltenen Funktion kommt.

Genau deshalb rate ich immer vor der Verwendung von " using namespace std; ab.

PS: Das Tutorial ist alt. Man sieht es an folgendem Code:

#include <iostream.h>    //Fehler 1: <iostream> wäre korrekt
void main()              //Fehler 2: int sollte der Rückgabewert sein
{
   int a=1;
   int &ra=a;
   cout<<"a: "<<a<<endl; //Fehler 3: cout ist im Namensbereich std
   ra=2;
   cout<<"a: "<<a<<endl;
};                       //Ich denke, dass das der vierte Fehler ist aber bin mir nicht sicher

Nicht zu vergessen ist, dass die STD auch Funktionen für das Austauschen ( std::swap aus <algorithm> ) und sogar für das Sortieren ( std::sort aus <algorithm> ) oder wenn die Sortierung stabil sein soll kann man auch die stabile Sortierung der STD nutzen ( std::stable_swap aus <algorithm> ).

stable_swap? Du meinst sicher stable_sort.

Edit: die Frage ging an die Experten im Forum.

Du meinst sicher stable_sort.

Oh man, wie peinlich. Ich habs gefixt.

s. Lektion 37

http://www.c-plusplus.net/cms/modules.php?op=modload&name=Downloads&file=index&req=getit&lid=15&nocache=1