ich hoffe ich bin hier im richtigen Bereich gelandet (falls nicht, würde ich mich über einen Link zum betreffenden bereich freuen )

Ich habe erst vor wenigen Tagen auf Grund meines Studiums angefangen mich mit C++ zu beschäftigen und nun stehe ich vor folgendem Problem:

Ich soll das Sortierverfahren "Bubblesort" in C++ implementieren:

die Funktion für das Sortierverfahren sieht wie folgt aus:

void bubblesort(int a[], unsigned la)
{

	for (int i = la - 1; i > 0; i--) {

	    // innere Schleife  
	    for (int j = 0; j < i; j++) {

	      // ggf. Korrektur der Ordnung zweier,
	      // aufeinanderfolgender Werte
	      if (a[j] > a[j + 1]) {

	        // Vertauschung
	        int temp = a[j];
	        a[j] = a[j + 1];
	        a[j + 1] = temp;
	      }
	    }
	  }

das Problem liegt bei mir nun daran, dass es mir nicht gelingt, diese Funktion ordnungsgemäß aufzurufen:

hier meine "main-Funktion:"

#include<stdio.h>
#include <iostream.h>
#include<malloc.h>
#include "bubble.h" // beinhaltet die Funktion bubblesort
#include <stdlib.h>
using namespace std;

int main()
{

unsigned n;
	unsigned i;
	int werte[100];

	cout << ("Wie viele Elemente sollen verglichen werden?");
	cin >> n;
	for (i=0; i< n; i++)
	{ cout << ("Bitte geben Sie eine Zahl ein:");  
	cin >> werte[i];

	bubblesort( werte[], n);
}
 return(0);

}

Der Compiler meldet, dass vor der Schließenden eckigen Klammer noch etwas folgen müsste... wenn ich nun jedoch eine Zahl angebe, meldet wer den gleichen Fehler...

ich hoffe es kann mir jemand helfen...

Vielen Dank & lg Dragon

Im Zweifelsfall bitte auch folgende Hinweise beachten:
C/C++ Forum :: FAQ - Sonstiges :: Wohin mit meiner Frage?

Dieses Posting wurde automatisch erzeugt.

...

Du würdest dir vieles Einfacher machen wenn du deinen Code sauber formatierst...
Für einen Anfänger ist wohl die Formatierung am besten, wo man öffnende und Schließende {} in eine separate Zeile auf jeweils die gleiche Höhe setzt.

Zuerst einmal deine Funktion anders formatiert (ich habe sie ansonsten nicht überprüft):

void bubblesort(int a[], unsigned la)
{
    for (int i = la - 1; i > 0; i--)
    {
        // innere Schleife  
        for (int j = 0; j < i; j++)
        {
            // ggf. Korrektur der Ordnung zweier,
            // aufeinanderfolgender Werte
            if (a[j] > a[j + 1])
            {
                // Vertauschung
                int temp = a[j];
                a[j] = a[j + 1];
                a[j + 1] = temp;
            }
        }
    }
} // <-- Fehlte hier, durch bessere Formatierung sichtbar

T-Dragonmaster XII schrieb:

das Problem liegt bei mir nun daran, dass es mir nicht gelingt, diese Funktion ordnungsgemäß aufzurufen:

Mehrere Anmerkungen:
a) malloc und free haben in C++ nichts zu suchen! (ganz davon abgesehen das du sie eh nicht verwendest aber Header includierst die nicht benötigt werden
b) Die Includes der Standardheader sind falsch! (Schon seit dem C++Standard von 98)
==> Es heißt z.B. #include <iostream> nicht #include <iostream.h>

Also reduzieren wir mal deine Includes auf das was du tatsächlich brauchst, gültiges C++ ist und zu deinem Projekt gehört. Und wieder formatieren wir deinen Code mal neu:

#include <iostream>
#include "bubble.h" // beinhaltet die Funktion bubblesort
using namespace std;

int main()
{
    unsigned int n; // Bitte ganzen Typ angeben, dies ist eine Unsitte!
    unsigned int i; // Zudem sprechende Namen wählen
    int werte[100];

    cout << "Wie viele Elemente sollen verglichen werden?";
    cin >> n;
    for (i=0; i< n; i++)
    {
        cout << "Bitte geben Sie eine Zahl ein:";  
        cin >> werte[i];

        bubblesort(werte[], n);
    }
    return 0;
}

Anmerkung: Ich würde noch einen Zeilenumbruch (cout << endl) an sinnvolle Stellen plazieren.

Dein Problem sollte die Schließende Klammer bei der Funktion gewesen sein.

cu André

vielen Dank & lg Dragon