Losowanko
Ostatnio zmodyfikowano 2016-08-13 10:53
damian955 Temat założony przez niniejszego użytkownika |
Losowanko » 2016-08-11 22:09:40 #include<iostream>
#include<ctime>
#include<cstdlib>
using namespace std;
int losuje()
{
return( rand() % 10 ) + 1;
}
bool sprawdzam( int a[], int b, int c )
{
if( b <= 0 ) return false;
int d = 0;
do
{
if( a[ d ] == c ) return true;
d++;
} while( d < b );
return false;
}
int main()
{
srand( time( NULL ) );
int a[ 10 ], b = 0;
do
{
int c = losuje();
if( sprawdzam( a, b, c ) == false )
{
a[ b ] = c;
b++;
}
} while( b < 10 );
b = 0;
do
{
cout << a[ b ] << endl;
b++;
} while( b < 10 );
return 0;
} W bolu po co jest: if(b<=0)return false; Bo bez tego i tak losowanie wychodzi pomyślnie nie kumam. :D A i kod zajumany z tego poradnika tylko troche inne literki powstawiane. |
|
carlosmay |
» 2016-08-11 22:20:04 | W bolu po co jest: if(b<=0)return false; |
Zabezpieczenie, gdyby przekazano ujemną wartość rozmiaru tablicy (oczywisty błąd). Gdy wylosowana jest pierwsza liczba indeks tablicy jest jeszcze równy zero (zapis do pierwszego elementu), więc nie ma sensu wykonie pozostałej części kodu (stąd return). | kod zajumany z tego poradnika tylko troche inne literki powstawiane. |
Zły pomysł. Literki nic nie mówią o swoim przeznaczeniu. Trzeba tracić czas na analizę kodu. |
|
damian955 Temat założony przez niniejszego użytkownika |
» 2016-08-13 02:14:23 Zmienne ustawiłem w sposób najbardziej wygodny dla mojej osoby.
Do zamknięcia. |
|
carlosmay |
» 2016-08-13 10:53:00 | Zmienne ustawiłem w sposób najbardziej wygodny dla mojej osoby. |
To dla nikogo nie jest wygodne, a jeśli szanujesz swój czas (innych też) to pomyśl o sensownym nazywaniu zmiennych. Jak przyjdzie ci pracować przy kilku plikach po kilkaset linii z powtarzającymi się literkami i nic nie mówiącymi nazwami, zgubisz się gdzieś przy piątym szóstym pliku. Będziesz zaczynał projekty po kilka razy nie mogąc zrozumieć, czemu tak się dzieje. Czemu projekty wymykają się spod kontroli. |
|
| « 1 » |