in dem Buch "Einstieg in C++" von Arnold Willemer ist folgender Code. In dem Programm geht es darum, Zahlen der Größe nach zu sortieren. Das klappt auch, aber
wenn es in dem Progra heisst "Durchlauf beendet", steht immer "4. Durchlauf beendet", auch wenn es "1. Durchlauf..." usw. heissen müsste.

Wa ist an dem Code also falsch?

#include <iostream>
#include <cstdlib>
using namespace std;

const int MAX=5;

int main()
{   
    int feld[MAX], hilf;
    int i, j, k;

    srand(time(0)); //Zufallsgenerator vorbereiten
    for (i=0; i<MAX; i++)
    {//Array besetzen und anzeigen
       feld [i] = rand() % 100 + 1;
       cout << feld [i] << " ";
    }
       cout << endl;

       for(i=MAX-1; i>0; i--) // MAX-1, weil die Karten immer mit dem rechten Nachbarn verglichen werden
       // d. h. nicht 4, sondern 3 Vergleiche
       {
           for(j=0; j<i; j++)
           {
                    cout << "(" << j << "-" << j+1 << "):" ;
                    if (feld[j]>feld[j+1])
                    { // Tauschen erforderlich
                      hilf = feld[j];
                      feld[j] = feld[j+1];
                      feld[j+1] = hilf;
                      }
                      cout << feld[j] << " - " << feld[j+1] << " ";
                      }
                      // Zeige das Array in diesem Durchlauf
                      cout << endl << MAX-1 << ". Durchlauf beendet: ";
                      for(k=0; k<MAX; k++)
                      {    
                           cout << feld[k] << " ";
                      }
                      cout << endl;

         }
     cin.clear();
     cin.get();
}

Liebe Grüße und vielen Dank schon mal

Schöne Grüße

muss ich das ändern, damit es richtig läuft?

Wenn du nicht jedes mal das Ergebnis von 5-1 ausgeben möchtest ...

Und vor allem das Buch beim lesen auch verstehen anstatt hier-rein-da-raus

Thx aber für die Antwort!

Weil Ausgabe ist am Anfang jedes Tutorials und Buches drin.

Und wenn ich volkard mal (sinngemäß) zitieren darf:
"Ich finde deine Fehler nicht, weil ich irgendwie viel besser oder intelligenter bin, sondern weil ich denselben fehler schon 10mal selbst gemacht hab und ihn daher kenne." (Sowas nennt man dann aus Erfahrung lernen.)

... aber was mache ich da genau?

Keine Ahnung. Jedenfalls nicht das, was man von einem lernenden Anfänger erwarten kann: selbst ausprobieren - verstehen - anwenden - oder ein Anfängerbuch lesen!

cout << endl << MAX-1 << ". Durchlauf beendet: ";

lieber

cout << endl << MAX-i << ". Durchlauf beendet: ";

schreiben solltest.

Und konsequenter Einrücken wäre was.

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;

const int MAX=5;

int main()
{
    int feld[MAX], hilf;
    int i, j, k;

    srand(time(0)); //Zufallsgenerator vorbereiten
    for (i=0; i<MAX; i++)
    {
        //Array besetzen und anzeigen
        feld [i] = rand() % 100 + 1;
        cout << feld [i] << " ";
    }
    cout << endl;

    for(i=MAX-1; i>0; i--) // MAX-1, weil die Karten immer mit dem rechten Nachbarn verglichen werden
        // d. h. nicht 4, sondern 3 Vergleiche
    {
        for(j=0; j<i; j++)
        {
            cout << "(" << j << "-" << j+1 << "):" ;
            if (feld[j]>feld[j+1])
            {
                // Tauschen erforderlich
                hilf = feld[j];
                feld[j] = feld[j+1];
                feld[j+1] = hilf;
            }
            cout << feld[j] << " - " << feld[j+1] << " ";
        }
        // Zeige das Array in diesem Durchlauf
        cout << endl << MAX-i << ". Durchlauf beendet: ";
        for(k=0; k<MAX; k++)
        {
            cout << feld[k] << " ";
        }
        cout << endl;

    }
    cin.clear();
    cin.get();
}

Das macht Deine IDE auch von selber.
Microsoft: Strg+A gefolgt von Alt+F8
CodeBlocks: Rechte Mauta "Format using AStyle"

Ist der Code im Buch auch so? Falls ja, wirf es.

Ich glaube, die Leute meinen eigenrlich, Daß Du statt

cout << endl << MAX-1 << ". Durchlauf beendet: ";

lieber

cout << endl << MAX-i << ". Durchlauf beendet: ";

schreiben solltest.

Und konsequenter Einrücken wäre was.

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;

const int MAX=5;

int main()
{
    int feld[MAX], hilf;
    int i, j, k;

    srand(time(0)); //Zufallsgenerator vorbereiten
    for (i=0; i<MAX; i++)
    {
        //Array besetzen und anzeigen
        feld [i] = rand() % 100 + 1;
        cout << feld [i] << " ";
    }
    cout << endl;

    for(i=MAX-1; i>0; i--) // MAX-1, weil die Karten immer mit dem rechten Nachbarn verglichen werden
        // d. h. nicht 4, sondern 3 Vergleiche
    {
        for(j=0; j<i; j++)
        {
            cout << "(" << j << "-" << j+1 << "):" ;
            if (feld[j]>feld[j+1])
            {
                // Tauschen erforderlich
                hilf = feld[j];
                feld[j] = feld[j+1];
                feld[j+1] = hilf;
            }
            cout << feld[j] << " - " << feld[j+1] << " ";
        }
        // Zeige das Array in diesem Durchlauf
        cout << endl << MAX-i << ". Durchlauf beendet: ";
        for(k=0; k<MAX; k++)
        {
            cout << feld[k] << " ";
        }
        cout << endl;

    }
    cin.clear();
    cin.get();
}

Das macht Deine IDE auch von selber.
Microsoft: Strg+A gefolgt von Alt+F8
CodeBlocks: Rechte Mauta "Format using AStyle"

Ist der Code im Buch auch so? Falls ja, wirf es.

so ist es aber auch falsch weil

cout << endl << MAX-i << ". Durchlauf beendet: ";

jetzt außerhalb der for schleife ist. hier würde auch nur immer wieder 4 ausgegeben werden oder vertu ich mich da jetzt?

Die äussere hat die Laufvariable i, die erste innere zählt mit j und die zweite innere mit k. Die Anweisung steht im Rumpf der äusseren Schleifen, d.h. dort i bekannt und wird pro Durchlauf auch geändert.

Du wirst also folgendes rausbekommen:

// ......
5. Durchlauf beendet.
4. Durchlauf beendet.
3. Durchlauf beendet.
2. Durchlauf beendet.
1. Durchlauf beendet.

Mit meinem Vorschalg zählt er genau umgekehrt, also von 1 (oder 0) hoch

so sieht das ganze doch schon viel schöner aus find ich zumindest^^

Komischen Geschmack hast du.

ich find das so übersichtlicher weil man nich mehr so viele schleifen ineinander verschachtelt hat und mit den klammern nich mehr durcheinander kommt

Dass man Teile in Funktionen auslagert ist auch gut.

Aber deine Formatierung ist Müll und deine globalen Variablen auch...