• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Losowanie 200liczb, sortowanie ich i zliczanie porównań, zamian

VPS Starter Arubacloud
0 głosów
176 wizyt
pytanie zadane 2 lutego 2017 w C i C++ przez adrian28 Użytkownik (540 p.)

Cześć

Jest to program losujący 200liczb z zakresu od 0 do 500. Potem sortuje je bąbelkowo, a następnie zlicza liczbę porównań i zamian. Problem jest jedynie taki, że liczba porównań jest zawsze taka sama (39800), w którym miejscu mam błąd? Jak mogę usprawnić działanie programu tak aby wynik był pokazywany od razu? Bo teraz wykonuje się linia po linii i zajmuje to trochę czasu. Prosiłbym o ewentualne ocenienie przejrzystości kodu, czy jakichś rzeczy brakuje, czy może niektóre są napisane niepotrzebnie wink

#include <iostream>
#include "losowanie.h"
using namespace std;

int main()
{
	Losowanie s1;
	s1.wypisz();

return 0;
}



//==============================================================

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
class Losowanie
{
public:
    void wypisz();
	int a[200];
	int lp,lz;

};

void Losowanie::wypisz()
{
    srand(time(NULL));
    for(int i=0; i<200; i++)
	{
    a[i]=rand()%500;
	 cout<<" "<<a[i];
	}

lp=lz=0;
for( int i = 0; i < 200; i++ )
    {
        for( int j = 0; j < 200 - 1; j++ )
        {
			lp++;
            if( a[ j ] > a[ j + 1 ] ){
                 swap( a[ j ], a[ j + 1 ] );
                 lz++;
            }
			cout<<a[j]<<" ";

        }
        cout<<"******************************************************************************************************"<<endl;
    }

 cout<<"Liczba porównań:"<<lp<<endl;
 cout<<"Liczba zamian:"<<lz<<endl;



}



 

1 odpowiedź

0 głosów
odpowiedź 2 lutego 2017 przez niezalogowany
edycja 2 lutego 2017
Liczba porównań zawsze będzie taka sama, bo masz dwie pętle w sobie i lp++ wykonuje się 200*199 razy czyli 39800. Zmieniać się może jedynie liczba zamian. Jeżeli chcesz by wynik był pokazywany od razu to ujmij w komentarz wszelkie wypisywanie liczb. To trwa bardzo długo. Sranda wrzuć do main'a (taka uwaga na przyszłość). Kod mógłby być bardziej dynamiczny, czyli różne rozmiary tablic...

Podobne pytania

0 głosów
0 odpowiedzi 449 wizyt
0 głosów
0 odpowiedzi 214 wizyt
pytanie zadane 26 maja 2018 w C i C++ przez Ala123456 Użytkownik (760 p.)
0 głosów
1 odpowiedź 200 wizyt

92,454 zapytań

141,262 odpowiedzi

319,089 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...