Dla odmiany dzisiaj chcę się pochwalić samodzielnie zrobionym programem.

Question

Cześć, po kilku postach, w których pytałam o sposób rozwiązania błędów w swoich kodach, chciałabym się dziś pochwalić krótkim, samodzielnie (tzn. z wykorzystaniem elementów z wcześniejszych kodów i lekcji) napisanym programikiem, który jak mi się wydaje działa dobrze.  Zadanie było następujące: 

"Napisz program, który wylosuje 999 liczb całkowitych z zakresu od 4 do 10 włącznie, wypisze te wartości na ekran, po czym zsumuje je i wynik wypisze na ekran. Program ma wykorzystywać tablicę, która zostanie najpierw wypełniona liczbami losowymi z określonego przedziału, a następnie wynik zostanie obliczony na podstawie zawartości całej tablicy."

#include <iostream>
#include <ctime>
#include <cstdlib>

int main()
{
    int liczba[999];
    int licznik = 0;
    std::srand(time(NULL));
    do
    {
       liczba[licznik] = (rand() % 7) + 4;
        licznik++;
    } while (licznik < 999);

    std::cout << "Podales nastepujace liczby: ";
    licznik = 0;
    do
    {
        std::cout << liczba[licznik] << ", ";
        licznik++;
    } while (licznik < 999);
    unsigned long long suma = liczba[0];
      suma = suma + liczba[licznik++];
        std::cout << "Suma wylosowanych liczb to " << suma << std::endl;

    return 0;
}

 

Comments

#include <iostream>
#include <ctime>
#include <cstdlib>
 
int main()
{
    int liczba[999];
    std::srand(time(NULL));
    for (int i=0; i<999; ++i) {
        liczba[i] = (rand() % 7) + 4;
    }
 
    std::cout << "Wylosowano nastepujace liczby:\n";
    unsigned long suma = 0;
    
    for (int i=0; i<999; ++i) {
        if (i % 20 == 0)  std::cout << "\n";
        std::cout << liczba[i] << ", ";
        suma += liczba[i];
    }
    
    std::cout << "\n\nSuma wylosowanych liczb to " << suma << std::endl;
 
    return 0;
}

 

---

Z pewnością tak to należało zrobić, ale.. otrzymywanie gotowców nie pomaga w nauce.  Ale dziękuję za ten fragment z "suma += liczba[i]", którego nie znałam. Moim sposobem pierwsza z wylosowanych liczb była sumowana 2 razy i fałszowało to wynik. A nie wiedziałam, jak to poprawnie zapisać. 

Answer 1

Można napisać to bez użycia tablic i z wykorzystaniem biblioteki random. (C++ 11)

#include <iostream>
#include <random>
#include <iomanip>

using namespace std;

int main()
{
    mt19937 gen(random_device{}());
    uniform_int_distribution<> distrib(4,10);

    cout << "Wylosowano nastepujace liczby:" << endl;

    size_t suma {0};
    for( auto i {0} ; i<999 ; ++i )
    {
        if( i%30 == 0 ) cout << endl;
        cout << setw(2) << [&suma]( int number ){ suma += number; return number; }(distrib(gen)) << ' ';
    }

    cout << endl << endl << "Suma wylosowanych liczb to " << suma << endl;

	return 0;
}

 

Comments

lambda nie musi być tu przypadkiem oznaczona jako mutable ?

---

Nie.

Słowo mutable odnosi się obiektów przechwyconych za pomocą [=] , tzn. obiektów kopiowanych (przekazywanych przez wartość). W takim przypadku mogą one być modyfikowane, oraz można przy ich pomocy wywoływać nie stałe funkcje składowe (jeżeli takie posiadają).

Przykład

#include <iostream>

using namespace std;

struct Foo
{
    void print(){ cout << "non const Foo" << endl; }
    void print_c() const { cout << "const Foo" << endl; }
};

int main()
{
    Foo foo;

    [=](){ foo.print_c(); }();
    [=]() mutable { foo.print(); }();
    [=]() { foo.print(); }(); // !!! does not compile !!!

	return 0;
}

 

---

@TOM_CPP, jeszcze nie znam takich rzeczy, które użyłeś w swoim kodzie. Uczę się dopiero od miesiąca.  A w treści zadania był wymóg zastosowania tablic, bo o tym była lekcja.

---

@TOM_CPP, Globalna przestrzeń nazw bije po oczach. Poza tym kod nie jest zgodny z c++11, a 17 i w górę. W pętli auto i {0} jest listą inicjalizującą.  

#include <iostream>
#include <random>
#include <iomanip>
  
int main()
{
    using std::cout, std::endl, std::setw, std::mt19937, std::uniform_int_distribution, std::random_device;
    auto gen = mt19937(random_device{}());
    auto distrib = uniform_int_distribution(4,10);
  
    cout << "Wylosowano nastepujace liczby:" << endl;
  
    size_t suma {0};
    for( auto i {0}, upperBound{999}; i < upperBound ; ++i )
    {
        if(const auto numberOfNumbersInOneLine {30}; i%numberOfNumbersInOneLine == 0 ) 
        {
            cout << endl;
        }
        cout << setw(2) << [&suma]( int number ){ suma += number; return number; }(distrib(gen)) << ' ';
    }
  
    cout << endl << endl << "Suma wylosowanych liczb to " << suma << endl;
}

Lambda też wydaje mi się zbędna.

---

@rain.deer,
Tak wiem, przykład pokazuje, że zadanie tego typu można napisać bez używania tablic, które w zależności od swojego rozmiaru, mogą zajmować dużo miejsca w pamięci.

---

@tkz,  

Kod jest zgodny z standardem C++11 zobacz  , natomiast Twój przykład już nie: zobacz

Zmienna zdefiniowana za pomocą auto i {0}; nie jest listą inicjalizującą  przykład

 

 

---

@TOM_CPP,

cpp.sh/3psry 

Wiem, że mój kod nie jest zgodny z standardem 11. Dlatego też o tym nie wspominałem. Dopiero w standardzie 17 zmieniono postrzeganie auto foo{1}; z listy na int'a. 

https://dsp.krzaq.cc/post/1417/artykul-cpp17-nowy-milosciwie-panujacy-nam-standard-c-z-programisty-66/

 

 https://stackoverflow.com/questions/25612262/why-does-auto-x3-deduce-an-initializer-list

---

Zgadza się, ale zmianę tę zastosowano wstecznie w standardzie C++11 (z uwagi na możliwe problemy z kompatybilnością kodu), stąd też dzisiaj auto foo{1} kompiluje się jako zmienna typu int w C++11.

Answer 2

Czy zwracany wynik nie wzbudza w Tobie absolutnie żadnych podejrzeń?

Comments

Ha, wiedziałam, że coś musi być jednak nie tak, skoro program się skompilował i nie wywalił żadnych błędów. Rzeczywiście - wynik jest bardzo duży.

---

Jak zatem trzeba by poprawnie przeprowadzić to sumowanie? Wydawało mi się, że właśnie w taki sposób, że jako sumę najpierw deklaruję pierwszą liczbę z tablicy, a następnie dodawane są do niej kolejne liczby.z rosnącym licznikiem.

---

Wynik jest bardzo duży? U mnie jest bardzo mały :)
Gdyby zwróciło mi coś dużego, to bez przyjrzenia się bym w to uwierzył.

Tak, właśnie mniej więcej tak to można zrobić, ale popatrz, co Ty zrobiłaś. Sumujesz poza pętlą, więc tam się nic nie zmienia, nic nie rośnie, wszystko jest stałe.

---

Właśnie przed chwilą zdałam sobie sprawę, że faktycznie, trzeba było tam zrobić jeszcze pętlę przy sumowaniu. Zmyliło mnie to, że program rzeczywiście wypisał wylosowane liczby na ekran i podał sumę. Jasne, nie dodawałam ręcznie 999 liczb, żeby sprawdzić, czy wynik jest prawdopodobny.

---

Można, ale też zastanów się, czy trzeba tak to sobie rozbrajać na części, czy może nie lepiej będzie wpisywać liczby do tablicy, sumować i wyświetlać je w jednej pętli :)

---

Nie trzeba aż tyle dodawać ręcznie, wystarczy zauważyć, że suma ~1000 liczb z zakresu <4,10> będzie w zakresie <4000, 10000>.  Uwagi:

1. 3 razy w kodzie masz stałą 999.
2. Do przeglądania tablic naturalne jest używanie pętli for. Oczywiście można to zrobić nawet na goto, ale program ma przede wszystkim być czytelny dla ludzi.
3. Deklarowanie dużych tablic na stosie jest nieco ryzykowne. Akurat te parę KB się zmieści, ale jedna funkcja może wywołać drugą, ta trzecią i zajętość stosu rośnie. Są też środowiska, gdzie stos jest dość mały.
4. Słowo "Podales" jest trochę nieodpowiednie. Raczej "Wylosowano".