Rozdział 22 , zadanie domowe . Poprawność
Ostatnio zmodyfikowano 2017-08-29 22:45
kundelko Temat założony przez niniejszego użytkownika |
Rozdział 22 , zadanie domowe . Poprawność » 2017-08-29 21:51:58 Witam . Prosił bym o ocenę ostatniego zadania z rozdziału 22 . O sprawdzenie poprawności kodu jak i wskazówki co robić lepiej / jak poprawić przejrzystość kodu . Z góry dziękuje za pomoc ;) #include <iostream> #include <cstdlib> #include <ctime>
using namespace std; void zapis( int t[] ) { for( int i = 0; i <= 9; i++ ) { t[ i ] =( rand() % 21 ) + 10; } } void wypisztablice( int tt[] ) { for( int i = 0; i <= 9; i++ ) { cout << tt[ i ] << endl; } } void odtylu( int tk[] ) { for( int i = 9; i >= 0; i-- ) { cout << tk[ i ] << endl; } } int maxim( int q[] ) { for( int l = 30; l >= 10; l-- ) { for( int i = 0; i <= 9; i++ ) { if( q[ i ] == l ) return i; } } } int mini( int q[] ) { for( int l = 10; l <= 30; l++ ) { for( int i = 0; i <= 9; i++ ) { if( q[ i ] == l ) return i; } } }
int main() { int k, tablica[ 10 ]; cout << "Hello world!" << endl; srand( time( NULL ) ); k =( rand() % 20 ) + 10; zapis( tablica ); cout << "liczby to:" << endl; wypisztablice( tablica ); cout << "odtylu" << endl; odtylu( tablica ); cout << "wartosc max" << endl; cout << tablica[ maxim( tablica ) ] << endl; cout << "warosc min" << endl; cout << tablica[ mini( tablica ) ]; return 0; }
|
|
karambaHZP |
» 2017-08-29 22:45:21 1. Używać znaczących nazw zamiast tt , tk itd. Mnie nic nie mówią te nazwy. 2. Używać nazwanych zmiennych zamiast Magic Number. 3. Funkcje wyszukujące max i min są zdrowo przekombinowane. W pesymistycznym przypadku zrobią 300 iteracji zamiast maks 10, które to wystarczą do znalezienia max i min. 3. W main jest nieużywana zmienna k . |
|
« 1 » |