Hi!

Also ich mache ne Ausbildung zum ITA und da haben wir natürlich auch Programmieren. Wir arbeiten im Moment mit Borland C++ und erstellen Konsolenprogramme.

Nun haben wir heute die Aufgabe bekommen ein Programm zu optimieren. Also ich bin der beste bei uns im Kurs und habe das Muster hier auch geschrieben, aber wie ich das jez noch, mit unserem Wissen, optimieren soll weiß ich nich. Wäre super wenn mir jemand helfen könnte!

#include<iostream.h>

#include<dos.h>


#define ANZ 8


void eingaben(int[]);

void sort(int[]);

void ausgabe(int[]);


int main()

{

    int eingabe[ANZ];

    char trash;


    system("color 02");


    cout <<"                       b                    	"<< endl;

    cout <<"                       b                		"<< endl;

    cout <<"                       bbb  y  y         		"<< endl;

    cout <<"                       b bb  yy            		"<< endl;

    cout <<"                       bbb   y              	"<< endl;

    cout <<"             EEEEEE         y        LL     	"<< endl;

    cout <<"            EE                       LL     	"<< endl;

    cout <<"            EE       gggggg  gggggg  LL     	"<< endl;

    cout <<"            EEEEE   gg   gg gg   gg  LL     	"<< endl;

    cout <<"            EE      gg   gg gg   gg  LL     	"<< endl;

    cout <<"            EE      gg   gg gg   gg  LL     	"<< endl;

    cout <<"            EE      gg   gg gg   gg  LL     	"<< endl;

    cout <<"             EEEEEE  gggggg  gggggg   LLLLLL	"<< endl;

    cout <<"                         gg      gg         	"<< endl;

    cout <<"                    gggggg  gggggg          	"<< endl;

    cout << endl;

    cout << endl;

    cout << endl;


    cout << "Bitte geben sie " << ANZ << " Zahlen ihrer Wahl ein:" << endl;

    eingaben(eingabe);


    sort(eingabe);


    cout << endl << "So lauten die von ihnen eingegebenen Zahlen in Reihenfolge:" << endl;

    ausgabe(eingabe);


    cin >> trash;

}



void eingaben(int eingabe[])

{

    int i;


    for(i=0;i<ANZ;i++)

    {

        cin >> eingabe[i];

        cout << endl;

    }

}


void sort(int eingabe[])

{

    int i=0,j,z;


    for(;i<ANZ-1;)

    {

        j=i+1;


        while(j<ANZ)

        {

            if(eingabe[i]<eingabe[j])

                ;

            else

            {

                z=eingabe[i];

                eingabe[i]=eingabe[j];

                eingabe[j]=z;

            }


            j++;

        }


        i++;

    }

}


void ausgabe(int eingabe[])

{

    int i;


    for(i=0;i<ANZ;i++)

        cout << eingabe[i] << endl;

}

THX!

GreeTzzZ

EggL

Hallo,

es ist zwar schon spät aber wenn ich es richtig sehe ist das ein Bubblesort.

Abgesehen davon dass es viele effektivere sortieralgorithmen gibt würde ich für diese Ausgabe erst folgendes tun:

Im Moment fängst Du bei der ersten Stelle an und lässt die großen Zahlen nach hinten wandern. Eine kleine Optiierung wäre es auf jeden fall das man auf dem 'Rückweg' die kleinen Zahlen nach vorne wandern läßt.

Ich glaub, es geht eher darum den code zu "säubern". d.h. die Lesbarkeit zu verbessern. (die sort Funktion ist ansich ein katastrophe bezüglich der Lesbarkeit)

z.b. "for(;i<ANZ-1;)" durch "while(i<ANZ-1)" zu ersetzen oder gleich

eine ordentliche for-schleife draus machen wie "for (int i=0; i<ANZ-1; i++)"

Den Bubblesort selbst kann man ansich nicht wirklich beschleunigen.

            if(eingabe[i]<eingabe[j])

                ;

            else

            {

                z=eingabe[i];

                eingabe[i]=eingabe[j];

                eingabe[j]=z;

            }[/CODE]
Eine leere Anweisung im if-Teil, aber dafür ein else-Block?


Dann kann man auch gleich die Bedingung umdrehen, den Code in den if-Block stecken und den else-Block weglassen.

Hi!

Also das mit dem Bubblesort stimmt nicht ganz weil bei nem Bubblesort beide Array-"Schubladen" weitergehen und bei diesem wird immer die 1. in einem Durchgang behandelt und wandert erst weiter wenn alle anderen durch sind ...

Ja die Kleinigkeiten hab ich beseitigt, aber das was Idexe meinte find ich ne gute Idee nur dies zu realisieren gelingt mir nicht. Kann da vllt jemand helfen ?

THX!

GreeTzzZ

EggL

Hi!

Ja die Kleinigkeiten hab ich beseitigt, aber das was Idexe meinte find ich ne gute Idee nur dies zu realisieren gelingt mir nicht. Kann da vllt jemand helfen ?

Suche im Internet nach Mergesort, Heapsort und Quicksort und suche Dir einen passenden Algorithmus aus. Mergesort hat die geringste Laufzeit in O-Notation, Quicksort ist jedoch meist einfacher zu implementieren (bzw. gehört in einer Variante auch schon zur C-Libraray).