Panel użytkownika
Nazwa użytkownika:
Hasło:
Nie masz jeszcze konta?

Rozdział 22 , zadanie domowe . Poprawność

Ostatnio zmodyfikowano 2017-08-29 22:45
Autor Wiadomość
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 ;)

C/C++
#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;
}
P-164378
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
.
P-164379
« 1 »
  Strona 1 z 1