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

Przekazywanie tablic do funkcji - nie rozumiem jak to działa

VPS Starter Arubacloud
0 głosów
156 wizyt
pytanie zadane 12 listopada 2020 w C i C++ przez rain.deer Początkujący (430 p.)

Cześć! Próbuję rozwiązać następujące zadanie:

"Napisz program, który wylosuje 9 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. Wszelkie operacje mają zostać zrealizowane w oparciu o funkcje.Tablica ma być zadeklarowana w funkcji main. Oczekiwane funkcje : Wypełnienie tablicy o podanym rozmiarze losowymi liczbami z przedziału podawanego przez argumenty funkcji. Sumowanie określonej ilości liczb znajdujących się w tablicy i zwrot wyniku. Wypisywanie zawartości tablicy."

Przy obecnym stanie kodu, którego nie wiem jak poprawić, program losuje 10 liczb zamiast 9 i zamiast je sumować wyświetla pierwszą z wylosowanych liczb jako sumę.

Drugi problem jest większy (zresztą chyba jest źródłowy dla złego funkcjonowania tego programu): nie potrafię dobrze wywołać tych 3 funkcji w funkcji main, mimo że czytałam o tym w książce "Język c++. Szkoła programowania", oglądałam filmik pana Zelenta na ten temat i czytałam inny kurs. sad

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

void wylosujLiczby(int tablica[9])
{
    int i = 0;
   

    for (int i = 0; i < 9; i++)
    {
       tablica[i] = (rand() % 7) + 4;
    }
}
void wypiszLiczby(int tablica[9])
{
    std::cout << "Wylosowano nastepujace liczby: ";
    int suma = 0;
    for (int i = 0; i < 9; i++)
    {
        std::cout << tablica[i] << ", ";
    }

}
int obliczSume(int tablica[9])
{
    int iSuma = 0;
    for (int i = 0; i < 9; i++)
    {
        std::cout << tablica[i] << ", ";
        iSuma += tablica[i] ;
        return iSuma;
    }
}
int main()
{
    int tablica[9];
    std::srand(time(NULL));
    wylosujLiczby(tablica);
    wypiszLiczby(tablica);
    int iSuma = obliczSume(tablica);
    std::cout << "Suma liczb wynosi: " << iSuma << std::endl;
    return 0;
}

 

1 odpowiedź

+1 głos
odpowiedź 12 listopada 2020 przez tangarr Mędrzec (154,780 p.)

Każda funkcja operuje na dziewięcioelementowej tablicy którą do nich poprawnie przekazujesz.

Twoje błędy wynikają z tego że kopiowałeś i przerabiałeś kod funkcji, ale nie pozostawiłeś zbędne elementy powodując tym chaos.

Dodałem komentarze do twojego kodu

#include <iostream>
#include <ctime>
#include <cstdlib>
 
void wylosujLiczby(int tablica[9])
{
    int i = 0;
    
 
    for (int i = 0; i < 9; i++)
    {
       tablica[i] = (rand() % 7) + 4;
    }
}
void wypiszLiczby(int tablica[9])
{
    std::cout << "Wylosowano nastepujace liczby: ";
    // nieużywana zmienna suma, usuń ją
    int suma = 0;
    for (int i = 0; i < 9; i++)
    {
        std::cout << tablica[i] << ", ";
    }
    // na koniec funkcji przydałoby się wypisać znak nowej linii
 
}
int obliczSume(int tablica[9])
{
    int iSuma = 0;
    for (int i = 0; i < 9; i++)
    {
        // wypisujesz i-ty element tablicy
        // dlaczego?
        // ponadto wypisujesz go w tej samej linii, w której pisała 
        // poprzednio wywołana funkcja
        std::cout << tablica[i] << ", ";
        iSuma += tablica[i] ;
        // kończysz wywołanie funkcji po zsumowaniu tylko jednego elementu
        // przesuń operację return poza pętlę
        return iSuma;
    }
}
int main()
{
    int tablica[9];
    std::srand(time(NULL));
    wylosujLiczby(tablica);
    wypiszLiczby(tablica);
    int iSuma = obliczSume(tablica);
    std::cout << "Suma liczb wynosi: " << iSuma << std::endl;
    return 0;
}

Ponadto zauważyłem, że dla zmiennej iSuma zastosowałeś notację węgierską.
Jeżeli zrobiłeś to celowo to mam dla ciebie dobrą radę: nie stosuj jej, najlepiej o niej zapomnij, notacja węgierska jest największą bzdurą jaką próbują nam wcisnąć twórcy książek o programowaniu. Używaj długich nazw zmiennych, dzięki którym nie będziesz musiał szukać w programie za co ta zmienna odpowiada. Jeżeli chcesz się dowiedzieć jakiego jest typu najedź na nią myszką, prawie każdy edytor kodu ci pokaże typ zmiennej.

komentarz 13 listopada 2020 przez rain.deer Początkujący (430 p.)

Dziękuję za wskazówki smiley Poprawiłam kod i teraz działa. Zastanawiam się, czy można by go uprościć i skrócić, zachowując wymagania zawarte w zadaniu?

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

void wylosujLiczby(int tablica[9])
{
    int i = 0;
       for (int i = 0; i < 9; i++)
    {
       tablica[i] = (rand() % 7) + 4;
    }
}
void wypiszLiczby(int tablica[9])
{
    std::cout << "Wylosowano nastepujace liczby: " << std::endl;
   
    for (int i = 0; i < 9; i++)
    {
        tablica[i];
    }
   }
int obliczSume(int tablica[9])
{
    int sumaLiczb = 0;
    for (int i = 0; i < 9; i++)
    {
        std::cout << tablica[i] << ", ";
        sumaLiczb += tablica[i] ;
    }
    return sumaLiczb;
} 
int main()
{
    int tablica[9];
    std::srand(time(NULL));
    wylosujLiczby(tablica);
    wypiszLiczby(tablica);
    int sumaLiczb = obliczSume(tablica);
    std::cout << "Suma liczb wynosi: " << sumaLiczb << std::endl;
    return 0;
}

 

Podobne pytania

0 głosów
2 odpowiedzi 865 wizyt
pytanie zadane 1 października 2017 w C i C++ przez qlucha Obywatel (1,790 p.)
+1 głos
2 odpowiedzi 394 wizyt
pytanie zadane 18 czerwca 2015 w C i C++ przez szymonn7 Początkujący (350 p.)
–2 głosów
5 odpowiedzi 466 wizyt
pytanie zadane 18 czerwca 2015 w C i C++ przez szymonn7 Początkujący (350 p.)

92,453 zapytań

141,262 odpowiedzi

319,087 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!

...