[C++]Problem z tablicami

[dejot44 0]

Witam!Ostatnio podczas mojego kursu dostałem taką pracę domową:"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.".Podczas pisania programu utknąłem po napisaniu tego:

#include <iostream>

#include <cstdlib>

#include <ctime>

using namespace std;

 

int main()

{

srand( time( NULL ) );

int a[ 999 ];

int losowanka;

do

{

losowanka = (std::rand() % 7 ) + 4;

a[ losowanka ];

}while( losowanka<999 );

 

 

return 0;

}

 

Co mógłbym dodać,żeby program działał tak jak powinien?Z góry dziękuje za pomoc

[Medivho 2]

a[ losowanka ];

Miałeś raczej na myśli:

a[i] = losowanka;

i nie pętla while losowanka<999 bo to raczej nie ma większego sensu skoro losowanka jest zawsze z przedziału [4-10] tylko:

for(int i=0;i<999;++i)

[dejot44 0]

Czyli w jaki sposób można by było to załatwić w praktyce?Możecie podać przykład z moim programem?Chodzi o to,że jestem niezbyt kumaty z reguły

[Ijoo 0]

1. Twoja pętla

do
{
losowanka = (std::rand() % 7 ) + 4;
a[ losowanka ];
}while( losowanka<999 );

nigdy nie zostanie zakończona, ponieważ

losowanka = (std::rand() % 7 ) + 4;

nigdy nie przekroczy wartości 999. Potrzebowałbyś jakiegoś licznika, który będzie dodawał co każdy obrót pętli 1.

Do tego celu najlepiej nadaje się pętla for, ponieważ wiesz ile razy ma być wykonana, przykład for'a z tablicą:

int a[999];
for(int i = 0; i < 999; i++)
     a[i] = i;

Pokombinuj z tym

 

2.Błąd także masz w:

a[ losowanka ];

powinno wyglądać tak:

a[JAKAS_ZMIENNA_LICZNIKOWA] = losowanka;

W ten sposób przypisujesz daną wartość we skazany indeks tablicy.

 

3. Taka mała uwaga, jak już masz

using namespace std;

to nie potrzebujesz używać std w:

losowanka = (std::rand() % 7 ) + 4;

[dejot44 0]

Dzięki udało mi się .Jest "losowanka = (std::rand() % 7 ) + 4;",ponieważ skopiowałem to z innego programu i nie chciało mi się usuwać "std::".

[cube0 0]

Nie ma sensu wywoływać pr zestrzeni std do tak krótkiego programiku. Ja wiem masz obszerną pamięć, ale tu bardziej chodzi mi o styl programowania. Wywołaj tą przestrzeń np. std::cout