Losowanie liczb - funkcje i tablice

Question

Hej. Piszę program który ma wylosować 10 liczb z zakresu od 4 do 10, zapisać je w tablicy, wypisać, a potem je dodać. Mają być wykorzystane funkcje. Program losuje wszystkie takie same liczby. Suma też jest jakaś dziwna. Pomoże ktoś?

#include <iostream>
#include <time.h>
#include <cstdlib>

using namespace std;

int suma(int liczby[],int s)
{
    int i=0;
    do
    {
        int sm;
        return sm+=liczby[i];
        s++;
    }while (s<11);

}

void losowacz(int liczby[])
{
    srand( time( NULL ) );
    int i=0;
    do
    {
       liczby[i] = rand()%7+4;
       i++;
    } while (i<11);

}

void wypisywacz (int liczby[])
{
    int i=0;
    int ile=1;
    do
    {
        cout << "Liczba " << ile << " to: " << liczby [i] << endl;
        ile++;
    }while (ile < 11);

}


int main()
{
    int ile=1;
    int i=0;
    int k=1;
    int liczby [10];
    losowacz(liczby);
    wypisywacz (liczby);
    cout << "Suma wynosi: " << suma(liczby,1) << endl;

    return 0;
}

 

Answer 1

1. srand( ) powinno być wywołane tylko raz (na początku programu)
2. funkcja suma do przepisania.
    - zmienną sm deklarujesz przed pętlą i inicjalizujesz zerem
    - dopóki i<s dodajesz liczbę z tablicy do sm
    - po zakończeniu pętli zwracasz sm

Comments

Dzięki bardzo. głupie błędy zrobiłam, muszę się bardziej skupić :p

Answer 2

srand( time( NULL ) );

Na początku w mainie powinien być i tylko raz się to pisze.

int suma(int liczby[],int s)

{

    int i=0;

    do

    {

        int sm;

        return sm+=liczby[i];

        s++;

    }while (s<11);


}

nie zwiększasz zmiennej "i" dlatego dodaje 10 razy 1 wylosowaną liczbę do NIE ZAINICJALIZOWANEJ zmiennej sm dlatego masz dziwny wynik. sm daj na zewnątrz pętli i dopisz = 0;

 

Przy wypisywaniu tak samo nie zwiększasz zmiennej i.

 

EDIT:

return w funkcji sumującej daj na zewnątrz pętli

 

 

Comments

int suma(int liczby[], int s)
{
	int i = 0;
	int sm = 0;
	do
	{
		sm += liczby[i];
		s++;
		i++;
	} while (s<11);

	return sm;
}

void losowacz(int liczby[])
{
	
	int i = 0;
	do
	{
		liczby[i] = rand() % 7 + 4;
		i++;
	} while (i<11);

}

void wypisywacz(int liczby[])
{
	int i = 0;
	int ile = 1;
	do
	{
		cout << "Liczba " << ile << " to: " << liczby[i] << endl;
		ile++;
		i++;
	} while (ile < 11);

}

Poprawiłem żeby działało, nie patrzyłem na inne błędy typu: definiowanie zmiennej i jej nie używanie :)

---

Dzięki bardzo :)

Answer 3

Dodatkowo wychodzisz poza zakres tablicy. 

do
{
// .....
}while("zmienna" < 10 );