Tablice jednowymiarowe

Question

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.

Tutaj jest mój kod: http://pastebin.com/Ztj8sXsa

Czy jest on poprawnie napisany? Czy mógłbym zapisaćgo przyjażniej dla oczu i w jakiś prostszy sposób?

 

 

Answer 1

Jeśli ten kod działa, to OK. Ale mi się pewne rzeczy nie podobają:

1. int losowanie(){

2.                                 int liczba;

3.                                 liczba = ( ( rand() % 7 ) + 4 );

4. } 

Ta funkcja nic nie zwraca, zmień to na:

int losowanie()
{
return  rand() % 7  + 4;
}

Może to i działa, ale ten kod czytelniejszy i jednoznaczny.

Ta pętla też trochę bez sensu:

do{
wylosowana = losowanie();
cout<< wylosowana<< ", ";
 
int tablica[licznik];
        licznik++;
} while (licznik < 999);

W każdym obrocie pętli deklarujesz tablicę... Powinno być tak:

do{
   wylosowana = losowanie();
   cout<< wylosowana<< ", ";
   tablica[licznik] = wylosowana;
   licznik++;
} while (licznik < 999);

Do policzenia sumy użyj pętli for. To ona jest najlepsza w iteracji po tablicach:

int suma = 0;
for(int licznik = 0; licznik < 999; licznik++)
{
    suma += tablica[licznik];
}

 

 

Comments

wlasnie jade kazda lekcje po kolei wedlug kursu i tam jeszcze do 'fora' nie doszedłem...

Jakby nie liczyc metody for, to obliczenie sumy za pomoca do while jest ok?

---

Nie potrzebnie  nawet są 2 pętle, wszystko idzie zrobić w max 2 funkcjach albo jednej. Czym więcej pętli w tak banalnym programie  tym kod bardziej jest nieczytelny.

---

W sumie jest też druga strona medalu. Im więcej instrukcji w jednej pętli tym kod jest bardziej spaghetti. Ale wydajność przemawia za tym żeby to umieścić w jednej.

Answer 2

Nie zwracasz wartości z funkcji

Nie wiem po co funkcja tylko losuje a główny program zajmuje się sumowaniem-bez sens

Funkcja robi wszystko, ewentualnie jedna losuje liczby a potem tą tablicę przekazujesz np jako wskaźnik do funkcji która sumuje elementy.

//bardzo prosto bo nie mam kompilatora pod ręką i nie chce mi się z palca pisać wszystkeigo

 int  Suma(int  *tab, int ile)
{ int suma=0;
       for(int i=0; i<ile; i++)
{
  tab[i]=( ( rand() % 7 ) + 4 );
 suma+=tab[i];
}
return suma;
}
 int main
{
int tab[999];
  int SumaDanych=Suma(tab,999);
}

 

Comments

co oznacza += ?

---

to samo co $zmienna=$zmienna+jakas wartosc (skrócony zapis)