[erwit]

Mirki, dlaczego case 3 i 4 ladnie mi sortuje, a w case 1 i 2 wychodzi "przed" tablice? (37 linijka)

http://pastebin.com/yncpPbUR

#cpp #programowanie

[Gorion103]

@erwit:

for(int i=0; i< ( sizeof(arr)/4) ; i++)

wut? czemu sizeof?

[erwit]

@Gorion103: tablica intow -> wiec rozmiar tablicy/rozmiar inta = ilosc elementow

[erwit]

@Gorion103: jezeli zamianie sizeof(arr)/4 na 4 (bo tyle jest elemtow) to dalej nie dziala...

[KrzaQ2]

@erwit: to rozmiar inta teƶ bierz sizeofem.

[KrzaQ2]

@erwit:

for(int j=0; j

if(arr[j]>arr[j+1])

std::swap(arr[j], arr[j+1]);

UB, dereferencja wskaźnika poza zakresem (wielkość 4, porównujesz na końcu 

arr[3] > arr[4]

)

[KrzaQ2]

@KrzaQ2: Btw to jest strasznie naiwna implementacja bubblesorta. Nie masz nawet optymalizacji kończenia sortowania jeśli nie dokonałeś ƶadnych zmian.

[erwit]

@KrzaQ2: dzieki, wiem jeszcze ja dopracuje

[patrolez]

@erwit: sort() w C++ działa pod kontenerami (kolekcjami) z STLa, albo z własnymi kontenerami z iteratorem i komparatorem :>

Innymi słowy najprościej jest to wsadzić w std::vector i zastosować sort() jeżeli chcesz tego tak używać.

[KrzaQ2]

@patrolez: sort na tablicy teƶ działa bezproblemowo, ale zakładam, ƶe erwit pisze własną implementację dla siebie aby zobaczyć jak to działa/whatever, a nie dlatego, ƶe nie wie o istnieniu

std::sort

std::sort(std::begin(arr),std::end(arr))

[k.....r]

@erwit:

Takie tipy:

public:

        int groesse, gewicht, alter, iq;

aaaaaaa, niemiecki!!!

a na serio: chyba private

Person(int groesse, int gewicht, int alter, int iq)

        {

                this-> groesse = groesse;

initialization list

int l=4;

czytelnosc wybitnie slaba, nie wiadomo do czego jest, a w dodatku wyglada jak 1

for(int i=0; i
                                for(int j=0; j

moze klamry?

std::swap(arr[j], arr[j+1]);

ja bym dal asserta na to, czy j+1 < rozmiar tablicy, bo tak to

[erwit]

@kuhar: dzieki, ale o co chodzi z DRY w switchu?

[k.....r]

@erwit: ze powtarzasz sie; 4 na 5 linijek jest identycznych w kazdym case'ie