Zaprogramowanie sortowania bąbelkowego

Question

Witam Was.

Jestem studentką informatyki stosowanej na pierwszym roku. Jednak wcześniej nie miałam do czynienia z programowaniem, a poszłam na te studia, bo bardzo mi się to podoba i chcę się tego nauczyć. Jednak profesorowie są przekonani, że podstawy programowania mamy w paluszku i gnają z materiałem, a ja niestety nie dam rady sama wszystko nadrobić i być na bieżąco w takim szybkim tempie. Ostatnio programowaliśmy sortowanie bąbelkowe, ale kiedy profesor sprawdzał na zaliczenie powiedział, że mam jeszcze błąd. Ja niestety nie potrafię go znaleźć, czy moglibyście mi pomóc?

#include <iostream>
#include <ctime>
#include <conio.h>
#include <cstdlib>

using namespace std;

double tab[500];
int i, j, licznik, n = 500;

int main()
{
    srand(time(NULL));

    for (i = 0; i < n; i++)
        tab[i] = double(rand()*0.01);

    for (i = 0; i < n; i++)
    {
        licznik = 0;
        for (j = 1; j < n - i; j++)
        {
            if (tab[j - 1]> tab[j])
                swap(tab[j - 1], tab[j]);
            licznik++;
        }
        if (licznik == 0) break;
            cout << licznik;
    }

    //for (i = 0; i < n; i++)
    //cout << tab[i] << endl;

    _getch();

    return 0;
}

 

Comments

Taka rada na przyszłość, proszę umieszczać kod źródłowy w specjalny bloczek. Dla Ciebie to kilka kliknięć więcej (razem z wybraniem odpowiedniego języka programowania do podświetlania składni), a na pewno chętniej Ci ktoś pomoże. Ja osobiście nie cierpię czytać kodu bez kolorowania składni i założę się że wiele innych osób też - po prostu zamkną pytanie, a Ty zostaniesz bez odpowiedzi.

No nie wspominając już, że narusza to regulamin forum

Answer 1

Podejrzewam, że chodzi tutaj o zmienną licznik. Iterujesz ją za każdym razem, kiedy wywoływana jest zagnieżdżona pętla, choć możliwa jest sytuacja, że żaden swap nie zachodzi (mamy posortowane dane) zanim pętla się skończy. Oznacza to, że licznik wcześniej mógłby wynosić zero. Rozwiązaniem problemu będzie:

    if (tab[j - 1]> tab[j]) {
                swap(tab[j - 1], tab[j]);
                licznik++;
    }

Edit: PS. Na przyszłość staraj się pisać kod tak aby nie używać instrukcji break, bo to nie świadczy dobrze o programiście.

 

Comments

Na przyszłość staraj się pisać kod tak aby nie używać instrukcji break, bo to nie świadczy dobrze o programiście.

A to niby dlaczego? W switch po case też zła praktyka?

---

Przerywanie pętli jest. Chyba, że to konieczne.

---

Ja nie widzę przeciwskazań, żeby używać break też w pętlach. Moim zdaniem, nie wpływa to negatywnie na czytelność kodu.